c++ - 构造函数重载。得到错误的解决方案

Question

  #include <iostream> 
  #include <conio.h>

  using namespace std;

  class Crectangle {
      int * height, * width;
      public: Crectangle();
      Crectangle(int, int);
      Crectangle();
      int area() {
          return (*height * *width);
      }
  };

  Crectangle::Crectangle() {
      *height = 3;
      *width = 5;
  }
  Crectangle::Crectangle(int a, int b) {
      height = new int;
      width = new int;
      *height = a;
      *width = b;
  }
  Crectangle::~Crectangle() {
      delete height;
      delete width;
  }

  int main() {
      Crectangle rect(1, 2);
      Crectangle rectb;
      cout << "rect = " << rect.area() << "\n";
      cout << "rectb = " << rectb.area();

      getch();
  }

我将矩形区域设为“6”，而不是“2”。有人可以指出错误。

Answer 1

这里：

Crectangle::Crectangle()
{
  *height=3; // height could point anywhere
  *width=5;  // width could point anywhere
}

您正在取消引用未初始化的指针。这是未定义的行为，因此结果可能是任何东西。

解决方案是不要使用指针 forheight和width。似乎没有任何理由使用它们。

class Crectangle
{
    int height;
    int width;
 ....
};

Answer 2

只有一个构造函数为宽度和高度分配内存。另一个具有未定义的行为。

Answer 3

#include <iostream> 
#include <conio.h>

using namespace std;

class Crectangle {
int * height, * width;
public: 
Crectangle();
Crectangle(int, int);
Crectangle();
int area() {
               return (*height * *width);
           }
};

Crectangle::Crectangle() {

   height = new int;
   width = new int;
  *height = 3;
  *width = 5;
}

Crectangle::Crectangle(int a, int b) {
  height = new int;
  width = new int;
  *height = a;
  *width = b;
}

Crectangle::~Crectangle() {
  delete height;
  delete width;
}

int main()
{
    Crectangle rect(1, 2);
    Crectangle rectb;
    cout << "rect = " << rect.area() << "\n";
    cout << "rectb = " << rectb.area();

    getch();
}