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

    • ava kropotor 09.06.2016 14:25 # 0

      Всё пучком, ошибок нет!
      Ответить

    • ava Her 09.06.2016 14:59 # +2

      и тут мельдоний
      Ответить

    • ava 3_14dar 09.06.2016 19:45 # 0

      Динамическая типизация, что ты хотел-то.
      Ответить

      • ava 3_dar 09.06.2016 21:13 # +1

        Я это же на любом другом языке напишу, кроме J

        Тут phpDoc не соответствует действительности
        Ответить

        • ava 3_14dar 09.06.2016 23:08 # 0

          Напиши на языке со статической типизацией.
          Ответить

          • ava 3_dar 09.06.2016 23:36 # +1

            Функцию, которая возвращает одно, а в жабадоке указано другое? Я не пойму при чём тут динамическая типизация.
            Ответить

            • ava 3_14dar 10.06.2016 00:28 # +1

              @return bool|array True = Keine Fehler, Array = Fehler-Array
              Ответить

              • ava inkanus-gray 10.06.2016 00:38 # 0

                Прикол в том, что функция Fehler-Array не возвращает никогда. Вместо него она возвращает false, а тот самый загадочный Fehler-Array... выводит в стандартный вывод, т. е. прямо дяденьке на шляпу юзеру в браузер.
                Ответить

                • ava 3_14dar 10.06.2016 01:27 # 0

                  Fehler-Array всего лишь массив ошибок. Предположим возвращал бы.

                  Вообще конечно говно. Во-первых, срет просто в текущую позицию (хотя может быть оно вызывается вовремя), во-вторых, это часть класса? Где оно хранит переменные?
                  Ответить

                  • ava inkanus-gray 10.06.2016 12:08 # 0

                    Код ещё и нерабочий. Тут явно две опечатки: fehler_meldung вместо $this->fehler_meldung в 12-й строке и $errori вместо $fehleri в 18-й строке.

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

                    А $this->fehler_meldung — это поле класса.
                    Ответить

                    • ava kropotor 10.06.2016 12:49 # +1

                      функция всегда возвращает true, так как переменная $fehler_meldung не определена и условие $fehler_meldung == null верно.
                      Ответить

                  • ava kropotor 10.06.2016 12:51 # 0

                    там класс на 5 тысяч строк, не буду же я всю кучу тут вываливать.
                    Ответить

        • ava kegdan 10.06.2016 05:30 # 0

          >> Я это же на любом другом языке напишу

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

      • ava kegdan 10.06.2016 05:29 # 0

        Питузация
        Ответить

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