I agree. At some point you have to admit to yourself that the hacks and workarounds you're piling on are getting too intricate, and could collapse under their own weight, and certain parts of your code are going to need a serious refactor.

I think the battle we wage in our minds is that smaller fixes and additions require a lot less time in and of themselves, and because they touch less of the code, are less likely to introduce bugs. However, the more of these fixes we implement, the more time we spend on them, and the harder they can be to understand. They can require a lot of documentation (because someone not familiar with the code might look at it and think "what the *^&^%'s going on here?!"), and generally cause a headache for everyone.

There's a not-easily-definable point when working around design flaws becomes a more time (and money) consuming endeavour than refactoring. (And even if you can define that point, a bigger task is often convincing management that only urgent bug fixes can be attended to while you rewrite several core modules).

I don't agree that these changes should never be applied to production code. A good release policy and regression tests will help to catch any introduced bugs, and it's often worth the pain in the long run at any rate.

In reply to Re: Structural Elegance by Mutant
in thread Structural Elegance by samizdat

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.