Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic

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

by blue_cowdawg (Monsignor)
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.

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://388541]
[marto]: LanX I'd rather commit sudoku :P
[marto]: trench humour folks, feeling rough.
[marto]: there is never a night when they sleep all the way through, but last night was something else
[marto]: gave in at 3:45 ish and let Charlie watch videos about spitfires, the battle of Britain. He's plane crazy
[1nickt]: marto soothing!
[marto]: Hiromi makes an appearance, let's hope Jools doesn't ruin it by joining in on the old Joanna
[1nickt]: karlgoethebier What is the issue with the semic-colon after the ellipsis? It's documented as proper syntax ...;
[karlgoethebier]: Crazy? What should i say? Son just started his third studies. Next week i have a session with my therapist ;-)
[marto]: and just as I type that, the boys begin playing the piano, the price I pay for using cockney slang :P

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (6)
As of 2017-11-18 18:56 GMT
Find Nodes?
    Voting Booth?
    In order to be able to say "I know Perl", you must have:

    Results (277 votes). Check out past polls.