PHPMD(PHP Mess Detector)を使ってバグ検出:未使用変数編


バグがない世界に行けるものなら行きたいけど、そんな世界は残念ながら存在しないです。
ただバグを少なくすることは可能で今日はつまらないバグを未然に防ぐことが出来るツールPHP Mess Detectorを見つけたので早速使ってみたのでまとめてみる。

PHP Messdetectorとは?

指定のソース、ディレクトリ内のPHPソースコードを解析し、バグの可能性になりうる箇所を検知しリストアップしてくれます。
* 命名ルール
* 未使用変数、関数
などなど

インストール

下記2つを追加してcomposer installするだけでOK。注意点としてはpdepend/pdependに依存してくるので追記を忘れないこと。

使い方

下記形式に手動実行するだけで可能。

使ってみる

下記のサンプルソースコードを使って一番分かりやすい未使用変数チェックをしてみる。

実行してみたら見事に検知できた。テストで自分のプロジェクトのソースファイルに対して実行してみると未使用変数がいくつか見つかりぞっとした・・・。明らかに無駄なデータ取得してるよね・・・。

unusedcodeは下記の4項目をチェックしてくれる。この項目はどれもよくあるあるでとても分かりやすいし修正も簡単な例ですね。その反面コードの量が増えるとその中にまぎれてしまい見つかりにくくなるのでこういうのこそ機械的にPHPMDで検知してしまうのがいいですね。
refs : http://phpmd.org/rules/unusedcode.html

未使用privateメソッド

  • 未使用のprivateメソッドを検出する。public/protecedは検出されない。

未使用privateメンバ変数

  • 未使用のprivateメンバ変数を検出する。public/protecedは検出されない。

未使用ローカル変数

未使用引数

今回はPHPMDでunusedcodeオプションを使い未使用変数を検知してみたが他にもオプションがあるので調べてみよう。


シェアする

  • このエントリーをはてなブックマークに追加

フォローする