Discussion :

[Jules_Courné]

Bonjour j'ai besoin de votre aide pour un code utilisant fopen. Celui ne me retourne pas "impossible d'ouvrir le fichier" donc j'en ai déduit que le fichier a bien été ouvert. Cependant rien de ce que j'ai met dans ma condition else ne s'affiche, ca me laisse un curseur en attente. En ayant activé les erreurs avec "gcc -Wall -W -Werror" rien ne s'affiche et valgrind 0 erreurs.
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
int main(){
 
	FILE* fichier = NULL; 
	fichier = fopen("json_test_files/test.json", "r");
	int size = 0;
 
 
	/* si fichier est vide on arrête tout*/
	if(!fichier){
		printf("impossible d'ouvrir le fichier\n");
		exit(1);
	}
	else{
		printf("fdsfsfds"); // j'ai essayé de mettre n'importe quoi dans le else mais rien ne s'affiche
		char r = fgetc(fichier);
		/*on calcule la taille du fichier*/
		while(r != EOF){
			size++;
 
		}
		char *chaine = malloc((size) * sizeof(char));//j'alloue une chaine de taille "size" de char
 
		/*tant que le fichier n'est pas fini, on le met dans chaine*/
		while(fgets(chaine, size, fichier)){
			analyseurLR(chaine);//fonction qui renvoie 0 si l'analyse a réussi, 1 sinon
		}
		free(chaine);
	}
	fclose(fichier);
	return 0;
}

Merci d'avance pour vos réponses!!

[Jules_Courné]

j'ai essayé d'une deuxieme facon où je tape l'executable avec le nom du fichier dans la commande gcc (pas sur du (void)argc ) qui me fait le même résultat.

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
int main(int argc, char* argv[]){
 
	(void)argc;
	FILE* fichier = NULL; 
	fichier = fopen(argv[0], "r");
	int size = 0;
 
 
	/* si fichier est vide on arrête tout*/
 
 
	if(fichier){
		printf("fdsfsfds"); // j'ai essayé de mettre n'importe quoi dans le else mais rien ne s'affiche
		char r = fgetc(fichier);
		/*on calcule la taille du fichier*/
		while(r != EOF){
			size++;
		}
		char *chaine = malloc((size) * sizeof(char));//j'alloue une chaine de taille "size" de char
 
		/*tant que le fichier n'est pas fini, on le met dans chaine*/
		while(fgets(chaine, size, fichier)){
			analyseurLR(chaine);//fonction qui renvoie 0 si l'analyse a réussi, 1 sinon
		}
		free(chaine);
	}
	else{
		printf("impossible d'ouvrir le fichier\n");
			exit(1);
	}
 
	fclose(fichier);
	return 0;
}

[Bousk]

Si tu ne passes pas dans le if alors tu passes forcément dans le else.
Par contre, il faut rewind le fichier après la boucle de fgetc, sinon tu es à la fin et les fgets suivants ne liront rien.

[dalfab]

Bonjour,

- Ligne 5, l'argument passé à l'executable est le second de argv, il faut donc mettre [1].
- Ligne 13, il n'y a pas de \n à la fin du printf donc il risque de ne jamais être visible. Mettre le \n ou forcer l'émission en appelant fflush(stdout);.
- Ligne 14, r est déclaré comme un char mais fgetc() retourne un int car c'est soit un caractère soit EOF. Il fait définir r comme étant un int.
- Ligne 16, le test risque d'être toujours faux (on compare l'int EOF à un char) et le programme bloque alors indéfiniment. Voir commentaire ligne 14.
- Ligne 19, il faut allouer un char de plus pour stocker le caractère de fin de chaine (le problème se verra sur un fichier d'une seule ligne.)
- Ligne 20, il faut faire un rewind(fichier) comme l'a expliqué Bousk.

[Sve@r]

Bonjour
As-tu remarqué que r ne change jamais dans ta boucle ? Il semble que je sois le premier à le voir

Sinon les autres erreurs dans ce code

• ton printf() de debug n'est pas forcé de s'afficher au moment où il s'exécute car il ne contient pas de '\n'. Certes il s'affichera à un moment ou à un autre (au pire quand ton programme se termine) mais pas forcément quand tu le souhaites => termine ta string par un '\n' (et en plus ton debug sera plus propre avec des lignes bien franches)
• tu utilises un char pour chercher EOF. Or EOF vaut 0xffff (soit un int). Et casté sur un char, 0xffff devient 0xff. Comment alors feras-tu la différence entre "0xff" (EOF casté) et "0xff" (caractère de valeur ascii 255, qui peut très bien se trouver dans ton fichier) ? Ce n'est pas pour rien si la fonction fgetc() retourne un int.
• la fonction fgets() lit un caractère de moins que la taille demandée, pour garder une place de libre pour y stocker le '\0' qu'elle positionne elle-même. Il te faut donc demander "size+1" pour être sûr de tout lire (et donc allouer "size+1" char pour être sûr d'avoir la place pour tout y compris le '\0' qui viendra en plus)
• la fonction fgets() s'arrête au premier '\n' rencontré (ce caractère symbolisant une fin de ligne, ce qui est alors marquant pour une fonction chargée de lire une ligne). Si tu veux lire un fichier à la "brute force" il te faut utiliser fread(). Et dans ce cas, il n'y aura pas de '\0' de positionné donc à moins de le mettre manuellement tu ne pourras pas traiter "chaine" comme une chaine (interdit d'y appliquer la moindre fonction "strxxx()")
• une fois que tu es à la fin du fichier, tout comme quand tu es à la fin d'un livre, ben tu as alors tout lu et il n'y a plus rien à lire. Si tu veux le relire il te faut revenir au début

Et sinon tu es sûr de vouloir lire argv[0] ?

Bref contrairement à ton titre, le vrai problème ce n'est pas fopen() mais tout le reste. Tu aurais d'ailleurs dû t'en douter que si la fonction n'a pas renvoyé NULL c'est qu'elle a réussi mais si le reste ne suit pas, alors c'est que cela vient du reste. Et dernier détail, le (void)argc, lui, restera pour moi à jamais une énigme.