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

    +49.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    bool ok;
double d;

QLocale c(QLocale::C);
d = c.toDouble( "1234.56", &ok );  // ok == true, d == 1234.56

    Ссылка на bool!

    Запостил: mock, 14 Апреля 2010

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

    • ava guest 14.04.2010 14:07 # +3

      Что такого странного в необходимости вернуть больше одного значения?
      И указатель != ссылка.
      Ответить

    • ava guest 14.04.2010 20:49 # +0.4

      Я дождался С++! Даже не верится!!!
      Ответить

    • ava guest 14.04.2010 21:28 # +0.2

      автор убейся, в C/C++ это стандартнейшая практика возвращать дополнительные значения через указатели на аргументы
      Ответить

      • ava guest 14.04.2010 21:31 # +1

        Правда, пиши я фреймворк, я бы сделал так:

        double d;
        if(c.parseDouble("1234.56", &d))
        {
        // success
        }
        else
        log("Oh noes!");
        Ответить

        • ava guest 14.04.2010 23:12 # +0.4

          Исключения нужно использовать, что-бы возвращать фэйл.
          Ответить

          • ava guest 15.04.2010 00:53 # +0.8

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

            исключения это когда на 100% уверен, что файл есть (ну должен же быть!), а его внезапно нету... для этого-то -- вместо того чтобы мудоёбиться с обработкой _редких случаев_, а чтоб тупо оставить обработку "на потом" -- и придумали исключения... но многие этого недопетривают и лепят их везде...

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

            • ava guest 15.04.2010 00:54 # 0

              s/что строка может не содержать неверное значение/что строка может не содержать верное значение
              Ответить

            • ava linux-hacker 16.04.2010 23:32 # 0

              расскажи это авторам boost::lexical_cast.
              Использование исключений здесь более, чем уместно. Но так как это куте, их здесь не используют. Куте старая библиотека и они отказались от исключений в пользу переносимости.
              Ответить

              • ava guest 17.04.2010 01:33 # +1.6

                я срал на авторов boost::lexical_cast, в С++ вообще извращена вся идея ООП, а всякие бусты - мёртвому припарки
                Ответить

                • ava linux-hacker 17.04.2010 19:08 # 0

                  причём тут ООП, вообще? Ну да ладно, всё с тобой понятно...
                  Ответить

                  • ava cfdev 17.04.2010 19:31 # 0

                    ага, а ещё исключения это несколько точек выхода, причём в С-дериватах кроме явы они не входят в часть контракта, неудивительно что оно постоянно валится потом.

                    хайль исключения везде где попало! больше проблем из ниччего!
                    Ответить

          • ava guest 15.04.2010 01:00 # +1.2

            Незря в .Net сделали две версии: Parse (кидает исключение) и TryParse (возвращает буль). В моём фреймворке я сделал так же. Иногда "ошибка" -- часть логики ("ах так! тогда мы поступим иначе!"), иногда -- её нарушение ("ВНЕЗАПНО!") Нельзя так просто судить возврат булем.

            Кстати, "чтобы" пишется слитно, без дефиса.
            Ответить

            • ava guest 15.04.2010 08:45 # +0.4

              Кстати, "незря" пишется раздельно, без дефиса:)
              Ответить

              • ava guest 15.04.2010 10:02 # +0.2

                граммарнаци негодуе
                Ответить

              • ava guest 16.04.2010 01:45 # +1.2

                ага, "несмотря" пишется слитно, а "не зря" - раздельно. Люблю русский язык.
                Ответить

                • ava guest 16.04.2010 08:18 # 0

                  Этог происходит от глубинного различия этих двух действий, активное и пассивное состояние ума, и еще tl;dr по буддизму.
                  Ответить

          • ava guest 15.04.2010 11:34 # 0

            -30% к производительности от включения исключений.
            Ответить

        • ava guest 15.04.2010 11:40 # 0

          Твой формат записи не позволяет сделать множественное присваивание одной строкой, т.е.
          a= b = c.toDouble("1234.56") - а передача параметра нужна отнюдь не всегда.
          Ответить

          • ava guest 16.04.2010 11:15 # 0

            и как часто это множественное присваивание встречается, чтобы её вообще учитывать, лол?
            Ответить

            • ava guest 16.04.2010 11:17 # 0

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

              • ava guest 17.04.2010 01:29 # 0

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

                ВНЕЗАПНОЕ решение, да:

                double d, e;
if(c.parseDouble("1234.56", &d))
{
    e = d;
}
                Ответить

    • ava mock 15.04.2010 05:35 # −1.4

      Все кто возвращают ссылки на бул - могут убить себя :) это пример из хелпа куте. Естественно надо поменять параметры bool QLocale::toDouble(const QString&, double&);
      Ответить

      • ava guest 15.04.2010 05:40 # +1.2

        конвенционно везде to%TYPE_NAME% обозначает, что возвращается %TYPE_NAME%

        так что не катит твой разрыв шаблонов. надо другое имя функции выбрать
        Ответить

    • ava guest 15.04.2010 20:40 # −0.2

      ох, неужели это с++, я не верю
      Ответить

      • ava guest 16.04.2010 02:33 # 0

        да, можешь подрочить, раз такое счастье... (омазавшись...)
        Ответить

        • ava guest 16.04.2010 11:30 # +2

          А я люблю обмазываться несвежим говнокодом и дрочить. Каждый день я хожу на говнокод.ру и вставляю комментарий "где здесь С++?" к любому говнокоду, который вижу. На сотню сообщений целый день уходит. Зато, когда после тяжёлого дня я прихожу домой, открываю визуал-студио, запускаю дебуггер...ммм и компелирую С++. Мне вообще кажется, что говнокоды на С++ умеют думать, у них есть свои семьи, города, чувства, не удаляйте их, лучше закомментируйте их, говорите с ними, ласкайте их.... А вчера за компьютером, мне преснился чудный сон, как будто я зашёл на говнокод.ру, и всё прератилось в С++, джава, питон, пхп, все из С++, даже небо, даже Аллах!
          Ответить

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