【PHPMD】else文の警告抑止対応！ヒステリックすぎ！ │ Traveler

PHPMD(PHP Mess Detector)というプラグインはヒステリックすぎる。。

PHPで開発をするにあたり、静的解析ツールを入れたく、PHPMDというプラグインをVSCodeにインストールしたのですが、このプラグインが、まあヒステリックで。

過剰なまでに「ダメ！！」って言ってくるので、正直うざいです。

ただ、警告自体は抑止もできるため、今回はその中の一つ、「else文」に対する警告に対しての抑止方法を解説します。

1. 前提
2. else文の警告
3. else文の警告抑止方法
4. 参考サイト
5. まとめ

前提

・「PHP Mess Detector」がインストールされていること

else文の警告

例えばこんなコードがあるとします。

if (isset($definition['metadata']['serviceIdentifier'])) {
    $this->serviceName = $this->getServiceName();
} else {
    $this->serviceName = $this->getEndpointPrefix();
}

if (isset($definition['metadata']['serviceIdentifier'])) {

$this->serviceName = $this->getServiceName();

} else {

$this->serviceName = $this->getEndpointPrefix();

}

VSCodeで見ると、こんな感じで警告が出ます。

else文の警告

※エラー内容は、「[PHP Mess Detector] The method resolve uses an else expression. Else is never necessary and you can simplify the code to work without else.」みたいな感じです。

else文の警告抑止方法

PHPMDの警告は基本的に「@SuppressWarnings」を付与することで抑止できます。

else文の警告を抑止するには、「ElseExpression」を設定します。

/**
 * hogehoge
 * 
 * @SuppressWarnings(PHPMD.ElseExpression)
 */
private function hogehoge()
{
    if (isset($definition['metadata']['serviceIdentifier'])) {
        $this->serviceName = $this->getServiceName();
    } else {
        $this->serviceName = $this->getEndpointPrefix();
    }
}