suppression retour chariot

Version imprimable

salut à tous

j'ai un problème avec le 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
 
int parse_config(char *line)
{
    char *key, *value;
 
    key = strtok(line, " \t\r\n");
    value = strtok(NULL, "");
 
    if (*value)
    {
	// suppression des espaces en debut de value
	while (isspace(*value))
	{
             value++;
	}
        // suppression du retour chariot
	while (*value != '\0')
	{
		if (*value == '\n')
		{
		      value--;
		}
	}
     }
     printf("%s|%s\n", key, value);
     return 1;
}

tout ce passe bien jusqu'au moment ou j'essai de supprimer le retour chariot, le programme reste bloqué.

Citation:
Envoyé par havoc50
salut à tous

j'ai un problème avec le 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 int parse_config(char *line) { char *key, *value; key = strtok(line, " \t\r\n"); value = strtok(NULL, ""); if (*value) { // suppression des espaces en debut de value while (isspace(*value)) { value++; } // suppression du retour chariot while (*value != '\0') { if (*value == '\n') { value--; } } } printf("%s|%s\n", key, value); return 1; }
tout ce passe bien jusqu'au moment ou j'essai de supprimer le retour chariot, le programme reste bloqué.
Dans ton while (*value != '\0'), si tu fais pas évoluer "value" ce while bouclera à l'infinie !!!

