这是正确的,但由于多种原因不是最佳解决方案。
- 整理这个 typedef 列表需要额外的工作。他们已经在
stdint.h
。
- 您的 typedef 在某些架构上不正确,并且您没有对此进行任何检查。如果有人看到
uint32_t
,他们希望它在任何架构上都是 32 位无符号整数;这将是一个令人讨厌的错误。
- 您的文件的用户不清楚
proto.h
它是否包含 stdint.h。有人会说你应该包含尽可能少的文件;在我看来,更重要的是要清楚。删除proto.h
用户的 C 文件中的包含应该只需要删除对其中声明的函数的引用,而不是添加 stdint.h 的包含。为清楚起见,您应该将其添加到.c
文件中,他们也希望这样做。
- 您在 typedef 周围添加了额外的包含保护,这些不是必需的 - stdint.h(以及您将使用的所有其他标头)已经包含包含保护。
由于这些原因,我建议在任何需要来自另一个头文件的定义的头文件中(例如,在函数原型中使用宏或 typedef),您应该按如下方式构建文件:
proto.h
#ifndef PROTO_H_INCLUDED
#define PROTO_H_INCLUDED
// Typedefs for prototypes
#include <stdint.h>
unit32_t proto(int32_t *value, size_t length);
#endif
proto.c
#include <stdint.h>
#include "proto.h" // Forward declare functions in this file
unit32_t proto(uint32_t *value, size_t length)
{
// Do something
}
main.c
#include <stdint.h>
#include "proto.h"
int main(int argc, char *argv[])
{
uint32_t values[] = { 1, 2, 3 };
uint32_t result;
// Could do 'uint32_t result, values[] = { 1, 2, 3 };' (one line)
// but this is better for clarity
size_t len = sizeof(values) / sizeof(values[0]);
proto(values, len);
}