W jaki sposób należy wcięcia nie robić konstruktorów listy inicjalizacji?

Question

Przykład: 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_; 
    } 
}; 

ja nie potrafię wymyślić sposób wcięcia rzeczy tak, że nie wygląda dziwnie ... a jednak jest to wspólny wzór. Jakie są typowe sposoby wcięcia tego?

Answer 1

Zawsze umieszczam puste klocki na jednej linii - tj. { } (zwróć uwagę na spację!).

Co więcej, zwykle umieszczam dwukropek i przecinki przed członami listy inicjalizacyjnej zamiast po niej - to sprawia, że ​​dodawanie członków później jest łatwiejsze.

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

Answer 2

To jak to zrobić:

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

Ale swoją drogą nie wygląda dla mnie dziwne.

Answer 3

Oto jak to zrobić

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

Google styl (conajmniej protobuf), byłoby:

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

Answer 4

Oto jak to zrobić, i dlaczego ja nic nie widzę źle z twoją próbką:

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

Jako Jeśli chodzi o mnie, zrób to po swojemu: tak długo, jak jesteś konsekwentny i spójny z projektem, nad którym pracujesz, nie ma to znaczenia w twoim stylu wcięcia.

Answer 5

Zalecam umieszczenie komentarza w pustym korpusie konstruktora, aby każdy czytający kod wiedział, że ma być pusty. W ten sposób mogą być pewni, że nie jest tak, że zapomniałeś wstawić tam kod.

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