1. PHP / Говнокод #16955

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    for($i = 0; $i < $db->max; $i++) {
    	if($i==1) continue;
    	.....
    }

    Почему бы и нет

    Запостил: UncaughtException, 28 Октября 2014

    Комментарии (10) RSS

    • а вдруг $db->max = 0 ?
      Ответить
    • А в чём "говнокодость"?
      0-ая итерация - выполняется
      1-ая итерация - пропускается (причин может быть множество, например, формат данных от внешнего источника)
      последующие итерации - выполняются
      Ответить
      • Магичность числа 1 только. Может "временный" костыль просто.
        Ответить
        • Плохо сформулировал, упустил главное, суть была в том, что проверялось ($db->max > 0), а 0-я итерация не имела смысла
          Ответить
      • в хороших языках есть для этого слайсинг

        в пыхе нет конечно же
        Ответить
        • Что такое слайсинг и какие языки считаются "хорошие"?
          Ответить
          • Слайсинг - взятие куска массива/списка/строки. Например в питоне:
            s = "foobar"
            t = s[3:6] # "bar"
            Ответить
            • Мило, но здесь было достаточно $i = 2
              Ответить
              • А при $i = 0 тело выполнять не надо?
                Ответить
                • Не всё перенёс, что нужно, там было условие вида if ($db->max > 0) ....
                  Хоть я и не поклонник php, но говнокода тут и вправду не так много
                  Ответить

    Добавить комментарий