Compilación de C++ cuando dos clases se refieren entre sí

Question

Estoy tratando de escribir un envoltorio simple alrededor de un puntero de conexión que lo devolverá al grupo cuando se destruya el envoltorio, pero no se compilará porque ConnectionPool y AutoConn se necesitan mutuamente para ser declarado.

Intenté usar la deceleración hacia adelante pero no funcionó. ¿Cómo puedo solucionar esto? (Con g ++)

class Connection {}; 

class ConnectionPool 
{ 
    Connection *m_c; 
public: 
    AutoConn getConn() 
    { 
     return AutoConn(this, m_c); // by value 
    } 

    void releaseConnection(Connection *c) 
    { 
    } 
}; 

class AutoConn 
{ 
    ConnectionPool* m_pool; 
    Connection *m_connection; 
public: 
    AutoConn(ConnectionPool* pool, Connection *c) : m_pool(pool), m_connection(c) {} 
    ~AutoConn() 
    { 
     m_pool->releaseConnection(m_connection); 
    } 
}; 

Answer 1

Combinación de declaración directa y separación de declaración de la definición de miembros con trabajos de dependencias circulares. Por ejemplo:

class Connection {}; 
class ConnectionPool ; 

class AutoConn 
{ 

    ConnectionPool* m_pool; 
    Connection *m_connection; 
public: 
    AutoConn(ConnectionPool* pool, Connection *c) : m_pool(pool), m_connection(c) {} 
    ~AutoConn() ; // Not defined here because it accesses unknown members of class Connection 
} ; 

class ConnectionPool 
{ 
    Connection *m_c; 
public: 
    AutoConn getConn() 
    { 
     return AutoConn(this, m_c); // by value 
    } 

    void releaseConnection(Connection *c) 
    { 
    } 
}; 

// Definition of destructor with class Connection member dependencies. 
AutoConn::~AutoConn() 
{ 
    m_pool->releaseConnection(m_connection); 
} 

Answer 2

Uso declaración adelantada:

class Connection {}; 

class ConnectionPool; //<<<<<<<<<<<<<<<forward declaration 

class AutoConn { 
//definitions 
}; 

class ConnectionPool { 
//definitions 
}; 

Answer 3

implementar las funciones después del punto en el que las clases se definen

Answer 4

Es posible que desee externalizar la definición de todos los ConnectionPool y AutoConn métodos , es decir,

class ConnectionPool; 
class AutoConn {…}; 

class ConnectionPool {…}; 

AutoConn ConnectionPool::getConn() { 
    … 
} 

Answer 5

The co rrecta sintaxis de declaración a seguir es:

class Connection; // no {} 

Si se escribe

class Connection {}; 

estamos definiendo la clase, y no se puede definir una clase dos veces.

Además, ¿no deberías estar adelante declarando AutoConn, no Connection?

Answer 6

No incluye el archivo de encabezado ConnectionPool en AutoConn. Solo use una referencia directa como class ConnectionPool; en el archivo de encabezado AutoConn.

Answer 7

La declaración directa solo le dice al compilador que "tal clase existe". En su

AutoConn getConn() 

desde AutoConn es un tipo de valor, toda la estructura de AutoConn debe ser conocido, por lo que la declaración de avance de la clase no va a funcionar. Por lo tanto, debe colocar la declaración real de AutoConn antes de ConnectionPool.

En su AutoConn, el tipo ConnectionPool solo se menciona mediante punteros. En este caso, no se requiere la estructura completa de ConnectionPool, por lo que la declaración directa de ConnectionPool es suficiente.

lo tanto es necesario reordenamiento, las clases en esto:

class Connection; 
class ConnectionPool; 
class AutoConn { ... }; 
class ConnectionPool { ... }; 

notar que

AutoConn(ConnectionPool* pool, Connection *c) : m_pool(pool), m_connection(c) {} 
~AutoConn() 
{ 
    m_pool->releaseConnection(m_connection); 
} 

estos métodos requieren el compilador para conocer a los miembros de ConnectionPool, por lo que es necesaria una estructura completa. Para resolver este problema, la definición debe colocarse después de ConnectionPool. Por lo tanto, solo los constructores y destructores deberían permanecer.

class AutoConn { 
    ... 
    AutoConn(ConnectionPool* pool, Connection *c); 
    ~AutoConn(); 
} 
class ConnectionPool { ... }; 
AutoConn::AutoConn(ConnectionPool* pool, Connection *c) : ... { ... } 
AutoConn::~AutoConn() { ... }