Сегментация Fault 11 в C++ при доступе к второму элементу в массиве

Question

Короче говоря, у меня есть программа на C++, состоящая из класса Car с подклассами SportsCar и SUV, а также CarInventory, которая хранит объекты Car.

Answer 1

Я думаю, что проблема в функции вставки

void CarInventory::insert(Car *car) { 
    for (int i = 0; i < inventorySize; i++) { 
     if (carArray[i]) { 
      i++; 
     } 

Когда вы найдете Автомобиль уже в вашем carArray Increase i, но тогда вы увеличиваете значение i снова в for-loop. Таким образом, вы пропустите любую другую позицию в массиве, и они останутся неинициализированными.

Answer 2

Вы назначили массив из 10 автомобилей. Но вы только инициализировали его 4 автомобилями.

Так что, когда ваша петля в displayVehichles обойдется, чтобы получить доступ к carArray[5], она, скорее всего, получит доступ к неинициализированной памяти.

Это утверждение кажется хлопотным:

carArray = new Car *[maxStock]; 

Это, вероятно, следует:

carArray = new Car *[maxStock]; 
for (int i = 0; i < maxStock; i++) 
{ 
    carArray[i] = NULL; 
} 
totalNumCars = 0; 

Таким образом, ваш метод вставки будет вести себя правильно. Но ваш метод вставки может быть гораздо проще:

void CarInventory::insert(Car *car) { 

    if (totalNumCars < maxStock) 
     carArray[totalNumCars] = car; 
     totalNumCars++; 
    } 
} 

Далее, в то время как это нормально так, как вы его автомобилей вы заявили выходить за рамки, прежде чем ваш CarInventory выходит из области видимости, это будет ссылки объектов автомобилей, которые уже удалены.

Использование станд :: вектор держать свои автомобили:

class CarInventory 
{ 
    std::vector<Car*> _cars; 
public: 
    void displayVehicles() 
    { 
     for (auto i = _cars.begin(); i != _cars.end(); i++) 
     { 
      i->printInfo(); 
     } 
    } 

    void insert(Car* car) 
    { 
     _cars.push_back(car); 
    } 
}; 

Теперь, по-прежнему не решает проблему своего класса CarInventory, держась указатели объектов стека.

Это даже лучше:

class CarInventory 
{ 
    std::vector<std::shared_ptr<Car>> _cars; 
public: 
    void displayVehicles() 
    { 
     for (auto i = _cars.begin(); i != _cars.end(); i++) 
     { 
      i->printInfo(); 
     } 
    } 

    void insert(std::shared_ptr<Car>& spCar) 
    { 
     _cars.push_back(spCar); 
    } 
}; 

Тогда ваш код, чтобы использовать класс:

std::shared_ptr<Car*> createCar(const char* vin, const char* make, const char* color, int year) 
{ 
    Car* car = new Car(vin, make, color, year); 
    return std::shared_ptr<Car>(car); 
} 

int main(int argn, char *argv[]) 
{ 
    CarInventory cars; 

    std::shared_ptr<Car*> toyota = createCar("2GCGC34M9F1152828", "Toyota", "Camry", "Green", 2012); 

    std::shared_ptr<Car*> honda = createCar("1C4BJWAG4DL602733", "Honda", "Civic", "Blue", 2015); 

    ... 

    CarInventory cars; 
    cars.insert(toyota); 
    cars.insert(honda); 
    cout << "\n"; 
    cars.displayVehicles(); 
}