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

    +150.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    /**
    	* gets the HTTP body for the current request.
    	*
    	* @param string $soapmsg The SOAP payload
    	* @return string The HTTP body, which includes the SOAP payload
    	* @access private
    	*/
    	function getHTTPBody($soapmsg) {
    		return $soapmsg;
    	}

    нашел где то в сорцах nusoap

    Запостил: nur, 07 Апреля 2010

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

    • Ну...... это как бы метод класса, его переопределять можно
      Ответить
      • а для чего?
        Ответить
        • ну хотя да, судя по описанию метода тут ничего страшного
          Ответить
        • gets the HTTP body for the current request

          Ответ может быть
          1. Обернут во что-то
          2. Зашифрован

          Поэтому напрашивается его применение
          1. Выдернуть из обертки
          2. Декодировать
          3. Вести лог или писать в БД
          Это как бы очевидные варианты

          Плюс взгляните в changelog, там есть такие строки
          - soapclient: add getHTTPBody, getHTTPContentType, getHTTPContentTypeCharset (anticipating MIME subclass)
          Т.е. разработчики просто так для развлечения добавили этот метод
          Ответить
          • function getHTTPContentType() {
            	return 'text/xml';
            }

            Тоже можно записать в говнокод. Но с точки зрения реализации, сделано правильно, так проще разработчику оптимизировать под себя и не надо лезть в исходный код и что-то править там, лишая себя обновлений.
            Ответить
    • Я смотрю тут народ потихоньку плюсует. Это, наверное, те, что не знаком с ООП.
      Ответить
      • по задумке автора метод приватный, так что не стоит лезть со своими оверрайдами.
        Ответить
        • Автор просто оставил себе место для изменений.

          По указу Министерства Бреда, тела SOAP запросов должны помещаться в тег <lol>.
          1) Глупый автор-говнокодер делает
          - return $soapmsg;
          + return "<lol>$soapmsg</lol>";
          и радуется жизни.
          2) Истинный кодер правит в N местах, которые еще надо найти
          Ответить
          • А что вы будете делать, если то же самое министерство выпустит указ о том, что цифра 9 теперь значит число 8 и соответственно наоборот:
            public const $NINE = 9;
            public const $EIGHT = 8;
            И дальше в том же духе? ;)
            Для того и существуют стандарты, чтобы не нужно было писать вот такую вот ерунду.
            Кроме того, решать несуществующую проблему в продакшн коде приложения - это теперь считается хорошим тоном?
            Ответить
    • Обыкновенная заглушка
      Ответить
    • vanished
      Ответить

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