Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl-Sensitive Sunglasses
 
PerlMonks  

Re^4: On Coding Standards and Code Reviews

by eyepopslikeamosquito (Archbishop)
on Feb 19, 2009 at 09:40 UTC ( [id://744992]=note: print w/replies, xml ) Need Help??


in reply to Re^3: On Coding Standards and Code Reviews
in thread On Coding Standards and Code Reviews

I'm not sure allowing each developer to set their own indentation level works very well in teams doing pair programming. FWIW, Michael G Schwern holds the same view as Damian:

Here's all I have to say about tabs. I expect the source to look the same no matter whose editor, pager, printer or utility I run it through. Literal tabs violate this. The end.
I also notice that the Parrot coding standard mandates that "Indentation must consist only of spaces".

Though I agree with Conway and Schwern on this, the main reason I chose spaces over TABs is that is we currently do that (mostly), and I don't want to change over a million lines of legacy code. :-)

  • Comment on Re^4: On Coding Standards and Code Reviews

Replies are listed 'Best First'.
Re^5: On Coding Standards and Code Reviews
by puudeli (Pilgrim) on Feb 19, 2009 at 10:39 UTC
    I'm not sure allowing each developer to set their own indentation level works very well in teams doing pair programming.

    Of course you let the team decide the indentation scheme and then everybody sticks to it. There is no reason to allow chaos :-)

    Here's all I have to say about tabs. I expect the source to look the same no matter whose editor, pager, printer or utility I run it through. Literal tabs violate this. The end.

    Well, that is one way to see it but it feels too dogmatic for me. To achieve it, you would have to dictate coding standards and enforce them brutally.

    The code never looks the same (unless I have total control over it, which I rarely do) and people tend to write code differently using different formatting (opening braces, spaces in array indexing etc.) and some editors provide syntax high-lightning and other tools forcing the looks to be different. The style to aim for is the one that can be changed/converted to the desired format, no matter what scheme or tool was used.

    --
    seek $her, $from, $everywhere if exists $true{love};

Log In?
Username:
Password:

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

How do I use this?Last hourOther CB clients
Other Users?
Others about the Monastery: (5)
As of 2024-04-24 10:09 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found