我试图在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文件中来解决此问题(首先我应该做。我不知道为什么不这样做),并在其中包括所需的标头,并在标头文件中使用前向声明。

我不确定这是否是正确的解决方案,但是它确实可以正常工作。

最佳答案

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

头文件中可能包含的包含保护措施可确保包含内容不会变为无限。但是它们无法解决数据类型之间的循环依赖关系。 CMatrix4x4在您的vector.h中是完全未知的,这会导致错误。

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

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

附言另外,

x, y, z, w = 0;


并没有做您可能认为的事情。它将为0分配w,但其他数据成员保持不变(有关C ++中的逗号运算符,请参见)。

关于c++ - 没有重载函数“CVector4::operator”的实例与指定类型匹配/项不求值为带有2个参数的函数,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/14166169/

10-11 12:19