Code Inspection: Unnecessary 'else' branch
Configure inspections: Settings | Editor | Inspections
Show intention actions: AltEnter
Reports the else
branches in if
statements whose bodies always execute a terminating statement such as return
or throw
. Such branches are redundant and can be safely omitted.
In the following example, if the condition evaluates to true
, $a
is returned and the function execution is immediately terminated. The statement return $b;
can only be reached if the condition evaluates to false
, so after the quick-fix is applied, the code is simplified by unwrapping the else
branch.
Before the quick-fix
function returnGreater($a, $b) { if ($a >= $b) { return $a; } else { return $b; }}
After the quick-fix
function returnGreater($a, $b) { if ($a >= $b) { return $a; } return $b;}
Place the caret at the highlighted line and press AltEnter or click
.
Click the arrow next to the inspection you want to suppress and select the necessary suppress action.