今日の罠

↓のようなコードがたくさんあって、

int result = 0;

result = <処理1>;

if( result != 0 ) {
    return result; 
} // end if

result = <処理2>;

if( result != 0 ) {
    return result; 
} // end if

result = <処理3>;

if( result != 0 ) {
    return result; 
} // end if

その中で以下を発見。

int result = <処理>;

if( result != 0 ) {
    // TODO:エラー処理
} // end if

return result;

お、TODOが残ってるなー、修正せねば。ログ出力とかは<処理>の中でされてるから、これでいいはず。

int result = 処理;

if( result != 0 ) {
    return result;
} // end if

return result;

って、あれ?これ、そもそもTODOじゃないじゃないかー。うれしげに修正してコミットしてしまった・・・。orz.