15
7

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?

More than 5 years have passed since last update.

PHP Code SnifferのSniffを自作するにあたって知っておいてよかったこと

Last updated at Posted at 2018-09-19

前稿でphpcs-psr4-sniff:クラスがPSR-4に準拠しているかチェックするツールを作ったという話をしたが、PHP Code SnifferのSniff(プラグイン)を作るのは今回初めてだったので、そこで得た知見を共有したい。

ディレクトリ構成に決まりがある

  • PSR-0のディレクトリ構成でないとならない。(PSR-4はだめ)
  • そして、ベンダーネームスペース(下でいう「StandardName」)にruleset.xmlを置かないとならない。
  • Sniffsというディレクトリが必要。
  • Sniffsディレクトリのは以下はネームスペースで改装を作って良い。
  • ファイル名はSniff.phpで終わる必要がある。
project-name
├── StandardName
│   ├── Sniffs
│   │   └── Classes
│   │   │    └── ClassNameSniff.php
│   │   ├── MyCustomSniff.php
│   ├── ruleset.xml
└── composer.json

テストにはslevomat氏作のTestCaseクラスを使うと楽

作ったSniffの単体テストをPHPUnitで行う場合、PHP Code Sniffer本家の書き方を真似したいところだが、それをしてしまうとテストコードが複雑になってしまう。

そこで、テストにはPHP Code Sniffer界の重鎮(?)である、slevomat氏作のTestCaseクラスを継承するのがおすすめ。テストコードを宣言的に書ける。

slevomat氏作のヘルパークラスが便利

カスタムSniffはPHP_CodeSniffer\Sniffs\Sniffインターフェイスを実装するわけだが、実装に渡ってくるPHP_CodeSniffer\Files\FileオブジェクトはAPIが低レイヤーなので、解析に必要な情報を取り出すのが大変だ。

それを補うために、slevomat氏がヘルパークラスを数多く作ってくれているので、それを利用するとコードがすっきりする。

Packagistで配布するならtypephpcodesniffer-standardにする

作ったSniffをPackagistで配布する場合は、typelibraryにするのではなく、phpcodesniffer-standardにすると良い。

composer.json
{
    "name": "owner/phpcs-standard-name",
    "description": "A set of phpcs sniffs for modern php development.",
    "type": "phpcodesniffer-standard",
}

そうしておくと、Dealerdirect/phpcodesniffer-composer-installerを使っているユーザが、あなたのSniffをインストールするとき、あなたのSniffが自動的にinstalled_pathsに入るようになる。

参考になるブログ記事

参考になるカスタムSniffリポジトリ

あまり参考にならないリポジトリ

  • Symplify/CodingStandard
    • phpcsではなくEasyCondingStandardというツール向けに実装されているため参考にならない
15
7
0

Register as a new user and use Qiita more conveniently

  1. You get articles that match your needs
  2. You can efficiently read back useful information
  3. You can use dark theme
What you can do with signing up
15
7

Delete article

Deleted articles cannot be recovered.

Draft of this article would be also deleted.

Are you sure you want to delete this article?