Question

我已将新文件添加到项目中：

#ifndef PLAYER_H
#define PLAYER_H
#include "enet/enet.h" //the problem
typedef struct Player
{
    ENetPeer * peer; //requires problematic include
    //void * peer; //works, since no include required
} Player;
const struct Player playerEmpty;
#endif //PLAYER_H

如果include存在，我会在不相关的文件中获得大量error: expected ';', ',' or ')' before numeric constant。如果我删除了include并使用void * peer，那么一切都很好。 enet库包含在其他地方的源文件中，并且工作正常。我正在使用enet 1.3.13（最新版），其标题保护似乎已到位。这是在gcc 4.9.2。

对于记录，错误发生在Point.h：

#ifndef POINT_H
#define POINT_H

#include <stdint.h>

#define X 0
#define Y 1
#define Z 2

typedef  int16_t  int16_Point2[2];
typedef  int32_t  int32_Point2[2];
typedef uint16_t uint16_Point2[2];
typedef uint32_t uint32_Point2[2];

typedef  int16_t  int16_Point3[3];
typedef  int32_t  int32_Point3[3];
typedef uint16_t uint16_Point3[3];
typedef uint32_t uint32_Point3[3];

#endif //POINT_H

我确定它很简单 - 不知道我做错了什么？

Answer 1

一般来说，使用单字母宏名称是一个好主意。它们可能很容易替换意外位置的字母（注意：在实际编译阶段之前，宏实际上是文本替换）。

你写的错误发生在Point.h中。我不认为他们实际上发生，但这里只是报告。 C在他们实际存在的地方检测语法错误是非常糟糕的。检查包含Point.h的文件

注意：标题中的const struct Player playerEmpty;也可能不需要，因为这将在每个编译单元中创建一个具有外部链接的对象。这与C ++不同：在C中，实际上没有常量，但只有常量变量：const只是程序员的承诺，变量一旦初始化就永远不会改变。更糟糕的是：你没有为它赋值，从而使它有效0 - 全局变量被初始化为所有位0.我很确定这不是预期的。

<强>更新

如果那是针对积分的，那么：

typedef union __attribute__ ((__packed__)) {
    struct {
        int16_t x,y,z;
    };    // anonymous union field (C99)
    int16_t vec[3];
} int16_Point3;

...

// usage:
int16_Point3 point = (int16_Point3){ .x = 5, .y = 3 }; // compound literal
point.z = point.x + point.vec[1]; // other word for point.y

摆脱#define并获得正确的语法。

注意__attribute__ ((__packed__))是特定于gcc的，以避免在struct字段之间填充字节。这是非标准的，但其他编译器通常具有类似的功能（例如pragma）。结构和数组必须具有相同的布局。

这可能比索引更具可读性。请注意，匿名结构和联合字段是标准的。

Answer 2

问题是单字符#define。永远不要这样做。

我几个月来一直在使用X，Y和Z，但在我今天加入Player.h之前一直没有问题，最后一定是 - 以迂回的方式 - 在预处理器/编译器中触发了一些问题。删除这些返回的编译为（相似的）正常性。

感谢那些在评论中提供帮助的人。

GCC报告包含的无关错误

2 个答案: