1

我有一段代码和一些错误和警告，我不明白它们为什么会发生。你能给我你的建议吗？

首先：

#pragma once

#include "Vector3.h"
#include <vector>

#include <GL/glew.h>

class BoundingSphere
{
public:
    float radius;
    Vector3 center;
    BoundingSphere(float radius, Vector3 center) : radius(radius),center(center) {};
    BoundingSphere() {};
};

class TriangleFace;

class MeshVertex
{

private:
    Vector3 position; 
    std::vector<TriangleFace *> faces;
    Vector3 normal;
    bool normalUpdateNeeded;

public:

    unsigned int index;

    MeshVertex(void);
    MeshVertex(Vector3 position);
    ~MeshVertex(void);

    Vector3 &getPos() {return position;};
    void addFace(TriangleFace *face);
    const std::vector<TriangleFace*>& getFaces() {return faces;  };
    Vector3 getNormal();

    void setPos(Vector3 & pos) {position = pos; }
    bool isSurfaceParticle() {return faces.size()>0;}
    void updateNormal();
};

class TriangleFace
{
private:
    Vector3 normal;
    bool normalUpdateNeeded;

public:
    MeshVertex* particles[3];
    TriangleFace(void);
    TriangleFace(MeshVertex *p1, MeshVertex *p2, MeshVertex *p3);
    MeshVertex& operator[](int i) {  return *(particles[i]); }
    Vector3 getNormal();
    ~TriangleFace(void);
    void updateNormal();
};

class TriangleMesh
{
protected:
    std::vector<MeshVertex> particles;
    std::vector<TriangleFace> faces;

public:
    TriangleMesh(string filename);
    ~TriangleMesh(void);

    void reserveNumberOfFaces(unsigned int n) { faces.reserve(n); };
    void addFace(TriangleFace &f) {faces.push_back(f);};
    void addFace(MeshVertex *p1, MeshVertex *p2, MeshVertex *p3) {faces.push_back(TriangleFace(p1,p2,p3));};

    std::vector<TriangleFace>& getFaces() {return faces;};
    std::vector<MeshVertex>& getParticles() { return particles; };

    void updateNormals();

    BoundingSphere getBoundingSphere();
};


class RenderTriangleMesh
{
private: 
    TriangleMesh &m;

    GLuint vboid[2];

    GLfloat *vertices;
    GLfloat *normals;

public:
    RenderTriangleMesh(TriangleMesh &m);
    void draw();

private:
    void generateVBOs();
    void updateVBOs();

};

错误：

错误 C2220：警告视为错误 - 未生成“对象”文件

警告 C4512：“RenderTriangleMesh”：无法生成赋值运算符

和另外一个：

 virtual short SimpleAddOK(const GeneralMatrix* gm) { return 0; }

错误 ：

错误 C2220：警告视为错误 - 未生成“对象”文件

警告 C4100：“gm”：未引用的形式参数

4

1 回答 1

7

首先是说编译器无法生成赋值运算符。发生这种情况是因为您的参考成员，因为参考不能被重新定位：

struct foo
{
    int& i;
};

int x, y;
foo f = { x };
foo g = { y };

f = g; // ???

您可以通过自己明确禁用警告来使警告静音：

struct foo
{
    int& i;

private:
    foo& operator=(const foo&); // declared but never defined
};

这是使赋值运算符不可用的老技巧，但会阻止编译器生成它。

在 C++11 中，您可以这样做：

struct foo
{
    int& i;

private:
    foo& operator=(const foo&) = delete;
};

这是更明确和更少的hacky。

另一个是因为您从未使用过您的论点，这通常是程序逻辑中的错误。如果您不使用它，您可能应该删除它的名称：

virtual short SimpleAddOK(const GeneralMatrix* /* gm */) { return 0; }

我个人将其完全删除。其他人这样做：

virtual short SimpleAddOK(const GeneralMatrix* gm)
{
    (void)gm; // this "uses" gm

    return 0;
}

它有时包含在一个名为的宏中USE：

#define USE(x) (void)x

virtual short SimpleAddOK(const GeneralMatrix* gm)
{
    USE(gm);

    return 0;
}

我认为这些不如简单地删除名称，因为如果警告实际上是正确的（也就是说，您的程序确实存在缺陷并且没有看到它），那么您就是在掩盖警告，违背了目的。

警告应始终固定，而不是静音。

于 2012-08-21T02:32:14.667 回答