3. コメントを書けというのは簡単だけど、これぞというコメントを書くのは難しいなあ。 5. システムハンガリアンはよくないと思う。$bIsMember は特に読みづらい。 7. getStatu...
3. コメントを書けというのは簡単だけど、これぞというコメントを書くのは難しいなあ。5. システムハンガリアンはよくないと思う。$bIsMember は特に読みづらい。
7. getStatus()の戻値がboolというのもよくわからんね。これは例が悪いのではないか。強いていえば isTwo() になるのかなあ。$bReturn を作って途中で代入するのは制御構造が見づらくなると思う。条件が多くなった場合は if (...) { return true; } の連続とかにした方がよさそうだし、可能であれば return するのは true; true; true; false; のように固めた方がよさそう。
11. if文は英語として読み下せるようにすべき。=== true が入ると意味がとれなくなる。
14. xdebugを入れて var_dump() を使えば？
15. 定数はトップレベルの名前空間なのでディレクトリ名とかに使うのはいかがなものか。
19. いくらなんでも例が悪すぎるのではないか。
より良いPHPerになるための20Tips - はてなブックマーク - 新着エントリー
Atom Feed