当成员覆盖内存时指向对象数组的指针

Question

这是交易。 我们有2个不同的Class F和Class O

class F {
    private:
        int x;
        int y; 
    public:
        int getXf(){ return x; }
        int getYf(){ return y; }
        f(int ,int);
};

class O {
    private:
        int n;
        int k;
        int x;
        int y;
        char type;
        int id;
        int t;
    public:
        O(int ,int ,int ,int ,int);
        int getX(){ return x; }
        int getY(){ return y; }
};

我们有第三类P，我们在其中初始化值。 在该类中，我们将创建两个对象数组。

class Prog {                                                                   
    public:
        int count;
        int fcount;
        O *o[]; //here we are declaring the arrays of objects
        F *f[];           
    public :
        //void init(); Here is the function where we initializing the values
};

现在，我们在其中创建对象的for语句的2。

for(int i=0;i<10;i++){
        randx = rand() % 10;
        randy = rand() % 20;

        o[i] = new O(100,50,i,randx,randy);
    }


    for(int i=0;i<3;i++){
        randx = rand() % 10;
        randy = rand() % 10;

        f[i] = new F(randx, randy);
    }

当我们打印所有对象时，都在这里，但是第一类的前3个被秒的对象替换。 分别是randx和randy的100和50 （第一）。

Answer 1

O *o[];

这声明了一个未知大小的数组，这是一个不完整的类型。 C ++不允许将其用作类成员，尽管某些编译器允许将其用作扩展，将其解释为零大小的数组。 无论哪种情况，这都不是您想要的。

如果您知道在编译时绑定的数组，则应指定它：

O *o[10];

否则，您将需要在运行时动态分配一个数组：

std::vector<O*> o;

for(int i=0;i<10;i++){
    randx = rand() % 10;
    randy = rand() % 20;

    o.push_back(new O(100,50,i,randx,randy));
}

我还建议您将对象或可能的智能指针（而不是原始指针）存储在数组中。 如果您确实出于某些原因想要原始指针，请记住在完成对象后删除它们，因为这不会自动发生，并且不要忘记三个规则 。

Answer 2

您在声明数组，但从未为它们分配内存。 您所看到的只是代码在堆栈中的遍历方式。

更合适的方法：

struct X {}; struct Y {};

class P {
public:
  P() : xs(new X*[10]), ys(new Y*[10]) { init(); }

  ~P() {
    // delete all objects
    for(std::size_t i = 0; i < 10; ++i)
      delete xs[i];
    for(std::size_t i = 0; i < 10; ++i)
      delete ys[i];

    delete[] xs;
    delete[] ys;
  }
private:
  void init() {
    // initialize
    for(std::size_t i = 0; i < 10; ++i)
      xs[i] = new X();
    for(std::size_t i = 0; i < 10; ++i)
      ys[i] = new Y();
  }

  // prevent assignment and copy
  P& operator=(const P& other);
  P(const P&);

  X** xs;
  Y** ys;
};

当然，如果仅使用std::vector来存储数据，那么所有这些魔力就变得不必要了。

Answer 3

问题是由于您声明数组的方式引起的：

O *o[/*No size here*/];
F *f[/*No size here*/];

由于您未说明数组的大小，因此这等效于

O **o;
F **f;

因此，您要声明两个类型分别为“指向O的指针”和“指向F的指针”的成员 ，但是这两个成员尚未初始化，因此您没有为它们分配任何内存。 也就是说，实际上您没有任何数组，只有指针可用于引用所需的数组类型。

如果在编译时知道要使用什么大小，则应在声明中指定该大小，这将为您提供一个适当分配的该大小的数组。 否则，请考虑使用std::vector 。