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

    −1

    1. 1
    2. 2
    3. 3
    4. 4
    // Read option name (can contain spaces)
     while (is >> token && token != "value")
-        name += string(" ", name.empty() ? 0 : 1) + token;
+        name += (name.empty() ? "" : " ") + token;

    terminate called after throwing an instance of 'std::length_error'
    what(): basic_string::_M_create

    Replacing string(" ", name.empty() ? 0 : 1) with (name.empty() ? "" : " ") and the same in the while() loop for value fixes the problem (for me).

    Does anyone know if "string(" ", 0)" is invalid C++ ?
    Кресты такие кресты.

    Запостил: 3.14159265, 18 Февраля 2018

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

    • ava Antervis 18.02.2018 05:51 # +4

      дело не в самих крестах, видимо что-то с компилятором. Строки вообще не должны length_error кидать, а для string(const char *s, size_t size) надо лишь чтобы s был корректным указателем.
      Ответить

    • ava Elvenfighter 18.02.2018 16:09 # 0

      https://wandbox.org/permlink/1ARI14UWTuo4iZwM
      https://wandbox.org/permlink/RFCaCRDx0R1ruyGZ

      Скажите своему производителю libstdc++ что он мудак
      Ответить

      • ava 3.14159265 18.02.2018 16:11 # 0

        #gcc, #O3, #windows-only
        Ответить

        • ava Antervis 18.02.2018 18:48 # 0

          MinGW, говоришь? какая версия то хоть?
          Ответить

        • ava bormand 18.02.2018 19:43 # +1

          Компилятор как бы намекает, что нефиг было выёбываться с этим конструированием стринга... А дизасма случаем нету?
          Ответить

          • ava 3.14159265 18.02.2018 20:10 # 0

            Винды под рукой нет. Взял отсюда:
            https://groups.google.com/forum/?fromgroups=#!topic/fishcooking/RjIPgeFFLPQ

            >Компилятор как бы намекает, что нефиг было выёбываться с этим конструированием стринга...
            Кстати да. У комилятора же задача: находить хуйню.
            Ответить

    • ava bormand 18.02.2018 19:34 # 0

      Ответить

    • ava inho 18.02.2018 22:57 # 0

      В чем прикол? Перепутал одинарные кавычки с двойными?

      string(name.empty() ? 0 : 1, ' ')
      Ответить

      • ava 1024-- 19.02.2018 07:40 # +2

        Не, вот в первом комментарии к этому говнокоду упомянули этот конструктор: string(const char *s, size_t size). Всё верно.
        Видимо, реализация попыталась выделить 0 байт под строку и сломалась.
        Ответить

        • ava Steve_Brown 19.02.2018 19:38 # +1

          http://www.cplusplus.com/reference/new/operator%20new/
          - size
          If this argument is zero, the function still returns a distinct non-null pointer on success

          http://www.cplusplus.com/reference/cstdlib/malloc/
          If size is zero, the return value depends on the particular library implementation (it may or may not be a null pointer)
          Ответить

          • ava Antervis 19.02.2018 21:00 # +1

            3.7.3.1/2:
            32. The intent is to have operator new() implementable by calling malloc() or calloc(), so the rules are substantially the same. C++ differs from C in requiring a zero request to return a non-null pointer.

            Видимо, этот пункт некорректно сделали. Откуда length_error - всё равно хз
            Ответить

            • ava g0_1494089131830 19.02.2018 21:12 # −1

              Всё равно твоей мамке, какой хер сосать - платят одинаково.
              Ответить

            • ava Steve_Brown 20.02.2018 10:39 # 0

              А, там же не bad_alloc... Видимо, просто проверку неправильно сделали: if (size <= 0) throw length_error(...)
              Ответить

              • ava g0_1494089131830 20.02.2018 11:00 # 0

                парень в сексуальном кружавчике обслужит парня или группу парней предпочтение кавказцам все позы секс-практики и секс-игрушки, любые желания за твои деньги! пиши прямо сейчас [email protected] антон ЖДУ
                Ответить

        • ava CaptainPlastid 23.02.2018 21:44 # +1

          Как GetOverlappedResult

          На мсдн нет инфы, что "0" это весь буфер, а не "пусто", в пайпах, долго искали...
          Ответить

          • ava SemaReal 25.02.2018 02:15 # +1

            кому нахй нужны оверлаппдрезул если есть completion ports?
            Ответить

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