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

    +163

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    $metka=0;
$reason = $form->getValue("reason");
if ($activedate < $datenow)
	{
		$metka++;
	}

if (trim($reason) == "")
	{
		$metka++;
	}
if ($conflict>0)
	{
		$metka++;
	}

if ($metka == 0)
	{
		for ($i=0;$i<count($srid);$i++)
		{
			$this->changeRates($srid[$i],$rate[$i],$datenow,$activedate,$reason);
		}
		pageAlert::nextPageAlert("Ставки по тарифам успешно сохранены!", PAGE_ALERT_SUCCESS);
	}

    Вот как кодят программисты, не слышавшие про исключения и return false при ошибках

    Запостил: 1Ad_Astra1, 18 Мая 2010

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

    • ava Lure Of Chaos 18.05.2010 15:50 # +1

      $metka и тут же $reason умиляют :)
      Ответить

      • ava HyperGeek 18.05.2010 16:41 # 0

        А $conflict, похоже, результат обработки ошибок в предыдущем коде ))
        Ответить

    • ava guest 18.05.2010 16:03 # +4

      >>"Ставки по тарифам успешно сохранены!"

      хорошее, годное продакшн гогно :)
      Ответить

      • ava xXx_totalwar 18.05.2010 16:04 # +1

        ставки сделаны, ставок больше нет))
        Ответить

        • ava 3.14159265 18.05.2010 16:33 # 0

          если вы о Snatche то там ставки были отозваны "bets are OFF"

          а тот кто писал наверно не слышал не только о return false но и о операторе ИЛИ и о else и о вложеных ifах тож
          Ответить

      • ava 1Ad_Astra1 18.05.2010 16:06 # 0

        и, да, это в продакшене %)
        Ответить

      • ava Lure Of Chaos 18.05.2010 16:33 # 0

        гогно... это не гогно, это lolно
        Ответить

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