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

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $pos = strpos($pp_req, '&FIRSTNAME=');
            if ($pos === false) {
                $pos = strpos($pp_req, '&SHIPTONAME=');
            }
            return $res;

    Запостил: kovel, 06 Июля 2011

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

    • Всю функцию/метод в студию.

      Может быть там function govnofunction ($govnovar, &$pos) {

      А то что написано не $pos = strpos($pp_req, '&FIRSTNAME=') or $pos = strpos($pp_req, '&SHIPTONAME='); так, strpos он и 0 вернуть может.
      Ответить
      • $pos -- переменная которая нигде не используется (это понятно из последующего return'a). Т.к. модифицирующих переменную $res функций не вызывается, то и сама по себе вся конструкция с strpos() бесполезна. То же самое касается и $pp_req.
        Ответить
        • Это не показано.
          Ответить
        • >это понятно из последующего return'a

          http://www.php.net/manual/en/language.references.pass.php
          Курить до просветления, потом перечитать мой коммент выше.
          Ответить
          • :) ok, просмотрел ваш пример.

            Вот сигнатура:
            private function getUserInfoFromPaypalRequest($pp_req) {

            globals и static тоже нет нигде.
            Ответить
            • Тогда да, говнокод в чистом виде.

              Программистус невнимательнус обыкновенус.
              Ответить

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