Code Inspection: Unnecessary 'else' branch
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.
function returnGreater($a, $b) {
if ($a >= $b) {
return $a;
} else {
return $b;
}
}
function returnGreater($a, $b) {
if ($a >= $b) {
return $a;
}
return $b;
}
Suppress an inspection in the editor
Place the caret at the highlighted line and press Alt+Enter or click .
Click the arrow next to the inspection you want to suppress and select the necessary suppress action.
Last modified: 25 March 2024