新人のとき、ほぼ新人のみで作成したWebアプリケーションに対して要望が投げられたため対応していたところ、目を疑うものを見つけてしまった。
びっくりだよ
foreach ($postData as $key => $item) {
$arr['email'] = $this->session->userdata('email');
//何行かの処理...
$arr = array(
'name' => $item['name'],
'email' => $arr['email'],
'text' => $item['text'],
);
//いろいろ処理...
}
いやこれ動くんかい。まじかい。自分が作ったとこじゃないから知らんかったぞ。
でも考えてみれば文末来るまでは変数宣言終わってませんもんね……そりゃまあうまくはいきますか……。
いやでも変数初期化しよ…?そもそも名前変えよ……?
社内でしか使われていないのでこっそり…こっそり修正します……。
こうみると新人の頃のコード本当にひどいなと……いっそ笑けるのでおっけいです(ダメ)。