[C++] コードのお作法

Question

いつもありがとうございます。
質問ですが、たとえばあるクラスAをトップダウン方式よりの考えで作ったとします。

// A.h
class A{
private:
// member

pubic:
// member
void model();
};

この時、クラスAのソースファイルである関数modelを実装中、
「この処理はmodel()内でのみ使用するものだが、何回か同様の処理をmodel()の他の場所（if-elseなど）で行う」
というコード群ができました。これを、

1. 関数化すべきか

1.1 関数化すべきならば（プライベート）メンバ関数に加えるのか
1.2 ソースファイルでのみ定義して、無名名前空間に置くのか（ ← 使い方まちがってるかもしれません）
2. そのままべた書きにしておくのか

のいずれで処理すべきなのでしょう？
3.の無名名前空間については、privateのように、アクセス権をそのファイルに書かれたモノのみに限定するための名前空間だと認識しています。

Answer 1

C++11なら、ラムダ式を使うのが便利でしょう。変数としてその場で宣言できるので、スコープを最小化できます。

Comments

[退会済みユーザー]

ラムダ式（に限らずスレッド処理やら例外処理もですが）にはまだ手を付けていないのでprivateで行こうと思います

Answer 2

1. 関数化すべきか

→
関数化すべきです。基本的に同じ内容を複数回ベタ書きはNGです。

1.1 関数化すべきならば（プライベート）メンバ関数に加えるのか
1.2 ソースファイルでのみ定義して、無名名前空間に置くのか（ ← 使い方まちがってるかもしれません）

→
他のかたも挙げられてますが、別のクラスで利用することがない限りはプライベートメソッドにすべきだと思います。
別のクラスで利用することになった時に、そのメソッドの処理内容に応じて、パブリックにするか、別クラス（Helperクラス、Utilクラス等）に移動するかを検討すればよいかと思います。

Comments

[退会済みユーザー]

やはり冗長性のあるコードの関数化は大前提なようですね。
TaMaMhyuさんのコメントにも記載させていただきましたが、1.2よりもプライベートのほうが良いのでしょうか？

Answer 3

ほかのクラスで使うことが想定されない処理であれば、プライベート関数で実装するのが良いのではないかと思います。

Comments

[退会済みユーザー]

そうさせていただきます。
疑問点で、とある本で、「ソースファイルはヘッダのようにほかのファイルへ影響を与えないので、class Aでのみ使用するクラス、class BがあるならソースファイルA.cppにclass Bの定義も含めた記述をしよう」と書いてあったのですが、これを関数に適用しようと思い、1.2の質問をしたのですがやはりプライベートのほうがいいのでしょうか？

[退会済みユーザー]

日本語おかしいのはお見逃しください

[TaMaMhyu]

おそらくそのクラスをまとめるというのは、ファイルスコープ的なことだとは思うのですが、ヘッダにその別のクラスを一切書かないという前提ではないかと思います。
なので、本質的にはプライベートになると思います。
もちろんそのクラスはプライベートにしない限りはどこかに自前で宣言してしまったら使えてしまうとは思いますが。

[退会済みユーザー]

ありがとうございました。