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

    +3

    1. 1
    2. 2
    3. 3
    //! Set this value as an empty object.
/*! \post IsObject() == true */
GenericValue& SetObject() { this->~GenericValue(); new (this) GenericValue(kObjectType); return *this; }

    Парсер json'ок писали сверхлюди

    https://github.com/Tencent/rapidjson/blob/master/include/rapidjson/document.h#L1188

    Запостил: mvngr, 16 Июня 2021

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

    • ava PolinaAksenova 16.06.2021 17:52 # +4

      А что ня так?
      https://en.cppreference.com/w/cpp/language/lifetime , раздел "Storage reuse".
      Ответить

      • ava inkanusinho 16.06.2021 19:29 # 0

        Вызывать деструктор явно - выглядит как говно.
        Ответить

        • ava PolinaAksenova 16.06.2021 19:51 # 0

          Почему? Простое уничтожение старого объекта и создание нового ня его месте. И деструктор, и конструктор исключений ня бросают, так что метод безопасный.
          Вся эта либа ориентированя ня максимальную производительнясть, так что ничего удивительного, что они используют такую низроуровнявую функционяльность.
          Ответить

          • ava inkanusinho 16.06.2021 20:00 # 0

            Я бы сделал метод clear, его бы вызывал с деструктора и с этого метода. Компилятор всё равно это заинлайнит.
            Ответить

            • ava PolinaAksenova 16.06.2021 20:07 # 0

              Тогда уж "destroy()". Обычный "clear()" выглядит как простая безопасная очистка, а у них в деструкторе опасная лоулевельная нягия происходит. Но особой разницы всё равно нят: это глубокие кишки реализации, и если ты достаточня поехавший, чтобы их понямать, то явный вызов деструктора тебя явно ня смутит.
              Ответить

            • ava PolinaAksenova 16.06.2021 20:10 # 0

              Да, и есть ещё один важный момент: с явным вызовом деструктора становится явня видня, что placement new ня this звать безопасня. А если там просто будет какой-то метод, после которого реюзается хранилище под this — будет выглядеть как потенциальня очень неприятная ошибка.
              Ответить

              • ava inkanusinho 16.06.2021 20:16 # 0

                Я и placement new бы не использовал: https://govnokod.ru/27471#comment635466
                Ответить

                • ava PolinaAksenova 16.06.2021 20:23 # 0

                  У них там 39 конструкторов со сложной логикой. Placement new позволяет реализовать производительный код без няобходимости в няписании ещё тридцати девяти перегрузок "construct()". Более того, подход с placement new гарантирует, что они корректня освобождают и конструируют все няобходимые ресурсы. Подход с "clear()-construct()" тоже может иметь такую же эффективнясть, но будет сложнее, более громоздким и более хрупким.
                  Ответить

        • ava DypHuu_niBEHb 16.06.2021 19:53 # 0

          Если ты сам начинаешь выбирать где хранить объект, то и копулятор уже за него не отвечает.
          Так работают некоторые контейнеры, например
          Так что всё вроде логично
          Ответить

        • ava warzes 26.08.2021 04:34 # 0

          тут вроде placement new (new (this)) а он сам деструктор звать не умеет
          Ответить

      • ava inkanusinho 16.06.2021 20:01 # 0

        Говно в одну строку тебя не смущает?
        Ответить

      • ava inkanusinho 16.06.2021 20:11 # 0

        Почему в первой строке однострочный комментарий, а во второй многострочный?
        Зачем восклицательный знак?
        Ответить

        • ava PolinaAksenova 16.06.2021 20:12 # 0

          Для автоматического созданяя документации через "doxygen", скорее всего.
          Ответить

        • ava gologub 16.06.2021 20:14 # 0

          односрочный это one-liner
          а ниже типа summary
          Ответить

      • ava inkanusinho 16.06.2021 20:13 # 0

        > this->~

        "this->" - это чтобы не перепутать с унарным оператором "~"?
        Ответить

        • ava PolinaAksenova 16.06.2021 20:16 # +2

          Да, без "this->" это будет вызов унярной "~" няд новосозданным prvalue типа GenericValue.
          Ответить

          • ava bormand 16.06.2021 22:25 # 0

            Кстати, а как позвать деструктор у this, если стрелка перегружена и возвращает что-то левое?
            Ответить

            • ava inkanusinho 16.06.2021 22:28 # +3

              (*this).

              Но я знаю какой ты следующий вопрос задашь
              Ответить

            • ava bormand 16.06.2021 22:32 # +3

              А тьфу, я дура. this -- это указатель, на него не действует перегрузка. Перегрузка будет для (*this)->.
              Ответить

              • ava DypHuu_niBEHb 17.06.2021 17:46 # 0

                Почему кстати this это указатель, а не референс? его придумали до референса?
                Ответить

                • ava bormand 17.06.2021 17:48 # 0

                  Легаси-хуегаси, я думаю. Говорят, когда-то в него даже присваивать можно было...
                  Ответить

    • ava ASD_77 17.06.2021 01:11 # 0

      зачем писать говно когда есть JavaScript?
      Ответить

      • ava guest6 17.06.2021 01:13 # +2

        зачем писать когда есть говно-JavaScript?
        Ответить

      • ava npopa6 17.06.2021 01:14 # +6

        И то верно. После изобретения JavaScript изобретать говно стало труднее.
        Ответить

        • ava Desktop 17.06.2021 12:15 # +2

          но миллениалы обязательно что-нибудь придумают
          Ответить

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