Discussion :

[batmat86]

Bonjour � tous, je viens vers vous afin d'optimiser mon code, car je le trouve tr�s r�p�titif et je pense que l'on doit pouvoir le r�duire tr�s fortement avec un switch ou autre mais je ne vois pas comment le mettre en place.

Voici mon code :

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
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("3 KVA ST"))
      {
            PrixAbo = Abo3KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP3;
            EditPrixHC->Enabled=false;
 
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("6 KVA ST"))
      {
            PrixAbo = Abo6KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("9 KVA ST"))
      {
            PrixAbo = Abo9KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("12 KVA ST"))
      {
            PrixAbo = Abo12KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("15 KVA ST"))
      {
            PrixAbo = Abo15KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("18 KVA ST"))
      {
            PrixAbo = Abo18KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("24 KVA ST"))
      {
            PrixAbo = Abo24KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("30 KVA ST"))
      {
            PrixAbo = Abo30KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("36 KVA ST"))
      {
            PrixAbo = Abo36KVA;
            TarifHC = PrixHC3;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=false;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("6 KVA DT"))
      {
            PrixAbo = Abo6KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("9 KVA DT"))
      {
            PrixAbo = Abo9KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("12 KVA DT"))
      {
            PrixAbo = Abo12KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("15 KVA DT"))
      {
            PrixAbo = Abo15KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("18 KVA DT"))
      {
            PrixAbo = Abo18KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("24 KVA DT"))
      {
            PrixAbo = Abo24KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("30 KVA DT"))
      {
            PrixAbo = Abo30KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }
      if (ComboPuissance->ItemIndex == ComboPuissance->Items->IndexOf("36 KVA DT"))
      {
            PrixAbo = Abo36KVADT;
            TarifHC = PrixHC;
            TarifHP = PrixHP;
            EditPrixHC->Enabled=true;
      }

Merci par avance de votre aide

[bandit boy]

Salut,
En effet, il est possible de simplifier la pr�sentation tout en gardant de la clareter. Le soucis est que tu auras autant de version que de programmeur

Il semble que tu connaisses l'item de la ComboBox et comme tu as �crit toi-m�me la comboBox, tu connais l'endroit des cha�nes:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
 
switch(ComboPuissance->ItemIndex)
{
    case 0://première chaine
          ...
          break;
    case 1://deuxième chaine
          ...
          break;
...
}

[henderson]

Salut !

C'est d'utiliser des tableaux (sans doute de pointeurs) pour �viter aussi le switch !

Pour ce qui est du Enabled, on peut faire une �valuation bool�enne.
Donc globalement on serait amen� � pouvoir r�sumer l'ensemble en faisant :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
 
int n = ComboPuissance->ItemIndex;
//séquence commune
TarifHC = PrixHC;
TarifHP = PrixHP;
//séquence propre
Val2 = *Valeurs2[n];
//le bool
EditPrixHC->Enabled = ( n > ...);

A plus !

[bandit boy]

Tu peux aussi faire une fonction qui teste la cha�ne et te renvoie un index (un int ou un type enum) qui te permet de faire ton switch.

[batmat86]

Bonjour et merci de vos r�ponses

Par contre je dois admettre que je ne vois pas trop comment les mettrent en place.

Pour ta solution bandit boy, je pense voir comment faire, mais au final cela va juste me supprimer tout mes If et les remplacer par les case, mais j'aurais toujours toutes les parties identiques selon les cas � copier coller comme c'est le cas actuellement.

Au final ce que je cherche � faire car � mon avis cela doit �tre possible, ce serait d'avoir le moins de ligne de code possible pour arriver au m�me r�sultat que j'ai actuellement, car je dois admettre que la syntaxe de mon code doit faire peur aux d�veloppeurs exp�riment�s que vous �tes


En ce qui concerne ta solution henderson, l� je dois admettre que les pointeurs a toujours �t� ma b�te noire en programmation pendant mon bts et je n'y avais jamais rien compris. Alors maintenant que j'ai eu mon bts il y a 2 ans et codant que tr�s tr�s peu pour me faciliter le boulot, je dois dire que je suis perdu pour une telle solution.

Donc si vous avez un exemple � me donner je pense que je pourrais m'adapter et ainsi faire l'optimisation souhaitait car selon moi on doit pour r�duire presque de moiti� mon code.

Merci en tout cas de votre aide, cela fait plaisir

[bandit boy]

Il faut que tu poses tout ca comme il faut, sachant que tu peux tester tr�s facilement des cha�ne de caract�res.

J'ai remarqu� que tu �cris souvent la m�me chose et que tu as en fait 4 param�tres � conficgurer. Pose un tableau (type table de bool) et simplifie au maximum le teste pour finalement travailler non pas sur une configuration, mais une option.

A toi de retourner le probl�me