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

    +55

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

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

    Запостил: domaster, 13 Июня 2012

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

    • ava hevding 13.06.2012 16:38 # +1

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

      • ava bormand 13.06.2012 16:40 # +1

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

        • ava hevding 13.06.2012 19:02 # +1

          Да, вы правы, не обратил внимания.
          Ответить

        • ava vistefan 14.06.2012 08:34 # +1

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

          • ava bormand 14.06.2012 09:01 # 0

            Теперь нет, вы вселили в меня тень сомнения.
            Ответить

            • ava vistefan 14.06.2012 09:20 # +1

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

          • ava domaster 14.06.2012 09:16 # 0

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

            • ava vistefan 14.06.2012 09:42 # +2

              Ок. Было бы разумно упомянуть об этом сразу.
              Ответить

    • ava Steve_Brown 13.06.2012 18:13 # +1

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

      • ava vistefan 14.06.2012 08:33 # +1

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

    • ava Lure Of Chaos 13.06.2012 23:24 # +4 

      if ($i > 1) {
  $i--;
}
      Ответить

    • ava guest 16.06.2012 21:26 # 0

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

      • ava bormand 17.06.2012 06:38 # 0

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

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