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

    • ava yasenhui 06.07.2011 16:52 # +1

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

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

      А то что написано не $pos = strpos($pp_req, '&FIRSTNAME=') or $pos = strpos($pp_req, '&SHIPTONAME='); так, strpos он и 0 вернуть может.
      Ответить

      • ava kovel 06.07.2011 16:59 # 0

        $pos -- переменная которая нигде не используется (это понятно из последующего return'a). Т.к. модифицирующих переменную $res функций не вызывается, то и сама по себе вся конструкция с strpos() бесполезна. То же самое касается и $pp_req.
        Ответить

        • ava gegMOPO4 06.07.2011 17:13 # 0

          Это не показано.
          Ответить

        • ava yasenhui 06.07.2011 17:27 # 0

          >это понятно из последующего return'a

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

          • ava kovel 06.07.2011 17:36 # 0

            :) ok, просмотрел ваш пример.

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

            globals и static тоже нет нигде.
            Ответить

            • ava yasenhui 06.07.2011 17:46 # 0

              Тогда да, говнокод в чистом виде.

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

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