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認証

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

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C#] Abstractクラスを名乗るが実は普通...

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

なぜ、君には、Abstractという名前...

internal class AbstractClass
{
  //......

鑑賞する »

[PHP] forとifの使い方覚えました!

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

うれしくて使いたかったんだと思います。 ...

for ($i = 1; $i <= 100; $i++) {
    if ...

鑑賞する »