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

Aparte de lo comentado por @Dolmenes...

void eliminarModelo(int pos)
{
    char* x = new char[50];
    x = arrCarros[pos]->getModelo();
    for (int i = 0;i< arrCarros.size(); i++)
    {
        if (/* comparacion del modelo */)
        {
            arrCarros.erase(begin(arrCarros) + i);
        }
    }
}

Cada vez que se llama a arrCarros.erase se elimina un elemento del vector... y todos los siguientes se desplazan una posición... pero esta última circunstancia no la tienes en cuenta, es decir, si entienes el siguiente vector:

A A B

Sucedería lo siguiente:

  • Encuentras A y eliminas el elemento de la posición 0, queda: A B
  • Compruebas el elemento de la posicion 1, B, lo ignoras. El vector final será A B

Lo que tienes que hacer es evitar el incremento si sacas un elemento del vector:

for (int i = 0;i< arrCarros.size(); i++)
{
    if (/* comparacion del modelo */)
    {
        arrCarros.erase(begin(arrCarros) + i);
        i--;
    }
}

Aparte de lo comentado por @Dolmenes...

void eliminarModelo(int pos)
{
    char* x = new char[50];
    x = arrCarros[pos]->getModelo();
    for (int i = 0;i< arrCarros.size(); i++)
    {
        if (/* comparacion del modelo */)
        {
            arrCarros.erase(begin(arrCarros) + i);
        }
    }
}

Cada vez que se llama a arrCarros.erase se elimina un elemento del vector... y todos los siguientes se desplazan una posición... pero esta última circunstancia no la tienes en cuenta, es decir, si en el siguiente vector:

A A B

Sucedería lo siguiente:

  • Encuentras A y eliminas el elemento de la posición 0, queda: A B
  • Compruebas el elemento de la posicion 1, B, lo ignoras. El vector final será A B

Lo que tienes que hacer es evitar el incremento si sacas un elemento del vector:

for (int i = 0;i< arrCarros.size(); i++)
{
    if (/* comparacion del modelo */)
    {
        arrCarros.erase(begin(arrCarros) + i);
        i--;
    }
}

Aparte de lo comentado por @Dolmenes...

void eliminarModelo(int pos)
{
    char* x = new char[50];
    x = arrCarros[pos]->getModelo();
    for (int i = 0;i< arrCarros.size(); i++)
    {
        if (/* comparacion del modelo */)
        {
            arrCarros.erase(begin(arrCarros) + i);
        }
    }
}

Cada vez que se llama a arrCarros.erase se elimina un elemento del vector... y todos los siguientes se desplazan una posición... pero esta última circunstancia no la tienes en cuenta, es decir, si tienes el siguiente vector:

A A B

Sucedería lo siguiente:

  • Encuentras A y eliminas el elemento de la posición 0, queda: A B
  • Compruebas el elemento de la posicion 1, B, lo ignoras. El vector final será A B

Lo que tienes que hacer es evitar el incremento si sacas un elemento del vector:

for (int i = 0;i< arrCarros.size(); i++)
{
    if (/* comparacion del modelo */)
    {
        arrCarros.erase(begin(arrCarros) + i);
        i--;
    }
}
Origen Enlace
eferion
  • 52.6k
  • 5
  • 35
  • 75

Aparte de lo comentado por @Dolmenes...

void eliminarModelo(int pos)
{
    char* x = new char[50];
    x = arrCarros[pos]->getModelo();
    for (int i = 0;i< arrCarros.size(); i++)
    {
        if (/* comparacion del modelo */)
        {
            arrCarros.erase(begin(arrCarros) + i);
        }
    }
}

Cada vez que se llama a arrCarros.erase se elimina un elemento del vector... y todos los siguientes se desplazan una posición... pero esta última circunstancia no la tienes en cuenta, es decir, si en el siguiente vector:

A A B

Sucedería lo siguiente:

  • Encuentras A y eliminas el elemento de la posición 0, queda: A B
  • Compruebas el elemento de la posicion 1, B, lo ignoras. El vector final será A B

Lo que tienes que hacer es evitar el incremento si sacas un elemento del vector:

for (int i = 0;i< arrCarros.size(); i++)
{
    if (/* comparacion del modelo */)
    {
        arrCarros.erase(begin(arrCarros) + i);
        i--;
    }
}