Discussion :

[shadowmoon]

Dans une fonction, pour des besoins métiers et fonctionnels, j'ai la structure suivante

Attention ! Ça pique les yeux !!!

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
 
if(A!= null && A.truc == true)
{
 ....if (B!= null && B.truc == true)
 ....{
 ........if (C!= null && C.truc == true)
 ........{
.............if (D == null ||!(D.machin == E.biduleA || D.machin == E.biduleB || D.machin == E.biduleC)) return false;
........}
........else
........{
.............if (D == null ||!(D.machin == E.biduleA || D.machin == E.biduleB )) return false;
........}
....}
....else 
....{ 
........if (C!= null && C.truc == true)
........{
.............if (D == null ||!(D.machin == E.biduleA || D.machin == E.biduleC)) return false;
.........}
.........else
.........{
.............if (D == null || D.machin != E.biduleA) return false;
.........}
.....}
}
else 
{
....if (B!= null && B.truc == true)
....{
.........if (C!= null && C.truc == true)
.........{
..............if (D == null ||!(D.machin == E.biduleB || D.machin == E.biduleC)) return false;
..........}
.........else
........{
.............if (D == null ||  D.machin != E.biduleB) return false;
.........}
....}
....else
....{
.........if D== null || D.machin != E.biduleC) return false;
....}
}

quelqu'un a-t-il une idée pour optimiser ce code ?

C'est du Winform classique, 100% "Client lourd"

[jopopmk]

Salut,

tu peux essayer de voir si des cas sont fusionables avec une table de vérité, mais le code résultant ne sera peut-être pas plus lisible (voire pire).
Et puis le code que tu nous montres n'est pas bien méchant niveau perf'.
Sinon, comme on comprend rien aux conditions, je me demande si y'a pas des cas sans retour

[DotNetMatt]

Sinon, une autre piste est d'isoler les "sous ifs" dans des methodes, par exemple :

Code C# :

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

1
2
3
4
5
6
7
8
if(A!= null && A.truc == true)
{
	CaseATrue(B, C, D);
}
else 
{
	CaseAFalse(B, C, D);
}

Ensuite dans CaseATrue et CaseAFalse tu reportes les conditions, etc. Ca va te donner un genre d'arbre de decision, qui sera plus facile a tester.

[shadowmoon]

tu peux essayer de voir si des cas sont fusionables avec une table de vérité, mais le code résultant ne sera peut-être pas plus lisible (voire pire).

J'ai déjà fusionné des cas pour améliorer le code fait par mon prédécesseur (surement en utilisant la métode Larache)

Sinon, comme on comprend rien aux conditions

En effet, j'ai fait une erreur monumentale en transcrivant mon code, c'est maintenant corrigé

Avec des mots ca donne :

Si A.truc, B.truc, et/ou C.truc est vrai alors on doit vérifier que D n'est pas null et que nous avons une égalité entre D.machin et l'attribut bidule de E correspondant respectivement à A, B et/ou C

Pas de cas sans retour, je n'ai pas mis tout le code de la fonction, juste le bloc qui me pose problème

Sinon, une autre piste est d'isoler les "sous ifs" dans des methodes ... tu reportes les conditions, etc.

C'est que ça sera un peu plus lisible comme ça, merci du conseil

[Pol63]

j'irais sur un truc du genre

Code :

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

1
2
3
4
5
6
7
8
if (D == null) {return false;}
var possibleValues = new List<toi même tu sais>();
 
if (A != null && A.truc) {possibleValues.Add(E.BiduleA);}
if (B != null && B.truc) {possibleValues.Add(E.BiduleB);}
if (C != null && C.truc) {possibleValues.Add(E.BiduleC);}
 
return possibleValues.Contains(D.machin);

A B et C sont du même type ?

[jopopmk]

HS : dans un bloc CODE les espaces sont conservées, tu n'es pas obligé de mettre des points pour l'alignement

[shadowmoon]

HS : dans un bloc CODE les espaces sont conservées, tu n'es pas obligé de mettre des points pour l'alignement

Pour des raisons de confidentialité, j'ai dû "masqué" les noms des classes et des attributs. Du coup, j'ai perdu l'indentation de Visual Studio

A B et C sont du même type ?

A, B et C sont trois classes différentes, mais avec une partie de leurs attributs identiques car elles héritent toutes les trois de la même classe.

Merci de ta proposition pour simplifier ce morceau de code, je vais m'en inspirer