読者です 読者をやめる 読者になる 読者になる

ここ3ヶ月間のコードレビューでついたコメント集(PHP編)

なんとWeb業界に転職してはや5ヶ月目突入しました。
今回はサービスに配属されてからこの3ヶ月間のうちに自分が出したPRのコードへのコメントをまとめてみました。 タイトルの通り内容はPHPの書き方に絞っています。

  • nullの判定
    • $foo['bar'] === nullではなくempty($foo['bar']) または isset($foo['bar'])を使う
  • nullだったら代入させたいパターン
    • ついついif (empty($now)) {と書いちゃいがちだが、if使わなくても1 sentenseでいけるよってパターン
    • nullでない時は$nowの値がそのままとなる
>>> $now || $now = date('Ymd')
>>> $now
=> "20170202"
  • 文字列判定
    • strpos()よりpreg_match()
    • 正規表現を使うことになるが、より細かい判定基準で文字列一致を判定できる
  • 戻り値をbooleanで返す時
    • ついついif ($bar === '何か文字列') {とかで判定した結果をreturnしがちだけど 1 sentenseでいけるよってパタン2
    function foo() {
        return ($bar === '何か文字列');
    }
  • ヒアドキュメントを使う
  • __DIR__を使う
    • require __DIR__.'/../../Example.php'


一応前職でCをかじっていたいので基本中の基本では突っ込まれることはなく、
PHP特有の標準関数を使うことや、こういう書き方まで書いてOKみたいなラインについて理解が進んできました。
とはいってもまだまだなので日進月歩で学んでいきます。

ではまた何ヶ月か後にPHPRails編で。