PC Lint 警告 537：重复包含文件

Question

如何处理 PC Lint 的此警告？

我在几个文件中都有#include。 PC Lint 向我显示消息警告 537：重复包含文件 'filepath\filename.h' 如果删除此声明，我将无法编译。

如果可能的话，我想取消这个警告。

您可以在此处查看相同的报告。

这是我的代码，我的编译器会发出警告：

checksum.h

#ifndef CHECKSUM_H
#define CHECKSUM_H

#include <GenericTypeDefs.h>

BOOL isCheckSumCorrect(const UINT8 *dataPointer, UINT8 len, UINT8 checkSumByte);

#ifdef  __cplusplus
extern "C" {
#endif

cryptography.h

#ifndef CRYPTOGRAPHY_H
#define CRYPTOGRAPHY_H

#include <GenericTypeDefs.h>

UINT8 encrypt(UINT8 c);
UINT8 decrypt(UINT8 c);

#ifdef  __cplusplus
extern "C" {
#endif

< code>crc8.h

#ifndef CRC_H
#define CRC_H

#include <GenericTypeDefs.h>

UINT8 generateCRC(const UINT8 *ptr, UINT8 Len);
BOOL isCRCValid(const UINT8 *ptr, UINT8 Len, UINT8 CRCChar);

#ifdef  __cplusplus
extern "C" {
#endif

显然，我没有在checksum.c上重复#include，cryptography.c 和crc8.c

Answer 1

您可以使用选项禁用该标头的警告：

-efile(537,GenericTypeDefs.h)

并将其添加到您的 Lint 配置中（如果有）。

Answer 2

您链接到的文章解释说，这更像是 PCLint 的奇怪现象，应该是注释而不是警告。

只需忽略它/禁用它即可。

Answer 3

忽略它

如果你有包含防护，这是一个虚假的警告，可以（应该）被忽略。我们使用包含防护，因为允许多次包含文件是一种很好的做法，可以实现更灵活的代码，防止人为错误，并避免在 #include 语句中具有顺序重要性。如果您想知道为什么会出现这种情况，请继续阅读。

多个包含不是问题，因为你有包含防护。

由于这些 .h 文件都非常相关，我猜测您将所有三个文件包含在同一个文件中的某个位置，也许是 main.c：

#include "checksum.h"
#include "cryptography.h"
#include "crc8.h"

main () { /* Do Stuff */ }

这将被扩展（当然，减去注释）到：

// CHECKSUM_H is not defined, so the preprocessor inserts:
#include <GenericTypeDefs.h>
BOOL isCheckSumCorrect(const UINT8 *dataPointer, UINT8 len, UINT8 checkSumByte);

// CRYPTOGRAPHY_H is not defined, so the preprocessor inserts:
#include <GenericTypeDefs.h>
UINT8 encrypt(UINT8 c);
UINT8 decrypt(UINT8 c);

// CRC_H is not defined, so the preprocessor inserts: 
#include <GenericTypeDefs.h>
UINT8 generateCRC(const UINT8 *ptr, UINT8 Len);
BOOL isCRCValid(const UINT8 *ptr, UINT8 Len, UINT8 CRCChar);

main () { /* Do Stuff */ }

所以，是的，#include 在预处理器步骤中的某个时刻确实出现了多次。表面上，该文件看起来像：

#ifndef GENERIC_TYPE_DEFS_H
#define GENERIC_TYPE_DEFS_H

#define UINT8 unsigned char

#ifdef  __cplusplus
extern "C" {
#endif

因此，经过更多预处理后，我们之前扩展的位（减去注释）变为：

// GENERIC_TYPE_DEFS_H is not defined, so the preprocessor inserts:
#define UINT8 unsigned char
BOOL isCheckSumCorrect(const UINT8 *dataPointer, UINT8 len, UINT8 checkSumByte);

// GENERIC_TYPE_DEFS_H IS defined, so the preprocessor inserts nothing.
UINT8 encrypt(UINT8 c);
UINT8 decrypt(UINT8 c);

// GENERIC_TYPE_DEFS_H IS defined, so the preprocessor inserts nothing.
UINT8 generateCRC(const UINT8 *ptr, UINT8 Len);
BOOL isCRCValid(const UINT8 *ptr, UINT8 Len, UINT8 CRCChar);

main () { /* Do Stuff */ }

进一步的预处理（未显示）在整个代码中复制 #define。

如果您留意其他预处理器和 linter 警告，则保证您拥有包含防护

。如果任何 .h 文件缺少标准包含防护，linter 将向您发出警告（错误 451 和 967）。如果多重包含的 GenericTypeDefs.h 没有包含保护，编译器将警告重复的符号定义。如果没有，请创建您自己的 MyGenericTypeDefs.h 或切换到 头文件，它是 C 标准的一部分，提供与您的类似的功能GenericTypeDefs.h。

警告：前面的解决方法不好

如果您确实坚持修复警告而不是忽略它，则必须从每个 .h 中删除 #include > 文件并在包含一个或多个文件之前输入一次，如下所示：

checksum.c

#include <GenericTypeDefs.h>
#include "checksum.h"

cryptography.c

#include <GenericTypeDefs.h>
#include "cryptography.h"

crc.c

#include <GenericTypeDefs.h>
#include "crc.h"

main.c

#include <GenericTypeDefs.h>
#include "checksum.h"
#include "cryptography.h"
#include "crc8.h"

main () { /* Do Stuff */ }

不推荐这样做，因为它会导致您做更多的工作，并且有更多的机会犯错误（无意冒犯，但您只是人类，而预处理器不是）。相反，只需要求预处理器完成其工作并按照设计方式使用包含防护即可。