Beefy Boxes and Bandwidth Generously Provided by pair Networks
Clear questions and runnable code
get the best and fastest answer
 
PerlMonks  

Re^3: The joys of bad code

by Limbic~Region (Chancellor)
on Oct 26, 2004 at 11:14 UTC ( #402536=note: print w/ replies, xml ) Need Help??


in reply to Re^2: The joys of bad code
in thread The joys of bad code

Happy-the-monk,
This isn't quite the same thing. Unless you make the total method lvaluable, neither

$self->total++; # nor $self->total = $self->total + 1;
will work. I think the point wasn't about incrementing a variable by one, but feeling the need to document it. I do see the point in documenting it (not in what the code does, but why it does it), but to each their own.

Cheers - L~R


Comment on Re^3: The joys of bad code
Download Code
Re^4: [OT] The joys of bad code
by Happy-the-monk (Monsignor) on Oct 26, 2004 at 11:27 UTC

    I do see the point in documenting it (not in what the code does, but why it does it)

    So this is off topic: bad documentation does not bad code make.

    Documenting what the code does isn't bad by itself, it just isn't needed at all if the people supposed to be reading the documentation can program already.

    Imagine they can't, that kind of documentation might help them understand what's happening.

    Cheers, Sören

      If they can't program already, why are they digging around in code?

      Over-documentation does make bad code. The documentation in the cases above is highly redundant with the actual code. Humans like some redundancy to keep communication robust, but there is a threshold where it just becomes silly. Flowerbox comments like the above go well past that point.

      "There is no shame in being self-taught, only in not trying to learn in the first place." -- Atrus, Myst: The Book of D'ni.

        why are they digging around in code?

        We can never know, can we?
        Just a happy guess: to learn from a code example?
        Another one: to do something we would not want them to do?

        I wasn't advocating that kind of documentation, if you thought I was -
        But some managers strongly do that.
        I do understand their views, even though I think they are flawed.

        Cheers, Sören

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://402536]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others chilling in the Monastery: (4)
As of 2014-11-27 22:18 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    My preferred Perl binaries come from:














    Results (188 votes), past polls