Simplifie algo last index

Version imprimable

Bonjour ,

J'ai fais un genre de convertisseur.

Dans celui-ci,on saisit un nombre dans un LineEdit : si on choisit dans la liste Degrés -> Farenheit , ça nous affiche le résultat dans le label , mais ce que j'ai fais , c'est une fonction permuter , qui , si on fait l'action précédente , puis qu'on choisit dans la liste la conversion inverse , car farenheit -> Degré , bah ça nout met ce qu'il y avait dans le label , dans le LineEdit , et vice-versa . J'ai donc implémenté ça avec un lastIndex.

Cependant , y a t il plus facile ? Car la fonction est un peu trop énorme :-)

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
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
121
122
123
124
125
126
127
128
void MaFenetre::convertir()
{
 
 
    if (myLineEdit->text().isEmpty())
    {
        results ->setText("--.--");
        return;
    }
    double valeur=myLineEdit->text().toDouble();
    switch (maListe->currentIndex()) {
    case 0:  dernierIndex= maListe->currentIndex(); results->setNum(c_DegreToFarenheit(valeur)); break;
    case 1:  dernierIndex= maListe->currentIndex(); results->setNum((c_FarenheitToDegre(valeur))); break;
    case 2:  dernierIndex= maListe->currentIndex(); results->setNum(c_MetreToYard(valeur)); break;
    case 3 :  dernierIndex= maListe->currentIndex(); results->setNum(c_YardToMetre(valeur)); break;
    case 4 :  dernierIndex= maListe->currentIndex(); results->setNum(c_KilometreToMileMarin(valeur)); break;
    case 5 :  dernierIndex= maListe->currentIndex(); results->setNum(c_MileMarinToKilometre(valeur));break;
    case 6 :  dernierIndex= maListe->currentIndex();  results->setNum(c_KilometreHToNoeud(valeur)); break;
    case 7 :  dernierIndex= maListe->currentIndex(); results ->setNum(c_NoeudToKilometreH(valeur));  break;}}
 
 
void MaFenetre::permuter(int)
{
    QString myValLabel=results->text();
    double LineEdit=myLineEdit->text().toDouble();
    switch (dernierIndex)
    {
    case 0:
        if (maListe->currentIndex()==1 || maListe->currentIndex()==0)
 
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
            //      myLineEdit->clear();
        }
 
        break;
    case 1:
        if (maListe->currentIndex()==0 || maListe->currentIndex()==1 )
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
            //   myLineEdit->clear();
        }
 
        break;
    case 2:
        if (maListe->currentIndex()==3 || maListe->currentIndex()==2)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
            //   myLineEdit->clear();
        }
 
        break;
    case 3:
        if (maListe->currentIndex()==2 || maListe->currentIndex()==3)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel); }
        else
        {
            results->setText("--.--");
            //   myLineEdit->clear();
        }
 
        break;
    case 4:
        if (maListe->currentIndex()==5 || maListe->currentIndex()==4)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
            // myLineEdit->clear();
        }
 
        break;
    case 5:
        if (maListe->currentIndex()==4 || maListe->currentIndex()==5)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);
        }
        else
        {
            results->setText("--.--");
            //   myLineEdit->clear();
        }
 
        break;
    case 6:
        if (maListe->currentIndex()==7 || maListe->currentIndex()==6)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
 
            //    myLineEdit->clear();
        }
 
        break;
    case 7:
        if (maListe->currentIndex()==6 || maListe->currentIndex()==7)
        {
            results->setNum(LineEdit);
            myLineEdit->setText(myValLabel);}
        else
        {
            results->setText("--.--");
            //     myLineEdit->clear();
        }
        break;
    }
 
}

Thanks in advance :-)

Pourquoi ne pas utiliser un tableau/vecteur de lambda ou de std::function ?

Code:

1
2
3
4
5
6
7
8
9
switch (maListe->currentIndex()) {
    case 0:  dernierIndex= maListe->currentIndex(); results->setNum(c_DegreToFarenheit(valeur)); break;
    case 1:  dernierIndex= maListe->currentIndex(); results->setNum((c_FarenheitToDegre(valeur))); break;
    case 2:  dernierIndex= maListe->currentIndex(); results->setNum(c_MetreToYard(valeur)); break;
    case 3 :  dernierIndex= maListe->currentIndex(); results->setNum(c_YardToMetre(valeur)); break;
    case 4 :  dernierIndex= maListe->currentIndex(); results->setNum(c_KilometreToMileMarin(valeur)); break;
    case 5 :  dernierIndex= maListe->currentIndex(); results->setNum(c_MileMarinToKilometre(valeur));break;
    case 6 :  dernierIndex= maListe->currentIndex();  results->setNum(c_KilometreHToNoeud(valeur)); break;
    case 7 :  dernierIndex= maListe->currentIndex(); results ->setNum(c_NoeudToKilometreH(valeur));  break;}}

Deviendrait :

Code:

1
2
dernierIndex= maListe->currentIndex();
foo[dernierIndex](results, valeur);

11/07/2013, 14h34
Invité

C'est surtout la fonction permuter que je veux simplifier ..

( J'ai oublié de préciser que c'est sous Qt )

Ce que fait ces deux fonctions ,c'est que si on clic sur Degrés -> Farenheit , ça nous affiche tel valeur , et si on prend Farenheit -> Degres , ça nous affiche l'inverse ( logique ) , mais aussi que si je fais degrés -> Mètre , ça ne doit pas marcher .

L'appli est disposé tel que les fonctions qui s'inverse sont l'une après l'autre , d'ou l'utilité du last index ( on peut tester lastIndex -1 pour vérifier ce qu'on a fait précédemment )
11/07/2013, 15h32
tom250908

salut,
Si tu cherche uniquement a rendre la fonction plus "courte", tu peut créer une nouvelle classe, par exemple condition.h, ou tu aura une fonction qui va contenir ton if/else.

de cette façon tu n'aura qu'a inclure cette classe dans celle contenant "inverser" (de mémoire) et a definir une variable de type condition qui interviendra dans chaque case du switch.