Ug :) I really hate that. I see no benefit in spaces both sides of the commas, or breaking the open paren away from the function name. And if you have to break the params across lines, at least balance their lengths :)
some_function(
some_variable, some_other_variable,
and_yet_another_variable, and_one_more_for, luck
);
That's not so bad for simple (void) calls as, but when you're retrieving data and checking you get something like
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
}
It's just a mess.The best I've come up with for this is
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
}
Which ain't great, but is better than most alternatives to my eyes.
And much better still is
if( someRv = fSome( some, oSome, AYAnother, OneMoreFor, luck )) {
// do some stuff here with someRv
}
else {
// report or otherwise handle the error
}
With the point being that whilst ths abbreviated variable names don't immediately make much sense, by the time a programmer has got familiar enough with the code to consider making changes, they will.
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.
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.