■セルフコードレビュー

果たしてコードレビューと言っていいかわからんが、

自分が半年前に作成して運用しているサービスのコードを自分でレビューする。

ちょうど、大きめな仕様変更のタイミングだったのと、機能追加が難しい構成になっていた。

やってみると色々見えるな…。

• スペルミス
• 関数名や変数が統一されてない
• 使ってない関数や変数がある
• 意味不明な継承
• コメントが意味不明
• ファイル構成とファイル名ルールが定まっているように見えて、無秩序。

などなど

自分は、最初から汎用性を持たせようとしてとっちらかってしまう癖がある。

整理し直したら結局３日位かかったな…。

こう言うタイミングじゃないと見直さないし、システム書き換えるのだってすごい労力がいる。

やっぱり経験だな。

Permalink | 記事への反応(0) | 17:14

ツイートシェア