コンストラクタの引数を減らす方法

コンストラクタの引数を減らす一般的な手法が知りたいです。

　例えばPlayerクラスを作成したとします。このPlayerクラスは必須のパラメータとしてHP、MP、初期位置、攻撃力、移動速度等沢山持つ必要があります。

　今、私はこういった状況に直面した時、コンストラクタの引数として取り初期化しています。

＜例＞

cpp
1class Player{
2
3private:
4  int m_hp;
5  int m_mp;
6  float m_power;
7  float speed;
8  Vec2 m_coord
9
10public:
11  Player(int hp, int mp, float power, float speed, Vec2 coord)
12    :m_hp(hp)
13    ,m_mp(mp)
14    ,m_power(power)
15    ,m_spped(speed)
16    ,m_cooord(coord){}
17
18};

この例くらいならギリギリなんとかなりますが、今後必須のパラメータを増やした時、どう考えても管理が大変になってしまいます。

そこで、こういったコンストラクタのパラメータが沢山増える時の一般的な対策を教えてほしいです。

大雑把な話になりましたが宜しくおねがいします。

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

回答4件

ベストアンサー

こんにちは。

コンストラクタの引数を減らす一般的な手法が知りたいです。

一般的なものはありません。常に状況に応じて対処するしかありません。

ただ、極端にコンストラクタのパラメータが増えること自体がレアと思います。
それらの多量のパラメータの値は、どのようにして決めるのでしょうか？
例えば、GUIを使って設定するのであれば、コンストラクタで渡す必要はないでしょう。
デフォルト・コンストラクタで何らかの固定のデフォルト値にしておいて、後でGUIで設定すればよいと思います。設定後にターゲットのオブジェクトへコピーすれば良いでしょう。
HDDに保存していたものを回復するようなケースでも同じですね。デフォルト・コンストラクタで初期設定後、load()メンバ関数的なもので回復すれば良いです。
逆に、定数テーブルで初期化したい場合はコンストラクタのパラメータを増やすのがベストと思います。

Windows APIはC言語I/Fでしたので、あまりに多量のパラメータが有る時は構造体に設定してから、それを渡してました。しかし、コンストラクタが自分とほぼ同じメンバ変数群を持つ構造体を受け取る方法はあまり意味がありません。その構造体のコンストラクタで同じ問題が発生しますし、なんと言っても二度手間です。

投稿2018/08/01 01:44

Chironian

総合スコア23274

１つの方法として、パラメータを個別に指定するのではなく、構造体としてまとめて渡すというやり方をよくやります。
そうすれば、構造体のメンバーを増やすだけで済みます。
構造体なので、渡すのは参照を渡します。

C++
1struct Parameter {
2  int m_hp;
3  int m_mp;
4  float m_power;
5  float speed;
6  Vec2 m_coord
7};
8
9class Player{
10
11private:
12  Parameter m_param;
13
14public:
15  Player(Parameter& p);
16
17};

構造体にコンストラクタや、代入演算子、コピーコンストラクタなどを作成しておけば、代入も楽に行なえます。

投稿2018/07/31 08:35

PineMatsu

総合スコア3579

pepperleaf

2018/07/31 11:34

同様にクラスにするのもありかも。

yuki23

2018/08/01 10:48

構造体を使ったら、初期化漏れが発生してしまいますので、よくありません。初期化漏れが発生してもいいのであれば、最初からコンストラクタでの初期化を必須でないとしてしまえばいいだけなので、解決になっていないと思います。

PineMatsu

2018/08/03 08:15

構造体にコンストラクタを記述して初期化すれば良いのでは？

yuki23

2018/08/03 10:42

それを元のクラスに書いたほうがいいのではないでしょうか？

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

コンストラクタの引数に渡せるのであれば、その引数を決めるための情報とアルゴリズムが必ずどこかに存在するはずです。
一つの一般的な解決策としては、単にその情報とアルゴリズムを、コンストラクタの外に置くのではなく中に置けばいいです。
例えば、引数をJSONから以下のようにパースしているのであれば、

JsonObject obj = parse(json_string);
new Player(obj["hp"], obj["mp"], ...);

それをコンストラクタ自体に入れてしまいます。

new Player(json_string);

...

Player(const string &json_string) {
    JsonObject obj = parse(json_string);
    hp = obj["hp"];
    mp = obj["mp"];
    ...
}

投稿2018/08/01 11:23

yuki23

総合スコア1459