204504bySE 2017-05-06 23: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 2017-05-20 00:57:25  

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

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

Twitter認証

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

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

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

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

PROCEDURE my_annonymous_arguments (
   ...

鑑賞する »

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C#] 7重の波括弧が織りなすハーモニー

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

(なお実物はもっとひどかった)

Task.Delay(0).ContinueWith(async (task) ...

鑑賞する »