質問編集履歴
2
軽微変更
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -106,7 +106,7 @@
|
|
|
106
106
|
|
|
107
107
|
#### (1)重複定義
|
|
108
108
|
|
|
109
|
-
重複定義は必ず避けられる問題．以下をしっかりやれば，
|
|
109
|
+
重複定義は必ず避けられる問題．以下をしっかりやれば，重複定義は避けられる場合が多い．
|
|
110
110
|
|
|
111
111
|
|
|
112
112
|
|
|
@@ -124,7 +124,7 @@
|
|
|
124
124
|
|
|
125
125
|
|
|
126
126
|
|
|
127
|
-
しかし，ヘッダに実装を書くことも往々にしてある．その場合は上記のような問題が起こることを認識し，相応の対応を取れば良い．
|
|
127
|
+
しかし，ヘッダに実装を書くことも往々にしてある．その場合は上記のような問題が起こることを認識し，相応の対応を取れば良いだけ．
|
|
128
128
|
|
|
129
129
|
|
|
130
130
|
|
1
回答いただいた内容のまとめを追記しました．
test
CHANGED
|
File without changes
|
test
CHANGED
|
@@ -89,3 +89,113 @@
|
|
|
89
89
|
以上長文になりましたが，ご回答いただけると幸いです．
|
|
90
90
|
|
|
91
91
|
よろしくお願いいたします．
|
|
92
|
+
|
|
93
|
+
|
|
94
|
+
|
|
95
|
+
|
|
96
|
+
|
|
97
|
+
## 【追記】ご回答頂いた内容のまとめ
|
|
98
|
+
|
|
99
|
+
#### 基本方針
|
|
100
|
+
|
|
101
|
+
* (1)重複定義，(2)循環参照をなくすというMUSTな課題を解決しつつ，可能な限り(3)コードの管理性・可読性，(4)コンパイル速度，(5)実行ファイルの速度を意識したコーディングを行う．
|
|
102
|
+
|
|
103
|
+
* (3)-(5)はトレードオフであり，要求に合わせてコーディングを変える．
|
|
104
|
+
|
|
105
|
+
|
|
106
|
+
|
|
107
|
+
#### (1)重複定義
|
|
108
|
+
|
|
109
|
+
重複定義は必ず避けられる問題．以下をしっかりやれば，基本的に重複定義は起こらない．
|
|
110
|
+
|
|
111
|
+
|
|
112
|
+
|
|
113
|
+
* ヘッダと実装を分離する
|
|
114
|
+
|
|
115
|
+
* インクルードガードをきっちり書く
|
|
116
|
+
|
|
117
|
+
* ヘッダ内で依存先ヘッダのインクルードを正しく指定する
|
|
118
|
+
|
|
119
|
+
|
|
120
|
+
|
|
121
|
+
ヘッダと実装を分離しておいたほうが良い理由は，
|
|
122
|
+
|
|
123
|
+
ヘッダで実体を定義した場合，複数の.cppからそのヘッダをインクルードすると，それぞれの.cppでその実体が定義されてしまい，重複定義となるから．（[C++ \- リンクエラー：既に定義されている｜teratail](https://teratail.com/questions/63207)）
|
|
124
|
+
|
|
125
|
+
|
|
126
|
+
|
|
127
|
+
しかし，ヘッダに実装を書くことも往々にしてある．その場合は上記のような問題が起こることを認識し，相応の対応を取れば良い．
|
|
128
|
+
|
|
129
|
+
|
|
130
|
+
|
|
131
|
+
重複定義を避ける方法の一つとして重要なのが，inline関数/変数である．
|
|
132
|
+
|
|
133
|
+
* [cpp17book/035\-cpp17\-core\-inline\-variables.md at master · EzoeRyou/cpp17book](https://github.com/EzoeRyou/cpp17book/blob/master/035-cpp17-core-inline-variables.md)
|
|
134
|
+
|
|
135
|
+
|
|
136
|
+
|
|
137
|
+
##### ※inlineに関する補足（上記記事より）
|
|
138
|
+
|
|
139
|
+
かつてinline関数の意味は，関数を強制的にインライン展開させるための機能だった．昔のコンパイラー技術が未熟だった時代のC++コンパイラーは関数をインライン展開するべきかどうかの判断ができなかったため，inlineキーワードが追加された．インライン展開してほしい関数をinline関数にすることで，コンパイラーはその関数がインライン展開するべき関数だと認識した．
|
|
140
|
+
|
|
141
|
+
しかしコンパイラが賢くなった現代ではむしろ，ODRの回避の目的で使用することが多い．
|
|
142
|
+
|
|
143
|
+
|
|
144
|
+
|
|
145
|
+
#### (2) 循環参照
|
|
146
|
+
|
|
147
|
+
循環参照の問題を語る場合は，以下の2つを区別することが大切．
|
|
148
|
+
|
|
149
|
+
|
|
150
|
+
|
|
151
|
+
* バグによる循環参照
|
|
152
|
+
|
|
153
|
+
* 意図的な避けられない循環参照
|
|
154
|
+
|
|
155
|
+
|
|
156
|
+
|
|
157
|
+
意図的な循環参照の解決方法は，例えば以下の様な記事を参照．
|
|
158
|
+
|
|
159
|
+
* [一週間で身につくC++言語の基本\|第２日目：クラス間の相互参照](http://cpp-lang.sevendays-study.com/ex-day2.html)
|
|
160
|
+
|
|
161
|
+
|
|
162
|
+
|
|
163
|
+
|
|
164
|
+
|
|
165
|
+
#### (3)コードの管理性・可読性
|
|
166
|
+
|
|
167
|
+
(4),(5)がそれほど問題にならないのであれば，最も優先したい項目．
|
|
168
|
+
|
|
169
|
+
|
|
170
|
+
|
|
171
|
+
#### (4)コンパイル速度
|
|
172
|
+
|
|
173
|
+
これは，ヘッダと実装を分けておくことの大きなメリットの一つ．
|
|
174
|
+
|
|
175
|
+
他には，pImplも（一番の目的は隠蔽だと思いますが）コンパイル速度を上げるテクニックの一つ．
|
|
176
|
+
|
|
177
|
+
ただしpImplは
|
|
178
|
+
|
|
179
|
+
* ソースコードが複雑になる
|
|
180
|
+
|
|
181
|
+
* 実行速度が少し落ちる
|
|
182
|
+
|
|
183
|
+
|
|
184
|
+
|
|
185
|
+
などのデメリットもあり，注意が必要．
|
|
186
|
+
|
|
187
|
+
|
|
188
|
+
|
|
189
|
+
|
|
190
|
+
|
|
191
|
+
#### (5)実行ファイルの速度
|
|
192
|
+
|
|
193
|
+
これは最初から意識するものというよりは，速度が問題になったときに対応すれば良い．
|
|
194
|
+
|
|
195
|
+
ヘッダと実装を分けた事による速度の違いは通常無視できる程度に小さいし，inline関数でヘッダに書いたほうが速いというのも対応策の一つというだけで，最初から考えるべきものでも無い．
|
|
196
|
+
|
|
197
|
+
|
|
198
|
+
|
|
199
|
+
|
|
200
|
+
|
|
201
|
+
問題になったときは，プロファイルによりボトルネックを探すなどして対応する．
|