ここ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でいけるよってパタン２

    function foo() {
        return ($bar === '何か文字列');
    }

• ヒアドキュメントを使う
• __DIR__を使う
  • require __DIR__.'/../../Example.php'

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

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