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

    +170

    1. 1
    include($b2bcontext_path."modules/"."costyl".".php");

    Самое интересное - инклюд безусловный. Он просто есть. Костыль - он такой костыль...

    Запостил: xStream, 22 Августа 2011

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

    • ava jQuery 22.08.2011 22:02 # −10

      показать все, что скрытоvse xujnja zadroti sranie
      Ответить

    • ava substr 22.08.2011 22:37 # −2

      Костыль?
      Ответить

    • ava Nemoden 22.08.2011 22:44 # +2

      require($b2bcontext_path."modules".DIREC TORY_SEPARATOR."costyl".".php");

      fixed.
      Ответить

      • ava xStream 22.08.2011 22:47 # 0

        Воистину :) Пойду исправлю xD
        Ответить

      • ava rO_ot 23.08.2011 14:17 # 0

        require($b2bcontext_path.CONST_MODULES.D IREC TORY_SEPARATOR.CONST_COSTYL.EXTENSIION_S EPARATOR.EXTENSION_PHP);
        ^)
        Ответить

    • ava Lure Of Chaos 23.08.2011 11:17 # +2

      > $b2bcontext_path
      business to business? божежмой
      Ответить

      • ava roman-kashitsyn 23.08.2011 11:18 # 0

        Вот такой вот суровый российский b2b.
        Ответить

      • ava xStream 23.08.2011 11:23 # 0

        Партнерский сервис на наш сайт, мать его... Там много гов... да нет, что это я? Там все - ГК
        Ответить

        • ava Lure Of Chaos 23.08.2011 11:26 # 0

          может, найдете что-нибудь еще интересное? из того же costyl.php было бы интересно глянуть, подо что и чем ставят :)
          Ответить

          • ava xStream 23.08.2011 11:34 # 0

            На самом деле говнисто, но не очень интересно, разве что имя переменной:
            $gaga = & new b2bcontexttemplate($b2bcontext_path.'templates/'.$template_var.'/costyl.tpl.php');
    $gaga->assign('costyl_mod',$costyl_mod);
    $gaga->assign('script_name',$script_name);
    $b2bcontext_costyl = $gaga->fetch();
    $b2bcontext_costyl = iconv('windows-1251', $charset,  $b2bcontext_costyl);


            Содержимое costyl.php
            Ответить

            • ava Lure Of Chaos 23.08.2011 11:41 # 0

              как же не очень? это гениально, полагать, что в шаблоне windows-1251, и конвертировать... наверное в утф8? дааа, расчет на исключительно кириллицу - это круто, очень круто!
              Ответить

              • ava xStream 23.08.2011 11:47 # 0

                А черт его знает, как оно там работает :)
                Меня аж дрожь пробила, когда внутренности этих скриптов захотелось посмотреть...

                Из "гениального": весь код - процедурный, но обернут в классы с конструктором (где строк 100-500 говна, делающего всю работу) и 0-5 методов (с таким же количеством строк). За что спасибо - не поломалась сама система, куда это все встраивалось. Верстке, JS и стилям повезло гораздо меньше.
                Ответить

                • ava Lure Of Chaos 23.08.2011 11:51 # 0

                  у стилей хуже с областью видимости - просто даже никто никогда не заморачивается, хотя ведь можно указывать четкую иерархию хоть от html
                  Ответить

            • ava roman-kashitsyn 23.08.2011 11:42 # +3

              > $gaga
              фанаты леди гаги жгут. Помню, в универе один парень давал переменным имена футбольных команд: spartak, dinamo, etc...
              Ответить

          • ava xStream 23.08.2011 11:52 # 0 

            // ����� �������� � �������
    if (!function_exists('indexOf')) {
        function indexOf($needle, $haystack) {
            for($i = 0; $i < count($haystack); $i++) {
                if ($haystack[$i] == $needle) {
                    return true;
                }
            }
            return false;
        }
    }


            Такое, как интересное пойдет? :)
            Ответить

            • ava Lure Of Chaos 23.08.2011 11:56 # +1

              да, надо бы отдельным постом )))
              1. indexOf, когда есть strpos
              2. аргументы "перепутаны" вопреки всем похожим реализациям
              3. наконец, возврат булевского значения, а не точного индекса - хотя функция называется indexOf, а не contains :D
              Ответить

              • ava xStream 23.08.2011 11:59 # 0

                На самом деле это просто in_array :)
                Ответить

                • ava Lure Of Chaos 23.08.2011 12:00 # 0

                  а по виду не скажешь ))))
                  хотя да, виноват, count для массивов, а strlen для строк = )
                  Ответить

                  • ava xStream 23.08.2011 12:02 # 0

                    Угу, семантика не соблюдена. Но это на самом деле оно (in_array) и по входу и по выходу, и никакой обфускации не надо - враги не пройдут!
                    Ответить

                    • ava Lure Of Chaos 23.08.2011 12:05 # +2

                      ага, http://php.net/manual/ru/function.in-array.php 1:1

                      а еще замечательна проверка if (!function_exists('indexOf')) - а вдруг нас уже опередили!
                      Ответить

            • ava roman-kashitsyn 23.08.2011 12:05 # 0

              Шикарно!
              Ответить

          • ava xStream 23.08.2011 12:23 # +1 

            // ������ ������ ������� � �������� ���
    if (!function_exists('parse_get')) {
        function parse_get($url) {
            if (preg_match("/id=([0-9]+)/", $url, $matches)) {
                return $matches[1];
            }
        }
    }


            Продолжаем выкладывать "интересности" :) Все оттуда же
            Ответить

            • ava Lure Of Chaos 23.08.2011 12:27 # 0

              :D уже (int)$_GET['id'] чем-то не угодил
              и опять проверка на существование функции ))) я понимаю, что этот костыль, наверное, инклудится повсюду - но, можно либо использовать require_once\include_once, либо сишным способом - проверять существование некоей константы :)
              Ответить

              • ava xStream 23.08.2011 12:32 # 0

                Ладно бы просто (int)$_GET['id'], но название функции!!! xD
                Ответить

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