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

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    Connection::Connection(const Options& opts): impl_(new Impl)
{
    impl_->isInitialized = false;
    impl_->options = std::move(*opts.impl_);
   
    if (!options().lazyInit)
        conn();
}

    Вся соль в строке 4: из объекта opts, переданного по константной ссылке, подло выжимают содержимое. Видимо, никто не пробовал создать два коннекта из одного объекта опций.

    Запостил: roman-kashitsyn, 28 Октября 2014

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

    • ava Soul_re@ver 28.10.2014 22:18 # 0

      Вот поэтому публичные данные — плохо. Был бы нормальный геттер — хрен вышло бы.
      Ответить

      • ava roman-kashitsyn 28.10.2014 23:12 # 0

        Там не в паблике дело - impl_ - приватное поле, а Connection объявлен френдом. Дело в нарушении семантики.
        Ответить

        • ava guest 28.10.2014 23:49 # −7

          показать все, что скрыто-
          Ответить

        • ava Dummy00001 29.10.2014 00:07 # 0

          дайте угадаю.

          там делаются миллионы соединений в секунду, почему кто-то и подумал что std::move() будет хорошей идеей, что бы съэкономить не копировании?

          "premature optimization" в ее чистой форме.
          Ответить

          • ava roman-kashitsyn 29.10.2014 13:12 # +1

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

            Правильным подходом было бы передавать входной параметр по значению. Тогда бы ни у кого не возникало иллюзий.
            Ответить

    • ava guest 29.10.2014 01:12 # −7

      показать все, что скрыто-
      Ответить

    • ava guest 29.10.2014 19:33 # −3

      -
      Ответить

    • ava miwomare 25.08.2021 07:30 # 0

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

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