Beefy Boxes and Bandwidth Generously Provided by pair Networks
Pathologically Eclectic Rubbish Lister
 
PerlMonks  

Re^5: What is code readability?

by Anonymous Monk
on Sep 10, 2009 at 09:26 UTC ( #794529=note: print w/ replies, xml ) Need Help??


in reply to Re^4: What is code readability?
in thread What is code readability?

if( some_return_value = some_function( some_variable, some_other_variable, and_yet_another_variable, and_one_more_for, luck ) ) { // do some stuff here with some_return_value } else { // report or otherwise handle the error }

My take on it is pretty close, I just see OK to open more than one paren in a line (and hence close them all in the same line):

if (some_return_value = some_function ( some_variable, some_other_variable, and_yet_another_variable, and_one_more_for, luck )) { // do some stuff here with some_return_value } else { // report or otherwise handle the error }


Comment on Re^5: What is code readability?
Select or Download Code
Re^6: What is code readability?
by BrowserUk (Pope) on Sep 10, 2009 at 10:52 UTC

    Yes, That's okay too, though I do prefer to have the indent level reflect the number of currently open parens. So:

    if( some_return_value = some_function( some_variable, some_other_variable, and_yet_another_variable, and_one_more_for, luck )) { // do some stuff here with some_return_value } else { // report or otherwise handle the error }

    but sensible length variable names (and camelCase :) are better.

    if( someRV = someFunc( some, someOther, yetMore, oneMore4, luck ) ) { // do some stuff here with some_return_value } else { // report or otherwise handle the error }

    Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
    "Science is about questioning the status quo. Questioning authority".
    In the absence of evidence, opinion is indistinguishable from prejudice.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others lurking in the Monastery: (7)
As of 2014-12-27 14:38 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    Is guessing a good strategy for surviving in the IT business?





    Results (177 votes), past polls