Transformer une condition if..else en switch

**Nyxin** · 14/09/2010, 11h34

Bonjour à tous,

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

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
 
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

**ram-0000** · 14/09/2010, 12h06

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

Ensuite quand je vois du code comme cela

Code :

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

*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 *, ++, =)

**Nyxin** · 14/09/2010, 12h17

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 :

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

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.

**ram-0000** · 14/09/2010, 12h20

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

Code :

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

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)
...

**Nyxin** · 14/09/2010, 12h24

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

**Sve@r** · 14/09/2010, 12h35

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 c :

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
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...