こんにちは。
今まで僕は何気なく実装ファイルの上の方にconst修飾子をつけて定数を宣言していましたが、本当にこれで良いのでしょうか？
const std::string CONST_STR("const_string");
const int CONST_NUM = 100;
検索してみると、こちらのサイトを見つけました。ここによると、以下のようにヘッダに宣言を書いて、
struct CConstParam
{
static const int RETCODE_SUCCESS;
static const int RETCODE_FAIL;
static const int RETCODE_WARN;
};
実装ファイルに実体と初期値を定義する（以下）とあるのですが、
const int CConstParam::RETCODE_SUCCESS = 0;
const int CConstParam::RETCODE_FAIL = 1;
const int CConstParam::RETCODE_WARN = 2;
この方法は最初の方法となにが違うのでしょうか？他のクラスから参照できるかどうかということでしょうか？
また、クラス内だけで使う定数なら最初の方法で問題ないのでしょうか？
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+1
実装ファイルの上の方にconst修飾子をつけて定数を宣言していましたが、本当にこれで良いのでしょうか？
const std::string CONST_STR("const_string"); const int CONST_NUM = 100;
大きく間違ってはいませんが、他の実装ファイルにたまたま同名の定数が存在すると、リンク時にエラーが発生するはずです（「同名エンティティが重複して存在する」といったニュアンスのメッセージになります）。
ある実装ファイル内部でしかその定数を使わないのであれば、定数宣言時にstatic修飾を行うか、無名名前空間に配置するか、あるいはその両方を行って下さい。
static const int CONST_NUM = 100;
// または
namespace {
const int CONST_NUM = 100;
}
以下のようにヘッダに宣言を書いて（中略）実装ファイルに実体と初期値を定義するとあるのですが、
この方法は最初の方法となにが違うのでしょうか？他のクラスから参照できるかどうかということでしょうか？
大きな違いは「コンパイル時定数か？」という点です。具体例の一つとして「配列の要素数として指定できるか否か」という違いがあります。前者CONST_NUMはint a[CONST_NUM]のように使えますが、後者はヘッダファイルをincludeしただけの実装ファイルでint b[CConstParam::RETCODE_WARN]としてもコンパイルエラーとなるはずです。
（注：その定数値が配列の要素数として意味があるかという観点は無視してください。あくまでもコンパイル定数の例示としてそのまま変数名を拝借しているだけです。）
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
+1
ファイル内でしか使わない定数であれば、前者の方法にさらにstaticを冠して、static const int ...のように定義しましょう。
staticがない場合、別なファイルに同じ定数があると衝突してしまいます。
あと、constexprの使えるコンパイラであれば、後者のようなクラス定数をconstexprとすることで、クラス内で定義してしまえます（参考）。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
+1
クラス/構造体の内側に定数を置くかグローバルな定義として置くかは定数定義の性質次第かと思います。
値定義を宣言と同時にするか宣言と定義を分離するかについては・・・
参考にされたページには「ヘッダーに直接定数を書くとそのヘッダーを修正して定数を変更すると参照しているヘッダーをincludeしているソースを全部コンパイルしなおさないといけない」と書いてあります。
開発中に調整したいような定数の場合はそうしたことを意識して定数の定義をソースファイルへ置いておくとコンパイル範囲を制限できるので便利といえるでしょう。ではヘッダーへ定数を書くとよくないかというと利点はあると思います。それはヘッダーに書いた定数はソースをコンパイルした時点で確定しているため値を参照した際にコンパイラーが最適化してくれると期待できるからです。
例えば以下のようなソースをコンパイルしてみると・・・
---a.h---
class A {
public:
static const int N;
static const int M = 10;
}
--a.cpp--
const int A::N = 10;
--main.cpp--
#include "a.h"
...
int v1 = A::N;
int v2 = A::M;
$ g++ -g -S main.cpp
-- main.s --
...
;以下の3行がv1への代入
.loc 1 9 0
movq .refptr._ZN1A1NE(%rip), %rax ; Nのアドレスをraxへ入れ
movl (%rax), %eax ; raxの中身をeaxへ入れ
movl %eax, -20(%rbp) ; eaxの中身をローカル変数v1へ設定
;以下の1行がv2への代入
.loc 1 10 0
movl $10, -24(%rbp) ; 値10を直接ローカル変数v2へ設定のようにNは実際に定数値が入っている変数Nの中身を参照するコードが生成されるのに対してMは定義した値10がそのままリテラルとしてコンパイルされていることが伺えます。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう！
- ただいまの回答率 88.13%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
質問への追記・修正、ベストアンサー選択の依頼
t_obara
2017/02/16 14:58
参考にされたページの意図はご理解されたのでしょうか？