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

    • ava murz 07.04.2010 12:12 # +1

      Ну...... это как бы метод класса, его переопределять можно
      Ответить

      • ava striker 07.04.2010 13:54 # 0

        а для чего?
        Ответить

        • ava striker 07.04.2010 13:56 # 0

          ну хотя да, судя по описанию метода тут ничего страшного
          Ответить

        • ava murz 07.04.2010 14:00 # 0

          gets the HTTP body for the current request

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

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

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

          • ava murz 07.04.2010 14:05 # +1 

            function getHTTPContentType() {
	return 'text/xml';
}

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

    • ava murz 07.04.2010 13:53 # +0.8

      Я смотрю тут народ потихоньку плюсует. Это, наверное, те, что не знаком с ООП.
      Ответить

      • ava nur 07.04.2010 14:48 # −0.2

        по задумке автора метод приватный, так что не стоит лезть со своими оверрайдами.
        Ответить

        • ava guest 07.04.2010 15:24 # +2.4

          Автор просто оставил себе место для изменений.

          По указу Министерства Бреда, тела SOAP запросов должны помещаться в тег <lol>.
          1) Глупый автор-говнокодер делает
          - return $soapmsg;
          + return "<lol>$soapmsg</lol>";
          и радуется жизни.
          2) Истинный кодер правит в N местах, которые еще надо найти
          Ответить

          • ava guest 07.04.2010 23:19 # −1

            А что вы будете делать, если то же самое министерство выпустит указ о том, что цифра 9 теперь значит число 8 и соответственно наоборот:
            public const $NINE = 9;
            public const $EIGHT = 8;
            И дальше в том же духе? ;)
            Для того и существуют стандарты, чтобы не нужно было писать вот такую вот ерунду.
            Кроме того, решать несуществующую проблему в продакшн коде приложения - это теперь считается хорошим тоном?
            Ответить

    • ava guest 07.04.2010 23:24 # +2.4

      Обыкновенная заглушка
      Ответить

    • ava a282750 24.08.2021 22:40 # 0

      vanished
      Ответить

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