隐藏C结构定义

Question

这是我的设置：

在public.h：

#ifndef PUBLIC_H_
#define PUBLIC_H_

#include "func.h"

/*extern typedef struct _my_private_struct PRIVATE_;*/
typedef struct _my_private_struct PRIVATE_; /* Thanks to larsmans and Simon Richter */
#endif

在struct.h中

#ifndef STRUCT_H_
#define STRUCT_H_

struct _my_private_struct {
    int i;
};
#endif

在func.h中：

#ifndef FUNC_H_
#define FUNC_H_
#include "struct.h"

/* typedef struct _my_private_struct PRIVATE_; */
extern PRIVATE_ * get_new(int);
#endif

在func.c中：

#include <stdlib.h>
#include "func.h"

PRIVATE_ * get_new(int i)
{
    PRIVATE_ *p = (PRIVATE_ *) malloc(sizeof(PRIVATE_));
    if (p == NULL) return NULL;

    p->i = i;

    return p; 
}

在main.c中：

#include "public.h"

int main(int argc, char ** argv)
{
    PRIVATE_ *p = get_new(2);
    return 0;
}

当我用GCC编译这些文件时，我收到此错误：

旧的编译错误

声明说明符中的多个存储类

编辑后的编译错误

在'*'标记之前预期'='，'，'，';'，'asm'或'__attribute__'

有人可以帮助我/解释为什么我得到这个以及如何解决它？

Answer 1

其他答案很好地解决了你的问题。 但是，请允许我添加并回答您的最新评论：

我收到编译错误：在public.h中：重新定义typedef PRIVATE _...

虽然错误是不言自明的，但可能不太清楚为什么会发生这种情况。 考虑一下当你包含public.h时会发生什么：

#include "struct.h"
#include "func.h"
typedef struct _my_private_struct PRIVATE_;

如果你追踪并完全展开预处理器，这就是你将得到的：

// struct.h
struct _my_private_struct
{
    int i;
};

// func.h
typedef struct _my_private_struct PRIVATE_;
extern PRIVATE_ * get_new(int);

// public.h
typedef struct _my_private_struct PRIVATE_;

现在应该很明显为什么你遇到了问题。 如果没有func.h中的typedef，你的get_new原型就会失败，因为它还没有看到PRIVATE 。 OTOH，如果你保留typedef，你已经定义了两次。

此外，看起来您正试图将该结构与其他代码和模块保持私有。 即使您确实修复了构建错误，也没有真正实现该封装。 考虑一下：

int main()
{
    PRIVATE_ *p = get_new(2);
    p->i = 1337;        // HAHA, I just modified your private i.
                        // what are you going to do about it?
}

如果您想在C中使用数据隐私，请考虑使用不透明的指针设计。 我建议像这样重构你的来源：

// public.h
#ifndef PUBLIC_H_
#define PUBLIC_H_

#include "func.h"

#endif

---

// func.h
#ifndef FUNC_H_
#define FUNC_H_

struct PRIVATE_NOT_ACCESSIBLE;
typedef struct PRIVATE_NOT_ACCESSIBLE myint_t;

// declare your struct methods here
myint_t* get_new(int);
// ..

#endif

---

// func.c
#include <stdlib.h>
#include "func.h"

// define this only with functions 
// that's suppose to work with its internal data
struct PRIVATE_NOT_ACCESSIBLE
{
    int i;
};

myint_t * get_new(int i)
{
  // ...
}

现在，如果你试试这个：

#include "public.h"

int main()
{
    myint_t *p = get_new(2);
    p->i = 1337;            // Aw, doesn't work anymore :(
}

编辑：回答以下OP的评论。

如果您在多个编译单元中实现了私有结构的方法，您仍然可以通过将私有定义移动到专用标头来使其工作：

// func_implementation.h
#include "func.h"
struct PRIVATE_NOT_ACCESSIBLE
{
    int i;
};
// internal methods, helper functions you don't want exposed should go here too.
// eg.
void helper_method(myint_t *);

实现struct private'object'的源文件将包含'func_implementation.h'。 使用私有的外部客户端代码仅包含'func.h'。

Answer 2

• 您必须使用;结束typedef语句;
• extern typedef毫无意义，只需要做一个typedef 。

Answer 3

当前语法不正确，您需要在typedef和结构后面加一个分号。

Answer 4

有一个';' 在typedef之后丢失。

编辑：

struct _my_private_struct {...};

不要使用带有前导下划线的名称。 它们保留用于语言或实现。