Question

我有一段代码和一些错误和警告，我无法理解它们为什么会发生。你能给我一些建议吗？

第一个：

#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：警告被视为错误 - 没有＆＃39;对象＆＃39;文件生成

警告C4512：＆＃39; RenderTriangleMesh＆＃39; ：无法生成赋值运算符

另一个：

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

错误：

错误C2220：警告被视为错误 - 没有＆＃39;对象＆＃39;文件生成

警告C4100：＆＃39; gm＆＃39; ：未引用的形式参数

Answer 1

第一个是说编译器无法生成赋值运算符。这是因为您的引用成员，因为无法重新引用引用：

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;
}

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

警告应该始终固定，而不是沉默。

这两个错误的原因是什么：未生成赋值运算符和未引用的形式参数？

1 个答案: