wisdom_speaking 2019-02-20 13: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認証)

コメント(3)

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

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

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

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

#3 icbrains 2021-09-11 18:50:12  

DoWorksに何も渡さないなら、foreachの意味はあるのか?

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

Twitter認証

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

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

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

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

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

鑑賞する »

[Java] もうコメント書くなお前

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

改修案件でぶち当たったソース マジでこ...

/**
  *
  *   〇〇の値を取得する
  *
  */
pu...

鑑賞する »

[C#] 無駄過ぎるSwitch文

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

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

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

鑑賞する »