blueskis382 blueskis382 2012-08-01 16:41:30

[JavaScript] とっても長いswitch/case文 このエントリーをはてなブックマークに追加

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

配列変数を全く理解できていなかった。

var rnd = Math.floor(Math.random() * 128) ;
var str = "" ;
switch(rnd){
  case 0 : 
    str = "hoge" ; 
    break ;
  case 1 : 
    str = "foo" ;
    break ;
  case 2 :

  ... //えんえんとcase文が続く

  case 128 :
    str = "bar" ;
    break ;
}

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

コメント(26)

#1 teramako teramako 2012-08-02 12:28:18  

if文で書かれるよりマシな感じがw 128個の配列を用意した方が可読性は高そうではある。

#2 seagull_kamome seagull_kamome 2012-08-05 23:23:01  

この間まで仕事で面倒見てた2万行位ある.cppの中身がだいたいこんな感じだった。面倒なのでそのまま行数増やして納品した。書いたのは客先プロパーだし俺シーラネ

#3 n_shinji n_shinji 2012-08-14 13:54:21  

これは解読したくないw

#4 mikahosi mikahosi 2012-08-14 15:19:57  

これは酷い。case 128は一体何のためにあるというのか?

#5 Moon_Speed Moon_Speed 2012-08-14 15:31:01  

こwれwはwひどすぎだろw

#6 Moon_Speed Moon_Speed 2012-08-14 15:31:13  

こwれwはwひどすぎだろw

#7 ZeroP2525 ZeroP2525 2012-08-14 15:36:28  

rndは0~127だよね。なぜcase128がある

#8 ZeroP2525 ZeroP2525 2012-08-14 15:36:50  

rndは0~127だよね。なぜcase128がある

#9 ZeroP2525 ZeroP2525 2012-08-14 15:37:04  

rndは0~127だよね。なぜcase128がある

#10 ZeroP2525 ZeroP2525 2012-08-14 15:37:11  

rndは0~127だよね。なぜcase128がある

#11 fuusuihy fuusuihy 2012-08-14 16:45:05  

腹w痛wイw

#12 ga_luck ga_luck 2012-08-14 18:30:04  

www よかれと思って下手に直すと不具合大量発生しそうな予感www

#13 s025236 s025236 2012-08-14 22:15:50  

そもそも何故128個もランダムが必要だったのか?本当に必要だったのか?

#14 BlackerPanda BlackerPanda 2012-08-15 13:06:42  

配列にしろよw

#15 akiyoshi_kamide akiyoshi_kamide 2012-08-15 17:41:43  

コピペ乙ww 配列使えばしなくて済んだのに

#17 tikal tikal 2012-08-15 17:51:38  

って思うじゃん?変数配列+関数つかうより巨大switch文の方がFirefoxだと速いんですねーーーーーーーーーーーーーーーーーーーーーーーー

#18 customX_156 customX_156 2012-08-17 03:50:02  

毎回配列生成してるからじゃね!?っておもったらJavaScriptかーwボトルネックになったらパフォーマンス優先ですしね。

#21 InternalServerE InternalServerE 2012-08-17 08:50:19  

意外と配列にしろ、な意見が多くて、ちょっと驚き。こんなものをコード内に静的に配列で持たせてたら、あとで保守する人か死にますよ。

自分なら「マスタに切り出して、そっちを引け」ですが、jsだと制約多くて厳しいかな…

#22 math_neko math_neko 2012-08-17 11:31:51  

無用な case 128 まで含めて全部書いたかと思うと尊敬すらするわw

#24 m_shige1979 m_shige1979 2013-10-19 10:20:52  

まあ、よくありそうなコード

#25 uuuuwoooyea uuuuwoooyea 2013-11-26 16:02:47  

あーすごい既視感

#26 komamonodaisuki komamonodaisuki 2015-03-14 16:51:01  

よく遭遇する。本当はいけないけど、納期に追われて修正できないパターンが多い。嫌だけど129個目を追加しなきゃいけないこともしばしば。

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

Twitter認証

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

[C] すげーちゃんと動く!

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

結合試験2日前に脱退したうんkプログラマ...

return 0;

鑑賞する »

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[その他] HOW TO ABC..

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

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »