У меня есть этот код в функции:
if ($route !== null) { // a route was found
$route->dispatch();
} else {
// show 404 page
$this->showErrorPage(404);
}
Теперь PHPmd выдает сообщение об ошибке:
Прогон метода использует выражение else. Else никогда не является необходимым и вы можете упростить работу кода без работы.
Теперь мне интересно, действительно ли это был бы лучший код, чтобы избежать else и просто добавить оператор return в часть if?
Я бы не стал беспокоиться о том, что говорит PHPmd, по крайней мере в этом случае.
Вероятно, они предназначены для использования условного оператора, потому что (по их мнению) его "чище".
$route !== null ? $route->dispatch() : $this->showErrorPage(404) ;
PHPMD ожидает, что вы используете оператор раннего возврата, чтобы избежать блокировки else. Что-то вроде следующего.
function foo($access)
{
if ($access) {
return true;
}
return false;
}
Вы можете подавить это предупреждение, добавив следующее в блок класса doc.
/**
* @SuppressWarnings(PHPMD.ElseExpression)
*/