Skip to main content
se añadieron 644 caracteres en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld) {
    terrorists = pTerrorists;
//  ^^^^^^^^^^

¿Qué es terrorists? ¿No debería ser pTerrorists? Fíjate que en mi respuesta yo te he dicho que hicieses esto:

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld)
  : pterrorists(pTerrorists)
{ }

Cuya alternativa sería:

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld) {
    this->pTerrorists = pTerrorists;

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld) {
    terrorists = pTerrorists;
//  ^^^^^^^^^^

¿Qué es terrorists? ¿No debería ser pTerrorists? Fíjate que en mi respuesta yo te he dicho que hicieses esto:

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld)
  : pterrorists(pTerrorists)
{ }

Cuya alternativa sería:

IA::IA(std::vector<Terrorist*> &pTerrorists, RigidBody *pPlayer, OgreBulletDynamics::DynamicsWorld *pWorld) {
    this->pTerrorists = pTerrorists;
se añadieron 45 caracteres en el cuerpo
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75
class IA
{
  std::vector<Terrorists*>vector<Terrorists*>& terrorists;
  //                      ^ Importante!!!!!

public:
  IA(std::vector<Terrorists*>& terrorists, /* ... */ )
    : terrorists{terrorists}
  { }

  void BorrarTerrorista()
  {
    // Algoritmo para elegir el terrorista a eliminar
    Terrorist* t = /* ... */

    auto it = std::find(terrorists.begin(),terrorists.end(),t);
    if( it != terrorists.end() )
    {
      delete *it;
      terrorists.erase(it);
      // Lanzar avisos
    }
  }  
};
class IA
{
  std::vector<Terrorists*> terrorists;

public:
  IA(std::vector<Terrorists*>& terrorists, /* ... */ )
    : terrorists{terrorists}
  { }

  void BorrarTerrorista()
  {
    // Algoritmo para elegir el terrorista a eliminar
    Terrorist* t = /* ... */

    auto it = std::find(terrorists.begin(),terrorists.end(),t);
    if( it != terrorists.end() )
    {
      delete *it;
      terrorists.erase(it);
      // Lanzar avisos
    }
  }  
};
class IA
{
  std::vector<Terrorists*>& terrorists;
  //                      ^ Importante!!!!!

public:
  IA(std::vector<Terrorists*>& terrorists, /* ... */ )
    : terrorists{terrorists}
  { }

  void BorrarTerrorista()
  {
    // Algoritmo para elegir el terrorista a eliminar
    Terrorist* t = /* ... */

    auto it = std::find(terrorists.begin(),terrorists.end(),t);
    if( it != terrorists.end() )
    {
      delete *it;
      terrorists.erase(it);
      // Lanzar avisos
    }
  }  
};
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

La IA puede, en vez de recibir una copia del vector, recibir una referencia a la PlayState:

class IA
{
  PlayState& ps;

public:

  IA(PlayState& ps, /* ... */ )
    : ps{ps}
  {
  }

  void BorrarTerrorista()
  {
    if( !ps.terrorists.empty() )
    {
      delete ps.terrorists.back();
      ps.terrorists.pop_back();
    }
  }
};

También puede suceder que no desees que terrorists sea un vector de acceso público. En cuyo caso necesitas una función que permita eliminar el elemento deseado. Esta solución permite lanzar eventos cada vez que cae un terrorista

class PlayState
{
  std::vector<Terrorist*> terrorists;

public:

  void NuevoTerrorista(Terrorist* t)
  {
    terrorists.push_back(t);
    
    // Avisamos de que hay un terrorista
  }

  void BorrarTerrorista(Terrorist* t)
  {
    auto it = std::find(terrorists.begin(),terrorists.end(),t);
    if( it != terrorists.end() )
    {
      delete *it;
      terrorists.erase(it);

      // Avisamos de que hay un terrorista menos
    }
  }
};

class IA
{
  PlayState& ps;

public:

  IA(PlayState& ps, /* ... */ )
    : ps{ps}
  {
  }

  void BorrarTerrorista()
  {
    // Algoritmo para elegir el terrorista a eliminar
    Terrorist* t = /* ... */

    ps.BorrarTerrorista(t);
  }
};

Por supuesto siempre puedes pasarle una referencia del vector... pero entonces los avisos para que el resto de componentes funcionen correctamente corre por cuenta de IA:

class IA
{
  std::vector<Terrorists*> terrorists;

public:
  IA(std::vector<Terrorists*>& terrorists, /* ... */ )
    : terrorists{terrorists}
  { }

  void BorrarTerrorista()
  {
    // Algoritmo para elegir el terrorista a eliminar
    Terrorist* t = /* ... */

    auto it = std::find(terrorists.begin(),terrorists.end(),t);
    if( it != terrorists.end() )
    {
      delete *it;
      terrorists.erase(it);
      // Lanzar avisos
    }
  }  
};