使用重载的加法运算符有困难

Question

这是我的程序。我必须制作一个 Student 类型的对象，然后让 Student "check out" 成为一个项目。我正在使用重载的加法运算符让用户签出该项目。

main.cpp:

#include <iostream>
#include "Student.h"

using namespace std;

int main() {

    Student s(54000, "JOHN", "DOE");

    cout << "main:" << endl << (s + "Frisbee") << endl << endl;

    system("pause");

    return 0;

}

我在头文件中定义了我所有的 class 定义，以尽量使这个程序保持最小和简化。

Student.h:

#ifndef STUDENT_H
#define STUDENT_H

#include <fstream>
#include <string>
#include <iostream>

using namespace std;

class Student {

public:

    string firstName;
    string lastName;
    int id;
    int itemsCheckedOut;
    int size;
    string *array;

    Student(int id = 0, string firstName = "", string lastName = "") {
        Student::firstName = firstName;
        Student::lastName = lastName;
        Student::id = id;
        itemsCheckedOut = 0;
        size = 10;
        array = new string[size];
    }

    Student(const Student &other) {
        itemsCheckedOut = other.itemsCheckedOut;
        array = new string[itemsCheckedOut];
        for (int i = 0; i < itemsCheckedOut; i++) {
            array[i] = other.array[i];
        }
    }

    ~Student() {
        delete[] array;
        array = NULL;
    }

    Student &operator=(const Student &rhs) {
        if (this != &rhs) {
            firstName = rhs.firstName;
            lastName = rhs.lastName;
            id = rhs.id;
            itemsCheckedOut = rhs.itemsCheckedOut;
            delete[] array;
            array = new string[size];
            for (int i = 0; i < itemsCheckedOut; i++) {
                array[i] = rhs.array[i];
            }
        }
        return *this;
    }

    void CheckOut(const string &item) {
        array[itemsCheckedOut] = item;
        itemsCheckedOut++;
    }

    friend ostream &operator<<(ostream &output, const Student &student) {
        output << student.id << "  " << student.firstName << " " << student.lastName << endl;
        if (student.itemsCheckedOut != 0) {
            output << student.itemsCheckedOut;
            for (int i = 0; i < student.itemsCheckedOut; i++) {
                output << " " << student.array[i] << endl;
            }
        }
        else {
            output << 0;
        }
        return output;
    }

    const Student operator+(const string &item) {
        Student s;
        s = *this;
        s.CheckOut(item);
        cout << "class:" << endl << s << endl << endl;
        return s;
    }

};

#endif

输出：

class:
54000  JOHN DOE
1 Frisbee

main:
-858993460
1 Frisbee

正如你所看到的，从主体来看，它输出了错误的东西。它不是输出 id 后跟两个空格然后是名字和姓氏，而是输出数字：-858993460。这一定是某种内存泄漏问题或其他问题，但我很确定我的复制构造函数、重载赋值运算符和解构函数都已正确定义，但你可以看一下它们。

我非常感谢任何帮助，因为我在这里变得非常绝望。谢谢

Answer 1

您应该用 std::vector 替换您的字符串* 数组。它将为您处理内存管理，使您的代码比您当前使用的手动内存管理更容易且更不容易出错。如果您担心它在添加项目时进行分配，您可以保留 10 的初始大小（尽管如此小的数据大小应该永远不会成为问题）。

Answer 2

您的实际 operator+ 看起来是正确的。但是您的复制构造函数和赋值运算符中存在会导致其出现故障的错误：

复制构造函数不设置 size、id 或名称。
复制构造函数应该分配 [size] 项，而不是 [itemsCheckedOut]。
赋值运算符不复制size。
赋值运算符分配了一个维度为旧大小的新数组，可能会立即导致缓冲区溢出。
checkOut 函数不检查它是否不写入超出 size。它需要检测这种情况并拒绝结帐，或者分配更多 space。（上次你发布关于这个项目的问题时我提到了这一点）

Answer 3

它调用复制构造函数：

    Student(const Student &other) {
    itemsCheckedOut = other.itemsCheckedOut;
    array = new string[itemsCheckedOut];
    for (int i = 0; i < itemsCheckedOut; i++) {
        array[i] = other.array[i];
    }
}

但您忘记复制其正文中的所有学生字段。您覆盖默认复制构造函数，因此您应该手动执行所有数据复制，如赋值运算符。

使用重载的加法运算符有困难

Difficulty with using an overloaded addition operator

c++

class

dynamic

addition

operator-keyword