Saisie clavier - séparer les arguments

Version imprimable

Bonjour,
lors d'unr saisie sur stdin je dois séparer la "commande" des differents arguments que l'utilisateur saisit. La saisie se fait sous forme suivante

Code:

1
2
 
COMMANDE arg1 arg2 ..

Que pensez vous du code suivant:

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
 
/*
 * Reads the command on stdin
 * 
 * */
void CO_get(char* buffer, char* cmd, char* args)
{
	cmd = fgets(buffer, MAX_STDIN_LENGTH, stdin);
 
	if (cmd != NULL)
	{
		// splits command from args
		args = strchr(buffer, ' ');
		if (args != NULL)
		{
			*args = '\0';
			// points to arguments 
			args++;			
		}
		char* tmp = strchr(args, '\n');
		if (tmp != NULL)
		{
			*tmp = '\0';
		}
 
		if( cmd != NULL ) printf("cmd: %s\n", cmd);		
		if( args != NULL ) printf("arg: %s\n", args);	
	}
}

merci

16/10/2006, 09h12
granquet
la seule chose que je vois:
Code:

1 2 3 4 5 char* tmp = strchr(args, '\n'); if (tmp != NULL) { *tmp = '\0'; {
d'abord les declarations au milieu du code, çaymal...
ensuite si tmp == NULL, ça veux dire que t'as un probleme ... y te manque des args.
mais peut etre plus grave, ils sont "restés dans le stdin", c-a-d que tu vas les retrouver a la prochaine lecture.

tu dois soit:
-agrandir ton buffer et strcat ce que tu viens de lire aux args que t'avais deja,
-vider le stdin (faire une recherche sur le forum a ce sujet)
16/10/2006, 09h30
Jean-Marc.Bourguet

Citation:

Envoyé par Dark_Ebola

d'abord les declarations au milieu du code, çaymal...

Pourquoi? Mettre les déclarations au même endroit que l'initialisation, ça permet de déclarer const plus de choses, donc de raisonner plus facilement sur le programme. Ça réduit la zone où une variable est visible, donc ça évite d'utiliser la même variable pour deux choses différentes et ça permet de préparer la restructuration qui viendra tôt ou tard (les fonctions ont tendance à grandir et il arrive généralement un moment où la bonne chose à faire c'est de les découper).
16/10/2006, 09h33
Jean-Marc.Bourguet

Citation:

Envoyé par Tex-Twil

Code:

void CO_get(char* buffer, char* cmd, char* args)

Le prototype m'inquiète,

Tu ne te sers pas de la valeur passée et tu modifies les arguments. Es-tu sûr de ne pas vouloir retourner cmd et args? Dans ce cas il faut passer des pointeurs vers pointeurs vers char et modifier le reste du code.

Beaucoup plus simple

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main(void)
{
    char buf[256];
    char *search = NULL;
 
    fgets(buf, sizeof buf, stdin);
 
    if(buf == NULL)
    {
        return EXIT_FAILURE;
    }
 
    search = strtok(buf, " ");
 
    if(search != NULL)
    {
        printf("Commande = %s\n", search);
 
        while((search = strtok(NULL, " ")) != NULL)
        {
            printf("Arg = %s\n", search);
        }
    }
 
	return 0;
}

Ne pas oublier de vider le buffer apres le fgets.
Après à toi d'ajouter un compteur ou faire ce que tu veux.

Citation:

Envoyé par Jean-Marc.Bourguet

Pourquoi?

tout simplement parce que ça permet ce genre de choses:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 #include <stdio.h> int main (void) { int i = 0; while (i < 10) { i++; int i = 5; while (i > 0) { printf (": %d\n", i); i--; } } return 0; }
c'est limpide comme code :aie:
(d'accord, si on est serieux et qu'on active le warning qui vas bien on s'en rend compte : -Wshadow sous gcc)

16/10/2006, 12h24
Médinoc

Là, le problème n'est pas "déclaration au milieu du code", mais "redéclaration de variable".
L'avantage suprême que peut avoir une déclaration au milieu du code est qu'on peut déclarer const la variable en question si on ne la modifie plus par la suite...

PS: Codez en C#, les redéclarations y sont interdites...
16/10/2006, 13h00
Jean-Marc.Bourguet

Citation:

Envoyé par Dark_Ebola

tout simplement parce que ça permet ce genre de choses:

Ah, tu n'utilises donc que des variables globales? Parce que les arguments et toutes les variables locales peuvent masquer des variables globales.

Ça me semble être un mauvais argument. Ce n'est pas parce que quelque chose peut être mésutiliser qu'il faut le banir. Il faudrait banir tout le C alors.
16/10/2006, 13h03
Jean-Marc.Bourguet

Citation:

Envoyé par Médinoc

PS: Codez en C#, les redéclarations y sont interdites...

Génial, quelqu'un dans un groupe modifie l'équivalent d'un en-tête et notre code est cassé? Ou bien le système de module est tellement contraint qu'il n'y a pas moyen de faire l'équivalent du using namespace de C++?
16/10/2006, 13h51
Médinoc

Euh, je ne crois pas que ce soit à ce point: La redéclaration est interdite pour variable locale sur variable locale, pas sur les trucs globaux:
http://msdn2.microsoft.com/en-us/library/ada6dw95.aspx
http://msdn2.microsoft.com/en-us/library/973aa6bt.aspx
17/10/2006, 14h32
Franck.H

Moi personnellement, j'aurais plutôt opté pour la façon que Skyrunner a montré ;)
17/10/2006, 15h48
Jean-Marc.Bourguet

Citation:

Envoyé par Franck.H

Moi personnellement, j'aurais plutôt opté pour la façon que Skyrunner a montré ;)

D'une part strtok n'est pas réentrant; même dans un programme qui n'utilise pas les threads pour le moment, j'éviterais. Ça fera un problème en moins à régler quand on les utilisera.

D'autre part le code de Skyrunner répond à un autre cahier des charges que le code initial: il découpe tous les arguments, le code initial sépare la commande des arguments. S'il faut découper tous les arguments, moi je reparts au cahier des charges pour spécifier un moyen d'avoir des blancs dans un argument.
17/10/2006, 15h51
Emmanuel Delahaye

Citation:

Envoyé par Jean-Marc.Bourguet

S'il faut découper tous les arguments, moi je reparts au cahier des charges pour spécifier un moyen d'avoir des blancs dans un argument.

http://emmanuel-delahaye.developpez.com/clib.htm
Module TOK