Потокобезопасная ленивая конструкция синглтона в C ++ - PullRequest
33 голосов
/ 10 августа 2008

Есть ли способ реализовать одноэлементный объект в C ++, а именно:

  1. Ленивое построение потокобезопасным способом (два пользователя могут одновременно быть первым пользователем синглтона - он все равно должен быть создан только один раз).
  2. Не полагается на то, что статические переменные создаются заранее (поэтому одноэлементный объект сам по себе безопасен при создании статических переменных).

(Я недостаточно хорошо знаю мой C ++, но это тот случай, когда целочисленные и постоянные статические переменные инициализируются до выполнения любого кода (т. Е. Даже до выполнения статических конструкторов - их значения уже могут быть «инициализированы») в образе программы)? Если это так - возможно, это можно использовать для реализации мьютекса синглтона - который, в свою очередь, можно использовать для защиты создания реального синглтона ..)


Отлично, похоже, у меня сейчас есть пара хороших ответов (позор, я не могу отметить 2 или 3 как ответ ). Похоже, есть два широких решения:

  1. Использовать статическую инициализацию (в отличие от динамической инициализации) статической переменной POD и реализовывать свой собственный мьютекс с помощью встроенных атомарных инструкций. Это был тип решения, на которое я намекал в своем вопросе, и я думаю, что я уже знал.
  2. Используйте некоторые другие библиотечные функции, такие как pthread_once или boost :: call_once . Об этом я, конечно, не знал - и очень благодарен за опубликованные ответы.

Ответы [ 9 ]

13 голосов
/ 10 августа 2008

К сожалению, ответ Мэтта включает в себя то, что называется двойной проверкой блокировки , которая не поддерживается моделью памяти C / C ++. (Это поддерживается Java 1.5 и более поздними версиями - и я думаю, что .NET - модель памяти.) Это означает, что между временем, когда выполняется проверка pObj == NULL, и когда блокировка (мьютекс) получена, pObj может иметь уже назначен в другой теме. Переключение потоков происходит всякий раз, когда этого хочет ОС, а не между «строками» программы (которые не имеют значения после компиляции в большинстве языков).

Кроме того, как признает Мэтт, он использует int в качестве блокировки, а не примитива ОС. Не делай этого. Правильные блокировки требуют использования инструкций барьера памяти, потенциально очистки строк кэша и т. Д .; используйте примитивы вашей операционной системы для блокировки. Это особенно важно, потому что используемые примитивы могут меняться между отдельными линиями ЦП, на которых работает ваша операционная система; что работает на CPU Foo, может не работать на CPU Foo2. Большинство операционных систем либо изначально поддерживают потоки POSIX (pthreads), либо предлагают их в качестве оболочки для пакета потоков ОС, поэтому часто лучше всего проиллюстрировать примеры с их использованием.

Если ваша операционная система предлагает соответствующие примитивы, и если вам это абсолютно необходимо для производительности, вместо выполнения этого типа блокировки / инициализации вы можете использовать операцию atomic сравнения и swap для инициализации общей глобальной переменной , По сути, то, что вы пишете, будет выглядеть так:

MySingleton *MySingleton::GetSingleton() {
    if (pObj == NULL) {
        // create a temporary instance of the singleton
        MySingleton *temp = new MySingleton();
        if (OSAtomicCompareAndSwapPtrBarrier(NULL, temp, &pObj) == false) {
            // if the swap didn't take place, delete the temporary instance
            delete temp;
        }
    }

    return pObj;
}

Это работает только в том случае, если безопасно создать несколько экземпляров вашего синглтона (по одному на поток, который вызывает одновременный вызов GetSingleton ()), а затем выбрасывать дополнительные элементы. Функция OSAtomicCompareAndSwapPtrBarrier, предоставляемая в Mac OS X - большинство операционных систем предоставляют аналогичный примитив - проверяет, является ли pObj значением NULL, и фактически устанавливает его на temp, если это так. При этом используется аппаратная поддержка, чтобы буквально выполнить своп только один раз и определить, произошло ли это.

Еще одно средство, которое можно использовать, если ваша ОС предлагает его, между этими двумя крайностями - pthread_once. Это позволяет вам настроить функцию, которая запускается только один раз - в основном, выполняя все блокировки / барьеры / и т.д. обман для вас - независимо от того, сколько раз он вызывался или сколько потоков он вызывал.

11 голосов
/ 10 августа 2008

По сути, вы запрашиваете синхронизированное создание синглтона, без использования какой-либо синхронизации (ранее построенные переменные). В общем, нет, это невозможно. Вам нужно что-то доступное для синхронизации.

Что касается вашего другого вопроса, то да, статические переменные, которые могут быть статически инициализированы (т. Е. Не требуется код времени выполнения), гарантированно будут инициализированы перед выполнением другого кода. Это позволяет использовать статически инициализированный мьютекс для синхронизации создания синглтона.

Начиная с версии 2003 года стандарта C ++:

Объекты со статической продолжительностью хранения (3.7.1) должны быть инициализированы нулем (8.5) перед любой другой инициализацией. Обнуление инициализации и инициализация с постоянным выражением вместе называются статической инициализацией; все остальные инициализации - это динамическая инициализация. Объекты типов POD (3.9) со статической продолжительностью хранения, инициализированные с помощью константных выражений (5.19), должны быть инициализированы перед любой динамической инициализацией. Объекты со статической продолжительностью хранения, определенной в области пространства имен в одной и той же единице перевода и динамически инициализированной, должны быть инициализированы в том порядке, в котором их определение появляется в единице перевода.

