Discussion :

[wallace27]

Bonjour,

Est-ce que quelqu'un saurait m'expliquer pour quelle raison cette fonction ne retourne pas le mot contenu dans le pointeur "*nomFich" :

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
char *lectureFichier(){
	FILE *fich;
	char ligne[LGL];
	char nomFichier[LGL];
	char *nomFich; // pointeur permettant de retourner le nom du fichier
	nomFich = ligne;
	printf("Nom du fichier a lire : ");
	fgets(ligne, LGL, stdin);
	sscanf(ligne, "%s", nomFichier);
	fich = fopen(nomFichier, "r");
	if (fich == NULL){
		printf("Ouverture impossible fichier %s\n", nomFichier);
	}
	else{
		printf("Fichier %s \n", nomFichier);
		fgets(ligne, LGL, fich); // Lecture 1ère ligne
		while (!feof(fich)) { // Tant que non fin de fichier
			printf("%s", ligne); // Imprimer ligne à l’écran
			fgets(ligne, LGL, fich); // Lecture ligne suivante
		}
		fclose(fich);
	}
	printf("Contenu de nomFich : ");
	puts(nomFich);
	return nomFich;
}

Merci pour votre aide.

[Invité]

Car nomFich est une variable local, et que ce qui est local, reste local ?

[wallace27]

mais le propre d'une fonction n'est-il pas d'affecter le contenu retourné d'une variable locale à une variable du "main" ?

[diogene]

Oui ...

... mais nomFich contient l'adresse du premier élément du tableau local ligne et c'est donc cette adresse qui est retournée. Comme les objets locaux sont détruits en sortie de la fonction, ce tableau est détruit et tu stockes alors l'adresse de quelque chose qui n'existe plus.

Moralité : On ne doit jamais retourner, de quelque façon que ce soit, l'adresse d'un objet local.

[wallace27]

Est-ce qu'avec un void qui par définition ne retourne rien je peux contourner le problème ?

[Dev 37C]

Bonjour, pour éviter ce problème alloue dynamiquement nomFichier

[wallace27]

Bonjour, pour éviter ce problème alloue dynamiquement nomFichier

Allouer dynamiquement comme ici par exemple ?

[Sve@r]

Bonjour

Tu devrais écrire encore plus gros.

Donc oui, écrit ainsi (renvoyer une adresse allouée via malloc) ça fonctionne. Charge alors à l'appelant de libérer la mémoire allouée une fois qu'il n'en aura plus besoin. Mais tu peux remplacer malloc+strcpy par strdup qui fait exactement le même travail.

Ce qui ne fonctionne pas, en revanche, c'est ta méthode pour enlever le '\n' car rien ne te garantit qu'il y en a un. Et si ce n'est pas le cas, tu enlèves alors un caractère utile de ta chaine.

Code c :

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

1
2
3
// Enlever '\n'
char *pt;
if ((pt=strchr(ligne, '\n')) != NULL) *pt='\0';

Est-ce qu'avec un void qui par définition ne retourne rien je peux contourner le problème ?

Tout à fait. Puisqu'un void ne retourne rien, aucun risque alors de retourner l'adresse d'une variable locale. Ainsi tu n'as plus de problème (ni de solution non plus d'ailleurs).

Code c :

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

1
2
3
4
5
fgets(ligne, LGL, fich); // Lecture 1ère ligne
while (!feof(fich)) { // Tant que non fin de fichier
	printf("%s", ligne); // Imprimer ligne à l’écran
	fgets(ligne, LGL, fich); // Lecture ligne suivante
}

Ca fonctionne mais il y a plus simple

Code c :

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

1
2
while (fgets(ligne, LGL, fich) != NULL)
	printf("%s", ligne);

Hé oui, puisque fgets() lit et aussi indique si la lecture a pu se faire, alors autant s'en servir...

[ternel]

En fait, while (feof()) ne fonctionne pas vraiment.
feof() indique si la dernière erreur était une fin de fichier, alors qu'il y a d'autre causes d'erreur.

Par contre, la bonne solution est bien celle que tu as proposée: tester la valeur de retour de la fonction de lecture.