Meilleur modèle pour la construction de cet objet?

Question

Je construis des valeurs enum qui ont une structure de données comme les suivantes:

enum MetaType { 
HUMAN(
    new MetaTypeAttribute(AttributeType.BODY, 1, 6), 
    new MetaTypeAttribute(AttributeType.AGILITY, 1, 6), 
    new MetaTypeAttribute(AttributeType.REACTION, 1, 6) 
), 
ORK(
    new MetaTypeAttribute(AttributeType.BODY, 4, 9), 
    new MetaTypeAttribute(AttributeType.AGILITY, 1, 6), 
    new MetaTypeAttribute(AttributeType.REACTION, 1, 6) 
); 

MetaType(MetaTypeAttribute body, MetaTypeAttribute agility, MetaTypeAttribute reaction) { 
} 
} 

Donc il y a un ENUM de metatypes; Chaque MetaType spécifie un ensemble fixe de MetaTypeAttributes, qui consiste en une valeur min et max pour cet attribut.

Je déteste cette syntaxe. C'est trop long, c'est moche, et ça ne m'empêche pas de passer dans le mauvais AttributeType (le AttributeType doit correspondre à l'ordre des arguments du constructeur de MetaType, AttributeType.BODY> body, etc.). Ce que je veux, c'est un meilleur modèle pour s'assurer qu'un objet est construit avec ce schéma exact.

Ce code est abrégé. En réalité, il y a 9 AttributeTypes.

Des suggestions d'amélioration?

Answer 1

enum MetaType { 
HUMAN(new MyRange(1,1), new MyRange(1, 10), new MyRange(1,10) 
), 
ORK(new MyRange(1,1), new MyRange(1, 10), new MyRange(1,10) 
); 

MetaType(MyRange body, MyRange agility, MyRange reaction) { 
     this.bodyAttrib = new MetaTaypAttibute(AttributeType.BODY, body); 
     this.agilityAttrib = new MetaTaypAttibute(AttributeType.AGILITY, agility); 
     this.reactionAttrib = new MetaTaypAttibute(AttributeType.REACTION, reactiony);} 
} 

Answer 2

Si vous considérez différents types d'attributs, vous devriez peut-être vraiment créer ces types d'attributs en tant que classes/sous-classes au lieu d'enums. Vous aurez tous les avantages de la POO pour gérer le comportement de chaque type (méthodes héritées, réécriture de méthodes, ...) et ils seront sécurisés.

abstract class AbstractTypeAttribute{ 
    private final int min, max; 
    public AbstractTypeAttribute(int min, int max){ 
     //... 
    } 
    //... 
} 

class Body extends AbstractTypeAttribute{ 
    public Body(int min, int max){ 
     super(min, max); 
     //... 
    } 
    //... 
} 

---

Vous devriez penser à le faire à la place des attributs meta et les méta-types aussi.