Thread Safe Singleton Class - Am I doing this Right? - c++

Question

I am creating a system state class that needs be thread safe in reading/writing. I am implementing it using the Singleton Design Pattern.
I have based it off of pointers from these Stackoverflow questions:
Singleton: How it Should be Used
C++ Singleton Design Pattern
Here is a condensed example of my code. Essentially it is just some state variables I need to keep track of and share across several threads. I know for a fact there will only ever needs be one state instance and I know that it needs to thread safe and constant. Will this implementation ensure that?
// System_State.h
class System_State
{
public:
static System_State &getInstance()
{
static System_State stateInstance;
return stateInstance;
}
System_State(System_State const&) = delete;
void operator=(System_State const&) = delete;
// Example Setter with mutex guard
void setWifiConnectedStatus(uint8_t _status)
{
std::lock_guard<std::mutex> lock(mtx);
Wifi.ConnectedStatus = _status;
}
private:
System_State()
{
initializeState();
}
void initializeState();
std::mutex mtx;
static struct WifiService
{
uint8_t ConnectedStatus;
std::string CurrentConnection;
std::string SavedNetworks;
std::string AvailableNetworks;
std::string ForgetNetwork;
std::string ConnectTo;
std::string DisconnectFrom;
} Wifi;
static struct ScanService
{
std::string PerformScan;
std::string ScanStatus;
uint8_t ScanProgress;
} Scan;
static struct UploadDataService
{
std::string PerformUpload;
uint8_t UploadProgress;
uint8_t ItemsWaitingToBeUploaded;
} UploadData;
static std::string LEDControlBrightness;
static uint8_t BatteryPercentage;
};
And here is some example main
//main.cpp
#include <thread>
void loop1()
{
System_State state = System_State::getInstance();
while(true)
{
//do randomness with state
}
}
void loop2()
{
System_State state2 = System_State::getInstance();
while(true)
{
//do randomness with state2
}
}
int main()
{
std::thread t1(loop1);
std::thread t2(loop2);
// do and join and all that!
return 0;
}

Answer 1

Yours is not idiomatic singleton, because it is still prone to so-called "static initialization order fiasco".
An idiomatic singleton doesn't have a static class member, instead it's instance function looks like
static MySingleton& instance() {
static MySingleton the_ton;
return the_ton;
}
More on the fiasco: static initialization order fiasco

Answer 2

No, this may not work in a more complex case, but not for thread-safety reason, but because of the undefined behavior of static object initialization.
Let's say that your object here:
static System_State stateInstance;
requires another static object that you retrieve with another static getInstance. You have no assurance that this will return a valid/constructed object.
If you can ensure that this object doesn't require another static object, then you should be fine. But this is the issue with lots of singleton constructions, you have to balance when it is constructed and thread-safety.