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 的输出与单个对象混淆,而该副本又被破坏。发生这种混淆的原因是因为您没有定义一个复制构造函数,它也打印出“创建”行。

因此,首先通过在代码中添加复制构造函数来采用最佳实践(提示:如果您有构造函数、析构函数或复制构造函数中的 1 个,则应该实现所有 3 个。如果您遗漏了任何一个,编译器会将其放入你)。

于 2013-03-29T13:20:26.213 回答