痛いソース・オブ・ザ・イヤー

中規模システムの保守を行っている同僚Mが頭を抱えていたのを見かけた。
面倒なので無視しようと思ったが、かわいそうなので話を聞いてあげた。

彼曰く、
あるenumに定義を追加したところ、全体的に動作がおかしくなった。
そのenumは、Webサイトのページの識別子で50個ほど定義がある。
仕方ないのでそれを使用している他のソースを調べてみたところ、
enumをわざわざ配列化し、それのインデックスを用いて値を指定していた。


以下のようなソースだ。

enum Itai {
    A,
    B,
    C,
};

・
・
if( ... ) {
  return Itai.values()[0];
} else if( ... ) {
  return Itai.values()[1];
} else {
  return Itai.values()[2];
}

※本物は、Itai.values()がメソッド化されていたり（それも痛い）複雑極まりない作りになっている

で、追加した定義はenumの途中だったので、インデックスがずれてしまい、バグってしまった。
これを直すには、ずれたインデックス指定をすべて+1しなくてはいけないが、
そもそも色々と納得できない！


間違いない。
今年見た中で最も痛いソースだ！
このソースを書いた人が誰かは知らないが、痛い・オブ・ザ・イヤーを進呈しよう
同僚Mよ、キミの憤りは正当だ。ご愁傷さま！（爆笑）

コメント

コメントを投稿

このブログの人気の投稿

nginxでlocalhostとしてアクセスをサーバーに転送する方法

Windowsにnginxをインストール 下記のDLページのstable版(Windows-1.2.7)をダウンロード http://nginx.org/en/download.html zipを展開し、適当なフォルダに置く nginxの設定ファイルの書き換え nginxの中の、nginx.confをエディタで開き、下記の記述を探す。 server {     listen       80;     server_name  localhost;     ... その最後に以下の記述を追加。これは、本物のサーバーに転送するための設定。 location /aaa {     proxy_pass http://www.xxxx.com/aaa; } ※"/aaa"は、転送したいパス名。 ※proxy_passは、転送先のURL ※上記２つのどちらも最後に"/"が無いのがミソ。 ちなみに、以下の記述で、ローカルPC内のd:\xxxxディレクトリに直結できる。 location /test {     alias d://xxxx; } nginxの開始 cmd.exeで、nginxがあるディレクトリまで行き、次のコマンドを実行 start nginx ブラウザでアクセスしてみる ブラウザを開く "http://localhost/aaa/..."というURLでアクセスし、本物のサーバーと同じレスポンスが返ってくることを確認してみよう。 nginxの終了 開始と同様にcmd.exeで、次のコマンドを実行 nginx -s stop
続きを読む

Android・・・テキスト描画あれこれ, ascent(), descent()等

Androidのテキスト描画に必要なTipsをメモ。 存在に忘れがちな横方向のAlignや、意外に正解がわかりにくい縦方向のセンター合わせ等。 ascentとdescent Paint及びFontMetricsで取得可能なascentとdescent。 テキスト描画の原点となるbaselineから上方向の高さがascentで、下方向の高さがdescentとなる。 baselineを割ってdescent領域に入るのは例えば小文字の"j"等だ。 しかし理解しづらいのは、ascentが負の値であることだ。 これは、画面の座標系を思い浮かべてbaselineを(0,0)と考えるとイメージしやすい。 つまり、画面上にはみ出すascentが負、画面内に入るdescentが正、となる。 テキスト用Paintの作成 TextPaint tp = new TextPaint(); tp.setColor( Color.WHITE ); // アンチエイリアスON tp.setAntiAlias( true ); // 塗りつぶしのみ。STROKEを入れると太くなる。STROKEだけだと中抜き tp.setStyle( Style.FILL ); // テキストのフォントサイズ。Paintに直接指定する場合、Density等の考慮が必要。 tp.setTextSize( 11f * density ); 縦方向にセンター合わせ @Override protected void onDraw(Canvas canvas) { // 領域の高さ int areaHeight = getHeight(); FontMetrics fm = tp.getFontMetrics(); // フォントの高さを求める float fh = fm.descent - fm.ascent; // センター合わせにしたあと、Baselineの位置を求めるためにdescentを引く float ty = (areaHeight/2f) + (fh/2f) - fm.descent; canvas.drawText( "text", 0, ty, tp ); } 横方向に原点を右端で設定する...
続きを読む

Android:stateに応じてTextの色を変更する・・・ColorStateList

テキストの色をButton等のstateに合わせて変えることを、 xmlでやらずに自前でやるにはColorStateListを作成し、それをsetTextColorすれば良い。 以下、select状態で色が変わるTextViewを作成するサンプル TextView text = new TextView(this); text.setTextColor( new ColorStateList( new int[][] { new int[]{ android.R.attr.state_selected }, new int[]{ -android.R.attr.state_selected }, }, new int[] { Color.argb(0xff,0xff,0xff,0xff), Color.argb(0xff,0x8f,0x8f,0x8f), } ) );
続きを読む