Если вы знаете , что будете использовать этот синглтон во время инициализации других статических объектов, я думаю, вы обнаружите, что синхронизация не является проблемой. Насколько мне известно, все основные компиляторы инициализируют статические объекты в одном потоке, так что потокобезопасность при статической инициализации. Вы можете объявить свой одноэлементный указатель равным NULL, а затем проверить, инициализирован ли он, прежде чем использовать его.

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

Редактировать: Предложение Криса использовать атомное сравнение и обмен, безусловно, сработало бы. Если переносимость не является проблемой (и создание дополнительных временных синглетонов не является проблемой), тогда это несколько более низкие издержки.

10 голосов
/ 19 мая 2010

Вот очень простой ленивый синглтон-геттер:

Singleton *Singleton::self() {
    static Singleton instance;
    return &instance;
}

Это лениво, и следующий стандарт C ++ (C ++ 0x) требует, чтобы он был поточно-ориентированным. На самом деле, я считаю, что, по крайней мере, в g ++ это реализовано безопасным способом. Так что, если это ваш целевой компилятор или , если вы используете компилятор, который также реализует это поточно-ориентированным способом (может быть, это делают более новые компиляторы Visual Studio? Я не знаю), тогда это может быть все, что вам нужно.

Также см. http://www.open -std.org / jtc1 / sc22 / wg21 / docs / paper / 2008 / n2513.html по этой теме.

8 голосов
/ 10 августа 2008

Вы не можете сделать это без каких-либо статических переменных, однако, если вы готовы допустить одну, вы можете использовать Boost.Thread для этой цели. Прочитайте раздел «Однократная инициализация» для получения дополнительной информации.

Затем в функции одноэлементного метода доступа используйте boost::call_once, чтобы создать объект и вернуть его.

6 голосов
/ 16 сентября 2008

Для gcc это довольно просто:

LazyType* GetMyLazyGlobal() {
    static const LazyType* instance = new LazyType();
    return instance;
}

GCC обеспечит, чтобы инициализация была атомарной. Для VC ++ это не так . : - (

Одной из основных проблем этого механизма является отсутствие тестируемости: если вам нужно сбросить LazyType на новый между тестами или вы хотите изменить LazyType * на MockLazyType *, вы не сможете. Учитывая это, обычно лучше использовать статический мьютекс + статический указатель.

Также, возможно, в сторону: лучше всегда избегать статических типов, не являющихся POD. (Указатели на POD в порядке.) Причин для этого много: как вы упоминаете, порядок инициализации не определен, равно как и порядок вызова деструкторов. Из-за этого программы заканчивают сбоем, когда они пытаются выйти; часто это не имеет большого значения, но иногда требуется всплывающее окно, когда профилировщик, который вы пытаетесь использовать, требует чистого выхода.

1 голос
/ 09 ноября 2009
  1. чтение на слабой модели памяти. Это может сломать дважды проверенные замки и спин-блокировки. Intel сильная модель памяти (пока), поэтому на Intel это проще

  2. осторожно используйте «volatile», чтобы избежать кеширования частей объекта в регистрах, иначе вы инициализируете указатель объекта, но не сам объект, а другой поток завершится сбоем

  3. порядок инициализации статических переменных в зависимости от загрузки общего кода иногда не тривиален. Я видел случаи, когда код для уничтожения объекта уже был выгружен, поэтому программа вылетала при выходе

  4. такие объекты трудно уничтожить должным образом

В общем, синглтоны трудно сделать правильно и их сложно отлаживать. Лучше их вообще избегать.

1 голос
/ 10 августа 2008

Вы могли бы использовать решение Мэтта, но вам нужно было бы использовать надлежащий мьютекс / критическую секцию для блокировки и проверки «pObj == NULL» как до, так и после блокировки. Конечно, pObj также должен быть статичным;). В этом случае мьютекс будет неоправданно тяжелым, лучше использовать критическую секцию.

OJ, это не работает. Как отметил Крис, это двойная проверка блокировки, которая не гарантируется в текущем стандарте C ++. См .: C ++ и опасности двойной проверки блокировки

Редактировать: нет проблем, OJ. Это действительно хорошо на языках, где это работает. Я ожидаю, что это будет работать в C ++ 0x (хотя я не уверен), потому что это такая удобная идиома.

1 голос
/ 10 августа 2008

Хотя на этот вопрос уже дан ответ, я думаю, что есть некоторые другие моменты, которые стоит упомянуть:

  • Если вам нужно ленивое создание экземпляра синглтона при использовании указателя на динамически размещенный экземпляр, вам нужно убедиться, что вы его очистили в нужной точке.
  • Вы могли бы использовать решение Мэтта, но вам нужно было бы использовать надлежащий мьютекс / критическую секцию для блокировки и путем проверки "pObj == NULL" как до, так и после блокировки. Конечно, pObj также должен быть static ;) , В этом случае мьютекс будет неоправданно тяжелым, лучше использовать критическую секцию.

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

Редактировать: Да, Дерек, ты прав. Виноват. :)

0 голосов
/ 20 августа 2008

Я полагаю, что не делайте этого, потому что это небезопасно и, вероятно, сломается чаще, чем просто инициализация этого материала в main() не будет такой популярной.

(И да, я знаю, что такое предположение означает, что вы не должны пытаться делать интересные вещи в конструкторах глобальных объектов. В этом суть.)

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