IL360 2015-05-26 13:25:45

[C++] パンドラ このエントリーをはてなブックマークに追加

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

参照番号配列がリテラル。switchをループしている意味もない。
配列要素数も考慮しないで[]アクセスしてるのでよくバグが埋まっている。
ほぼ専用文字列なのに、番号で管理されててコードだけでは何が表示されるか分からない。
現実はファイル間接参照で、これはほんの一部に過ぎない。封印を解くべきじゃなかった。

// hogehoge =構造体配列
int position[20] = {9,3,1,0,2,4,/*以降略*/};
for( int i=0 ; i<20 ; i++ ) {
  switch( i ) {
    case 0:
      int a = atoi(hogehoge[m_count].foo[position[i]].str);
      処理1;
      break;
    case 1:
      文字列 = hogehoge[m_count].bar[position[i]].str;
      処理2;
      break;
    // 以下case続く
  }
}

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

コメント(1)

#1 3higo3 2015-05-26 23:03:47  

シンプルに書けなかったプログラマを問い詰めたい事例ですね

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

Twitter認証

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

[Java] 曰く、「getterアクセサを呼ぶ時にn...

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

**これは或るメンバに対するGetter...

// なんか20000行くらいあるクラス

// ...

// 600...

鑑賞する »

[JavaScript] dは良い奴だったよなぁ

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

もう初っ端からおかしい
コ...

//js用のおまじない
var use_strict = "use stric...

鑑賞する »

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »