Beefy Boxes and Bandwidth Generously Provided by pair Networks
Do you know where your variables are?
 
PerlMonks  

Re: Ninja style coding... is it bad?

by blue_cowdawg (Prior)
on Sep 04, 2004 at 23:39 UTC ( #388541=note: print w/ replies, xml ) Need Help??


in reply to Ninja style coding... is it bad?

      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.


Comment on Re: Ninja style coding... is it bad?

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others perusing the Monastery: (15)
As of 2014-07-28 20:00 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    My favorite superfluous repetitious redundant duplicative phrase is:









    Results (207 votes), past polls