Comment vérifier une classe a toutes les propriétés configurées avant d'être utilisé?

Question

Je me demande quelle est la meilleure approche pour vérifier qu'une classe a tous les détails requis avant de pouvoir l'utiliser. Par exemple, disons que j'ai une classe Book et que je souhaite transmettre un ensemble de propriétés à propos du livre au constructeur Book.

Book book = new Book(bookProperties); 

Ce que je veux vous assurer que BookProperties est complète, à savoir a toutes les informations.

Disons que dans cet exemple, je donne les résultats suivants:

• Titre du livre
• Auteur
• livre original Publishing Date de

Une façon est que je pourrais créer un constructeur par défaut qui accepte seulement les 3 items:

BookProperties bookProperties = new BookProperties("2001: A Space Odyssey", 
                "Arthur C. Clarke", 
                1968); 

Maintenant c'est ok, si nous avons seulement trois valeurs, mais disons que ma classe a 10 propriétés ou plus qui doivent être initialisées par l'utilisateur avant que la classe Book puisse être créée.

Une chose que je pensais était d'avoir une méthode dans le BookProperties appelé isValid. Puis dans le constructeur de la classe Book je verrais si bookProperties.isValid et j'affirmerais si le retour est faux.

Est-ce une bonne idée ou est-ce que je vais tout à fait faux?

Answer 1

Si nécessite 10 valeurs, alors il n'est pas rare de passer 10 valeurs dans le constructeur - mais ce que vous décrivez ressemble plus à un objet entité. Pour plus de commodité, il est courant d'utiliser simplement les setters de propriété. Avec 3.0 C#, vous pouvez utiliser la syntaxe initialiseur pour ce faire facilement et clairement:

Book book = new Book { 
    Title = "2001: A Space Odyssey", 
    Author = "Arthur C. Clarke", 
    PublishedYear = 1968 
}; 

Vous pouvez utiliser un objet BookProperties (comme une sorte de constructeur) - mais dans ce cas, pourquoi ne pas faire la construction d'une méthode sur le constructeur? Ensuite, le constructeur fait la validation avant de créer un Book (vraisemblablement via un constructeur internal) - et déclenche une exception s'il y a un problème. Je suppose l'utilisation de BookProperties est (par exemple) pour permettre une Book immuable - mais il n'y a pas besoin pour le constructeur être immuable:

var bookProperties = new BookProperties(); // builder? 
bookProperties.Title = "2001: A Space Odyssey"; 
... 
Book book = bookProperties.CreateBook(); 

Personnellement, je viens d'utiliser l'approche Book (le plus haut ci-dessus) dans la plupart des cas - cette approche fonctionne bien avec les moteurs de sérialisation .NET et les frameworks de liaison communs (comme les constructeurs sans paramètres et les propriétés get/set), par exemple.Si vous voulez une validation, alors ajoutez simplement une méthode Validate() (ou la propriété IsValue {get;}) à Book. Ou mettre en œuvre IDataErrorInfo.

Answer 2

Si ces propriétés sont vraiment essentielles, elles devraient être dans le constructeur. Vous pouvez bien sûr utiliser les surcharges pour omettre les propriétés qui ont une valeur par défaut acceptable.

10 ou même 20 paramètres peuvent ne pas être joli mais c'est mieux que les alternatives.