amélioration de code à base de if imbriqués

Version imprimable

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

Attention ! Ça pique les yeux !!!

Code:

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"

19/01/2017, 19h47
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 :P
19/01/2017, 22h21
DotNetMatt
Sinon, une autre piste est d'isoler les "sous ifs" dans des methodes, par exemple :
Code:

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.
20/01/2017, 09h59
shadowmoon

Citation:

Envoyé par jopopmk

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)

Citation:

Envoyé par jopopmk

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

Citation:

Envoyé par DotNetMatt

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

j'irais sur un truc du genre

Code:

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 ?

20/01/2017, 10h20
jopopmk

HS : dans un bloc CODE les espaces sont conservées, tu n'es pas obligé de mettre des points pour l'alignement ;)
20/01/2017, 12h07
shadowmoon

Citation:

Envoyé par jopopmk

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

Citation:

Envoyé par Pol63

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