研究室で引き継いでいるウンコード

Leko / れこ @L_e_k_o

おいちょっとまてこれ色々と酷いぞ

Leko / れこ @L_e_k_o

全リファクタも辞さないレベル

Leko / れこ @L_e_k_o

**.txtをAjaxで取ってくる、その中身はxml。

Leko / れこ @L_e_k_o

xmlでインデント無くされるとこうも読みにくいとは。タグの名前で何となく把握出来ないから無理

Leko / れこ @L_e_k_o

これ最初素のjsで書いてたけど途中でjQueryにスイッチして適応中にタイムアップって感じがするなぁ

Leko / れこ @L_e_k_o

（まさかコーディングから修正が必要だとは・・・）

Leko / れこ @L_e_k_o

metaタグのcharset宣言がDOCTYPE宣言より手前に書かれている。text/javascriptとlang=Javascriptが混在。文字コードがShift-JIS。去年作られたものとは思えない・・・

Leko / れこ @L_e_k_o

閉じタグ対応してない・・・

Leko / れこ @L_e_k_o

titleタグもheadの外に出てるじゃねえか！どうしてこうなった！

Leko / れこ @L_e_k_o

iframeタグがまさかのキャメルケース。IframeとかiPhoneみたいだね

Leko / れこ @L_e_k_o

CSSは//でコメントぉ！！！

Leko / れこ @L_e_k_o

CSSは修正じゃなくて作りなおすほうが早そう

Leko / れこ @L_e_k_o

そもそもデフォルトで動かないからどうしようもないけど

Leko / れこ @L_e_k_o

ここでグローバル使いますか。。。

Leko / れこ @L_e_k_o

それもうっかりじゃなくて意図的に使ってるのがたちが悪い

Leko / れこ @L_e_k_o

関数の内部から別の関数を呼び出して作業を渡すのは良いけど、その間の値をグローバル変数で渡すのはやめてくれ

Leko / れこ @L_e_k_o

ここではグローバル変数を引数として渡すのか。引数無しでもグローバルならアクセスできるじゃん。使い方統一してくれ

Leko / れこ @L_e_k_o

aと出力する関数の中でbを出力する関数を読んで、その中でcを出力する関数を読んでいる無駄感

Leko / れこ @L_e_k_o

ウンコード・マニアとかに乗せても全く受けないタイプのネタじゃなくてガチのウンコード

Leko / れこ @L_e_k_o

IBの人だから仕方ないのかも知れんが、仮にもPHP教えてる人なんだしこのコードで終わりはさすがに口出ししようよ・・・

Leko / れこ @L_e_k_o

ifとelseに同じコードが多々含まれているのは何故でしょうね

Leko / れこ @L_e_k_o

キャメルケースで統一してるかと思えば突然のアンダースコア

Leko / れこ @L_e_k_o

@ken_zookie うちの学部の"プログラミングの"コース出身の子がjquery1.8.3の頃に書いたコードらしいですぞ

Leko / れこ @L_e_k_o

http://t.co/TfhjedqbNf この発想は無かった

拡大

Leko / れこ @L_e_k_o

むしろこっちをよく思いついたな