Beefy Boxes and Bandwidth Generously Provided by pair Networks
Welcome to the Monastery
 
PerlMonks  

Comment on

( #3333=superdoc: print w/ replies, xml ) Need Help??

      When one opens an existing file written by a different party, puts in a fix that matches the style of the initial author with no traces left behind (except cvs knows :) )

I see good and bad with "Ninja Coding."

the good

When maintaining code other folks code I'm a very ferverent believer in the concept of "First, do no harm." If patching you do improves the code and you end up leaving no footprints behind then all is well. Having said that leaves me a very good seque into the bad

the bad

If you patch somone else's code and an unforseen consequence of that patch comes up then you need to be in a position where you can be accountable for that patch. Not for the purpose of guilt, punishment or anything negative like that, but more a case of being able to answer the question "OK, what did you do and where?" Trust me, I have a PHB that modifies my code (and the code of other co-workers of mine) without telling anybody what he did and things break usually in the wee hours of the morning. PHB doesn't even use CVS to track his change.

When coding in a cooperative environment it is a good thing to maintain lines of communication. So if I change someone else's code I usually put comments in that explain that a) I did it, b) what I did and c) what my rationale was for doing it. Sometimes I've had to modify someone's code due to a bug report or a service ticket so I put the MR or ticket number in the comments as well as in the CVS log message.


Note: for those not exposed to Dilbert®, PHB means Pointy Haired Boss.


In reply to Re: Ninja style coding... is it bad? by blue_cowdawg
in thread Ninja style coding... is it bad? by thaigrrl

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!
  • 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, 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, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • Outside of code tags, you may need to use entities for some characters:
            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.
  • Log In?
    Username:
    Password:

    What's my password?
    Create A New User
    Chatterbox?
    and the web crawler heard nothing...

    How do I use this? | Other CB clients
    Other Users?
    Others rifling through the Monastery: (8)
    As of 2014-07-29 21:22 GMT
    Sections?
    Information?
    Find Nodes?
    Leftovers?
      Voting Booth?

      My favorite superfluous repetitious redundant duplicative phrase is:









      Results (228 votes), past polls