我只是在编写一个小的OOP应用程序,并通过setter在设置类的私有字符串变量上运行(而不是编译)该应用程序时崩溃,这是头文件:

class Car
{
private:
int year;
std::string brand;
std::string model;
int price;
std::string currency;
public:
int setYear(int x){this->year = x;}
std::string setBrand(std::string x){this->brand = x;}
std::string setModel(std::string x){this->model = x;}
int setPrice(int x){this->price = x;};
std::string setCurrency(std::string x){this->currency = x;}
};


这是主要的:
n-对象数
temp-用于传递整数的临时变量
temp1-用于传递字符串的临时变量

ifstream fd("input.in");
int n;
fd >> n;
int temp;
string temp1;
Car A[n];
for(int i = 0; i < 3; i++)
{
    fd >> temp;
    A[i].setYear(temp);
    fd >> temp1;
    A[i].setBrand(temp1);  //Crashes Here
    fd >> temp1;
    A[i].setModel(temp1);
    fd >> temp;
    A[i].setPrice(temp);
    fd >> temp1;
    A[i].setCurrency(temp1);
}


经过小小的测试,我发现它崩溃了,然后代码尝试设置“ brand”变量。有什么问题?

最佳答案

数组尺寸必须在编译时已知,因此:

C A[n];


是错的。

GCC支持将可变长度数组作为非标准扩展,但是即使您不小心使用了可变长度数组,您的循环也会假设n == 3,而没有明显迹象表明这一定是正确的。

而是使用向量:

std::vector<C> A(n);


并对其进行适当的迭代:

std::vector<C>::iterator it = A.begin(), end = A.end();
for ( ; it != end; ++it) {
   // your for loop stuff with *it
}


或者,在C ++ 11中:

for (auto& a : A) {
   // your for loop stuff with a
}

关于c++ - 代码在设置类变量时崩溃,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/14102964/

10-11 23:02
查看更多