Discussion :

[Nightfall]

Bonjour,

Pour apprendre C# je viens d'écrire un premier code source sur le thème classique des formes géométriques: classe abstraite qui représente n'importe quelle forme, puis les classes dérivées rectangle, cercle, etc... Par contre il y a pas de winform.

J'ai essayé de faire en sorte d'utiliser un maximum de principes de la POO et de choses courantes en .net, pour me familiariser avec: héritage, polymorphisme, classe abstraite, interface et implémentation de méthode explicite, exceptions, évènements, opérateur as, clonage, liste générique.

Je sais pas si ce source peut avoir un intérêt quelconque. Si oui je peux rajouter les commentaires dans un but didactique.

[tomlev]

Je vois que tu n'es pas tombé dans le piège de créer une classe Carré qui hérite de Rectangle, et qui casse le principe de substitution de Liskov

2 petites remarques, aussi bien du point de vue OOP que du point de vue des bonnes pratiques en C# :

- Pourquoi une énumération TypeObjet ? Ca n'apporte pas grand chose, c'est redondant avec le type (la classe) qui indique déjà ce que c'est, et ça complique les évolutions ultérieures (si je veux créer une classe Pentagone, il faut que je l'ajoute à l'énumération, ce qui n'est possible que si j'ai la main sur les sources). Si c'est juste pour l'implémentation de la méthode ToString, chaque classe peut la rédéfinir à son niveau, ou alors implémenter une méthode qui indique le type :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
public abstract string GetTypeObjet();
 
public override string ToString()
{
    // return GetTypeObjet() + " " + Nom;
    // Plus propre avec String.Format :
    return string.Format("{0} {1}", GetTypeObjet(), Nom);
}

- Le champ Nom devrait être une propriété. Il est généralement déconseillé d'exposer des champs publics (sauf à la rigueur s'ils sont readonly, et encore...), car cela permet à du code externe de modifier l'état interne de la classe sans que celle-ci puisse le contrôler ou même le détecter.