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

    +3

    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
    /**
    	* Gibt ein Array mit den Fehlermeldungen zurück
    	*
    	* @todo Test
    	* @param void
    	* @return bool|array True = Keine Fehler, Array = Fehler-Array
    	* @access public
    	* @since version 4.0
    	*/
    public function get_meldung()
    	{
    		if($fehler_meldung == null)
    		{
    			return true;
    		}
    		else
    		{
    			for($errori=0;$fehleri<count($this->fehler_meldung);$fehleri++)
    			{
    				echo $this->fehler_meldung[$fehleri]."<br>\n";
    			}	
    			return false;
    		}
    	}

    Ещё немного немецкого говнокода.
    По описанию ЭТО возвращает true когда ошибок нет или array с ошибками.

    Запостил: kropotor, 09 Июня 2016

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

    • Всё пучком, ошибок нет!
      Ответить
    • и тут мельдоний
      Ответить
    • Динамическая типизация, что ты хотел-то.
      Ответить
      • Я это же на любом другом языке напишу, кроме J

        Тут phpDoc не соответствует действительности
        Ответить
        • Напиши на языке со статической типизацией.
          Ответить
          • Функцию, которая возвращает одно, а в жабадоке указано другое? Я не пойму при чём тут динамическая типизация.
            Ответить
            • @return bool|array True = Keine Fehler, Array = Fehler-Array
              Ответить
              • Прикол в том, что функция Fehler-Array не возвращает никогда. Вместо него она возвращает false, а тот самый загадочный Fehler-Array... выводит в стандартный вывод, т. е. прямо дяденьке на шляпу юзеру в браузер.
                Ответить
                • Fehler-Array всего лишь массив ошибок. Предположим возвращал бы.

                  Вообще конечно говно. Во-первых, срет просто в текущую позицию (хотя может быть оно вызывается вовремя), во-вторых, это часть класса? Где оно хранит переменные?
                  Ответить
                  • Код ещё и нерабочий. Тут явно две опечатки: fehler_meldung вместо $this->fehler_meldung в 12-й строке и $errori вместо $fehleri в 18-й строке.

                    А так $fehleri — это локальная переменная (за пределами метода её не видно). В JS хотя бы есть слово var для объявления переменных, а в PHP переменная автоматически объявляется при первом использовании.

                    А $this->fehler_meldung — это поле класса.
                    Ответить
                    • функция всегда возвращает true, так как переменная $fehler_meldung не определена и условие $fehler_meldung == null верно.
                      Ответить
                  • там класс на 5 тысяч строк, не буду же я всю кучу тут вываливать.
                    Ответить
        • >> Я это же на любом другом языке напишу

          "Программист на PHP на любом языке пишет как на PHP"
          Ответить
      • Питузация
        Ответить

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