204504bySE 204504bySE 2017-05-07 08:25:43

[C#] 変数を初期化したり分けないと気が済まないらしい このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

今修正してるコードがこういうのがいっぱいあってつらいめう

//変数名はてきとーに捏造した

bool aaa = false;
//なんか処理する
if (なんとか)
{
//やっぱりいろいろやる
    if(あああああああ)
    {
        aaa = true;
        //いろいろ(ry
    }
}

bool bbb = false;

if (true == aaa)
{
    bbb = true;
}

return bbb;

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(1)

#1 DsYochibe DsYochibe 2017-05-20 09:57:25  

もともとcccやddd以降のフラグもあるコードで、要らなくなって削った際にGitなどで変更差分を追いやすいよう効率を犠牲にした可能性も。 あと、bool型をtrueと比較しているのはうんこだと思います。

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[その他] [PL/SQL]自動生成?いいえ、手書き...

このエントリーをはてなブックマークに追加

処理本体はあえて除外したくなるほどの衝撃...

PROCEDURE my_annonymous_arguments (
   ...

鑑賞する »

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[PHP] 値を返すかどうかはあなた次第

このエントリーをはてなブックマークに追加

某Wordpressのプラグインより。 ...

function get_item_custom( $post_id, $typ...

鑑賞する »