"PHP" / Говнокод #10939 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
$i = 0;
foreach($_POST['union'] as $key=>$value) {
	$i++;
	if ($i == 1) {
		$fid = $key;
	}
	if ($i > 1) {
		[код поскипан]
	}
}

Ну почему не используется if { } else { }?

Запостил: domaster domaster, (Updated )

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

  • ava hevding # Ссылка на оригинал 0

    Может нужно было задать точное условие, когда переменная больше единицы.
    Ответить

    • ava bormand # Ссылка на оригинал 0 ⇈

      Выше она инициализирована нулем, и затем обязательно инкрементируется. Меньше единицы она уже никак не станет.
      Ответить

      • ava vistefan # Ссылка на оригинал 0 ⇈

        А вы уверены, что в [код поскипан] с i ничего не происходит?
        Ответить

          • ava vistefan # Ссылка на оригинал 0 ⇈

            В любом случае, даже если что-то происходит, else вместо второго условия был бы не лишним.
            Ответить

        • ava domaster # Ссылка на оригинал 0 ⇈

          В [код поскипан] находится пара селектов и один апдейт. $i больше нигде не изменяется.
          Ответить

  • ava Steve_Brown # Ссылка на оригинал 0

    Может, предполагается, что так нагляднее? Если единица, то... а если больше, то... Хотя как-то неуклюже.
    Ответить

    • ava vistefan # Ссылка на оригинал 0 ⇈

      Лишние проверки в цикле, количество которых растёт пропорционально длине входного массива, а вот наглядности сильно не поприбавилось...
      Ответить

  • ava guest # Ссылка на оригинал 0

    Блин ну как так можно?
    Ведь так лучше:
    if ($i >= 1) {
    [действия]
}
    Ответить

    • ava bormand # Ссылка на оригинал 0 ⇈

      Так смысл же другой:
      if ($i == 1) {
  [одни действия]
} else {
  [другие действия]
}
      Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