org.sonar.l10n.java.rules.java.S1107.html Maven / Gradle / Ivy

Go to download
Why is this an issue?
Shared coding conventions make it possible for a team to collaborate efficiently.
This rule makes it mandatory to place closing curly braces on the same line as the next else, catch or
finally keywords.
Noncompliant code example
public void myMethod() {
  if(something) {
    executeTask();
  } else if (somethingElse) {
    doSomethingElse();
  }
  else {                               // Noncompliant
     generateError();
  }

  try {
    generateOrder();
  } catch (Exception e) {
    log(e);
  }
  finally {                            // Noncompliant
    closeConnection();
  }
}

Compliant solution
public void myMethod() {
  if(something) {
    executeTask();
  } else if (somethingElse) {
    doSomethingElse();
  } else {
     generateError();
  }

  try {
    generateOrder();
  } catch (Exception e) {
    log(e);
  } finally {
    closeConnection();
  }
}