cppファイルとヘッダファイルのどちらでincludeするか

includeの書き方について悩んでいます。
以下に例を示します。
悩んでいるのは、"cat.h"と<iostream>のinclude文をdog.cppに書くか、dog.hに書くかです。
以下の例では、dog.cppに書いています。
どちらに書くのが良いのでしょうか？
理由なども教えて頂けると幸いです。

dog.h

  1 #ifndef DOG_H
  2 #define DOG_H
  3 
  4 //#include <iostream>
  5 //#include "cat.h"
  6 
  7 class Cat;
  8 
  9 class Dog
 10 {
 11 public:
 12     Dog();
 13     ~Dog();
 14     Cat *cat;
 15 };
 16 
 17 #endif

dog.cpp

  1 #include "dog.h"
  2 #include "cat.h"     // dog.hに書くべき？
  3 #include <iostream>  // dog.hに書くべき？
  4 
  5 Dog::Dog() :
  6     cat(new Cat)
  7 {
  8     std::cout << "wan" << std::endl;
  9 }   
 10 
 11 Dog::~Dog()
 12 {
 13     delete cat;
 14 }

行動規範の内容に同意します

回答3件

ベストアンサー

<iostream>

これはdog.cppにてinclude．
dog.hをincludeする側に<iostream>を強制的にincludeさせる理由が全く無いので．

Cat.h

これは状況次第．

仮に，dogクラスのメンバであるCat *cat;がprivateだったならば，dog.cppにてinclude．
その理由は<iostream>と同じ．

しかし，現状はpublicなので，dogクラスを利用する側がCatクラスの定義を必要とする場合にはCat.hをincludeする必要が生じる．
これを，
dog.hに #include "Cat.h" とか書いておくことで面倒が無いようにしておくか，
それとも，dog.h の利用側が必要に応じて Cat.h をincludeするという話にするのかは，方針に依る．

[補足追記]

※何やら他者の回答のコメント欄にて「privateがどうの」という騒ぎが起きているのは
この私の回答のせいなのかもしれない(?)ので，念のため補足しておく．

もしも，Dogクラスがこのような形だったとしたら…

C++
1//dog.h
2class Cat;
3
4class Dog
5{
6//publicかprivateかはともかくとして
7  Cat cat;  //Cat型のメンバ変数
8};

メンバ変数cat が publicだろうがprivateだろうが，Cat型の定義を与えないとコンパイルエラーになる．
「Cat型ってのは何なんだよ？」というわけだ．
このdog.hをincludeする場所ではCat型の定義が必ず必要（≒Cat.hのincludeが必要）．
（この場合，include順とかも面倒だし，dog.hに#include "Cat.h"とか書いておくのがまぁ無難であろう．）

しかし本件の例では，メンバはCat型ではなくて Cat *cat; である．

C++
1//dog.h
2class Cat;
3
4class Dog
5{
6//publicかprivateかはともかくとして
7  Cat *cat;  //Cat型ではなく Cat*
8};

「"Cat"っていうのは，何らかのclassの型名なんだけどさー」（←前方宣言による）
「で，Cat*型のポインタ変数catがあってさー」
というだけの話をしている限りは，Cat型の具体的な定義が無くても良いから，とりあえずはコンパイルエラーにはならない．（つまり，Cat.hのincludeはとりあえず必要ではない）

↓

で，この回答で「privateが云々」と言っているのは，
【もしも cat がprivateであれば，dog.hをincludeしてDogクラスを使う側では cat に触れることは無いハズなのだから，Cat型の定義が追加で必要になることも無い（Cat.hのincludeが必要になることは無い）ハズ】
という話．
Dogクラスを使う側では Cat.hのinclude が必要になることは無いハズなのだから，dog.hに#include "Cat.h"とか書いても何の利便性ももたらさないし，
それどころかこれを書いてしまうと，余計な依存関係を作り出してしまうから，書くべきではない．

投稿2021/03/17 09:48

編集2021/03/18 02:24

fana

総合スコア11985

torimingo

2021/03/17 12:25

ご回答をありがとうございました。 privateとpublicに分けて考えるという発想はありませんでした。

fana

2021/03/18 01:25

dog.h内に他のヘッダのincludeを書くか否かは dogクラスを使う側，すなわち dog.h をincludeする側の観点で考えると良いかと．「コンパイルエラー？　…なんだよDog.hの他にCat.hのincludeも必要なのかよ…だったらあらかじめ dog.h にinclude を書いておいてくれよ」と思うようなものであれば，書いておけば良い（使うのが面倒でないという意味で）と思いますし，「俺はDogクラスを使いたいだけなのに，何故そのことに不要な（無関係な）ものまでincludeされなきゃならんのか？」というようなものは書かないでおくと良いかと．