C++动态数组访问冲突

Question

****对不起原始文章中有关numCars的混淆。我修改了代码以与原始代码保持一致

以下学术程序是原始问题的简化版本，但它着重解决了我尚未解决的问题。这个问题有两个类和一个主要方法，这两个类由Dealer类和Car类组成。 Dealer类有一个私人Car *指针，它在Dealer的构造函数中被初始化为一个动态数组。经销商的addCar方法被调用时，主方法发生错误。 在主要方法中，我故意将Dealer变量传递给addCar（Dealer & d）方法来模拟原始应用程序的结构。 addCar方法然后调用经销商的addCar（const Car & car）方法，当我执行cars [numCars ++] = car;你能解释为什么汽车[numCars ++] =车导致访问冲突

/**********************************Dealer.h**************************/ 
#include <cstdlib> 
#include "Car.h" 

using namespace std; 

class Dealer 
{ 
    public: 
     Dealer(int maxCars = DEFAULT_MAX_CARS) 

：numCars（0） {汽车=新车[maxCars];}

 ~Dealer(){delete [] cars;} 

     int getTotalCars() const { return numCars;} 

     void addCar(const Car& car) 
     {  
      cars[numCars++] = car; // Access Violation 
     } 

     Car* begin(){return cars;}; 

     Car* end(){ return cars + numCars;} 

setNumCars（诠释计数） {numCars =计数;}

private: 
     static const int DEFAULT_MAX_CARS = 10; 
     Car* cars; 
     int numCars; 
}; 

/**********************************Car.h**********************/ 
#include <cstdlib> 
#include <string> 

using namespace std; 


class Car{ 
    public: 

     Car() 
      : year(0), make(""), model("") 
     {} 

     Car(int year, string make, string model) 
      : year(year), make(make), model(model) 
     {}  

     string getMake() const {return make;} 
     void setMake(string make){this->make=make;} 

     string getModel() const {return model;} 
     void setModel(string model){this->model=model;} 

     int getYear() const {return year;} 
     void setYear(int year){this->year=year;} 

    private: 
     int year; 
     string make; 
     string model;  
}; 


ostream& operator<< (ostream& out, const Car& car) 
{ 
    out << car.getYear() << " " << car.getMake() << " " << car.getModel(); 
    return out; 
} 

/**********************************Main.cpp**********************/ 
#include &lt;cstdlib&gt; 
#include &lt;iostream&gt; 
#include "Dealer.h" 

using namespace std; 

void addCar(Dealer& d); 

int main(int argc, char *argv[]) 
{ 
    Dealer d; 

    addCar(d); 

    system("PAUSE"); 
    return EXIT_SUCCESS; 
} 

void addCar(Dealer& d) 
{ 
    d = Dealer(); 

    d.addCar(Car(2007, "Honda", "Civic")); 

    cout << d.getTotalCars() << " total cars" << endl; 
} 

Answer 1

void addCar(const Car& car) 
{ 
    cars[numCars++] = car; // Access Violation 
} 

你永远不会初始化numCars - 它包含这堆一定的价值几乎肯定不是零。这会导致您读取超出汽车阵列的尾部并进入无法访问的内存。您应该在您的构造函数中将numCars设置为0。

最重要的是，您应该在addCar中进行一些检查，以便不超出汽车数组。

编辑：

没有与代码中的一些其他问题 - 例如， “d =经销商（）;”创建一个新的经销商并覆盖您通过引用传递给addCars的那个，这似乎不是您想要做的。

尝试向构造函数/析构函数添加一些额外的跟踪来验证您认为被调用的构造函数实际上是 - Dealer（）应该使用您指定的默认参数调用构造函数，但如果不是获取默认的构造函数。

Answer 2

你没有初始化numCars的任何地方，你应该把它设置为0：

Dealer(int maxCars = DEFAULT_MAX_CARS) : 
numCars(0) 
{ 
    cars = new Car[maxCars]; 
} 

你有使用原始指针？为什么不把它包起来并用std::vector代替？

Answer 3

上述代码中没有任何内容初始化Dealer :: numCars。因此它可以是任何垃圾。

Answer 4

也许我没有看到它，但你最初设置numCars在哪里？

Answer 5

这看起来像一个内存泄露到我，因为，你不释放被汽车举行指针以前的记忆：

setNumCars(0) {cars = new Car[maxCars];} 

这个代码应该真正要防溢出条件：

void addCar(const Car& car)   
{         
    cars[numCars++] = car; // Access Violation  ' 
} 

做这样的事情：

void addCar(const Car& car)   
{         
    if (numCars < maxCars) 
     cars[numCars++] = car;  ' 
    else 
     // throw and exception ..... 
     // or better still grow the cars buffer 
} 

Answer 6

cars[numCars++] = car; // Access Violation 

我从发布的代码中看不到任何问题。可能是其他地方的问题？

也许你可以尝试以下操作：

• 改变阵列载体，尝试使用在（）来捕捉out_of_range例外。 类似：

  std::vector<int> myVec; 
      try 
      { 
      int x = myVec.at(0); 
  
      } 
      catch(std::out_of_range& oor) 
      { 
       printf("\nout of range "); 
      }