1. C++ / Говнокод #5508

    +165

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    static string GetRequestParam(string request)
        {
            string result="";
            int i = 4;
            for(char c=request[i];c!=' ';c=request[++i])
                result+=c;
            return result;
        }

    Запостил: Говногость, 03 Февраля 2011

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

    • ava Lure Of Chaos 03.02.2011 12:57 # 0

      субстринг с 4ого символа до пробела?
      БСК.
      Ответить

    • ava Dummy00001 03.02.2011 13:55 # +4 

      string result="";

      вот это моя любимая часть. некоторые компилеры оптимят это - но многие нет. а фишка заключается в том что STL вполне корректно попытается сделать копию пустой строки со всеми вытекающими.

      как то раз на одном проекте, часто используемый класс такую инициализацию в конструкторе использовал. после удаления этих бессмысленых `=""` производительность апликухи улучшилась на более чем 5%.
      Ответить

      • ava gegMOPO4 03.02.2011 14:18 # 0

        По сравнению с 6-й строчкой это мелочи.
        Ответить

        • ava Dummy00001 03.02.2011 14:26 # 0

          ежику понятно что это все говно, но просто с `std::string s = "";` наболело ;)

          не говоря уже о том что тривиальной проверки на конец строки не делается. что для меня - говно №0.
          Ответить

          • ava gegMOPO4 03.02.2011 14:56 # 0

            Страшно подумать, что написано в более сложных функциях.
            Ответить

        • ava TarasB 03.02.2011 15:29 # 0

          А разве строка не умеет резервировать место заранее?
          Ответить

          • ava pushkoff 03.02.2011 15:46 # 0

            умеет, но она это делает как делфи, по определенным коеффициентам, что не всегда оптимально...
            хотя мы могли указать что вероятнее всего строка, которую мы ждем, влезет например в 32 символа и избавиться от одной лишней аллокации в большинстве случаев...
            Ответить

            • ava TarasB 03.02.2011 16:18 # 0

              Тогда алгоритмическая сложность при наращивании по элементу будет та же, что и при моментальном выделении (о(н)).
              А на константу здесь, скорее всего, пофиг.
              Ответить

          • ava gegMOPO4 03.02.2011 16:32 # 0

            Не обязана. И вообще, все эти телодвижения легко заменить на одну строчку:
            return request.substr(4, request.find(' ', 4));
            Ответить

    • ava Говногость 03.02.2011 15:58 # 0

      Сглупил... Это код на C#. :( Впрочем, в раздел С++ тоже подходит...
      Ответить

      • ava guest 03.02.2011 16:32 # −1

        Может это 
        string result="";
        , дабы 
        string result=new string();
        не писать?
        Ответить

        • ava koodeer 03.02.2011 19:11 # 0

          Вообще, правильным (по MSDN) считается
          string result = string.Empty;
          Ответить

    • ava Мистер Хэнки 03.02.2011 17:59 # 0

      C тех пор как я открыл для себя Perl, меня стало воротить от таких конструкций
      "строка1 += строка2". В том смысле, что оператор сложения выполняет конкатенацию.
      Ответить

      • ava Lure Of Chaos 03.02.2011 18:32 # 0

        а что, по вашему, должен делать этот оператор со строками?
        Ответить

        • ava gegMOPO4 03.02.2011 18:37 # 0

          В PHP? Вы не поверите...
          Ответить

        • ava Мистер Хэнки 03.02.2011 18:54 # −1

          этот оператор вообще не должен ничего делать со строками, он предназначен для сложения чисел
          Ответить

          • ava koodeer 03.02.2011 19:09 # 0

            Сложение чисел - это математика. А в математике нет операции +=
            Ответить

            • ava Мистер Хэнки 03.02.2011 21:17 # −2

              невелика разница.
              X += Y это сахар для X = X + Y.
              Такая-то операция есть в математике, математик вы наш дорогой?
              Или string1 = string1 + string2 + "blabla" не встречали ни разу?
              Ответить

              • ava koodeer 03.02.2011 23:35 # −1

                X = X + Y - это должно быть НЕравенство :р
                Ответить

                • ava guest 03.02.2011 23:39 # 0

                  Y=0
                  Ответить

                • ava TarasB 04.02.2011 10:20 # 0

                  Поэтому в нормальных языках это пишется как X := X + Y;
                  Ответить

                  • ava Lure Of Chaos 04.02.2011 11:55 # +3

                    мы знаем, какие для вас "нормальные"
                    Ответить

                  • ava eth0 04.02.2011 19:18 # 0

                    Интересно. Подобный сахар там бы выглядел как
                    X +:= Y

                    Занятный смайлик.
                    Ответить

                    • ava bugmenot 04.02.2011 20:50 # 0

                      3~~@
                      Ответить

                    • ava ctm 04.02.2011 21:51 # 0

                      в лазарусе, если поставить соотв. опции,
                      это выглядит в точности как в С/С++:
                      X+=Y;
                      Ответить

                      • ava bugmenot 04.02.2011 22:01 # −1

                        > в лазарусе
                        ЛОЛШТО?
                        Ответить

                        • ava istem 04.02.2011 22:10 # 0

                          по данной терминологии у Тараса Б проконсультируйтесь...
                          Ответить

                        • ava TarasB 05.02.2011 12:02 # −1

                          лол то
                          := - присвоить как есть (двоеточие отвечает за "как есть"), += - присвоить и сложить со старым. Удобно, логично.
                          Ответить

                          • ava bugmenot 05.02.2011 12:07 # +4

                            в паскале такого не сделаешь
                            говно этот ваш лазарус
                            Ответить

                            • ava guest 05.02.2011 15:39 # −1

                              Невидимую надпись я то сразу и не приметил...
                              Ответить

                              • ava bugmenot 05.02.2011 16:42 # +1

                                чорт, паливо!
                                там целый трикалор:
                                1) оригинал пост - говно, поскольку фича fpc, а не lazarus (который является IDE, имитирующим Delphi)
                                2) инициатива введения shorthand операторов в fpc - засахаренное говно
                                3) lazarus сам по себе говно, т.к. требует установки в рут
                                Ответить

                                • ava eth0 05.02.2011 18:54 # 0

                                  Лазарус глупый, пытается во все щели делфай копировать, и глючный вдобавок.
                                  Для установки компонентов ему потребна пересборка самого себя.
                                  Забавная красноглазая безделушка.
                                  С ен. мовы перевод слова вполне символизирует.
                                  Ответить

                                  • ava bugmenot 05.02.2011 19:29 # +1

                                    just as planned, хотели открытые дельфи, их и сделали, даже слоупочный интерфейс склонировали. а глючит потому, что молодой еще.
                                    в статической линковке не вижу ничего плохого, даже наоборот, шароварные макаки, работающие методом проб и ошибок должны страдать. броланд решил вопрос высером своего формата шаред объектов, вот говнище так говнище.
                                    еще там проститутская лицензия на библиотеки.
                                    лазарь вроде с прожидью был, нет?
                                    Ответить

          • ava Lure Of Chaos 03.02.2011 19:10 # +2

            это может иметь смысл для динамических языков, что бы определить "2"+"2", это "22" или "4"?
            здесь же нет такой неоднозначности
            Ответить

            • ava Мистер Хэнки 03.02.2011 21:19 # −3

              >здесь же нет такой неоднозначности
              для компилятора нету.
              для человекообразного писателя/читателя есть.
              Ответить

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