タグ

SCAとPHPに関するhiroomiのブックマーク (3)

  • [PHP]新卒入社してからコードレビューで指摘された内容まとめ - Qiita

    概要 タイトルの通り、入社してからコードレビューで指摘された基的なコードの書き方や、実装方法について、自分の戒めとしてまとめたいと思います。 なお、言語はPHPを使用しています。 コードの書き方に関する内容 インデントは揃える 基中の基ですが、インデントは揃えるようにしましょう。 また、エディタの設定によっては自動フォーマットされる場合もあるため、チームで開発する際は改行やインデントのルールを統一しましょう。 Atomの自動フォーマットの設定に関しては下記のような記事を書いています。 【Atom】エディタの自動フォーマットの設定を統一しよう - Qiita Docコメントは必ず書く 新たなメソッドの作成や、返すパラメータの変更などの作業をした際は、必ずDocコメントの作成・修正を行いましょう。 /** * メソッドの説明 * * @param int $id * @return boo

    [PHP]新卒入社してからコードレビューで指摘された内容まとめ - Qiita
  • Hudson+phpcpdで継続的に重複コードをチェックする

    Hudson等を使った継続的インテグレーションでは、テストの自動実行の他にも、ドキュメントの自動作成、コーディング規約の自動チェック、重複コードのチェック(DRY原則のチェック)等を行うことができるし、実行するべきである。 今回は、PHP+Hudsonの環境でコードの重複を継続的にチェックできるようにしてみた。 phpcpdPHPでコードの重複を検査するには、phpcpdというツールを使うのが定番である。 phpcpdはpearコマンドでインストール可能だ。 なお、phpcpdを利用するためには、pearが1.9.1以上である必要がある。 インストール手順 pear upgrade pear pear channel-discover pear.phpunit.de pear channel-discover components.ez.no pear install phpunit/ph

    Hudson+phpcpdで継続的に重複コードをチェックする
  • SonarQubeでPHPのコードを静的解析する - Qiita

    2019-02-10 追記: SonarQubeではなくPHPStanで同様のことをする記事も書きました。いま個人開発ならこちらがを選ぶことも多そうです。 LaravelプロジェクトのコードをCircleCI上のPHPStanで静的解析してreviewdogにコメントしてもらう SonarQubeはオープンソースの静的解析ツール 一言で言うと「コードのダメな部分を教えてくれるやつ」です。LGPLライセンス。 下のようなこんな感じでコード品質の推移が一覧できます。また、コードごとに何が悪いのか、どう直すべきなのかを見ることができます。(画像はDrupalの解析結果) オンプレ環境で使えるのがメリットで、GitHubで個人のプロジェクトを解析するなら、各種静的解析SaaSを使うのが簡単なのでおすすめです。 PHP向け静的解析SaaSの主観的比較 (Scrutinizer, SensioLabs

    SonarQubeでPHPのコードを静的解析する - Qiita
  • 1