[Article] Bonnes pratiques objet en .net : Introduction aux principes SOLID

Version imprimable

25/02/2009, 17h47
Philippe Vialatte

[Article] Bonnes pratiques objet en .net : Introduction aux principes SOLID

Dans cet article, je vais essayer de vous présenter les principes SOLID, tels que décrits dans le livre de Robert Martin, en Agile Software Development, Principles, Patterns, and Practices.

On va voir l'intérêt de ces principes, et comment les appliquer, de façon abordable par tout le monde.

Accéder à l'article

N'hésitez pas à laisser vos commentaires à la suite ;)

Bonjour et merci pour cet article très intéressant.
Je viens d'envoyer le lien à 2 de mes collègues auxquels j'essayais d'expliquer ces principes il y quelques temps.

Une petite remarque sur le code exemple du chapitre II :

Les méthodes Save() et GetById() du WorkItemDataAccess ne devrait-elle pas être static ? Je me base là sur la façon que tu as d'appeler ce code ici:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
           public class WorkItemFactory{     
 
                public void Save(WorkItem item){
                    WorkItemDataAccess.Save(item.Id, item.Name);
                }
                public void GetWorkItemById(string id){
                    Datarow dr =  WorkItemDataAccess.GetById(id);
                    if (dr == null){
                        return null;
                    }
                    return new WorkItemData(dr);
                }
            }

03/03/2009, 08h31
djflex68

Super article effectivement, qui résume bien la plupart des notions présentés sur d'autres sites. :king:. Au passage j'aime bien la subtile référence au pattern activerecord plébiscité par les puristes RoR et que je trouve à titre personnel, parfaitement dégueulasse.
Pour répondre à la question de papy214, moi je ne mettrais personnellement pas les méthodes en static. Eventuellement je ferais de la factory un singleton pour éviter d'avoir à l'instancier à chaque fois, mais il me semble pertinent de laisser les méthodes d'instance car :
-> On peut imaginer que chaque factory hérite d'une interface commune de type ICRUD (pour lui permettre de gérer les opérations d'accès aux données élémentaires). Or il n'est pas possible d'hériter de méthodes statiques.
-> On peut aussi imaginer une factory de base dont le but est le même

a+
03/03/2009, 09h47
Papy214

djflex68 :

Dans mon message, je ne voulais pas dire qu'il "fallait" dans le sens obligatoire. :)

Mais il me semble que la seule manière d'utiliser le code suivant:

Code:

WorkItemDataAccess.Save(item.Id, item.Name);

est que la méthode WorkItemDataAccess.Save soit une méthode de classe.
Mais peut-être que quelque chose m'a échappé.

C'était une simple remarque sur le code, pas sur le fond de l'article que je trouve par ailleurs très intéressant. :)
03/03/2009, 10h58
Philippe Vialatte

Salut,

Citation:

Mais il me semble que la seule manière d'utiliser le code suivant:

Code :

WorkItemDataAccess.Save(item.Id, item.Name);

est que la méthode WorkItemDataAccess.Save soit une méthode de classe.
Mais peut-être que quelque chose m'a échappé.

Non, non, bien vu ;)

Comme quoi, malgré toute l'attention qu'on peut y porter et les relectures successives, il reste toujours des coquilles :oops:

Je vais essayer de le modifier dans la journée (pas accès au kit pour le moment)

Et merci des appréciations ;)
03/03/2009, 11h14
djflex68

Ah ok autant pour moi...
je ne pensais pas que tu faisais référence à

Citation:

WorkItemDataAccess.Save(item.Id, item.Name);

mais plutôt à :

Citation:

public void Save(WorkItem item)

On est donc d'accord sur le fond :king:
04/03/2009, 20h24
tehcyse
merci pour l'article :king:

Si je peux me permettre:

Premier cadre de code:

Citation:

cmd.CommandTex = "SELECT Id, Name FROM WorkItem where Id = '" + id + "'";

CommandTex à la place de CommandText

Et tu dis que tu passes sur le couplage mais c'est pas beaucoup plus long de faire:
Code:

1 2 cmd.CommandText = "SELECT Id, Name FROM WorkItem where Id = @Id"; cmd.Parameters.AddWithValue("Id", id);
qui est quand même plus jolie :D

Deuxième cadre: manque le T aussi +

Citation:

return new WorkItemData(dr);

au lieu de

Citation:

return new WorkItem(dr);

Voilà rien vu d'autre.

Ca y'est, j'ai mis a jour le document

@tehcyse, j'ai corrige les t manquants (je vais copier 100 fois "je n'ecrirais plus d'articles dans le train a 6h du mat'" :D)

Par contre, je n'ai pas change la requête en requête paramétrée...pourquoi ? juste parce que j'ai peur que ca brouille un peu les pistes (je sais, excuse a 2 frcs.), mais dans l'absolu, je ne ferais même pas :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 SqlConnection cnx = new SqlConnection(ConfigurationManager.ConnectionStrings["database"]); cnx.Open(); SqlCommand cmd = new SqlCommand(); cmd.Connection = cnx; cmd.CommandText = "SELECT Id, Name FROM WorkItem where Id = '" + id + "'"; DataTable dt = new DataTable(); using (SqlDataAdapter da = new SqlDataAdapter(cmd)) { da.Fill(dt); } return dt.Rows.Count == 0 ? null : dt.Rows[0];
Mais plutôt:
Code:

