2015-10-08 33 views
0

enter image description here我的定义有什么问题? C++

我相信这是一个简单的错误,因为每个其他文件都抱怨同样的错误。但是我试图将这些帮助函数放在一个名为Tools的名称空间中,它又如何报告这个错误?整个项目可以在这里找到https://github.com/luming89/MyGameEngine。顺便说一句,如果你知道如何用glm来代替这些助手,请告诉我。我会很感激!

// This is the Tools.h file 
#ifndef TOOLS_H 
#define TOOLS_H 

#define MATH_PI 3.1415926535897932384626433832795 
#define ToRadians(x) (float)(((x) * MATH_PI/180.0f)) 
#define ToDegrees(x) (float)(((x) * 180.0f/MATH_PI)) 

typedef glm::detail::tquat<float, glm::precision::highp> Quaternion; 

namespace Tools 
{ 
    glm::mat4 initRotationFromVectors(const glm::vec3& n, const glm::vec3& v, const glm::vec3& u) // forward, up, right 
    { 
     glm::mat4 res; // Identity? 
     res[0][0] = u.x; res[1][0] = u.y; res[2][0] = u.z; res[3][0] = 0; 
     res[0][1] = v.x; res[1][1] = v.y; res[2][1] = v.z; res[3][1] = 0; 
     res[0][2] = n.x; res[1][2] = n.y; res[2][2] = n.z; res[3][2] = 0; 
     res[0][3] = 0;  res[1][3] = 0;  res[2][3] = 0;  res[3][3] = 1; 
     return res; 
    } 

    glm::vec3 transformToVec3(const glm::mat4& m, const glm::vec3& r) 
    { 
     glm::vec3 ret; 

     for (unsigned int i = 0; i < 3; i++) 
     { 
      ret[i] = 0; 
      for (unsigned int j = 0; j < 3; j++) 
       ret[i] += m[j][i] * r[j]; 
     } 

     return ret; 
    } 

    glm::vec3 GetRight(const Quaternion r) 
    { 
     return glm::rotate(r, glm::vec3(1, 0, 0)); 
    } 

    glm::mat4 initTranslation(glm::mat4& m, const glm::vec3& r) 
    { 
     for (unsigned int i = 0; i < 4; i++) 
     { 
      for (unsigned int j = 0; j < 4; j++) 
      { 
       if (i == 3 && j != 3) 
        m[i][j] = r[j]; 
       else if (i == j) 
        m[i][j] = 1; 
       else 
        m[i][j] = 0; 
      } 
     } 
     m[3][3] = 1; 
     return m; 
    } 

    glm::mat4 initScale(glm::mat4& m, const glm::vec3& r) 
    { 
     for (unsigned int i = 0; i < 4; i++) 
     { 
      for (unsigned int j = 0; j < 4; j++) 
      { 
       if (i == j && i != 3) 
        m[i][j] = r[i]; 
       else 
        m[i][j] = 0; 
      } 
     } 

     m[3][3] = 1; 

     return m; 
    } 

    glm::mat4 toRotationMatrix(const Quaternion& rot) 
    { 
     glm::vec3 forward = glm::vec3(2.0f * (rot.x * rot.z - rot.w * rot.y), 2.0f * (rot.y * rot.z + rot.w * rot.x), 1.0f - 2.0f * (rot.x * rot.x + rot.y * rot.y)); 
     glm::vec3 up = glm::vec3(2.0f * (rot.x*rot.y + rot.w*rot.z), 1.0f - 2.0f * (rot.x*rot.x + rot.z*rot.z), 2.0f * (rot.y*rot.z - rot.w*rot.x)); 
     glm::vec3 right = glm::vec3(1.0f - 2.0f * (rot.y*rot.y + rot.z*rot.z), 2.0f * (rot.x*rot.y - rot.w*rot.z), 2.0f * (rot.x*rot.z + rot.w*rot.y)); 

     return initRotationFromVectors(forward, up, right); 
    } 
} 

#endif 
+2

不要发布错误的图片。发布实际的错误测试。 – NathanOliver

+0

看起来像是在头文件中实现函数并在多个源文件中包含头文件。相反,在源文件的头文件和实现中有声明。 – Mahesh

+0

有点建议;当使用常量值比如PI时,更多的是个人偏好,尽量不要使用#define宏,而是在头文件'static const float PI;'中使用这个声明,并在相应的cpp文件中定义它为'const float PI = 4.0 f * atan(1.0f); // tan(pi/4)= 1'只要确保包含atan()函数的正确标题即可。 –

回答

7

您正在将定义放在头文件中,这意味着每个翻译单元(.cpp文件)都会得到一份副本。将定义放在.cpp文件中,并将声明放入.h文件中。 (或者你可以使功能inline。)

+0

哦,我明白了。谢谢!!! – Asgard