in reply to Re^4: Legacy Code: "Dominoes = Bad"
in thread Legacy Code: "Dominoes = Bad"

"Does the job". Requirements always change.
Yeah, but you either know what's going to change (and then do it right the first time), or you don't (and then, how do you anticipate?)
Maintainability lends agility to code
Eh, no. That's quite the opposite. Doing stuff now that isn't required is not agile. It's overhead. From The Agile Manifesto: Simplicity--the art of maximizing the amount of work not done--is essential. Anticipating to unknown future requirements does not lead to simpler code. Nor to less amount of work now.
Only maintainable, flexible code has any chance at all of being delivered on time while requirements keep changing.
Wait. By doing more work now, it's going to be finished sooner? How does that work? In the same way that paying more taxes increases my spending power?
Only maintainable code can be easily tuned and adjusted to suit performance or resource constraints. Non-maintainable code locks in specific algorithms and inflexible use-cases.
You got any data to back that up? Basically, you are saying it's impossible to write code that's fast enough, while it's not maintainable. So, code that's running fast enough, but of which the sources get lost (a real downer on its maintainability) suddenly slows down?
Poorly written code is just poorly written code.
Sure. "X == X" statements (even if one does rephrase the second X -- which you don't even bother to) are known as tautologies. They are a rhetoric device, and actually contribute nothing to the discussion.

Replies are listed 'Best First'.
Re^6: Legacy Code: "Dominoes = Bad"
by armstd (Friar) on Apr 29, 2011 at 15:55 UTC

    The assertion that writing maintainable code implies more work is a falacy. It only implies different work, not more work.

    Maintainability does not imply anticipating all future requirements either. It only implies anticipating that requirements will change, and therefore the code already developed will need to change as well.

    You cite the Agile Manifesto, but miss that the basic definition of agility is ability to change (direction, speed, etc). The basic definition of maintenance (bugfix, enhancement) is change. Agility is maintainability.

    The Agile Manifesto also says "Continuous attention to technical excellence and good design enhances agility."

    The most common aspect of non-maintainable code that you'll find is poor design.

    Good design will breed better code reuse, higher cohesion, and looser coupling. All of these are also fundamentals of maintainability.

    Maintainable code is less work, not more. Its different work, requiring design expertise. Expertise that not all programmers have, and some programmers just develop naturally, and don't even realize they're doing so. For programmers without design expertise, talent, or aesthetic, yes maintainability is a lot of hard work, perhaps of questionable value. The Dunning–Kruger effect in action.

    --Dave

      The assertion that writing maintainable code implies more work is a falacy. It only implies different work, not more work.
      So, code that isn't written to be maintainable actually requires an additional effort? Curious. Can you elaborate, with some non-trivial examples? For instance, show me a method that does something non-trivial in the simplest possible way, then a maintainable version of it, and then show that it required no more work to write.
      You cite the Agile Manifesto, but miss that the basic definition of agility is ability to change (direction, speed, etc).
      Ditching, rewrite, copy-paste-and-change-one-line are all change as well. It's usually not what people consider "maintainable".
      The most common aspect of non-maintainable code that you'll find is poor design.
      Ah, yes, "poor design". What is poor design? More often than not people say "poor design" but they mean "I would have done it differently", or "I'm not smart enough to understand it". It's highly subjective.
      Maintainable code is less work, not more.
      I'm calling your bluff. Show it.
        Could it be that monks of your experience will see almost any example as trivial, because it's something you've been doing for years?

        If not, what about CGI.pm? Would you agree that using CGI.pm is less effort and more maintainable? I posit that this is a non-trivial example because it is so frequent (still!), particularly roll-your-own query-string parsing.

        How about descriptive varible names? One might argue that typing $target_hostname is more physical effort than $th, but descriptive hostnames can reduce cognitive effort, and if you know your way around your editor, the physical difference can be minimal.



        - Boldra

        Quality of design that adheres to principles is not subjective. It can be measured against those principles. Good design is based on analysis.

        Yes, copy-paste is an example of change, and its not considered an "as" maintainable approach as reusing code, in the general case. Maintainability, like all abilities, is measured on a scale, not in black and white.

        --Dave