คุณจะย่อยสลายนวกรรมิกได้อย่างไร?

ให้บอกว่าฉันมีคลาสศัตรูและตัวสร้างจะมีลักษณะดังนี้:

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

สิ่งนี้ดูไม่ดีนักเนื่องจากตัวสร้างมีพารามิเตอร์มากมาย แต่เมื่อฉันสร้างอินสแตนซ์ของศัตรูฉันต้องระบุทุกสิ่งเหล่านี้ ฉันต้องการคุณลักษณะเหล่านี้ในคลาส Enemy เพื่อให้สามารถวนซ้ำรายการของพวกเขาและรับ / ตั้งค่าพารามิเตอร์เหล่านี้ ฉันคิดว่าอาจจะ subclassing Enemy เป็น EnemyB, EnemyA ในขณะที่ hardcoding maxHp ของพวกเขาและคุณลักษณะเฉพาะอื่น ๆ แต่จากนั้นฉันก็จะสูญเสียการเข้าถึงคุณลักษณะ hardcoded ของพวกเขาหากฉันต้องการย้ำผ่านรายการ EnemyA ของ EnemyA และ EnemyB EnemyC ของ)

ฉันแค่พยายามเรียนรู้วิธีการเขียนโค้ดให้สะอาด ถ้ามันสร้างความแตกต่างฉันทำงานใน Java / C ++ / C # จุดใดในทิศทางที่ถูกต้องจะชื่นชม

การแก้ปัญหาคือการรวมพารามิเตอร์เป็นประเภทคอมโพสิต ความกว้างและความสูงเกี่ยวข้องกับแนวคิด - พวกเขาระบุขนาดของศัตรูและมักจะต้องการร่วมกัน พวกเขาอาจถูกแทนที่ด้วยDimensionsประเภทหรืออาจเป็นRectangleประเภทที่มีตำแหน่ง ในทางกลับกันมันอาจสมเหตุสมผลมากกว่าที่จะจัดกลุ่มpositionและspeedเป็นMovementDataประเภทโดยเฉพาะอย่างยิ่งหากการเร่งความเร็วเข้ามาในรูปภาพ จากบริบทผมถือว่าmaxHp, attackDamage, defenseฯลฯ นอกจากนี้ยังอยู่ด้วยกันในStatsประเภท ดังนั้นลายเซ็นที่แก้ไขอาจมีลักษณะเช่นนี้:

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

รายละเอียดเกี่ยวกับตำแหน่งที่จะวาดเส้นจะขึ้นอยู่กับส่วนที่เหลือของรหัสของคุณและข้อมูลที่ใช้ร่วมกัน

คุณอาจต้องการที่จะดูที่รูปแบบการสร้าง จากลิงก์ (พร้อมตัวอย่างของรูปแบบและทางเลือก):

[รูปแบบ] ตัวสร้างเป็นตัวเลือกที่ดีเมื่อออกแบบคลาสที่คอนสตรัคเตอร์หรือโรงงานคงที่จะมีมากกว่าหนึ่งพารามิเตอร์โดยเฉพาะถ้าพารามิเตอร์เหล่านั้นส่วนใหญ่เป็นตัวเลือก รหัสลูกค้านั้นง่ายต่อการอ่านและเขียนด้วยผู้สร้างมากกว่ารูปแบบตัวสร้าง telescoping แบบดั้งเดิมและผู้สร้างปลอดภัยกว่า 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 หรือคลาสเพื่อเป็นตัวแทนของสไปรต์ที่มีหน้าจอที่มีความกว้าง ความสูงตำแหน่ง

ฉันไม่เห็นอะไรผิดปกติกับ Constructor ที่มีพารามิเตอร์อินพุตจำนวนมาก แต่ถ้าคุณต้องการแยกมันออกมาเล็กน้อยคุณสามารถมี Constructor หนึ่งตัวที่ตั้งค่าพารามิเตอร์ส่วนใหญ่และ Constructor (Overloaded) อื่นที่สามารถใช้งานได้ เพื่อตั้งค่าเฉพาะและตั้งค่าอื่นเป็นค่าเริ่มต้น

ขึ้นอยู่กับภาษาที่คุณเลือกใช้บางคนสามารถตั้งค่าเริ่มต้นสำหรับพารามิเตอร์อินพุตของตัวสร้างของคุณเช่น:

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();