我应该包含另一个标题中包含的文件吗？

Question

Most often when creating multiple classes inside a program that use each other, I like to include only the minimum number of header files I need to reduce clutter.

For example, say class C inherits from class B, which contains class A. Now of course since class B contains class A as a member, it needs to include ah in bh . However, let's say C also needs to include ah . Being lazy as I am, I just include bh (which C needs to include anyways), and since bh already includes ah , I don't need to include anything more, and it compiles fine. Same for my .cpp files: I just include the header, and anything that's included in the header will be automatically included in my .cpp file, so I don't include it there.

Is this a bad habit of mine? Does it make my code less readable?

Answer 1

我坚持使用这个简单的规则：包括完全声明给定类所需的所有内容，但不要更多，并且不要假设包含从其他来源引入，即确保您的文件是自给自足的。

Answer 2

Include what's necessary for the header file to be parsed without relying on external include ordering (in other words : make your headers self-sufficient).

In your case, if ch declares a class C which inherits from class B , obviously you must include Bh . However, if class A never appears in ch , I believe there is no reason to include it. The fact that bh mentions A means that bh must make what's necessary to be parsed, either through forward declaring A or including ah .

So from my point of view, you're doing what should be done.

Also note that if for some reasons ch starts mentioning A , I would add the appropriate include or forward declaration so I wouldn't depend on the fact that bh does it for me.

Answer 3

It's best to include every header with definitions that you are using directly.

Relying on one of the other headers to include stuff makes your code more fragile as it becomes dependent on the implementation of classes that are external to it.

EDIT:

A short example:

• Class B uses class A, eg a hash table implementation B that uses a hashing mechanism A

• You create a class C that needs a hash table (ie B) and a hash algorithm (ie A) for some other purpose. You include Bh and leave out Ah since Bh includes it anyway.

• Mary, one of your co-workers, discovers a paper about this new fabulous hashing algorithm that reduces the probability of collisions, while it needs 10% less space and is twice as fast. She (rightly) rewrites class B to use class D, which implements that algorithm. Since class A is no longer needed in B, she also removes all references to it from Bh

• Your code breaks.

EDIT 2:

There are some programmers (and I've occasionally been guilty of this too, when in a hurry) who deal with this issue by having an "include-all" header file in their project. This should be avoided, since it causes namespace pollution of unparalleled proportions. And yes, windows.h in MSVC is one of those cases in my opinion.