我正在使用一个开源项目,并认为使用phpmd实现自动代码修订是个好主意。
它向我展示了许多我已经修复的编码错误。但是其中一个让我很好奇。
考虑以下方法:
/**
*
* @param string $pluginName
*/
public static function loadPlugin($pluginName){
$path = self::getPath()."plugins/$pluginName/";
$bootPath = $path.'boot.php';
if(\is_dir($path)){
//Autoload classes
self::$classloader->add("", $path);
//If theres a "boot.php", run it
if(is_file($bootPath)){
require $bootPath;
}
}else{
throw new \Exception("Plugin not found: $pluginName");
}
}
在这里,phpmd 说Else 从来没有必要
...永远不需要带有 else 分支的 if 表达式。您可以以不需要 else 的方式重写条件,并且代码变得更易于阅读。...
is_dir
只要给定路径是文件或根本不存在,就会返回 false,因此,在我看来,此测试根本无效。
有没有办法解决它,或者可能只是忽略这样的情况?