Skip to content

Instantly share code, notes, and snippets.

@ronocod

ronocod/50Columns.java

Last active August 29, 2015 14:20
Show Gist options
  • Save ronocod/da14c059ad42097602d0 to your computer and use it in GitHub Desktop.
Save ronocod/da14c059ad42097602d0 to your computer and use it in GitHub Desktop.
Download ZIP
Formatting Discussion
Raw
50Columns.java
public ResultType arbitraryMethodName(
FirstArgumentType firstArgument,
SecondArgumentType secondArgument,
ThirdArgumentType thirdArgument)
{
LocalVariableType localVariable =
method(firstArgument, secondArgument);
if (localVariable.isSomething(
thirdArgument, SOME_SHOUTY_CONSTANT))
{
doSomething(localVariable);
}
return localVariable.getSomething();
}
Raw
80Columns.java
public ResultType arbitraryMethodName(FirstArgumentType firstArgument,
SecondArgumentType secondArgument,
ThirdArgumentType thirdArgument) {
LocalVariableType localVariable = method(firstArgument, secondArgument);
if (localVariable.isSomething(thirdArgument, SOME_SHOUTY_CONSTANT)) {
doSomething(localVariable);
}
return localVariable.getSomething();
}
@zegnus
Copy link

zegnus commented Apr 29, 2015

I would prefer something like this

public ResultType arbitraryMethodName(
        FirstArgumentType firstArgument,
        SecondArgumentType secondArgument,
        ThirdArgumentType thirdArgument) {

    LocalVariableType localVariable = method(firstArgument, secondArgument);

    if (localVariable.isSomething(thirdArgument, SOME_SHOUTY_CONSTANT)) {
        doSomething(localVariable);
    }

    return localVariable.getSomething();
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment