{ "componentChunkName": "component---src-templates-blog-post-jsx", "path": "/post/create-git-hook-to-develop-modern-php/", "result": {"data":{"site":{"siteMetadata":{"title":"WEB EGG","author":"Leko - CTO at Yuimedi"}},"markdownRemark":{"id":"7e12e82b-c398-53a0-b1c2-fb186b54eed9","excerpt":"こんにちは。 久々の更新です。 最近Schemaというフレームワークを作っており、 PHPでの開発について色々と新たな知識を得る機会がたくさんあります。 そこでまずは開発環境の構築編として、最低限のお作法を担保する Gitのhook を作成してみました。 目的 –…","html":"

こんにちは。
\n久々の更新です。

\n

最近Schemaというフレームワークを作っており、
\nPHPでの開発について色々と新たな知識を得る機会がたくさんあります。

\n

そこでまずは開発環境の構築編として、最低限のお作法を担保する Gitのhook を作成してみました。

\n\n

目的 – 5箇条

\n

それくらいきちんとやれよ という話に尽きるのですが、
\n自動で確認してくれるに越したことはありません。
\nプログラマは得てして機械的な確認が苦手な生き物です。楽することに全力であるべきです。

\n

（マジレスをすると、人的要素を排除することで、うっかりや人為的なミスが無くなるので、より確実にコードの品質を担保できます。
\nただし、このチェックが通れば完璧というわけではないので、加えて動作確認やコードレビュー等の人的チェックも必要かと思います）。

\n

と能書きはこのくらいにしておいて、具体的にやりたいことは以下の通りです。

\n\n

上記5点の確認をコミットする前に行い、どれか１つでもコケた場合は コミットできない という縛りを加えようと思います。
\nさすがに業務や複数人でやるプロジェクトでこれは相当キツイので、
\nいくらか縛りを外せば、ちょうどいい感じの制約になると思います。

\n

それぞれの項目について説明していきます。

\n

1. PHPの構文チェック

\n

これは5番目のテストとかぶるのですが、
\nたまに 自分が書いたコードを一度も実行・確認せずにコミットしやがる輩 が居て、pullした途端に動かなくなるということがあります。

\n

そんなゴミクズ野郎にならないように、最低限構文エラー起こしてるコードはコミットしないためにチェックします。

\n

2. PSRのお作法に反したコードが無いか

\n

PHPのコーディング標準として、PSRというものが挙げられています。

\n\n

コミットの制約に加えるか否かはさておき、読んでおいて損はない規約だと思います。

\n

あと3と4もあるのですが、とりあえずこれだけ知っておけば問題ないので割愛します。 これら一般的なコーディング規則に則ったコードになっているかをチェックしていきます。

\n

3. 使用してない変数、あまりに長い命名、複雑なループ等を実装してないか

\n

書いてる途中で気付けよ というお話ではありますが、
\nもしこのようなコードをうっかり書いてしまい、それに気づかずコミットしてしまわないようにコードの品質チェックを行います。

\n

4. composer.jsonはvalidになっているか

\n

PHPにはcomposerという依存関係の解決ツールがあります。
\nRubyでいうとこのBundlerみたいなものです。

\n

ナウいPHP開発をしているならcomposerの存在はほぼマストになっていると思います。
\n昨今では、だいたいのライブラリがcomposer経由でインストールできるようになっています。

\n

私がよくやらかすケツカンマ問題とシングルクオートつけちゃう問題のチェックをしてもらいたいがためのチェックです。

\n

「ケツカンマ」とは、配列の末尾要素のあとにカンマをつけたりオブジェクトの末尾にカンマをつけてしまうアレです。
\njsではIE9以降ならケツカンマがきちんとパースされてしまうので、つい忘れがちですがJSONの書式としては不正です。
\nシングルクオートは言わずもがな、JSONでは不正な形式です。

\n
\n

\n 「ケツカンマ問題」\n

— Sotaro KARASAWA© (@sotarok) \n 2012, 5月 2\n
\n

5. テストは通っているか

\n

「テストを書かずに実装だけドバドバ貯めていって、あとからテストを埋めてく」のはいささか危険です。
\n私の経験談では、趣味でのコーディングではほぼ確実に途中で心が折れます。

\n

最近上司の教えで、コードとテストコードを併せてレビューして頂いているのですが、
\n実装と実装した分テストを書くのであれば、記述量がそこまで多くないのと、実装した直後なので割と仕様や考慮漏れに気づくことが出来ます。
\nこれは良い習慣だ と思ったので強制的に癖をつけようということで制約に追加しました。

\n

あとはREADMEの日本語チェックとかも入れたかったのですが、
\n色々やりすぎるとコミットできなくなって飽きる
\nと思ったので制約はこれくらいにしておきます。

\n

上記の制約なら、普通にコード書いていれば問題ないはずの制約です。
\n自分自身の「当たり前」のクオリティを上げるために矯正ギプスをはめます。

\n

学習 – Git hookについて

\n

Gitプロジェクトであれば、./.git/hooksの中に色々とファイルが入っています。

\n

bareリポジトリでGit pushされたら云々する、とかにも使うやつです。

\n

今回はコミット前にチェックをしたいので、pre-commitを使用します。
\nhooksディレクトリの中にpre-commit.sampleというファイルがあるので、.sampleを除去したファイルを複製します。

\n
ls .git/hooks\ncp .git/hooks/pre-commit.sample .git/hooks/pre-commit
\n

エディタはなんでもいいので、複製したpre-commitを開いて下さい。

\n
#!/bin/sh\n#\n# An example hook script to verify what is about to be committed.\n# Called by \"git commit\" with no arguments.  The hook should\n# exit with non-zero status after issuing an appropriate message if\n# it wants to stop the commit.\n#\n# To enable this hook, rename this file to \"pre-commit\".\n\nif git rev-parse --verify HEAD >/dev/null 2>&1\nthen\n    against=HEAD\nelse\n    # Initial commit: diff against an empty tree object\n    against=4b825dc642cb6eb9a060e54bf8d69288fbee4904\nfi\n\n# If you want to allow non-ASCII filenames set this variable to true.\nallownonascii=$(git config --bool hooks.allownonascii)\n\n# Redirect output to stderr.\nexec 1>&2\n\n# --- 長いので割愛 ---
\n

という感じのファイルになっていると思います。

\n

このファイルを実行した時の終了コードが1なら、コミットが失敗するようにできています。

\n

すでに先駆者の方がいらっしゃったので、そちらをパクりつつ、拡張とカスタマイズを加えます。

\n
\n

gitのpre-commit hookを使って、綺麗なPHPファイルしかコミットできないようにする
\n— http://blog.manaten.net/entry/645

\n
\n

先ほどの長いので割愛以下に、このコードを貼り付けて下さい。

\n
IS_ERROR=0\n# コミットされるファイルのうち、.phpで終わるもの\nfor FILE in `git status -uno --short | grep -E '^[AUM].*.php$'| cut -c3-`; do\n    # シンタックスのチェック\n    if php -l $FILE; then\n        # PHPMDで未使用変数などのチェック\n        if ! ./vendor/bin/phpmd $FILE text unusedcode,codesize,naming; then\n            IS_ERROR=1\n        fi\n        # PSR準拠なコードかチェック\n        if ! ./vendor/bin/php-cs-fixer fix $FILE --dry-run -v --diff; then\n            IS_ERROR=1\n        fi\n    else\n        IS_ERROR=1\n    fi\ndone\n\n# composer.jsonのバリデーション\nif ! ./composer.phar validate; then\n    IS_ERROR=1\nfi\n\n# テストを実行\nif ! ./vendor/bin/phpunit -c tests/phpunit.xml; then\n    echo \"　　　　 ,、,,,、,,, \"\n    echo \"　　　 _,,;' '\\\" '' ;;,, \"\n    echo \"　　（rヽ,;''\\\"\\\"''゛゛;,ﾉｒ）　　　　 \"\n    echo \"　　 ,; i ___　、___iヽ゛;,　　テスト書いてないとかお前それ@t_wadaの前でも同じ事言えんの？\"\n    echo \"　 ,;'''|ヽ・〉〈・ノ |ﾞ ';, \"\n    echo \"　 ,;''\\\"|　 　▼　　 |ﾞ゛';, \"\n    echo \"　 ,;''　ヽ ＿人＿  /　,;'_ \"\n    echo \" ／ｼ、    ヽ  ⌒⌒  /　 ﾘ　＼ \"\n    echo \"|　　 \\\"ｒ,,｀\\\"'''ﾞ´　　,,ﾐ| \"\n    echo \"|　　 　 ﾘ、　　　　,ﾘ　　 | \"\n    echo \"|　　i 　゛ｒ、ﾉ,,ｒ\\\" i _ | \"\n    echo \"|　　｀ー――-----------┴ ⌒´ ） \"\n    echo \"（ヽ  _____________ ,, ＿´） \"\n    echo \" （_⌒_______________ ,, ィ \"\n    echo \"     T                 |\"\n    echo \"     |                 |\"\n\n    IS_ERROR=1\nfi\n\nexit $IS_ERROR\n</code>
\n

テストを書かない人にはサバンナからお導きがあるようなので、そちらを使わせていただきました。
\nカバレッジの話になると閾値の調整が難しいのでカバレッジは見てません。判断基準は、テストに通るか否かです。テストがない場合も通ってしまいますが、そこは人力チェックで問題ない範囲でしょう。

\n

参考元の記事ではphp-cs-fixerは勝手に修正を加えるようになっていたのですが、
\n勝手に直されるのは性分じゃないので、自動的な修正はせず怒ってもらって、自分で修正するようにします。

\n

このpre-commitの実行にはいくつかコマンドが必要なので、
\ncomposer.jsonにこれらを追加しておいて下さい。

\n
\"require-dev\": {\n  \"phpmd/phpmd\": \"@stable\",\n  \"fabpot/php-cs-fixer\": \"@stable\",\n  \"phpunit/phpunit\": \"4.*\"\n}
\n

プロジェクト作成コマンドまとめ

\n

これらを考慮し、新規プロジェクトを作成するときの最低限のコマンドをまとめます。

\n
mkdir SOME_PROJECT\ncd SOME_PROJECT\ngit init\n# --- composer\ncurl -sS https://getcomposer.org/installer | php\n./composer.phar init     # 必要事項を入力\nvim composer.json # phpmd,php-cs-fixer,phpunitの指定を追加\n./composer.phar install\necho vendor >> .gitignore\n# --- PHPUnit\ngit clone  tests\nrm -rf tests/.git\n# --- git hook\ncp .git/hooks/pre-commit.sample .git/hooks/pre-commit\nvim .git/hooks/pre-commit # 先ほどのhookを貼り付け\n# --- initial commit\ntouch README.md\ngit add .\ngit commit -m \"initial commit\"
\n

PHPUnitの設定は結構面倒なので、gistにスケルトンを作っておき、それをtestsというディレクトリ名でcloneしてます。
\nほかは特に変わったところはないと思います。

\n

まとめ

\n

いかがでしたでしょうか。

\n

PHPのプロジェクト作成と一口にとっても、色々と改良の余地がありそうです。 こんな感じで、少しでもウンコード発生率を下げましょう！ ！ ！

\n

言い訳

\n

冒頭に紹介したschemaはまだこの記事を書く前に作成されたプロジェクトなので、PSR0準拠できていません。
\nぼちぼちと修正中です。

\n
\n

PHPMD – PHP Mess Detector
\n— http://phpmd.org/

\n
\n\n
\n

The PHP Coding Standards Fixer for PSR-1 and PSR-2
\n— http://cs.sensiolabs.org/

\n
","timeToRead":11,"frontmatter":{"title":"PHPで開発するためのgit hookを作った","tags":["Git","PHP"],"date":"October 19, 2014","featuredImage":{"childImageSharp":{"fluid":{"tracedSVG":"data:image/svg+xml,%3csvg%20xmlns='http://www.w3.org/2000/svg'%20width='400'%20height='135'%20viewBox='0%200%20400%20135'%20preserveAspectRatio='none'%3e%3cpath%20d='M332%2013l-3%203-1%201c-1-1-1%200-1%201%200%202%200%202-1%201-1-3-4%201-9%2010l-3%206%202%204c3%204%203%205-1%202l-2-2v4c1%206%201%206-1%202-2-2-2-2-4%205-3%206-3%207-1%209v3l-1-2h-1c-1%202%200%204%201%203l1%201c-2%202%201%205%204%206l2%202%201%201%202-9%203-10-1-2c-2%200-2-1-2-2l2-2c2%200%203-2%205-7l4-8c1-2%202-2-2%208-3%207-3%207-2%208%203%202%202%204-1%204-1-1-2%200-3%209-2%2012-3%2032-1%2033l1-2%206-3%202-2c-4%201-4-1%201-2%202%200%203%200%202-1-2%200-1-1%201-2%203-2%204-3%204-6l-1-3v-1c2-1%202-2%201-5-2-4-2-4-4-2l-4%201%203-3c6-3%206-2%207-13l2-4c0-2%200-2-2-1l-3%201%201-2c3-1%206-8%204-10v-9c2-2%201-6-2-10l-2-4-3%201m-126%204l-1%204v2c-3%200-2%203%202%207s4%204%202%204l-5%201c-1%202-1%202%202%203l5%202%206%203c5%203%206%208%201%208-2%200-1%202%201%206%202%202%202%204%202%205l-2-1c-1-3-7-5-9-5l-4-2c-6-4-18-1-23%206l-2%203v-4c0-3%200-4-2-4-3%200-3-4%201-7l4-6%205-6%202-5c-1-1%200-2%204-4l4-4-8%204c-8%205-9%205-9%202%200-2-1-3-2-1h-4l3-2%203-2-3%201h-5c-1%200-3%201-4%204l-3%203-2-4c-4-6-5-2-2%207l2%208v2c2%201%202%202%202%205%200%202-1%202-2%201-3%200-3%200-3%205l1%205c1%200%202%203%202%2012%200%2013%200%2013%2011%2021l5%204v1c1%201%200%202-5%201h-6l3%202%203%201-1-1%208%201%207%202h-19l7%201c16%201%2021%203%2021%205l-1%204-1%201v-3c0-3-1-4-4-1h-1c0-3-2-2-3%200l-2%202v-2c-1-5-8-2-8%204%200%202%200%202%203%202h5c1-1%202-2%203-1%200%201%201%201%202-1l2-2v2c0%203%203%203%207-1l3-2v2c0%203%200%203%203%203l9-1c5%201%209%200%208-1v-2l-1-1%201-1%203-1-3-1c-3%200-5%202-5%204s0%202-2%201c-2-3-3-4%200-4%201%201%202-1%200-2s-5%203-3%204v2c-5-4-2-8%2012-11%209-3%209-3%207-4v-1l1-1h-2v-2c2-2%202-2%200-2s-2%200-1-3l3-10c0-4%202-8%203-11%202-4%202-11%200-14v-2l2-9-4%201c-3%201-3%201-5-3l-1-8c0-6-6-19-9-19l-1%203%201%204-5-4c-6-3-7-4-9-2m1%203l4%205c6%206%206%207%200%204-6-4-6-4-2%201%203%203%205%204%206%203%202%200%202%200%201%201l-5%201c-5%201-8%202-5%202l7%202%206%203%203%204c1%203%202%203%205%202%203%200%204-1%202-4l-2-9-2-11c-3-5-5-6-3-1l1%204c-1%200-4-1-3-2%200-1-6-5-8-5l3%203c5%205%203%204-3%200l-5-3m-22%2013l-17%2011%201%203v3c-1%202%200%203%204%203%202%200%203%200%203-3l4-5%209-13-4%201m-63%2034c0%202%200%202-1%200-1-1-1-1-1%201-2%204-7%2011-10%2012l-1%201-1%202-1%208%201-3c0-3%202-3%202-1%202%205%204%205%2017%203%208-1%2012-3%2011-3l1-1c1%201%204-3%204-5l-2-2v-1l2-1-1-1h-2l2-1%201-1h-1l-2-1c-1-1%200-1%201-1s2%200%201-2h-1c-1%202-1%201-3-1%200-2-1-2-1-1l-4%201h-4l5%201%205%201h-5l-8%201h-3l1-4c0-5-1-5-2-1m161%206c-3%202-3%202-2%204v1l-1%202v2c-2%200-1%203%201%205h-1c-2%200-2%200-1%201v4c0%203%200%203%203%205%203%201%203%201%201%201-4-1-5%201-1%202l9%202-2-1v-2c2-1%202-2%202-5v-2l1%2011v8l5-6v-2c2-1%202-1%201-2l1-2c3-2%203-3%203-6-1-4-2-5-3-4l-1-3c1-2-3-10-5-11l-2-1c-3-3-6-3-8-1m-78%202l-4%202-16%202c-3%201-3%201-1%203%201%201%202%203%200%203h-1l2%202%207%206%205%204%202-3%203-2h-3c-4%201-4%200-1-2l1-1v-3c0-3%200-3-3-2h-5c-2-1-2-1%203-1%204%200%204%200%202-1h-6c-2%200-2%200-1-1%201-2%202-2%206-1l5-1%201-1v7c0%204%200%204%202%202s3%200%200%202c-1%202-1%202%201%202h3l-2%201c-2%200-2%200%200%202%201%201%201%201-1%201-1-1-3%200-5%202s-2%202%200%203c4%202%2011-1%2014-5v-3l-1-1%201-1%204-2c0-2%202-3%203-3s1-1-1-3-2-2-6%200c-2%201-2%201%200%201%202-1%203%200%200%201-2%201-4%201-7-1-4-3-4-5%200-3h8v-1l-4-3c-3-3-3-3-5-1'%20fill='%23d3d3d3'%20fill-rule='evenodd'/%3e%3c/svg%3e","aspectRatio":2.9557522123893807,"src":"/static/5690df49e4df5f831089a6cf2c0b7efb/32d53/featured-image.png","srcSet":"/static/5690df49e4df5f831089a6cf2c0b7efb/1ec58/featured-image.png 334w,\n/static/5690df49e4df5f831089a6cf2c0b7efb/32d53/featured-image.png 614w","srcWebp":"/static/5690df49e4df5f831089a6cf2c0b7efb/9b99b/featured-image.webp","srcSetWebp":"/static/5690df49e4df5f831089a6cf2c0b7efb/cd98f/featured-image.webp 334w,\n/static/5690df49e4df5f831089a6cf2c0b7efb/9b99b/featured-image.webp 614w","sizes":"(max-width: 614px) 100vw, 614px"}}}}}},"pageContext":{"slug":"/create-git-hook-to-develop-modern-php/","previous":{"fields":{"slug":"/useful-libraries-in-develop-taskloud/"},"frontmatter":{"title":"Taskloudを制作する際に使用したライブラリまとめ","tags":["Backbone.js","Firebase","JavaScript","Marionette"]}},"next":{"fields":{"slug":"/how-to-implement-fulltext-search-with-fuelphp/"},"frontmatter":{"title":"FuelPHPでInnoDBの全文検索を利用してみる","tags":["PHP","FuelPHP","MySQL"]}}}}, "staticQueryHashes": ["2585454260","2954598359"]}