wisdom_speaking wisdom_speaking 2019-02-20 22:54:47

[C#] 微妙な位置のif文は本当に面倒 このエントリーをはてなブックマークに追加

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

ループしまくってるなかに突如として現れたif文 なんかもう少しどうにかできなかったのか

foreach(var nest1 in array){
  foreach(var nest2 in nest1) {
     foreach(var nest3 in nest2) {
        foreach(var nest4 in nest3){
           DoWorks();
        }
     }
     if(nest2.Count() % 2 == 0) continue; 
  }
}

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

コメント(2)

#1 kuma056 kuma056 2019-02-21 23:59:33  

}の直前でcontiune;したところで、無いときと同じ動きになるのではないかと。

#2 InternalServerE InternalServerE 2019-03-10 10:19:41  

その前に、ループの多重入れ子について考察があっても良さげ。
全てが悪とは言わないまでも、実装者の未熟さや設計のイケてなさに起因する「本質的に不要なコード」であることがやたらと多い。

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

Twitter認証

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

[C#] 無駄過ぎるSwitch文

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

大学の同期のコード こんな回り道をしな...

//落第しているかの判定
bool failFlg = false;
//も...

鑑賞する »

[C] 何がしたかったのか

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

10年位前、炎上しているプロジェクトの支...

void funcX() {
    int a;
    a = func...

鑑賞する »

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

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

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

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

鑑賞する »