Discussion :

[Ganondorf]

salut à tous

j'ai un problème avec le code suivant:

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

[Sve@r]

salut à tous

j'ai un problème avec le code suivant:

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

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

[jeroman]

Salut

Cette boucle

Code :

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

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 :

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

    value = strtok(NULL, "");

puisse fonctionner. La chaine "delim" est vide !

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

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

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:

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

[jeroman]

Si le format des lignes est

Code :

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

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

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

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

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 :

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

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.

[Ganondorf]

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

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

remoteAddr|192.168.1.3|
remotePort|7252|
remotePass|link|

ca fonctionne a merveille
bonne journée

[Sve@r]

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

j'ai aussi essayé ce code:

Code :

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

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 :

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

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