vector iteradores de reparto

Question

Hey, en C++, I tienen un vector de tipo:

vector<BaseClass*> myVector; 

En la que, I inserto (push_back) punteros de clases derivadas en ella.

Ahora, yo quiero a pop volver sus elementos por lo que hago esto:

vector<ADlgcDev*>::iterator iter; 

for (iter = myVector.rbegin(); iter != myVector.rend(); iter++) 
{ 
// but before I pop it, I need to shutdown it down 
// so I cast this 
// but this way, I'm unable to call the function 
(DerivedClass*(*iter))->Shutdown(); 

myVector.pop_back(); 
} 

sino como una mención en los comentarios antes de que el pop, necesito llamar a su apagado() método y el reparto no funciona correctamente también ¿Alguna resolución? o es imposible?

Answer 1

while (!myVector.empty()) 
{ 
    ((DerivedClass*)(myVector.back()))->Shutdown(); 
    myVector.pop_back(); 
} 

Notas:

• Probablemente debería utilizar dynamic_cast en lugar de la fundición dura. (Si está seguro de que solo hay DerivedClass objetos en el vector, ¿por qué no es std::vector<DerivedClass>?)
• Probablemente no deba tener que lanzar nada, ya que Shutdown() debe declararse en la clase base.
• Probablemente también deba eliminar los objetos, antes de que se salgan del vector. (Pero puede que no sea así.)
• Probablemente deba utilizar un puntero inteligente que llame al Shutdown() (y delete, probablemente).

Editar: Usando std::vector<T>::clear(), as shown by markh44 es probablemente mejor que la pop_back().

Answer 2

La conversión del constructor no funciona para los punteros. Utiliza static_cast si estás seguro o dynamic_cast y comprueba.

Answer 3

Si Shutdown() es un método virtual de la clase Base, es decir, BaseClass :: ShutDown() debe llamar directamente a iter-> ShutDown();

De lo contrario, si el método no es virtual, debe usar dynamic_cast.

vector<ADlgcDev*>::iterator iter; 

for (iter = myVector.rbegin(); iter != myVector.end(); iter++) 
{ 
DerivedClassA* a = dynamic_cast<DerivedClassA*>(*iter) ; 
if (a) a->ShutDownA(); 
else 
{ 
DerivedClassB* b = dynamic_cast<DerivedClassB*>(*iter); 
if (b) b->ShutDownB(); 
// ... repeat for every class in hierarchy that might be in the vector. 
} 
myVector.pop_back(); 
} 

De todos modos es probable que se escapa de la memoria, a no ser apagado() elimina el objeto de la misma (que es generalmente una mala idea) o que está manteniendo punteros duplicados y eliminarlos en otro lugar, lo que es otra idea arriesgada.

Answer 4

¿Podría hacer de Shutdown una función virtual en BaseClass? Entonces no necesitarías un elenco.

También es probable que tenga problemas para eliminar elementos de un vector mientras itera. Yo lo haría así:

vector<BaseClass*>::iterator iter; 

for (iter = myVector.rbegin(); iter != myVector.rend(); iter++) 
{ 
    (*iter)->Shutdown(); 
} 
myVector.clear(); 

Editar: y otra cosa, ++ iter se prefiere generalmente durante ++ ITER.