Tu aurais dû étudier la façon dont on supprime les espaces en début de chaine au lieu d'essayer d'adapter bêtement un code analogue pour supprimer le retour charriot final.
Tant que le début de chaine est un espace, alors on décale ce début. Ainsi, on obtient en final un pointeur pointant vers le premier caractère utile et les espaces du début ne sont pas traités. Essayer le même algo sur le return final ne servira absolument à rien car le traitement des chaines se fait dans le sens "début vers '\0'" et donc ce qui marche au début ne marche pas pour la fin.
Ce qu'il faut faire, c'est remplacer chaque return par un '\0'. Comme ça, la chaine se termine juste avant.
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 int parse_config(char *line) { char *key, *value, *fin; key = strtok(line, " \t\r\n"); value = strtok(NULL, ""); if (*value) { // suppression des espaces en debut de value while (isspace(*value)) { value++; } // suppression des retour chariot en fin de value fin=value + strlen(value) - 1; while (*fin == '\n') { *fin='\0'; fin--; } } printf("%s|%s\n", key, value); return 1; }

28/08/2010, 08h49
jeroman
Salut

Cette boucle
Code:

1 2 3 4 5 6 7 while (*value != '\0') { if (*value == '\n') { value--; } }
ne supprime rien, elle ne fait au maximum que décrémenter la valeur du pointeur.
De plus, elle a toutes les chances de tourner à l'infini (d'où ton programme qui se bloque). Normal : tu compares si le caractère situé à l'adresse "value" est différent de '\0'. Or, derrière un '\n', un '\0' n'est pas obligatoirement censé s'y trouver. Par exemple, juste avant, tu testes la présence d'espaces, une espace peut donc s'y trouver. Si tel est le cas, comme "value" ne change plus par la suite, la boucle tourne alors à l'infini sans rien faire.

Tu as oublié de nous donner le principal : le format de ta chaîne à parser. On comprendrait mieux ce que tu cherches à faire et cela nous aiderait à corriger tes erreurs.

Sinon, je ne suis pas certain que

Code:

value = strtok(NULL, "");

puisse fonctionner. La chaine "delim" est vide !
28/08/2010, 15h15
Ganondorf
merci à vous pour vos réponses
je tient quand même a préciser que je n'ai rien recopié "bêtement" ... le code pour la suppression des espaces en début de chaine vient de moi.

j'ai aussi essayé ce code:
Code:

1 2 3 4 5 6 7 fin=value + strlen(value) - 1; while (*fin == '\n') { *fin='\0'; fin--; }
ca boucle toujours à l'infini ...
jeroman ce que je cherche a faire c'est parser un fichier de ce style:

Citation:

remoteAddr 192.168.1.3
remotePort 7252

remotePass crypted
remoteName Test de service

key correspond a 1er argument et value correspond au reste de la chaine, les espaces entre key et value peuvent differer. line correspond a une chaine recuperé via fgets

Si le format des lignes est

Code:

>=0 espaces + key + >=1 espaces + value (qui peut contenir des espaces) + \n ou \r\n

(car sous Windows une ligne d'un fichier texte se termine par CR+LF, ces deux caractères se retrouvant tous les deux dans la chaine remplie par fgets)

... tu peux essayer et adapter ce bout de code à tes besoins :

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
43
44
45
46
47
48
49
50
51
52
#include <stdio.h>
#include <string.h>
#include <ctype.h>
 
int main(void)
{
	char ligne[128] ;
	char key[128] ;
	char value[128] ;
	FILE * fichier ;
 
	fichier = fopen ("test.txt" , "rb" ) ;
	if (fichier == NULL)
		return 0;
 
	while ( fgets(ligne,128,fichier) != NULL )
	{
		char * position_token ;
		char * cherche_caractere ;
 
/* on supprime \r et/ou \n */
		cherche_caractere = strchr(ligne , '\r');
		if (cherche_caractere != NULL)
			*cherche_caractere = '\0' ;
		cherche_caractere = strchr(ligne , '\n');
		if (cherche_caractere != NULL)
			*cherche_caractere = '\0' ;
 
		key[0] = '\0' ;
		value[0] = '\0' ;
 
		position_token = strtok(ligne , " "); /* on cherche le premier token ("key") */
		if ( position_token != NULL ) /* si c'est OK... */
		{
			strncpy(key,position_token,128) ; /* on le copie dans "key" */
			position_token = position_token + strlen(key) + 1 ; /* on se positionne après le premier token */
			while ( *position_token && isspace(*position_token) ) /* on vire tous les espaces éventuels entre "key" et "value" */
				position_token++;
			strncpy(value,position_token,128); /* on copie ce qui reste dans "value" */
		}
 
		if (*key && *value) /* on affiche si les deux chaines sont remplies */
		{
			printf("key : %s\n" , key);
			printf("value : %s\n\n" , value);
		}
	}
 
	fclose(fichier);
 
	return 0;
}

Contenu de "test.txt" :

Code:

1
2
3
4
5
   remoteAddr   192.168.1.3   
remotePort       7252
 
  remotePass crypted
    remoteName    Test de service

J'ai volontairement rajouté plein d'espaces un peu partout pour tester toutes les possibilités.

Cela donne chez moi :

Code:

1
2
3
4
5
6
7
8
9
10
11
key : remoteAddr
value : 192.168.1.3
 
key : remotePort
value : 7252
 
key : remotePass
value : crypted
 
key : remoteName
value : Test de service

Y a peut-être moyen de l'améliorer, je l'ai fait à la va-vite.

salut jeroman et encore merci pour ton aide, j'ai réussi a obtenir ce que je voulais de cette maniere:

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
 
int parse_config(char *line)
{
    char *key, *value, *tmp;
 
    key = strtok(line, " \t");
    value = strtok(NULL, "");
 
    if (*value)
    {
		// suppression des espaces en debut de value
		while (isspace(*value))
		{
			value++;
		}
 
                tmp = value;
		while (*tmp)
		{
			if (*tmp == '\n')
			{
				*tmp++ = 0;
			}
			tmp++;
		}
	}
	printf("%s|%s|\n", key, value);
	return 1;
}

résultat:

Citation:

ca fonctionne a merveille
bonne journée

28/08/2010, 18h27
Sve@r
Citation:

Envoyé par havoc50

je tient quand même a préciser que je n'ai rien recopié "bêtement" ... le code pour la suppression des espaces en début de chaine vient de moi.

Ok. Pas de souci. J'ai conclu brièvement à tort et j'en suis désolé.
Citation:
Envoyé par havoc50

j'ai aussi essayé ce code:

Code:

1 2 3 4 5 6 fin=value + strlen(value) - 1; while (*fin == '\n') { *fin='\0'; fin--; }

ca boucle toujours à l'infini ...
Bizarre. La seule raison qui puisse faire que ça boucle à l'infini serait que la chaine "value" ne contienne que des \n. T'es sûr que ça boucle à cet endroit ?

Suffit de vérifier avec du printf. Et, pour plus de sûreté, empêcher "fin" de dépasser le début de "value"
Code:

1 2 3 4 5 6 7 fin=value + strlen(value) - 1; while (fin > value && *fin == '\n') { printf("value=0x%04x, fin=0x%04x [0x%02x]\n", value, fin, *fin); *fin='\0'; fin--; }