Beefy Boxes and Bandwidth Generously Provided by pair Networks
Don't ask to ask, just ask

Re: How do you critique another person's code?

by perrin (Chancellor)
on Dec 19, 2001 at 23:37 UTC ( #133233=note: print w/replies, xml ) Need Help??

in reply to How do you critique another person's code?

Code reviews are hard. That's because it's really hard to take someone else picking at your code the first few times it happens. I've had a hard time accepting criticisms in code reviews, and I know how important it is to have them and to listen.

My advice is to think about how you would feel on the other side of the table and act accordingly. Be as nice as you can about it, or the guy will reject everything you say out of spite.

It's also a good idea for your company to invest some money into training resources for this guy. Maybe a good book about style and maintenance issues, like Code Complete. Maybe let him ride a more competent coder's coattails for a bit, doing maintenance on some solid code. I learned a lot that way.

  • Comment on Re: How do you critique another person's code?

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://133233]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others chilling in the Monastery: (5)
As of 2018-04-20 07:18 GMT
Find Nodes?
    Voting Booth?