0

次のクラスを含むプロジェクトを実行すると、実行時エラーが発生します:
Shape- 抽象
Polygon :public Shape- 抽象
Triangle :public Polygon- 通常のクラス

を作成vector<Shape*> shapesし、コードで試しているときに次のことを行います。

shapes[i] = new Triangle(****);

実行時エラーが発生しています。
二重継承と関係がありますか、それとも私のコードの問題ですか?
私も授業に行っているのでCircle :public Shape、やってるときは

shapes[i] = new Circle(*****);

それはすべて正常に動作します..

編集：ほとんどの人が推測したとおり
です。 変更しvector<Shape*> shapes;たボクターの境界から出ることはありませんが、それでも崩壊します。
.push_back(new Circle())

表示される実行時エラーにはエラー コードは表示されず、exe ファイルが閉じられるだけです。

すべてのメソッドがPolygon のように純粋な仮想であるため、形状は明らかに抽象的です。
私は独自のコンストラクターを（シェイプとポリゴンで）作成しましたが、デフォルトのコンストラクターの問題を防ぐためにスコープを空白にするだけで、何もしません。

シェイプ.h

#include "Point.h"

    class Shape  
    {  
    public:  
        Shape();  

        //Methods
        virtual double get_Perimeter() = 0; //Returns shape's perimeter
        virtual double get_Area() = 0; //Returns shape's area
        virtual void move(point p) = 0; //Moves the shape
    };

シェイプ.cpp

#include "Shape.h"

shape :: Shape()
{
}

Polygon.h

#include "Point.h"
#include "Shape.h"
#include <vector>
using namespace std;

class Polygon :public Shape
{
protected:
    //Fields
    vector<point> points; //Vector of the polygon's points

public:
    //Constructors
    Polygon();


    //Methods
    virtual int getNumOfPoints() = 0; //Returns number of points
    virtual vector<double> get_Sides() = 0; //Returns vector of side's length
    virtual vector<point> get_Points() = 0; //Returns vector of points

    virtual double get_Perimeter() = 0; //Returns shape's perimeter
    virtual double get_Area() = 0; //Returns shape's area
    virtual void move(point p) = 0; //Moves the shape
};

Polygon.h

#include "polygon.h"
polygon :: polygon()
{
}

Point.h

class point
{
protected:
    //Fields
    double x; //the X value of point
    double y; //the Y value of point

public:
    //Constructors
    point(double x, double y); //Creates new point with given params
    point(const point &other); //Creates new point with other point's params

    //Methods
    double get_X() const; //Returns X field
    double get_Y() const; //Returns Y field
    void move(int dx, int dy); //Adds given params to current params
    void move(point p); //Adds p's params to current params
};

main.cpp

vector<shape*> shapes;

    //Creating new tirangle
    point* p1 = new point(1,1);
    point* p2 = new point(5,1);
    point* p3 = new point(3,4);
    shapes.push_back(new triangle(*p1,*p2,*p3));

三角形.h

#include "point.h"
#include "polygon.h"
#include <vector>
using namespace std;

class triangle :public polygon
{

public:
    //Constructors
    triangle(point p1, point p2, point p3); //Creates new triangle with given params
    triangle(const triangle &other); //Copies other's params to new triangle

    //Methods
    point get_P1() const; //Returns p1
    point get_P2() const; //Returns p2
    point get_P3() const; //Returns p3

    int getNumOfPoints(); //Returns number of points
    vector<double> get_Sides(); //Returns vector of side's length
    vector<point> get_Points(); //Returns vector of points

    double get_Perimeter(); //Returns shape's perimeter
    double get_Area(); //Returns shape's area
    void move(point p); //Moves the shape
};

三角形.cpp

