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

    +165

    1. 1
    2. 2
    3. 3
    4. 4
    if (($pos = strpos($_SERVER["REQUEST_URI"], "?")) !== false)
{
	$params = substr($_SERVER["REQUEST_URI"], $pos+1);
	parse_str($params, $_GET);

    Bitrix, 404.php

    Посоны не слышали про $_SERVER[QUERY_STRING], зато, сука, неймспейсы знают!

    Запостил: Boolean, 26 Января 2014

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

    • ava iproger 27.01.2014 20:43 # +1

      Уже несколько недель мучаюсь с ним. Просто тихий ужас.
      Ответить

    • ava Lure Of Chaos 28.01.2014 00:02 # +3

      ребята еще и не слишком доверяют $_GET, конструируя заново... молодцы!
      Ответить

    • ava proweber1 02.02.2014 20:12 # 0

      Ребят, я не битриксовод, я хотел спросить, в чем прикол про namespace?
      Ответить

      • ava inkanus-gray 02.02.2014 20:23 # 0

        В том, что используются всякие заумные конструкции типа: 
        namespace Bitrix\Main;

use Bitrix\Main\Context;
use Bitrix\Main\Localization\LanguageTable;
        Но при этом игнорируются более простые.
        Ответить

        • ava bormand 02.02.2014 20:28 # +1

          Что ж в неймспейсах заумного?

          А что касается QUERY_STRING vs REQUEST_URI - а вдруг там с ней какие-то подводные камни связаны? :) Например какой-нибудь сервер не выставляет эту переменную, или при каких-то условиях она кривая.
          Ответить

          • ava inkanus-gray 02.02.2014 20:34 # +2

            Предлагаю пойти дальше: поднимать сервер прямо из PHP с помощью socket_listen и парсить строку запроса и все заголовки самостоятельно.

            Мало ли что там лежит в этих глобальных переменных! Нельзя подвергать себя зависимости от кривых серверов!
            Ответить

            • ava Vasiliy 02.02.2014 22:29 # 0

              пых и так умеет сервер. http://www.php.net/manual/ru/features.commandline.webserver.php
              Ответить

              • ava inkanus-gray 03.02.2014 08:27 # 0

                Знаю, 5.4 умеет, но это не то. Ощущения не те, всё равно придётся переменную $_SERVER использовать.
                Ответить

                • ava Vasiliy 03.02.2014 11:41 # 0

                  О извиняюсь я в первом сообщении сарказм не разглядел.
                  Ответить

        • ava proweber1 03.02.2014 11:57 # 0

          Ааа, понял, спасибо =)
          Ответить

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