"PHP" / Говнокод #2951 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 nur, (Updated )

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

  • ava murz # Ссылка на оригинал 0

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

      • ava striker # Ссылка на оригинал 0 ⇈

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

      • ava murz # Ссылка на оригинал 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 # Ссылка на оригинал 0 ⇈ 

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

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

  • ava murz # Ссылка на оригинал 0

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

    • ava nur # Ссылка на оригинал 0 ⇈

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

      • ava guest # Ссылка на оригинал 0 ⇈

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

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

        • ava guest # Ссылка на оригинал 0 ⇈

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

  • ava a282750 # Ссылка на оригинал 0

    Наконец-то парень взвыл! Это музыка для его ушей! Прокусил губу до крови. Но взвыл!..
    Ответить

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