仕様書もだいたい終わったので、今日は新人のソースコードレヴューをする。
初心者が多いためか、なかなかユニークだ。
Nothing との比較に "=" 演算子を使ってしまっているな。
まあ、これは VB の方言だからしかたないか。正解は "Is" 演算子。
Try ~ Catch の Catch の中で、Throw したあとにも処理がある。
これは実行されないよなぁ。
あとは戻り値のあるメソッドで Return がないとか、コメントが少ないとか、ありがちと言えばありがちな指摘だけど。
レヴュー結果がまとまってから、それをみんなに説明した。
レヴューだけで 4 時間くらいかかっちゃったなぁ。
残業。
親会社の担当者からまた追加の依頼メールが来た。
もう追加はないと言っていたはずなんだけどなぁ…。