Discussion :

[drayif]

salut,

je travailles sur un projet en C.
et pour le bout de code , quand je compile , il me donne des erreurs du type suivant:

commande_client.c:41: error: case label does not reduce to an integer constant.
je ne sais pas ou est le probleme.

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
 
void commande_client(int sockfd, char** list_commande){
 
switch (list_commande[0])
{
case "set" : 
/*traitement de la commande set*/
break;
case "help" :
/*traitement de la commande help*/
break;
case "list":
/*traitement de la commande list*/
break;
case "get":
/*traitement de la commande get*/
break;
case "info":
/*traitement de la commande info*/
break;
case "download":
/*traitement de la commande download*/
break;
case "upload":
/*traitement de la commande upload*/
break;
case "connect":
/*traitement de la commande connect*/
break;
case "raw":
/*traitement de la commande raw*/
break;
default:
break;
}
}

[straasha]

on ne peut faire un switch que sur un type entier comme int ou char mais surement pas sur une chaine de caractere

[drayif]

comment on peut faire alors? pour eviter trop de if .... else

[Médinoc]

En C, pas grand-chose de trivial.
En C++, on utilise un conteneur associatif, genre une table de hachage...

[el magnifico]

Bonjour

Une solution consisterait à faire un tableau dans lequel tu places tes différents nom.

En tab[0], tu aurais "set"
En tab[1], tu aurais "help"
En tab[2], tu aurais "list"

Et ainsi de suite.

Avant de rentrer dans ton switch, tu fais une recherche séquentielle dans ton tableau et une fois que tu auras trouvé l'indice correspondant au nom cherché, tu entres dans ton switch.

Ex : tu cherches "help".

Tu rentres dans ta boucle et tu constates qu'il est à l'indice 1.

Tu sors de ta boucle et tu entres dans le switch et le tour est joué.

[souviron34]

tu peux faire le test avec la première lettre (qui est en C comme un entier), et ensuite si il y a ambiguité tu testes sur la 2ième , voire la 3ième lettre :

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
 
 
void commande_client(int sockfd, char** list_commande)
{
 
switch ( list_commande[0][0] )
 {
    case 's' : 
/*traitement de la commande set*/
   break;
   case 'h' :
/*traitement de la commande help*/
   break;
   case 'l' :
/*traitement de la commande list*/
   break;
   case 'g' :
/*traitement de la commande get*/
   break;
   case 'i' :
/*traitement de la commande info*/
   break;
   case 'd' :
/*traitement de la commande download*/
   break;
   case 'u' :
/*traitement de la commande upload*/
   break;
   case 'c' :
/*traitement de la commande connect*/
   break;
   case 'r' :
/*traitement de la commande raw*/
   break;
 
   default:
   break;
  }
}

[Invité]

avec strcmp() ou mieux strncmp() ?

[scriptoff]

est ce une bonne pratique d'additionnner tous les caractères de la chaine puis de faire un switch de ce nombre ?

[droggo]

Hua,

est ce une bonne pratique d'additionnner tous les caractères de la chaine puis de faire un switch de ce nombre ?

Non.

Comment distinguer "allo" de "lola" ?

[scriptoff]

Dans ce cas précis , il est quand même rare d'avoir des annagrammes , non ?

[Emmanuel Delahaye]

comment on peut faire alors? pour eviter trop de if .... else

• if else if avec strcmp()
• une recherche linéaire dans une table non triée
• une recherche linéaire dans une table triée
• une recherche dichotomique dans une table triée (bsearch())
• hashage etc...

[droggo]

Koi,

Dans ce cas précis , il est quand même rare d'avoir des annagrammes , non ?

Oui, mais as-tu un moyen de garantir que cela n'arrivera pas ?

Il y a d'autres moyens, Emmanuel vient d'en citer quelques uns.

Pourquoi ne pas vouloir s'en servir ? C'est ceux-là que tout le monde utilise.

[souviron34]

j'veux pas paraître nul, mais qu'est-ce qu'il a mon switch ? y vous plait pas ???

[straasha]

moi je le trouve tres bien mais reserve a un cas particulier
en tout cas bien vu