• Home
• org.codehaus.sonar-plugins.java
• java-checks
• 2.7
• source code
• S1862.html

×

Project download

Many resources are needed to download a project. Please understand that we have to compensate our server costs. Thank you in advance.
Project price only 1 $

You can buy this project and download/modify it how often you want.

org.sonar.l10n.java.rules.squid.S1862.html Maven / Gradle / Ivy

  A chain of if/else if statements is evaluated from top to bottom. At most, only one branch of the chain will be executed: the first one with a condition that evaluates to true.


  Therefore, duplicating a condition in a sequence of if/else if statements automatically leads to dead code. Usually, is due to a copy/paste error. Obviously it could lead to unexpected behavior.


Noncompliant Code Sample
if (param == 1)
  openWindow();
else if (param == 2)
  closeWindow();
else if (param == 1)  // For sure this is a bug
  moveWindowToTheBackground();


Compliant Solution
if (param == 1)
  openWindow();
else if (param == 2)
  closeWindow();
else if (param == 3)
  moveWindowToTheBackground();