1

我再次为学校做任务，我正在慢慢实现它，我不知道为什么我的park_car功能不起作用，我只是想做一个测试并且程序崩溃了......这是我的代码。

PS：我无法更改，***p2parkboxes因为它像大多数其他变量一样在启动文件中给出。我只想将第 0 层的第一个元素视为：HH-AB 1234。非常感谢您的帮助。PS2：我不能使用 std::string 以及它不允许用于该任务。

#include <iostream>
#include <cstring>
using namespace std;

#define EMPTY "----------"
class Parkbox{
    char *license_plate; // car's license plate
    public:
    Parkbox(char *s = EMPTY); // CTOR
    ~Parkbox(); // DTOR
    char *get_plate(){return license_plate;}
};
class ParkingGarage{
    Parkbox ***p2parkboxes;
    //int dimensions_of_parkhouse[3]; // better with rows,columns,floors
    int rows,columns,floors; // dimensions of park house
    int total_num_of_cars_currently_parked;
    int next_free_parking_position[3];
    // PRIVATE MEMBER FUNCTION
    void find_next_free_parking_position();
    public:
    ParkingGarage(int row, int col, int flr);// CTOR,[rows][columns][floors]
    ~ParkingGarage(); // DTOR
    bool park_car(char*); // park car with license plate
    bool fetch_car(char*); // fetch car with license plate
    void show(); // show content of garage floor
    // by floor
};

Parkbox::Parkbox(char *s ) { // CTOR
    license_plate = new char[strlen(s)+1];
    strcpy(license_plate, s);
    //cout << "ParkBox CTOR" << endl;
}
Parkbox::~Parkbox() { // DTOR
    delete [] license_plate;
    //cout << "ParkBox DTOR" << endl;
}

ParkingGarage::ParkingGarage(int row, int col, int flr){
    rows = row; columns = col; floors = flr;
    p2parkboxes = new Parkbox**[row];
    for (int i = 0; i < row; ++i) {
        p2parkboxes[i] = new Parkbox*[col];

        for (int j = 0; j < col; ++j)
            p2parkboxes[i][j] = new Parkbox[flr];
    }

}

ParkingGarage::~ParkingGarage(){

    for (int i = 0; i < rows; ++i) {
        for (int j = 0; j < columns; ++j)
            delete [] p2parkboxes[i][j];

        delete [] p2parkboxes[i];
    }
    delete [] p2parkboxes;
}

void ParkingGarage::show(){
    int i,j,k;
    for (i = 0 ; i < floors; i++){
        cout << "Floor" << i << endl;
        for (j=0;j<rows;j++){
            for (k=0;k<columns;k++){
                cout << p2parkboxes[j][k][i].get_plate() << "  ";
            }
            cout << endl;
        }
    }
}

bool ParkingGarage::park_car(char*s){

    p2parkboxes[0][0][0] = Parkbox(s); //test
    //p2parkboxes[0][0][0] = s; //test

    return true;
}


int main(void) {
    // a parking garage with 2 rows, 3 columns and 4 floors
    ParkingGarage pg1(2, 3, 4);
    pg1.park_car("HH-AB 1234");
    /*pg1.park_car("HH-CD 5678");
      pg1.park_car("HH-EF 1010");
      pg1.park_car("HH-GH 1235");
      pg1.park_car("HH-IJ 5676");
      pg1.park_car("HH-LM 1017");
      pg1.park_car("HH-MN 1111"); */
    pg1.show();
    /*pg1.fetch_car("HH-CD 5678");
      pg1.show();
      pg1.fetch_car("HH-IJ 5676");
      pg1.show();
      pg1.park_car("HH-SK 1087");
      pg1.show();
      pg1.park_car("SE-AB 1000");
      pg1.show();
      pg1.park_car("PI-XY 9999");
      pg1.show(); */
    return 0;
}
4

3 回答 3

6

您没有为 Parkbox 类声明复制构造函数。所以，线

p2parboxes[0][0][0] = Parkbox(s)

在堆栈上创建一些东西（带有 char* 指针的 Parkbox 实例）（并几乎立即将其删除）。要纠正这个问题，您可以定义

Parkbox& operator = Parkbox(const Parkbox& other)
{
    license_plate = new char[strlen(other.get_plate())+1];
    strcpy(license_plate, other.get_plate());
    return *this;
}

让我们看看工作流程

p2parboxes[0][0][0] = Parkbox(s)

线。

  1. 首先，调用构造函数并在堆栈上创建一个 Parkbox 的实例（我们称之为 tmp_Parkbox）。
  2. 在此构造函数中分配了 license_plate ，假设它指向 0xDEADBEEF 位置。
  3. 复制发生（这很明显，因为这是用代码编写的东西）并且 p2parboxes[0][0][0] 现在包含 tmp_Parkbox 的精确副本。
  4. tmp_Parkbox 的作用域现在结束并调用 tmp_Parkbox 的析构函数，其中 tmp_Parkbox.license_plate (0xDEADBEEF ptr) 被释放。

  5. p2parboxes[0][0][0] 仍然包含 Parkbox 的“有效”实例，并且 p2parboxes[0][0][0].license_plate 仍然是 0xDEADBEEF ，如果在您调用之前发生任何分配，则会导致未定义的行为这

    cout << p2parboxes[0][0][0].license_plate;

底线：行本身没有问题，问题隐藏在'='运算符的实现细节中。

在这一点上，你最好将 std::string 用于字符串，而不是使用带有隐式 C++ 复制/构造语义的锋利、棘手和显式的 C 风格直接内存管理。如果您将 std::vector 用于动态数组，代码也会更好。

于 2012-05-20T23:53:31.523 回答
2

这里的问题是您没有深拷贝分配语义。当您将临时 Parkbox 分配给停车场中的 Parkbox 时，编译器生成的赋值运算符会制作指针 license_plate 的浅表副本，使两个 Parkbox 都指向相同的内存位置。然后临时 Parkbox 超出范围并删除 license_plate。由于另一个 Parkbox 指向同一个位置，它的 license_plate 也被删除。

有几个解决方案。解决该问题的一种方法是定义提供适当语义的赋值运算符和复制构造函数，即执行车牌字符串的深拷贝。更好的选择，也是更好地利用 C++ 的选择，是使用 std::strings 而不是手动分配的 C-strings。我强烈建议第二种方法，尽管通过第一种方法可能具有指导意义。

于 2012-05-21T00:21:44.707 回答
0

从OP：

我解决了这个问题：

void Parkbox::change_plate(char *s){
delete [] license_plate;
license_plate = new char[strlen(s)+1];
strcpy(license_plate, s);
}
于 2012-12-07T12:49:17.693 回答