Transformer une condition if..else en switch

Version imprimable

Bonjour à tous,

Tout est dans le titre, si vous avez des suggestions, je suis preneur :)

Voici mon code :

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
 
void function_test_natural(char * function_name, IDL_LONG n_q_values, double *q_values, double *p_values, double *results)
{
 
    int i;    
 
    sasfit_param param;
 
    // Sets the parameters according to user-defined values.
    for (i = 0; i < MAXPAR; i = i + 1)
    {
        param.p[i] = *p_values++;
    }
 
 
    if (strcmp(function_name, "sphere") == 0)
 
        // Loop over the q values.        
        for (i = 0; i < (int)n_q_values; i = i + 1)
        {
            *results++ = (double) sasfit_ff_sphere(*q_values++, &param);
        }
 
    else
        *results++ = -9999.999;
 
}
 
void function_test(int argc, void *argv[])
{  
    double *q_values, *p_values, *results;
    IDL_LONG n_q_values;
    char * function_name;
 
    function_name = argv[0];
    n_q_values    = (IDL_LONG) argv[1];
    q_values      = argv[2];
    p_values      = argv[3];
    results       = argv[4];
 
    function_test_natural(function_name, n_q_values, q_values, p_values, results);
}

Merci d'avance

14/09/2010, 12h06
ram-0000

Je ne comprends pas trop ton problème, qu'est ce que tu veux faire ?

Ensuite quand je vois du code comme cela

Code:

*results++ = -9999.999;

je me demande toujours si cela fait bien ce que c'est censé faire (à cause des priorités des différents opérateurs *, ++, =)

L'idée étant de rendre mon code plus lisible, à l'intérieur de ma boucle if..else, je vais avoir une multitude de conditions, un switch case est à mon sens plus approprié afin d'éviter des centaines de ligne de code "inutile".
Code:

1 2 3 4 5 6 7 8 9 10 11 if (strcmp(function_name, "sphere") == 0) // Loop over the q values. for (i = 0; i < (int)n_q_values; i = i + 1) { *results++ = (double) sasfit_ff_sphere(*q_values++, &param); } else *results++ = -9999.999;
La modification est à faire à ce niveau là, le problème est que j'ai pas mal de difficultés avec ce langage, et je ne sais pas vraiment comment modifier ce code pour avoir une condition switch au lieu de mon if...else.

Tu veux dire que tu auras plusieurs valeurs possible pour function_name ?

Code:

1
2
3
4
5
6
7
8
if (strcmp(function_name, "sphere") == 0)
...
if (strcmp(function_name, "carre") == 0)
...
if (strcmp(function_name, "rectangle") == 0)
...
if (strcmp(function_name, "ellipse") == 0)
...

14/09/2010, 12h24
Nyxin

Exactement, désolé j'aurais du le préciser :p

Citation:

Envoyé par Nyxin

L'idée étant de rendre mon code plus lisible, à l'intérieur de ma boucle if..else, je vais avoir une multitude de conditions, un switch case est à mon sens plus approprié afin d'éviter des centaines de ligne de code "inutile".

Non. A ce niveau là, un tableau associatif mot-clef <=> fonction est encore plus approprié...
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 typedef struct { char *mot_clef; double (*fct)(int, void *); } t_choix; t_choix tabChoix[]={ {"sphere", sasfit_ff_sphere}, {"carre", sasfit_ff_carre}, {"cube", sasfit_ff_cube}, {NULL, NULL}, } t_choix *ptChoix; for (ptChoix=tabChoix; ptChoix->mot_clef != NULL; ptChoix++) { if (strcmp(ptChoix->mot_clef, function_name) == 0) { for (i = 0; i < (int)n_q_values; i++) { *results++ = (double) (ptChoix->fct)(*q_values++, &param); } break; } } if (ptChoix->mot_clef == NULL) // Rien n'a été trouvé *results++=-9999.999;
PS: j'ai recopié tel quel le *results++=... pour que tu piges le principe de l'algo mais je suis d'accord avec ram-0000 quand il dit que c'est pas génial comme instruction...

14/09/2010, 13h15
ram-0000

Citation:

Envoyé par Nyxin

Tout est dans le titre, si vous avez des suggestions, je suis preneur :)

50 minutes plus tard ...

Citation:

Envoyé par Nyxin

Exactement, désolé j'aurais du le préciser :p

Comme quoi, ce qui va sans dire, va mieux en le disant. Expliquer ton problème concrètement et du premier coups t'aurais permis de gagner 1 heure.
14/09/2010, 14h04
Nyxin

Ouep désolé, j'aurais du mieux m'exprimer la première fois =/

Je vous remercie pour vos réponses, je vais essayer de faire fonctionner tout ça :)