[PHP] いちいちfor文でまわすな-ウンコード・マニア

glassonion1 2012-11-26 02:47:30

[PHP] いちいちfor文でまわすな

for文の中にあるif文には疑いの目向けることが多いですがここまでわかりやすいのは久しぶりに見ました。

for ($i = 0; $i < 2; $i++) {
  if ($i === 0) {
   // 処理...
  }
  // その他処理...
}

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

#1 math_neko 2012-11-26 02:48:28

for 文が泣いている

#2 InternalServerE 2012-11-26 16:14:08

お、ループ2回しかしないあたり、まさにウンコード職人の仕業ですな。
いい仕事してますねえ

#3 okadabasso 2012-12-01 10:43:18

ここまで直接的なのではなく、$i ではなくif($foo[$i]["bar"] == "buz")...はよくみる

#4 builtlast 2012-12-09 14:13:41

0 との比較に === 使ってるくせになww

#5 jkr_2255 2013-12-25 11:58:51

2回ならともかく、初期化にも実データが必要なために、foreachの1回めで初期化、というのはやったことがあります。

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

Twitter認証

« PHP一覧に戻る

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

何でもやってくれる関数って、便利ですよね...

class Processor
{
	public bool process...

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

create table item_master (
 A varchar(2...