#include "triangle.h"
#include <cmath>

    //Constructors
    triangle :: triangle(point p1, point p2, point p3)
    {
        points[0] = p1;
        points[1] = p2;
        points[2] = p3;

    }
    triangle :: triangle(const triangle &other)
    {
        points[0] = other.get_P1();
        points[1] = other.get_P2();
        points[2] = other.get_P3();
    }

    //Methods
    point triangle :: get_P1() const
    {
        return points[0];
    }
    point triangle :: get_P2() const
    {
        return points[1];
    }
    point triangle :: get_P3() const
    {
        return points[2];
    }

    int triangle :: getNumOfPoints()
    {
        return points.size();
    }
    vector<double> triangle :: get_Sides()
    {
        vector<double> sides;
        sides[0] = sqrt(pow(points[0].get_X()-points[1].get_X(),2)+pow(points[0].get_Y()-points[1].get_Y(),2));
        sides[1] = sqrt(pow(points[1].get_X()-points[2].get_X(),2)+pow(points[1].get_Y()-points[2].get_Y(),2));
        sides[2] = sqrt(pow(points[2].get_X()-points[0].get_X(),2)+pow(points[2].get_Y()-points[0].get_Y(),2));
        return sides;
    }
    vector<point> triangle :: get_Points()
    {
        return points;
    }

    double triangle :: get_Perimeter()
    {
        vector<double> sides = this->get_Sides();
        return sides[0]+sides[1]+sides[2];
    }
    double triangle ::get_Area() //By Heron's Formula
    {
        vector<double> sides = this->get_Sides();
        double area = this->get_Perimeter()/2;
        double tmp = area;
        for(int i=0;i<3;i++)
            area*=(tmp-sides[i]);
        return sqrt(area);

    }
    void triangle :: move(point p)
    {
        for(int i=0;i<3;i++)
        {
            points[i].move(p);
        }
    }
4

3 に答える 3

3

ベクトルは、同じタイプのオブジェクトのみを格納できます。必要なのは、所有するポインターのコンテナーです。

#include <vector>
#include <memory>

std::vector<std::unique_ptr<Shape>> shapes;

shapes.emplace_back(new Triangle);
shapes.emplace_back(new Circle);
于 2012-11-15T16:40:14.610 に答える
0

問題は：のコンストラクターですtriangle。

triangle :: triangle(point p1, point p2, point p3)
{
    points[0] = p1;
    points[1] = p2;
    points[2] = p3;
}

これは、空のベクトルの要素に書き込もうとします。[]ベクトルが十分に大きい場合にのみ、ベクトル要素にアクセスするために使用できます。push_back()ベクトルを成長させるために使用する必要があります：

points.push_back(p1);
points.push_back(p2);
points.push_back(p3);

または、C ++ 11を使用できる場合は、初期化子リストから割り当てます。

points = {p1, p2, p3};

コピーコンストラクタについても同様です。ただし、暗黙的に生成されたコンストラクターはポイントのベクトルを正しくコピーするため、コピーコンストラクターを宣言する意味はまったくありません。

少し話題から外れていますが、生のポインタのコンテナを使用してオブジェクトの存続期間を管理している場合は、十分に注意してください。オブジェクトを削除せずに誤ってポインタを削除してメモリリークを発生させるのは非常に簡単です。スマートポインタを使用することをお勧めします。

別の潜在的な問題もありShapeます。仮想デストラクタがないため、ベクトル内のどの形状も削除できません。これを行うと、クラッシュが発生する可能性があります。次のいずれかを行う必要があります。

virtual ~Shape() {}
于 2012-11-15T17:00:30.740 に答える
0

基本的にケレスが言ったこと。同様のケースに関連する可能性のある追加のメモ。

Shapeクラスが抽象的ではなく、使用した場合

std::vector<Shape> shapes;

それからこれ

shapes[i] = Circle(***);

有効ですが、オブジェクトがスライスされます! つまり、さらに派生したクラスのすべての情報が失われます。これは覚えておくべき重要な事実です！

于 2012-11-15T16:47:26.423 に答える