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--;
}
}