<実績のロックが解除されました>
if ( i > CONST_VALUE)
と
if ( CONST_VALUE < i)
評価すべき変数が左辺にあった方が、何を評価してるのか分かりやすいと思うんですけど
不等号の向きを統一したところで、そんなにソースコードの可読性は上がるんだろうか?
個人的には、右辺により静的な値を入れた方がいいような気がするんだけど
オブジェクトをNewするのと、hoge.Newってメソッドを呼び出している所の区別がついていなくて、単にNewって単語をgrepしてるだけという…
ソースコード読めないプロパーと、ソースコード読めない上司が、パフォーマンス改善案件でソースコード解析してるのウケる