コンストラクタをどのように分解できますか？

私にはEnemyクラスがあり、コンストラクタは次のようになります。

public Enemy(String name, float width, float height, Vector2 position, 
             float speed, int maxHp, int attackDamage, int defense... etc.){}

コンストラクターには非常に多くのパラメーターがあるため、これは悪いように見えますが、Enemyインスタンスを作成するときは、これらすべてを指定する必要があります。また、これらの属性をEnemyクラスに含めて、それらのリストを反復処理し、これらのパラメーターを取得/設定できるようにします。EnemyをEnemyB、EnemyAにサブクラス化して、maxHpなどの特定の属性をハードコーディングすることを考えていましたが、Enemyのリスト（EnemyA、EnemyB、およびEnemyC's）。

私はただきれいにコーディングする方法を学ぼうとしています。違いがあれば、Java / C ++ / C＃で作業しています。正しい方向の任意のポイントが高く評価されています。

解決策は、パラメーターを複合型にバンドルすることです。幅と高さは概念的に関連しています-それらは敵の寸法を指定し、通常一緒に必要になります。これらは、Dimensionsタイプ、またはRectangle位置を含むタイプに置き換えることができます。一方、特に加速度が後で画像に入る場合は、グループposition化speedしてMovementDataタイプに分類する方が理にかなっている場合があります。私は仮定コンテキストからmaxHp、attackDamage、defense、なども一緒に属しているStatsタイプ。したがって、改訂された署名は次のようになります。

public Enemy(String name, Dimensions dimensions, MovementData movementData, Stats stats)

線を描画する場所の詳細は、コードの残りの部分と、通常一緒に使用されるデータによって異なります。

Builderパターンを確認してください。リンクから（パターンと代替案の例を示します）：

[コンストラクタ] Builderパターンは、コンストラクターまたは静的ファクトリーに少数のパラメーターが含まれるクラスを設計する場合、特にそれらのパラメーターのほとんどがオプションである場合に適しています。クライアントコードは、従来のテレスコープコンストラクターパターンよりもビルダーを使用して読み書きする方がはるかに簡単であり、ビルダーはJavaBeansよりもはるかに安全です。

サブクラスを使用して一部の値を事前設定することは望ましくありません。新しいタイプの敵の行動や属性が異なる場合にのみサブクラスを作成します。

工場出荷時のパターンは、通常使用される正確なクラスの上に抽象化するために使用されているが、また、オブジェクトの作成のためのテンプレートを提供するために使用することができます。

class EnemyFactory {

    // each of these methods is essentially a template for a kind of enemy

    Enemy enemyA(String name, ...) {
        return new Enemy(name, ..., presetValue, ...);
    }

    Enemy enemyB(String name, ...) {
        return new Enemy(name, ..., otherValue, ...);
    }

    Enemy enemyC(String name, ...) {
        return new EnemySubclass(name, ..., otherValue, ...);
    }

    ...
}

EnemyFactory factory = new EnemyFactory();
Enemy a = factory.enemyA("fred", ...);
Enemy b = factory.enemyB("willy", ...);

独立して使用したいオブジェクトを表すクラスへのサブクラスを予約します。たとえば、敵だけでなく、すべてのキャラクターが名前、速度、maxHpを持つキャラクタークラス、または画面上に幅のある存在を持つスプライトを表すクラス、高さ、位置。

多くの入力パラメーターを持つコンストラクターに本質的に問題はありませんが、少し分割したい場合は、ほとんどのパラメーターを設定する1つのコンストラクターと、使用可能な別の（オーバーロードされた）コンストラクターがあります特定の値を設定し、他の値をデフォルト値に設定します。

使用する言語に応じて、コンストラクターの入力パラメーターに次のようなデフォルト値を設定できるものがあります。

Enemy(float height = 42, float width = 42);

Rory Hunterの回答に追加するコード例（Java）：

public class Enemy{
   private String name;
   private float width;
   ...

   public static class Builder{
       private Enemy instance;

       public Builder(){
           this.instance = new Enemy();
       }


       public Builder withName(String name){
           instance.name = name;
           return this;
       }

       ...

       public Enemy build(){
           return instance;
       }
   }
}

これで、次のようにEnemyの新しいインスタンスを作成できます。

Enemy myEnemy = new Enemy.Builder().withName("John").withX(x).build();