1 2 3 4 5 6 7 8 9 IList<SqlParameter> parameters = new List<SqlParameter>(); parameters.Add(new SqlParameter("@Id", id)); string strSql = "SELECT Id, Name FROM WorkItem where Id = @id"; DataTable dt = DataBaseHelper.GetDataTable(strSql, parameters) return dt.Rows.Count == 0 ? null : dt.Rows[0];
Par exemple...

Voire un GetDataRow au lie de GetDataTable, ou autre ;)

29/03/2010, 17h25
mitnick2006
salut,
un article très intéressant pour bien appliquer les principales bonnes pratiques en DotNet, juste une petite remarque dans la partie ISP: Interface Segregation Principle, V - C. Exemple, je pense que le code suivant correspond à une interface pas une classe ?!
Code:

1 2 3 4 5 public class IWorkItem { ... bool IsDeadLineExceeded(); }
et merci en avance :)
29/03/2010, 17h44
Philippe Vialatte
Salut,
Citation:
Envoyé par mitnick2006

salut,
un article très intéressant pour bien appliquer les principales bonnes pratiques en DotNet, juste une petite remarque dans la partie ISP: Interface Segregation Principle, V - C. Exemple, je pense que le code suivant correspond à une interface pas une classe ?!

Code:

1 2 3 4 5 public class IWorkItem { ... bool IsDeadLineExceeded(); }

et merci en avance :)
oops, j'ai oublié un public devant bool ;)

Faut que je pense à modifier :mrgreen:
29/03/2010, 17h55
mitnick2006

Citation:

Envoyé par Philippe Vialatte

Salut,
oops, j'ai oublié un public devant bool ;)

Faut que je pense à modifier :mrgreen:

mais est ce que c'est une classe ou bien une interface? car dans les différents exemples que vous avez cités IWorkItem est une interface !
29/03/2010, 22h12
Philippe Vialatte

bah oops 2, alors, j'étais pas réveillé cet aprèm.

c'est effectivement une interface, et corrigé ;)
30/03/2010, 11h04
mitnick2006

ok merci, maintenant c'est claire :ccool:
30/03/2010, 13h16
AsPrO

C'est peut-être hors-sujet mais ce ne sont pas également de bonnes pratiques pour d'autre langage OO, tel que C++ ?
Ou tout simplement pour tout langage OO.
30/03/2010, 14h39
Philippe Vialatte

Citation:

Envoyé par AsPrO

C'est peut-être hors-sujet mais ce ne sont pas également de bonnes pratiques pour d'autre langage OO, tel que C++ ?
Ou tout simplement pour tout langage OO.

Si, bien sur, ce sont des pratiques mutualisables a tous les langages OO, c'est juste qu'étant plus orienté .Net, les exemples sont dans ce langage ;)

D'ou d'ailleurs le placement dans la rubrique conception
31/03/2010, 11h32
AsPrO

Je n'ai pas encore tout lu mais je te fais part des deux fautes de frappes que j'ai remarqué dans la version pdf.

III - C - Exemple : apres le 2e cadre de code " à caque création d'un nouveau..."

IV - B - Comment l'appliquer : debut 2e paragraphe " il faut le valider pout tous les clients"

Sinon, super bon résumé. Le fait que parfois tu ne veuilles pas rentrer trop dans les détails est par moment assez frustrant mais ca donne une bonne vue d'ensemble. Peut-être un jour tu les approfondiras un par un ? :p
31/03/2010, 12h51
Philippe Vialatte

Citation:

Envoyé par AsPrO

III - C - Exemple : apres le 2e cadre de code " à caque création d'un nouveau..."

IV - B - Comment l'appliquer : debut 2e paragraphe " il faut le valider pout tous les clients"

Va falloir que je corrige

Citation:

Sinon, super bon résumé. Le fait que parfois tu ne veuilles pas rentrer trop dans les détails est par moment assez frustrant mais ca donne une bonne vue d'ensemble. Peut-être un jour tu les approfondiras un par un ? :p

Si j'ai le temps :aie:

enfin, j'ai deja fait un gros bout (specifique .net) sur l'injection de dependances
12/03/2019, 18h15
Shade

Contresens sur la traduction au niveau de la substitution de Liskov

Bonjour,

Il me semble qu'il y a un contresens au niveau de la traduction de la phrase de Robert C. Martin sur la substitution de Liskov :

Citation:

"Les sous-types doivent être remplaçables par leur type de base."

C'est peut-être dû à une petite erreur de traduction mais la phrase devrait plutôt ressembler à "Les sous-types doivent être substituables à leur type de base". Car sinon, cela contredit la traduction de la phrase de Barbara Liskov au-dessus, ainsi que votre explication, très bonne au demeurant, au-dessous.

Ce que dit Liskov c'est bien que S est un sous-type de T seulement si quand on attend un T on peut passer un S sans que cela ne change le comportement du programme et pas l'inverse : on ne veut pas remplacer S par T.

Ceci étant dit, merci pour votre travail, très bon article :)
03/09/2019, 01h02
Malick

Salut,

Citation:

Envoyé par Shade

la phrase devrait plutôt ressembler à "Les sous-types doivent être substituables à leur type de base". Car sinon, cela contredit la traduction de la phrase de Barbara Liskov au-dessus, ainsi que votre explication, très bonne au demeurant, au-dessous.

Cela a été corrigé.