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

    • $metka и тут же $reason умиляют :)
      Ответить
      • А $conflict, похоже, результат обработки ошибок в предыдущем коде ))
        Ответить
    • >>"Ставки по тарифам успешно сохранены!"

      хорошее, годное продакшн гогно :)
      Ответить
      • ставки сделаны, ставок больше нет))
        Ответить
        • если вы о Snatche то там ставки были отозваны "bets are OFF"

          а тот кто писал наверно не слышал не только о return false но и о операторе ИЛИ и о else и о вложеных ifах тож
          Ответить
      • и, да, это в продакшене %)
        Ответить
      • гогно... это не гогно, это lolно
        Ответить

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