Discussion :

[pek.plus.ultra59]

Bonjour,

Je suis en train de développer un programme en C qui me permet d'enregistrer un fichier dans un tableau de structure. J'obtiens une erreur de compilation qui est:
erreur incompatible types in return.

Voici quelques bouts de code.

Déclaration de ma structure:

Code :

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

1
2
3
4
5
6
 
typedef struct {
                       char ville[50];
                       int nb;
                    } rmi;
rmi tab[160];

Ma fonction de copie:

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
 
rmi file_copy(char nomfich[])
{
	rmi tab[160];
	FILE *fiche = NULL;
	char *p_temp = NULL;
	char *p_temp_word = NULL;
 
	char buffer[1024];
	int i=0, cpt = 0;
 
 
 
	/* ----- Ouverture du fichier ----- */
 
	fiche = fopen (nomfich, "r");
 
   	if (!fiche)
	   {
	      /* Erreur: impossible d'ouvrir le fichier. */
	      printf("Erreur: impossible d'ouvrir le fichie");
	   }
 
	while (fgets(buffer,BUFSIZ,fiche) )
		{
		 char *p = NULL;
		 p=strstr(buffer, "\n");
		    if (p != NULL)
			{
			 *(p-1) = 0;
			}
		    else
			{
			 memset (buffer, 0, sizeof (buffer));
			}
		 strcpy(tab[i].ville,buffer);
		 tab[i].nb = 0;
	         p_temp = NULL;				 
	      	 i++;cpt++;
	        }
fclose(fiche);
return tab;
 
}

Pouvez vous m'aider please?

[Médinoc]

Ta fonction a un type de retour structure, et tu cherches à lui faire retourner un tableau (l'adresse d'un tableau local, qui plus est...)

[pek.plus.ultra59]

Donc il faut que je déclare mon tableau en global comme ce ci?
Sinon comment je peux retourner mon tableau de type structure rmi?

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
 
rmi file_copy(char nomfich[],rmi tab[];)
{
 
	FILE *fiche = NULL;
	char *p_temp = NULL;
	char *p_temp_word = NULL;
 
	char buffer[1024];
	int i=0, cpt = 0;
 
 
 
	/* ----- Ouverture du fichier ----- */
 
	fiche = fopen (nomfich, "r");
 
   	if (!fiche)
	   {
	      /* Erreur: impossible d'ouvrir le fichier. */
	      printf("Erreur: impossible d'ouvrir le fichie");
	   }
 
	while (fgets(buffer,BUFSIZ,fiche) )
		{
		 char *p = NULL;
		 p=strstr(buffer, "\n");
		    if (p != NULL)
			{
			 *(p-1) = 0;
			}
		    else
			{
			 memset (buffer, 0, sizeof (buffer));
			}
		 strcpy(tab[i].ville,buffer);
		 tab[i].nb = 0;
	         p_temp = NULL;				 
	      	 i++;cpt++;
	        }
fclose(fiche);
return tab;
 
}

[Médinoc]

Si tu connais à l'avance la taille maximale du tableau, du peux passer le tableau en paramètre en effet.
Dans ce cas, je te conseille de passer aussi sa taille en paramètre, c'est une bonne pratique :

Code C :

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
#define MY_BUFFER_SIZE 1024
 
/* Une pratique commune en C:
   On retourne 0 si OK, -1 si erreur. */
int file_copy(char const nomfich[], rmi tab[], size_t taille)
{
	FILE *fiche = NULL;
 
	char buffer[MY_BUFFER_SIZE];
	size_t i=0;
 
	/* ----- Ouverture du fichier ----- */
	fiche = fopen (nomfich, "r");
 
   	if (!fiche)
	{
		/* Erreur: impossible d'ouvrir le fichier. */
		printf("Erreur: impossible d'ouvrir le fichie");
		return -1;
	}
 
	while (fgets(buffer, MY_BUFFER_SIZE, fiche) && i<taille )
	{
		/* Cherche le caractère de nouvelle ligne (Line Feed) */
		char *pLF = NULL;
		pLF = strchr(buffer, '\n');
		if (pLF != NULL)
		{
			*p = '\0';
		}
		else
		{
			/* On finit de lire la ligne jusqu'au LF */
			int c;
			while((c=fgetc(fiche)) != '\n' && c != EOF)
			{ }
 
			/* ??? */
			memset (buffer, 0, sizeof (buffer));
		}
		strcpy(tab[i].ville, buffer);
		tab[i].nb = 0;
	      	i++;
	}
	fclose(fiche);
	return 0;
}

[pek.plus.ultra59]

Je te remercie cela fonctionne à merveille. En tout cas chapeau
Ça faisait 5 ans que je n'avais plus programmé donc je ne te dis pas comment je suis un peu rouillé

J'ai du modifié quand même un oubli et une érreur que je ne comprend pas vraiment. (voir le texte en rouge)

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
while (fgets(buffer, MY_BUFFER_SIZE, fiche) && i<taille )
	{
		/* Cherche le caractère de nouvelle ligne (Line Feed) */
		char *pLF = NULL;
		pLF = strchr(buffer, '\n');
		if (pLF != NULL)
		{
                    /* Petite erreur mais je ne sais pas pourquoi il faut mettre -1*/
			*(pLF-1) = '\0';
		}
		else
		{
			/* On finit de lire la ligne jusqu'au LF */
			int c;
			while((c=fgetc(fiche)) != '\n' && c != EOF)
			{ }
 
		}
		strcpy(tab[i].ville, buffer);
		tab[i].nb = 0;
	      	i++;
	}

[souviron34]

Code C :

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

1
2
/* Une pratique commune en C:
   On retourne 0 si OK, -1 si erreur. */

Euh....

Une pratique commune en C....

Il arrive de nombreuses fois qu'on fasse comme un booléen, c'est à dire retourner 0 si erreur ou 1 si ok :

Code :

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

1
2
3
4
5
6
if ( Ouverture_Fichier(....) && Lecture_Fichier(...) )
{
   /* OK */
}
else
 return ERROR ;

[Médinoc]

pek.plus.ultra59 : Es-tu sûr qu'il faut vraiment mettre un -1 ? Je l'ai retiré volontairement, tu sais: Avec le -1, on ne supprime pas seulement le \n, mais aussi le caractère qui le précède!

souviron34 : En effet, Windows fait ça. Mais généralement dans ce cas-là, on spécifie d'une autre manière que le résultat est booléen (avec un typedef, par exemple).

Mais j'ai été un peu restrictif, je l'admet: Généralement, quand une fonction standard C ou POSIX retourne un int, c'est plutôt "non-négatif pour succès, négatif pour échec" (et c'est pareil pour les HRESULT, au passage).