09/10/16 01:02:35
>>935-936
設計と命名以外でダメ出してみる (×:MUST be fixed、△:SHOULD be fixed)
×Form2.ZassekiEventイベントのソースがForm2のインスタンスじゃない
→ ソースはそのイベントを定義している型かその派生にする
×Form2でZassekiEventがnullのときにNullReferenceException
→ nullチェックを入れるかnullにならないようにする
△ZassekiEventHandlerのシグネチャがEventHandlerと同じ
→ 単に無駄というだけでなく、EventHandlerには「イベントデータが無い」という意味がある
△Form1.Sekigaeメソッド内のWhere(...).Single()は無駄
→ Single、Firstなどには条件を指定できるオーバーロードがある