0

我一直在尝试使用动态创建的对象数组创建一个类。我重载了运算符 + 以向一组对象（类到类）添加（暂时仅此）一个新对象。问题是当我读取数组中的数据时，我得到了各种各样的大数字。这是代码：

#include<iostream.h>

class Figura {
    public: 
        int x, y, poz;
        int tip; //1 = punct ; 2 = dreapta; 3 = dreptunghi
        Figura() { };
        Figura(const Figura&) { };
};

class Grup {
    private:
        int nr_elemente;
        Figura *figuri;
    public:
        int i;
        Grup(int nr_el) {
            nr_elemente = nr_el;
            figuri = new Figura[nr_elemente];
            i = 1;
        }
        ~Grup() {};
        Grup(const Grup&) {};
        int _nr_elemente() {
            return i;
        }

        void adauga_element(Figura fig) {
            if( i <= nr_elemente)
                figuri[i++] = fig;
            else
                cout<<"Grupul a atins numarul maxim de figuri.";
        }

        void afiseaza_elemente() {
            for(int j = 1; j <= i; j++)
                cout<<"Figura nr : "<<j<<"tip: "<<figuri[j].tip<<figuri[j].x<<" "<<figuri[j].y<<" "<<figuri[j].poz;
        }
    friend Grup operator+(const Figura& fig1, const Figura& fig2) {
        return fig1.poz + fig2.poz;
    };
    friend Grup operator+(const Grup& gr1, const Grup& gr2) {};
    void operator+(const Figura& fig);
    friend Grup operator*(const Grup& fig) {};

};

void Grup::operator+(const Figura& fig) {
    Grup::adauga_element(fig);
}

class Punct : public Figura
{
    public: 
        Punct(int poz) {
            Punct::tip = 1;
            Punct::poz = poz;
        }
};

class Segment : public Figura
{
    public:
        Segment(int poz, int x) {
            Segment::tip = 2;
            Segment::poz = poz;
            Segment::x = x;
        }
};

class Dreptunghi : public Figura
{
    public:
        Dreptunghi(int poz, int x, int y) {
            Dreptunghi::tip = 3;
            Dreptunghi::poz = poz;
            Dreptunghi::x = x;
            Dreptunghi::y = y;
        }
};

void main(void) {

    Grup gr(1);
    Punct pct(1);
    Segment sgm(3, 5);

    gr + pct;
    gr + sgm;
    //cout<<gr.i;
    cout<<sgm.x;
    gr.afiseaza_elemente();

}
4

3 回答 3

4

数组索引从0到N - 1，当N是数组大小。以下代码将导致越界数组访问：

void adauga_element(Figura fig) {
    if( i <= nr_elemente)
        figuri[i++] = fig;
    else
        cout<<"Grupul a atins numarul maxim de figuri.";
}

改成：

    if( i < nr_elemente)

中的同样问题afiseaza_elemente()。

由于您有动态分配的成员，因此必须正确实现析构函数delete[]，动态分配的数组和复制构造函数和赋值运算符需要正确实现，或声明private以防止复制。由于这是 C++，请考虑使用std::vector<Figura>而不是数组。

于 2012-04-23T21:02:25.987 回答
1

一个更简单的例子可以提高理解。

无论如何，您的问题被称为对象切片。

简而言之，您不能将派生对象放入基础对象数组中。

要解决此问题，您可以将 figuri 数据成员更改为指向 Figura 的指针数组：

Figura ** figuri;
// ...
figuri = new Figura*[nr_elemente];
于 2012-04-23T21:02:34.317 回答
0

您没有初始化任何成员。您应该在构造函数中执行此操作。

例如：

Punct pct(1);

仅初始化成员tipand pos，但xandy将包含垃圾值，您可能最终会看到这些值。

您也不应该使用动态分配的数组作为成员 -Figura *figuri;而是使用std::vector. 这也将使您免于添加新图形时遇到的未定义行为。

另外，请注意您的析构函数和复制构造函数没有实现，这是错误的来源。

void adauga_element(Figura fig)

传递对象fig是按值传递的，所以应该实现一个拷贝构造函数。

于 2012-04-23T21:00:00.213 回答