Wie übergeben Sie den Zeiger eines Klassenmembers in einer Elementinitialisierungsliste?

Question

Ich habe eine Klasse mit dem Namen HighWaterDetector:

class HighWaterDetector 
{ 
public: 
    HighWaterDetector(Device* device); 
    Device * devicePtr; 
    Output * output1Ptr; 
    CloudMsgParser * cloudMsgParserPtr; 
    Output output1; 
    NCD2Relay ncd2Relay; 
    CloudMsgParser cloudMsgParser; 
}; 

mit Konstruktor:

HighWaterDetector::HighWaterDetector(Device* device): ncd2Relay(), output1(1, &ncd2Relay){ 
} 

Ich versuche, eine Instanz von Output in der Initialisierungsliste für HighWaterDetector aber Output zu initialisieren, die Sie übergeben müssen Zeiger auf eine Instanz von NCD2Relay, die ebenfalls ein Mitglied der Klasse HighWaterDetector ist. Mein Programm stürzt im Ausgabekonstruktor ab. Ist das der falsche Weg, dies zu tun? Was mache ich falsch?

Ausgabeklasse:

class Output 
{ 
public: 
    Output(ushort relayNum, NCD2Relay* ncd2RelayPtr); 
    ushort relayNum; 
    OutputStatus outputStatus; 
    int setOutputOn(void); 
    int setOutputOff(void); 
    void process(void); 
    NCD2Relay* ncd2RelayPtr; 
}; 


//Output Constructor 
Output::Output(ushort relayNum, NCD2Relay* ncd2RelayPtr) { 
    this->relayNum = relayNum; 
    this->ncd2RelayPtr = ncd2RelayPtr; //DOESNT CRASH IF I COMMENT THIS OUT 
    this->outputStatus.outFail = 0; 
    Serial.print("Initializing output "); 
    Serial.println(this->relayNum); 
    this->setOutputOff(); 
} 

Answer 1

in C++ die Initialisiererliste nicht der Reihenfolge folgen, in dem Sie die Elemente in dem initializer schreiben. Es folgt vielmehr der Reihenfolge, in der die Mitglieder in Ihrer Klasse deklariert sind. Da Sie Ihre NCD2Relay nach Ihrer Output innerhalb der Klasse setzen, wird NCD2Relaynach Ausgang initialisiert, obwohl Sie in der Initialisierung NCD2Relay zuerst setzen. Also, bewegen Sie einfach NCD2Relayvor die Output innerhalb Ihrer Klassendeklaration.

Answer 2

Haben Sie die Warnungen Ihres Compilers beachtet? oder sind sie maximal eingeschaltet? Die Reihenfolge der Deklaration Ihrer Mitglieder kann die Ursache sein:

class HighWaterDetector 
{ 
public: 
    HighWaterDetector(Device* device); 
    Device * devicePtr; 
    Output * output1Ptr; 
    CloudMsgParser * cloudMsgParserPtr; 
    Output output1;      // <- This is constructed before 
    NCD2Relay ncd2Relay;    // <- This... 
    CloudMsgParser cloudMsgParser; 
}; 

aber Konstruktor geht so:

HighWaterDetector::HighWaterDetector(Device* device): ncd2Relay(), output1(1, &ncd2Relay){ ... } 

In dem obigen Zusammenhang die Adresse ncd2Relay im Konstruktor von output1 Verwendung einfach ist Verwenden eines Zeigers auf ein nicht initialisiertes Objekt, bei dem es sich um ein nicht definiertes Verhalten handelt, wenn Sie es vor der Konstruktion dereferenzieren. Daher Sie müssen die Bestellung in Ihrer Klassendefinition erzwingen ...

gerade aus dem C++ Standard zitieren: (Hervorhebung von mir) [class.base.init/13]

In einem nicht delegieren Konstruktor Initialisierung erfolgt in der folgende Reihenfolge:

• zuerst und nur für den Konstruktor der meisten abgeleiteten Klasse ([intro.object]), werden in der Reihenfolge virtuelle Basisklassen initialisiert sie auf einem tiefen ersten links erscheinen -Richtiges Traversal des gerichteten azyklischen Graph der Basisklassen, wobei "von links nach rechts" die Reihenfolge Aussehen der Basisklassen in der abgeleiteten Klasse Base-Specifier-Liste ist.

• Dann werden direkte Basisklassen in der Deklarationsreihenfolge initialisiert, wie sie in der Basisspezifiziererliste erscheinen (unabhängig von der Reihenfolge der mem-Initialisierer).

• Dann nicht-statische Datenelemente werden in der Reihenfolge initialisiert sie in der Klassendefinition deklariert wurden (wieder unabhängig von der Reihenfolge des mem-initializers).

• Schließlich wird die Compound-Anweisung des Konstruktors ausgeführt.