我试图在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.h
和matrix.h
-它们相互包含。这是循环包含,没有任何有意义的作用。
头文件中可能包含的包含保护措施可确保包含内容不会变为无限。但是它们无法解决数据类型之间的循环依赖关系。 CMatrix4x4
在您的vector.h
中是完全未知的,这会导致错误。
在CMatrix4x4
中向前声明vector.h
是朝着正确方向迈出的一步。但是,无论如何,您都必须摆脱无用的循环包含。您必须记住,CMatrix4x4
在vector.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/