Wie sollte ich einrücken tun nichts Initialisierung Liste Konstruktoren?

Question

Beispiel: Thread::Thread:

class Thread 
{ 
    Process * parent_; 
    unsigned __int32 id_; 
    void * entryPoint_; 
public: 
    Thread(Process * parent, unsigned __int32 id, void * entryPoint) : 
     parent_(parent), 
     id_(id), 
     entryPoint_(entryPoint) 
    { 
    } 
    unsigned __int32 GetId() const 
    { 
     return id_; 
    } 
    void * GetEntryPointAddress() const 
    { 
     return entryPoint_; 
    } 
}; 

Ich kann nicht mit einem Weg zu kommen einzurücken Dinge scheint so dass es nicht seltsam aussieht ... und doch ist dies ein weit verbreitetes Muster. Was sind gängige Methoden, um dies einzudämmen? Diese

Answer 1

Ich lege immer leere Blöcke auf einer einzigen Zeile - d. H. { } (beachten Sie den Platz!).

Darüber hinaus habe ich in der Regel den Doppelpunkt und Kommas vor die Initialisierungsliste Mitglieder statt nach - das macht das Hinzufügen von Mitgliedern später leichter.

Thread(Process * parent, unsigned __int32 id, void * entryPoint) 
    : parent_(parent) 
    , id_(id) 
    , entryPoint_(entryPoint) 
{ } 

Answer 2

ist, wie ich es tun:

Thread(Process * parent, unsigned __int32 id, void * entryPoint) 
    :parent_(parent), 
    id_(id), 
    entryPoint_(entryPoint) 
{} 

Aber Ihre Art und Weise sieht nicht mir fremd.

Answer 3

Hier ist, wie ich es tun

public: 
    Thread(Process * parent, unsigned __int32 id, void * entryPoint) : 
    parent_(parent), 
    id_(id), 
    entryPoint_(entryPoint) 
    { } 

Google-Stil (atleast protobuf), wäre:

public: 
Thread(Process * parent, 
     unsigned __int32 id, 
     void * entryPoint) 
    : parent_(parent), 
    id_(id), 
    entryPoint_(entryPoint) { } 

Answer 4

Hier ist, wie ich es tun, und warum ich sehe nichts falsch mit der Probe:

Thread(Process * parent, unsigned __int32 id, void * entryPoint) : 
     parent_(parent), 
     id_(id), 
     entryPoint_(entryPoint) { } 

Soweit es mich betrifft, tun Sie es auf Ihre Art und Weise: Solange Sie selbstkonsistent und konsistent mit dem Projekt, an dem Sie arbeiten, ist es nicht Angelegenheit, was Ihr Eindruck Stil ist.

Answer 5

Ich würde empfehlen, einen Kommentar in den leeren Konstruktor-Körper zu setzen, damit jeder, der den Code liest, weiß, dass Sie ihn leer haben wollten. Auf diese Weise können Sie sicher sein, dass Sie nicht vergessen haben, dort Code einzufügen.

Thread(Process * parent, unsigned __int32 id, void * entryPoint) : 
    parent_(parent), 
    id_(id), 
    entryPoint_(entryPoint) 
{ 
    // empty 
}