Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl Monk, Perl Meditation
 
PerlMonks  

Re: Perl Style Guides for Large Projects

by dws (Chancellor)
on Nov 01, 2002 at 04:25 UTC ( #209634=note: print w/ replies, xml ) Need Help??


in reply to Perl Style Guides for Large Projects

I've worked on several mid-sized projects (including one 80KLOC Perl application suite) that used fairly minimal style guidelines, and a couple of projects that used coding standards.

What I've found works best is a combination of light-weight guidelines coupled with "educational" code reviews. Reviewing work by senior team members demonstrates what "good" is to junior team members. Reviewing work by junior team members is an opportunity for gentle correction. Even in the absence of standards or guidelines, senior people can usually form a concencus about where the boundaries around "good" are, and the rest of the team can learn by example.

This approach might not scale to a situation where all of the coders are peers, without an acknowledged senior/junior distinction.


Comment on Re: Perl Style Guides for Large Projects
Re^2: Perl Style Guides for Large Projects
by adrianh (Chancellor) on Nov 01, 2002 at 11:05 UTC
    This approach might not scale to a situation where all of the coders are peers, without an acknowledged senior/junior distinction.

    I've found it still works well for groups of peers as long as there is collective code ownership.

    I'm not sure if this is just because CCO is just a sensible idea, or if the... individuals... who cannot take constructive criticism can't stand to work in an environment with CCO :-)

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (11)
As of 2014-10-22 16:08 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    For retirement, I am banking on:










    Results (119 votes), past polls