Re^5: What is code readability?

[Anonymous Monk]

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 
}

[BrowserUk]

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 
}