0

我正在尝试在 C++ 中同时实现 Vector4 类和 Matrix4x4 类,以便更好地处理该语言。我环顾四周,似乎没有什么能真正回答我遇到的问题,但如果我错过了什么,我深表歉意。

编辑:原来的错误似乎不再发生(它是由循环包含引起的)。但是,现在我收到以下错误:

1>main.cpp(35): error C2064: term does not evaluate to a function taking 2 arguments

我只能想象这是因为我在 CMatrix4x4 中重载了 () 运算符,但是当我从 main() 调用时,它并没有出现在我之前的代码中。

请求的 SSCCE 案例:

#include <assert.h>
#include <cmath>
#include <iostream>

class CMatrix4x4;

class CVector4
{
public:
    float x, y, z, w;

    CVector4();
    CVector4(float, float, float, float);
    ~CVector4();

    CVector4    operator*(CMatrix4x4&);
};

CVector4::CVector4()
{
    x, y, z, w = 0;
}
CVector4::CVector4(float cx, float cy, float cz, float cw)
{
    x = cx, y = cy, z = cz, w = cw;
}

//No instance of overloaded function "CVector4::operator" matches the specified type
//<error-type> m
//DOES NOT occur with forward declaration of class, only when including matrix.h
//from a separate file.
//Now causes "term does not evaluate to a function taking 2 arguments" at lines: 35-38
//Whenever I call the overloaded operator ()
CVector4 CVector4::operator*(CMatrix4x4& m)
{
    CVector4 v;
    v.x = x*m(0, 0) + y*m(1, 0) + z*m(2, 0) + w*m(3, 0);
    v.y = x*m(0, 1) + y*m(1, 1) + z*m(2, 1) + w*m(3, 1);
    v.z = x*m(0, 2) + y*m(1, 2) + z*m(2, 2) + w*m(3, 2);
    v.w = x*m(0, 3) + y*m(1, 3) + z*m(2, 3) + w*m(3, 3);
    return v;
}

class CMatrix4x4
{
public:
    CMatrix4x4();
    ~CMatrix4x4();

    void SetRow(int r, CVector4);

    float operator()(int r, int c);

    private:
    float   matrix4x4[4][4];
};

CMatrix4x4::CMatrix4x4()
{
    for(int r = 0; r < 4; r++)
    {
        for(int c = 0; c < 4; c++)
        {
            matrix4x4[r][c] = 0;
        }
    }
}

CMatrix4x4::~CMatrix4x4()
{
}

float CMatrix4x4::operator()(int r, int c)
{
    assert(r >= 0 && r < 4);
    assert(c >= 0 && c < 4);
    return matrix4x4[r][c];
}

void CMatrix4x4::SetRow(int r, CVector4 v)
{
    assert(r >= 0 && r < 4);
    matrix4x4[r][0] = v.x;
    matrix4x4[r][1] = v.y;
    matrix4x4[r][2] = v.z;
    matrix4x4[r][3] = v.w;
}

int main()
{
    CMatrix4x4 m;
    CVector4 vec1(1, 2, 3, 4);
    CVector4 vec2;

    m.SetRow(0, CVector4(1, 0, 0, 0));
    m.SetRow(1, CVector4(0, 1, 0, 0));
    m.SetRow(2, CVector4(0, 0, 1, 0));
    m.SetRow(3, CVector4(0, 0, 0, 1));
    vec2 = vec1 * m;
    std::cout << vec2.x;
    std::cin.ignore();
    return 0;
}

编辑:感谢所有提供帮助的人。我设法通过将函数实现移动到单独的 .cpp 文件来解决这个问题(我应该从一开始就这样做。我不知道为什么我没有这样做),并在其中包含所需的头文件,并在头文件中使用前向声明.

我不确定这是否是正确的解决方案,但它似乎确实有效。

4

1 回答 1

2

与之前提出的许多问题相同的问题:您的代码的原始版本显然有两个头文件 -vector.h并且matrix.h- 相互包含。这是循环包含,没有任何意义。

您可能在头文件中包含的包含保护确保包含不会变得无限。但是它们对解决数据类型之间的循环依赖没有任何作用。您的CMatrix4x4中的 完全未知vector.h,这会导致错误。

CMatrix4x4in的前向声明vector.h是朝着正确方向迈出的一步。但是,无论如何,您都必须摆脱这种无用的循环包含。而且你必须记住,这CMatrix4x4将是一个不完整的类型vector.h,这意味着你将无法访问它的内部vector.h

后者意味着你CVector4 CVector4::operator*(CMatrix4x4& m)必须在定义之后定义CMatrix4x4,而不是之前。在您的代码中,它是之前定义的CMatrix4x4。此时类型CMatrix4x4仍然不完整,这意味着您不能使用它的()运算符。像这样的表达式m(0, 0)不会因为这个原因而专门编译。这就是您遇到错误的原因。

PS 另外,

x, y, z, w = 0;

没有做你可能认为它会做的事情。它将分配0w,但保持其他数据成员不变(阅读 C++ 中的逗号运算符)。

于 2013-01-04T23:51:46.227 回答