¿Cómo debo sangrar para no hacer nada en los constructores de listas de inicialización?

Question

Ejemplo: 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_; 
    } 
}; 

Me parece que no puede llegar a una forma de sangría cosas para que no se ve extraño ... y sin embargo, este es un patrón común. ¿Cuáles son las formas más comunes de sangrar esto?

Answer 1

Siempre pongo bloques vacíos en una sola línea - es decir, { } (nótese el espacio!).

Además, pongo los dos puntos y las comas delante de los miembros de la lista de inicialización en lugar de después, esto hace que agregar miembros más tarde sea más fácil.

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

Answer 2

Esto es cómo lo hago:

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

Pero su camino no parece extraño para mí.

Answer 3

Así es como lo hago

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

Google estilo (al menos protobuf), sería:

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

Answer 4

Así es como lo hago, y por qué no veo nada mal con su muestra:

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

Como lo que a mí respecta, hacerlo a tu manera: Mientras usted está auto-consistente y coherente con el proyecto que está trabajando, no importa lo que su estilo de sangría es.

Answer 5

lo recomiendo poner un comentario en el cuerpo constructor vacío, sólo para que cualquiera que lea el código sabe que la intención es estar vacío. De esta forma, pueden estar seguros de que no se trata de que hayas olvidado insertar el código allí.

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