糞ソース作者へのお願い - 週記くらい(BTS開発記)

php仕事で、別のところの人が作ったシステムに手を入れることは多いんだけど、酷いソースは本当に酷い。加えて仕様が酷い場合は相乗効果で、もう手をつけられない状態になる。修正する人が病気になるくらい破壊力がある。 良くないソースの特徴と対策(糞ソース作者へのお願い)を書いてみた。複数ある検索条件などフォームから受け取って、個々の条件毎別々にセッションスコープに保存している検索条件は、連想配列とかクラスのインスタンスに入れて(纏めた上で)セッションスコープに入れるようにして欲しい。セッションスコープはある意味、グローバルスコープになってしまうんだから、なるべく減らすべき。ifで条件分岐しているけど、if節とelse節でやってることがほとんど同じ同じ部分は、ifの外に出して欲しい。おんなじ事書いてて変だと思わないのか。メソッド名が変外から見て、何をするメソッドかがわかるようにして欲しい。変数名が変