blueskis382 2012-08-01 07: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認証)

コメント(27)

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

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

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

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

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

これは解読したくないw

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

腹w痛wイw

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

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

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

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

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

配列にしろよw

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

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

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

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

#18 customX_156 2012-08-16 18:50:02  

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

#21 InternalServerE 2012-08-16 23:50:19  

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

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

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

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

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

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

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

あーすごい既視感

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

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

#27 _rainfs 2017-06-14 17:11:40  

みんな違うんだ...こいつは悪くない 配列を知らなかっただけなんだ...

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

Twitter認証

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

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

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

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

return 0;

鑑賞する »

[Java] 連番

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

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

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[Java] is禁止令

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

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »