Eviter 25 if else

Version imprimable

19/05/2006, 16h26
lechewal

Eviter 25 if else

Salut !

pour lancer des commandes a partir du term, je parse l'entrée dans un buffer et je fais des strcmp a la suite pour savoir a quelle commande sa correspond et quelle fonction lancée.

mais bon, j'ai une dizaine de commandes donc ca commence a etre moche..

j'ai eu l'idée de faire 2 tableaux, un avec les noms des commandes et un avec des pointeurs sur fonction

parcourir le tableau avec strcmp et lancer la fonction correspondante dans l'indice

c'est plus classe mais le probleme de cette méthode c'est que les fonctions doivent toutes avoir le meme prototype.. ce qui n'est pas mon cas..

autres techniques ??
a+
19/05/2006, 16h49
miron

Citation:

Envoyé par lechewal

Salut !

pour lancer des commandes a partir du term, je parse l'entrée dans un buffer et je fais des strcmp a la suite pour savoir a quelle commande sa correspond et quelle fonction lancée.

mais bon, j'ai une dizaine de commandes donc ca commence a etre moche..

j'ai eu l'idée de faire 2 tableaux, un avec les noms des commandes et un avec des pointeurs sur fonction

parcourir le tableau avec strcmp et lancer la fonction correspondante dans l'indice

c'est plus classe mais le probleme de cette méthode c'est que les fonctions doivent toutes avoir le meme prototype.. ce qui n'est pas mon cas..

autres techniques ??
a+

Je supose que si c'aitais possible tu aurais utilisé switch()...
19/05/2006, 16h52
damienlann

Code:

le probleme de cette méthode c'est que les fonctions doivent toutes avoir le meme prototype

pourquoi?
19/05/2006, 16h57
Swoög

Citation:

Envoyé par damienlann

Code:

le probleme de cette méthode c'est que les fonctions doivent toutes avoir le meme prototype

pourquoi?

parce que pour définir un pointeur de fonction, il faut spécifier le prototype entier de la fonction

Si tu veux utiliser switch, tu peux avoir un tableau de commandes dont l'indice correspond à une valeur d'enum

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
enum {
  COMMANDE1 = 0,
  COMMANDE2,
  /* */
  COMMANDEN
};
#define NB_COMMANDES (COMMANDEN + 1)
 
static const char * tab_commandes[NB_COMMANDES] = {
  "commande1",
  "commande2",
  /*... */
  "commanden"
};

Apres, il faut rechercher le commande fournie dans le tableau

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
for (k = 0, num_commande == -1; k < NB_COMMANDES; k++) {
   if (!strcmp(commande, tab_commandes[k]))
      num_commande = k;
}
 
switch (num_commande) {
  case COMMANDE1 : /* */break;
  case COMMANDE2 : /* */break;
  /* */
  case COMMANDEN : /* */break;
  default : /* commande incorrecte */
}

Mais bon:aie:
C'est un peu plus le bordel qu'une serie de if(!strcmp())

Salut,
pourquoi pas une structure

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
typedef struct {
   char CmdName[10];
  void (*pfonc) (arg_list toto);
} S_cmd;
const s_cmd[NBER_MAX_CMD] = { { "cmd1", fct1},
                                                {"cmd2", fct2}};
 
void launchCmd( char* Mycmd, arg_list *.....
{ 
  int index;
  do
  {
   if (strcmp (s_cmd.Cmd_name ........
    {
      s_cmd.pfonc .......
   }
  while (index < NBER_MAX_CMD)
}

et pourquoi pas quelque chose comme ca.....

19/05/2006, 17h29
gege2061
Pour le problème des arguments :
- tu peux utiliser des fonctions à nombres variables de paramètre (les pointeurs de fonction recevront donc un paramètre de type va_list)
- Un tableau de pointeur de type void
- Une union
19/05/2006, 17h51
Emmanuel Delahaye

Citation:

Envoyé par lechewal

pour lancer des commandes a partir du term, je parse l'entrée dans un buffer et je fais des strcmp a la suite pour savoir a quelle commande sa correspond et quelle fonction lancée.

mais bon, j'ai une dizaine de commandes donc ca commence a etre moche..
<...>
autres techniques ??

tokeniser la commande sous la forme d'un tableau de pointeur sur char et d'un compteur de tokens : int argc, char **argv

forcer la commande (argv[0]) en minuscule.
chercher la commande dans un tableau dont les éléments contiennent :

- la commande en minuscules
- l'adresse de la fonction à appeler

Le prototype des fonctions est unique est évidemment (air connu)

Code:

int f (int argc, char **argv);

personellement, j'ajoute un contexte utilisateur, ça évite les immondes globales...

Code:

int f (int argc, char **argv, void *p_user);

appeler la fonction correspondante avec les paramètres qui vont bien.

Exemple d'implémentation (un peu 'avancé', désolé...)

http://emmanuel-delahaye.developpez.com/clib.htm
Module CI (Command Interpretor)