Question

我正在尝试编写一个名为student的类，其中包含一个paramaterized和一个默认构造函数，并且paramaterized版本工作正常，但是在我手动分配值后尝试运行默认构造函数时控制台崩溃。 p>

Student.cpp：

Student::Student()
{
    this -> firstName = firstName;
    this -> lastName = lastName;
    this -> maxGrades = maxGrades;
    grades[maxGrades];
}

Student::Student(string fName, string lName, int mGrades)
{
    firstName = fName;
    lastName = lName;
    maxGrades = mGrades;
    grades[maxGrades];
}

Student::~Student()
{

}

void Student::setFirstName(string fName)
{
    firstName = fName;
}

void Student::setLastName(string lName)
{
    lastName = lName;
}

void Student::setMaxGrades(int mGrades)
{
    mGrades = maxGrades;
    grades[maxGrades];
}

string Student::getFirstName()
{
    return firstName;
}

string Student::getLastName()
{
    return lastName;
}

void Student::addGrade(int currentGradeNumber, double addedGrade)
{
    if(currentGradeNumber < maxGrades)
    {
        grades[currentGradeNumber] = addedGrade;
        cout << "grade " << currentGradeNumber << "is " << grades[currentGradeNumber] << endl;
    }
}

double Student::calcAvg()
{
    double sum = 0;
    double avg = 0;
    for(int i=0; i < maxGrades;i++)
    {
        sum += grades[i];
    }
    avg = sum/maxGrades;
    return avg;
}

studentTest.cpp：

int main()
{
    Student student1("Bill", "Nye", 3);
    cout << "First Name: " << student1.getFirstName() << endl;
    cout << "LastName: " << student1.getLastName() << endl;
    student1.addGrade(0, 90);
    student1.addGrade(1, 95);
    student1.addGrade(2, 80);
    cout << "Average is " << student1.calcAvg() << endl;

    Student student2;
    student2.setMaxGrades(2);
    student2.setFirstName("Frank");
    student2.setLastName("West");
    cout << "\nFirst Name: " << student2.getFirstName() << endl;
    cout << "Last Name: " << student2.getLastName() << endl;
    student2.addGrade(0,50);
    student2.addGrade(1,100);
    cout << "Average is: " << student2.calcAvg();
    return 0;
}

student.h：

class Student
{
    private:
        string firstName;
        string lastName;
        int maxGrades;
        int numGrades;
        double grades[];
    public:
        Student();
        Student(string, string, int);
        ~Student();
        void setFirstName(string);
        void setLastName(string);
        string getFirstName();
        string getLastName();
        void addGrade(int, double);
        double calcAvg();
        void setMaxGrades(int);
};

student1对象运行正常，但是当我尝试对student2使用addGrade（）或calcAvg（）时出现错误。任何帮助将不胜感激。

Answer 1

执行此操作时，两个构造函数都不正确：

grades[maxGrades]; // This does not do what you think it does

此行不会在参数化构造函数中崩溃，因为maxGrades具有已知值。但是，您的默认构造函数会重用未初始化的maxGrades值，从而导致未定义的行为。

您应该使用初始化列表重写构造函数。假设grades是std::vector<int>，您可以这样做：

Student::Student() : maxGrades(0)
{
// The remaining members will be initialized, because they have constructors.
}

Student::Student(string fName, string lName, int mGrades)
:   firstName(fName)
,   lastName(lName)
,   maxGrades(mGrades)
,   grades(mGrades, 0)
{
}

Answer 2

这没什么用处：

this -> firstName = firstName;
this -> lastName = lastName;
this -> maxGrades = maxGrades;

因为您使用的是类的方法，this->firstName和firstName都引用相同的变量 - 相同的成员变量。同样，对于其他两个陈述。

如果没有看到这些成员是如何宣布的，很难说这是否是您崩溃的原因。但是，这绝对是虚假的代码。

此外，这一行也没有用处：

grades[maxGrades];

根据maxGrades和grades的宣传方式，很可能是您崩溃的原因。

编辑：您对grades的声明看起来并不好，因为它没有为成绩分配任何存储空间。您应该考虑在此处使用std::vector<double>，或者至少指定一个大于您将被要求使用的最大数据集的最大大小。（例如double grades[100];）但严重的是，请在此处考虑std::vector<double>。

在默认构造函数中，您应该为初始化列表中的每个成员分配合理的默认值。像这样：

Student::Student() :
    firstName(""),
    lastName(""),
    maxGrades(0),
    numGrades(0)
{
}

Answer 3

一旦创建了对象，就会调用构造函数，并且其中的任何内容都会被初始化。您只能在创建对象后将值分配给first，maxgrade等。我不认为这是对的。

Answer 4

由于成绩只是一个指针，因此您不会使用构造函数行“grade [maxGrades];”为其分配内存。您必须执行grade = new double [maxGrades]之类的操作，并且只能在已定义的maxGrades（未在默认构造函数中定义）上执行。您可以在平均值中除以零（如果maxGrades = 0），并访问未初始化的指针，等级。您需要一种方法将默认值设置为maxGrades，然后您需要为成绩进行“新”内存分配。

你的功能

void Student::setMaxGrades(int mGrades)
{
    mGrades = maxGrades;
    grades[maxGrades];
}

应该阅读

maxGrades = mGrades; 
grades = new double[maxGrades];

如何在同一个类中同时具有参数化和默认构造函数？

4 个答案: