How to initialize a static member object?

Question

I didn't know that I didn't know this :) . and a similar question here didn't help much.

So here i am asking. Please Consider the following class:

//in Agent.h class Agent : public ns3::Object{ private: //... static BaseWifi m_wifi; //... };

is this :

//Agent.cpp BaseWifi temp; BaseWifi Agent::m_wifi = temp;

very much different from this:

//Agent.cpp BaseWifi Agent::m_wifi = BaseWifi();

The second approach doesn't work for me. why and how?

I don't want to trouble you with more codes coz I faced this problem deep in my program. The program generate seg faults as things(members) inside BaseWifi's constructor are not initialized correctly. when those uninitialized internal members are used, seg faults occur.

Thank you in advance for your kind comments and answers.

p.s.: In fact I found this issue when I hadn't yet initialized this static member and I was deleting an extra line :

BaseWifi temp;

in my main(), which added more to my confusion!!!(this one could be dependent on what I put in BaseWifi's constructor, so dont mind it for now)

Update-1: For those who would like to see the BaseWifi:

class BaseWifi { ns3::WifiHelper m_wifiHelper; // a wifi helper apply to setup vehicles Wifi ns3::NqosWifiMacHelper m_wifiMacHelper; // a wifi mac helper apply to setup vehicles Wifi ns3::YansWifiPhyHelper m_wifiPhyHelper; // a wifi phy helper apply to setup vehicles Wifi std::string m_phyMode; double m_rss; // -dBm bool m_init_done; public: BaseWifi(); virtual void init(); ns3::NetDeviceContainer Install(ns3::NodeContainer &c); virtual ~BaseWifi(); }; BaseWifi::BaseWifi() { m_init_done = false; m_rss = -80; m_phyMode ="DsssRate1Mbps"; // TODO Auto-generated constructor stub init(); } void BaseWifi::init() { NS_LOG_UNCOND("inside BaseWifi::init()"); m_wifiHelper.SetStandard (ns3::WIFI_PHY_STANDARD_80211b); m_wifiPhyHelper = ns3::YansWifiPhyHelper::Default (); // This is one parameter that matters when using FixedRssLossModel // set it to zero; otherwise, gain will be added m_wifiPhyHelper.Set ("RxGain", ns3::DoubleValue (0) ); // ns-3 supports RadioTap and Prism tracing extensions for 802.11b m_wifiPhyHelper.SetPcapDataLinkType (ns3::YansWifiPhyHelper::DLT_IEEE802_11_RADIO); ns3::YansWifiChannelHelper wifiChannel; wifiChannel.SetPropagationDelay ("ns3::ConstantSpeedPropagationDelayModel"); // The below FixedRssLossModel will cause the rss to be fixed regardless // of the distance between the two stations, and the transmit power wifiChannel.AddPropagationLoss ("ns3::FixedRssLossModel","Rss",ns3::DoubleValue (m_rss)); m_wifiPhyHelper.SetChannel (wifiChannel.Create ()); // Add a non-QoS upper mac, and disable rate control m_wifiMacHelper = ns3::NqosWifiMacHelper::Default (); m_wifiHelper.SetRemoteStationManager ("ns3::ConstantRateWifiManager", "DataMode",ns3::StringValue (m_phyMode), "ControlMode",ns3::StringValue (m_phyMode)); // Set it to adhoc mode m_wifiMacHelper.SetType ("ns3::AdhocWifiMac"); m_init_done = true; } //Install the class's embedded settings on the nodes in this node container. ns3::NetDeviceContainer BaseWifi::Install(ns3::NodeContainer &nc) { return m_wifiHelper.Install(m_wifiPhyHelper, m_wifiMacHelper, nc); }

For the second aprproach, do you get compilation errors? Mind telling us what those errors are in that case? Because normally that would be fine. — Some programmer dude
– Some programmer dude, Commented Aug 13, 2013 at 11:43
@JoachimPileborg no compilation error. it generate seg faults as things(members) inside BaseWifi's constructor are not initialized correctly. when those internal members are used, seg faults occur. — rahman
– rahman, Commented Aug 13, 2013 at 11:45
If you have pointers, or really any kind of complex data, you should follow the rule of three. — Some programmer dude
– Some programmer dude, Commented Aug 13, 2013 at 11:53
If you don't provide a copy-constructor and does a deep copy, the compiler will generate only shallow copying, which means that for pointers only the pointers will be copied and not what they point to. That means that you then have multiple pointers pointing to the same data, and if one object then deletes that data the other pointers will now point to deleted data. — Some programmer dude
– Some programmer dude, Commented Aug 13, 2013 at 11:55

Benjamin Leinweber · Accepted Answer · 2017-10-20 16:00:06Z

I have run into these kind of issues before. Apparently the initialization of static member objects very much depends on where the implementation is done in your code and (probably) on how the whole thing is compiled. The solution that I found (somewhere) to the problem was to wrap the whole thing into a static member function like this:

//in Agent.h class Agent : public ns3::Object{ private: //... static BaseWifi& m_wifi(); //... };

and:

//in Agent.cpp BaseWifi& Agent::m_wifi() { static BaseWifi TheObject=BaseWifi(); return TheObject; }

This way the object is initialized properly the first time the static member function is called.

This is a nice "workaround", though in many instances in C++ such workarounds are the only SOLUTION. Thanks
Still, I will try the suggestions commented below my question regarding copy constructors and will get back to you guys tomorrow.

JackyZhu · Accepted Answer · 2013-08-13 13:01:02Z

The difference here is the first approach is using copy constructor to initialize the object while the second is using the default constructor.

pmw1234 · Accepted Answer · 2024-01-25 15:54:08Z

Since C++11 a static member can be declared and initialized inside a class with the inline keyword.

Class X { inline static int n = 1; };

This is both a declaration and a definition with initialization.

Collectives™ on Stack Overflow

How to initialize a static member object?

3 Answers 3

2 Comments

Comments

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

Comments

Linked

Related