Возможно, следующий код считается безопасным?Ссылка на статическую локальную переменную функции
Foo& GetFoo()
{
static std::once_flag fooInitFlag;
static Foo f;
std::call_once(fooInitFlag, [&Foo]()
{
Foo.Init();
});
return std::ref(f);
}
Вы спросили:
Является ли следующий код считается безопасным?
Ответ на этот вопрос да.
Другая реализация:
// Not exposed to the outside.
static Foo& getInitializedFoo();
{
static Foo f;
f.Init();
return f;
}
Foo& GetFoo()
{
static Foo& f = getInitializedFoo();
return f;
}
как насчет, 'static Foo &' (reference), чтобы избежать копирования. Я думаю, что это просто опечатка. –
@ Cheersandhth.-Alf, я имел в виду вспомогательную функцию для возврата объекта по значению. –
Ну, это будет работать так же, как и ссылка (та же самая инициализационная гарантия), 'static' Foo и более эффективная + меньшая вероятность побочного эффекта. Нижняя сторона, что 'getInitializedFoo', является внутренней детализацией реализации, которую можно вызвать только один раз. –
Вместо этого отправил код:
Foo& GetFoo()
{
static std::once_flag fooInitFlag;
static Foo f;
std::call_once(fooInitFlag, [&Foo]()
{
Foo.Init();
});
return std::ref(f);
}
сделать это:
struct Initialized_foo
{
Foo item;
Initialized_foo() { item.Init(); }
};
auto get_foo()
-> Foo&
{
static Initialized_foo the_foo;
return the_foo.item;
}
Это не более безопасно, но это проще, и, следовательно, более безопасным в отношении непреднамеренно введенных ошибок.
Обратите внимание, что стандарт гарантирует единственную инициализацию здесь, даже в контексте многопоточности.
лучше, в идеале каждый Foo insteance должен быть надлежащим образом инициализируется Foo конструктора, а не с помощью метода Init. Но это не всегда возможно.
Вы забыли 'the_foo.item' –
@ThePhD: спасибо, исправлено. –
И 'static Initialized_foo the_foo;'. –
Вам не нужно 'once_flag' вещь. Единая инициализация гарантирована. Использовать конструктор, а не метод 'Init' - –
Да. Обратите внимание, что вам все еще нужно, что в msvc 2013, но не в компиляторе C++ 11, где статическая инициализация является потокобезопасной. (не возвращайте std :: ref (f), просто f) –
Здесь вам не нужно 'std :: ref'; просто верните 'f'. И это безопасно, так как 'f' останется. –