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 
}

[download]

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 
}

[download]

Replies are listed 'Best First'.
Re^6: What is code readability?
by BrowserUk (Patriarch) 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 
}

    [download]

    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 
}

    [download]
    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.
[reply]
[d/l]
[select]

In Section Meditations