おそらく重複していますが、簡単に検索することはできません...

次のようなヘッダーがあるとします。

namespace ns1
{
 class MyClass
 {
  void method();
 };
}

私はmethod().cppファイルでいくつかの方法で定義されているのを見ました：

バージョン1：

namespace ns1
{
 void MyClass::method()
 {
  ...
 }
}

バージョン2：

using namespace ns1;

void MyClass::method()
{
 ...
}

バージョン3：

void ns1::MyClass::method()
{
 ...
}

それを行う「正しい」方法はありますか？これらはすべて同じ意味ではないという点で「間違った」ものはありますか？

バージョン2は不明確であり、理解するのは簡単ではありません。これMyClassは、どの名前空間が属しているかがわからず、非論理的だからです（クラス関数が同じ名前空間にないのですか？）

バージョン1が正しいのは、名前空間で関数を定義していることを示しているためです。

::スコープ解決演算子を使用MyClass::method ()して名前空間のを参照しているため、バージョン3も正しいですns1。私はバージョン3を好みます。

名前空間（C ++）を参照してください。これはこれを行うための最良の方法です。

5年後、私はこれに言及したいと思いました、それはどちらも見た目が良く、邪悪ではありません

using ns1::MyClass;

void MyClass::method()
{
  // ...
}

私はバージョン4（以下）を使用しています。バージョン1（resoective定義の簡潔さ）とバージョン3（最大限に明示的）の利点のほとんどを組み合わせているためです。主な欠点は、人々がそれに慣れていないことですが、私はそれが技術的に他の選択肢より優れていると思うので、気にしません。

バージョン4：名前空間エイリアスを使用して完全修飾を使用します。

#include "my-header.hpp"
namespace OI = outer::inner;
void OI::Obj::method() {
    ...
}

私の世界では、すべてが明示的に修飾されている（変数名など）か、既知のカスタマイズポイント（関数テンプレートのswap（）など）でない限り、名前空間エイリアスを頻繁に使用しています。

バージョン3では、クラスと名前空間の間の関連付けが、より多くのタイピングを犠牲にして非常に明示的になっています。バージョン1はこれを回避しますが、ブロックとの関連付けをキャプチャします。バージョン2はこれを隠す傾向があるので、私はそれを避けます。

GoogleのC ++スタイルガイドは、インデントなしでバージョン1を指示します。

私はNum.3（別名詳細バージョン）を選択します。それはよりタイピングですが、意図はあなたとコンパイラに正確です。あなたが現状で投稿した問題は、実際には現実の世界よりも単純です。現実の世界では、クラスメンバーだけでなく、定義のスコープが他にもあります。ネームスペースやグローバルスコープなどとは異なり、スコープが再度開かれることがないため、クラスだけで定義が複雑になることはありません。

Num.1これはクラス以外のスコープで失敗する可能性があります-再開できるものなら何でも。したがって、このアプローチを使用して名前空間で新しい関数を宣言するか、インラインがODRを介して置き換えられる可能性があります。これは、一部の定義（特に、テンプレートの特殊化）で必要になります。

Num.2これは非常に壊れやすく、特に大規模なコードベースでは、ヘッダーと依存関係がシフトするため、プログラムのコンパイルに失敗します。

Num.3これは理想的ですが、タイプすることはたくさんあります-あなたの意図は何かを定義することです。これはまさにそれを行い、コンパイラはあなたが間違いを犯していないか、定義がその宣言と同期していないなどを確認するために起動します。

「コーディングスタイルの問題」だけではないことがわかります。Num。2は、ヘッダーファイルでexternと宣言された変数を定義および初期化するときに、リンクエラーを引き起こします。私の質問の例を見てください。cppファイルの名前空間内の定数の定義

すべての方法が正しく、それぞれに長所と短所があります。

バージョン1では、各関数の前に名前空間を記述する必要がないという利点があります。欠点は、特に複数のレベルの名前空間がある場合は、退屈なIDを取得することです。

バージョン2では、コードをよりクリーンにしますが、CPPに実装されている名前空間が複数ある場合、他の名前空間の関数と変数に直接アクセスして、名前空間を（そのcppファイルに対して）使用できなくする可能性があります。

バージョン3では、さらに入力する必要があり、関数の行が画面よりも大きくなる可能性があり、デザイン効果には良くありません。

一部の人々はそれを使用する別の方法もあります。これは最初のバージョンと似ていますが、識別の問題はありません。

こんな感じです：

#define OPEN_NS1 namespace ns1 { 
#define CLOSE_NS1 }

OPEN_NS1

void MyClass::method()
{
...
}

CLOSE_NS1

状況に応じてどちらを選ぶかはあなた次第です=]