ソースレビュー

雑記

オフショア先から送られてくるソースをレビューする毎日ですが、どこまでチェックすれば良いかサジ加減に悩みます。あからさまなバグやネーミングルール違反は良いのですが、変数のスコープを狭くしてくれとか、無駄なループを回すなとか、どんどん細かい点が気になって困ります。今後のことも考えて教育として指摘しているのですが、あまり教育色を出さない方が良いのかしら・・・