Discussion :

[kissmytoe]

Bonjour
J'essaie de lire un fichier dont chaque ligne est structurée de cette manière :

Renner-Nantz, Jody (530) 752-2906 (530) 753-4759 jjrennernantz@ucdavis.edu

Voici mon code :

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
 
while (fgets(ligne,MAX_LIGNE,f) != NULL)
	{
		LigneAnnuaire employe;
		employe.nom = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.prenom = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.indTelephone = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.telephone = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.indFax = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.fax = (char*) malloc(sizeof(char)*TAILLE_MAX);
		employe.email = (char*) malloc(sizeof(char)*TAILLE_MAX);
 
		mot = strtok(ligne, " ,()");
		strcpy(employe.nom,mot);
 
		mot = strtok(NULL, " ,()");
		strcpy(employe.prenom,mot);
 
		mot = strtok(NULL, " ,()");	
		strcpy(employe.indTelephone,mot);
 
		mot = strtok(NULL, " ,()");
		strcpy(employe.telephone,mot);
 
		mot = strtok(NULL, " ,()");
		strcpy(employe.indFax,mot);
 
		mot = strtok(NULL, " ,()");
		strcpy(employe.fax,mot);
 
		mot = strtok(NULL, " ,()");
		strcpy(employe.email,mot);

Bien entendu mon code ne fonctionne pas comme il le faut, il ne coupe pas les mots comme il le faut.
Par exemple je lis Renner-Nantz, ensuite Jody (donc tout va bien) et apres je devrais lire 530 mais je lis un caractere inconnu ! :S

Merci pour tout aide
Mary

[fearyourself]

Bonsoir,

Quelques remarques :

1) Pas de cast après un malloc
2) sizeof(char) retourne toujours 1, c'est donc un pléonasme lorsqu'on l'utilise dans une multiplication
3) faire un malloc d'une taille fixe est une erreur de conception, déclare ta structure comme ceci sauf si quelque part tu alloues des tailles différentes...

Code :

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

1
2
3
4
5
6
7
8
9
10
typedef struct sla {
	char 	employe[TAILLE_MAX],
		prenom[TAILLE_MAX],
		nom[TAILLE_MAX],
		indTelephone[TAILLE_MAX],
		telephone[TAILLE_MAX],
		indFax[TAILLE_MAX],
		fax[TAILLE_MAX],
		email[TAILLE_MAX];
}LigneAnnuaire;

4) strtok est une fonction à éviter à tout prix (c'est même écrit dans le man).
5) si tu l'utilise, teste au moins le retour
6) Ce code :

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define TAILLE_MAX 64
 
typedef struct sla {
	char 	employe[TAILLE_MAX],
		prenom[TAILLE_MAX],
		nom[TAILLE_MAX],
		indTelephone[TAILLE_MAX],
		telephone[TAILLE_MAX],
		indFax[TAILLE_MAX],
		fax[TAILLE_MAX],
		email[TAILLE_MAX];
}LigneAnnuaire;
 
int main()
{
	char ligne[] = "Renner-Nantz, Jody (530) 752-2906 (530) 753-4759 jjrennernantz@ucdavis.edu";
	char *mot;
	LigneAnnuaire employe;
 
	mot = strtok(ligne, " ,()");
 
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.nom,mot);
  	printf("%s\n",employe.nom);
 
	mot = strtok(NULL, " ,()");
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.prenom,mot);
  	printf("%s\n",employe.prenom);
 
	mot = strtok(NULL, " ,()");	
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.indTelephone,mot);
  	printf("%s\n",employe.indTelephone);
 
	mot = strtok(NULL, " ,()");
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.telephone,mot);
  	printf("%s\n",employe.telephone);
 
	mot = strtok(NULL, " ,()");
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.indFax,mot);
  	printf("%s\n",employe.indFax);
 
	mot = strtok(NULL, " ,()");
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.fax,mot);
  	printf("%s\n",employe.fax);
 
	mot = strtok(NULL, " ,()");
	if(mot == NULL) {
		printf("Erreur avec la ligne\n");
		return EXIT_FAILURE;
	}
	strcpy(employe.email,mot);
 	printf("%s\n",employe.email);
 
	return EXIT_SUCCESS;
}

M'affiche :

Renner-Nantz
Jody
530
752-2906
530
753-4759
jjrennernantz@ucdavis.edu

Jc

[Médinoc]

1. +1
2. sizeof(*uneChaine) peut être utile, mais sizeof(char) est toujours inutile en effet.
3. +1
4. strtok() est une fonction à éviter quand on le peux. Sous un Visual Studio datant d'avant la version 2005, strtok_r() n'est pas disponible et ne possède pas d'équivalent (ou alors, il faut le coder soi-même).
5. +1
6. TLDR.

[kissmytoe]

J'ai demandé à personne de commenté les problèmes de mon code.
J'apprécie toujours des commentaires pertinents sur mon code qui pourraient m'aider mais quand il ya une réponse a mon probleme.

Ca change absolument rien je sais deja que strtok n'est pas conseillé mais ca m'aide pas plus a lire mon fichier correctement

Maryse

[kissmytoe]

Voici la résolution du problème trouvé par un de mes amis
mot = strtok(ligne, " ,()\t-");

Il manquait un \t pour enlever les tabulations.
C'est pour ca qu'il ne lisait pas les espaces comme il le fallais car c'était un caractere tabulations qui formait un espece de carré noir dans le debugger.

Mary

[zooro]

Voici la résolution du problème trouvé par un de mes amis
mot = strtok(ligne, " ,()\t-");

Il manquait un \t pour enlever les tabulations.
C'est pour ca qu'il ne lisait pas les espaces comme il le fallais car c'était un caractere tabulations qui formait un espece de carré noir dans le debugger.

Mary

Pas de chance pour nous... Nous aurions eu du mal à trouver, il n'y en avait pas dans la chaîne d'exemple.