2

2 つの 3D ベクトルの外積が機能するかどうかをテストする簡単なプログラムがあります。

#include <iostream>
#include "math\Vec3.h"

using namespace std;

int main(int argc, char **argv)
{
    Vec3 v1(1, 23, 5);
    Vec3 v2(7, 3, 4);
    cout << "Crossing v1 and v2" << endl;
    Vec3 v3 = v1.cross(v2);
    cout << "crossed" << endl;
    return 0;
}

変数を作成した直後にデストラクタが呼び出されたのはなぜですか?
印刷したものは次のとおりです。

Created: Vec3[1, 23, 5]
Destroy: Vec3[1, 23, 5]     // Why is the vector destroyed here?
Created: Vec3[7, 3, 4]
Destroy: Vec3[7, 3, 4]      // And here?
Crossing v1 and v2
Created: Vec3[77, 31, -158]
Destroy: Vec3[77, 31, -158] //And here??
crossed
Destroy: Vec3[77, 31, -158]
Destroy: Vec3[7, 3, 4]
Destroy: Vec3[1, 23, 5]

Process returned 0 (0x0)   execution time : 0.090 s
Press any key to continue.

Vec3.h は次のとおりです。

#include <iostream>
#include <string>

struct Vec3
{
    float x, y, z;

    Vec3():
        x(0), y(0), z(0) { std::cout << "Created: " << *this << std::endl; };
    Vec3(float i, float j, float k):
        x(i), y(j), z(k) { std::cout << "Created: " << *this << std::endl; };

    //...

    double dot(const Vec3&);
    Vec3 cross(const Vec3&);


    friend std::ostream& operator<<(std::ostream&, const Vec3);

    //...

    ~Vec3();
};

Vec.cpp:

Vec3 Vec3::cross(const Vec3& v)
{
    return Vec3(y * v.z - z * v.y,
                z * v.x - x * v.z,
                x * v.y - y * v.x);
}

std::ostream& operator<<(std::ostream& out, const Vec3 v)
{
    out << "Vec3[" << v.x << ", " << v.y << ", " << v.z << "]";
    return out;
}

Vec3::~Vec3()
{
    std::cout << "Destroy: "
        << "Vec3[" << x << ", " << y << ", " << z << "]"
        << std::endl;
}
4

2 に答える 2

7

デバッグ出力 ( を使用operator<<) により、コピー (値で「Vec3」を受け取るため) と追加の破棄が発生します。

コピーコンストラクターを提供していないため、それを見ることはできません。しかし、そうすれば、実際には構造よりも破壊の方が多くないことがわかるでしょう。

于 2013-03-29T13:16:36.970 に答える
1

出力を見てください - Vec3 を作成すると、それが破棄され、最後に Vec3 が破棄されます... うーん、途中で別の Vec3 が作成されていることは明らかで、それは Vec3 のコピーですあなたが作りたかった。

それが問題のように見えます.コード内の何かがコピーを作成しているときに、Vec3s の出力を単一のオブジェクトと混同しています。この混乱が発生する理由は、「作成された」行も出力するコピー コンストラクターが定義されていないためです。

したがって、最初にコピー コンストラクターをコードに追加してベスト プラクティスを採用します (ヒント: コンストラクター、デストラクタ、またはコピー コンストラクターのいずれかがある場合は、3 つすべてを実装する必要があります。あなた）。

于 2013-03-29T13:20:26.213 に答える