Probleme d'arret du programme

Hmm j'ai un probleme quand j'execute mon code le programme se ferme des que j'ai entrer le nom de fichier et il n'y as pas l'air d'avoir un probleme avec l'ouverture du fichier puisqu'il ne m'affiche pas "Erreur sur fichier" .
Si quelqu'un as une idée sur l'origine de cette fermeture précipitée voici le bout de code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 char nomfichier[50]; printf("\nIndiquez l'emplacement et le nom du fichier a verifier (ex:C:\\..\\nom.txt)\n"); fgets(nomfichier, sizeof nomfichier, stdin); clean(nomfichier, stdin); FILE *fichier = fopen(nomfichier,"r"); if (fichier==NULL) { printf("Erreur sur fichier"); } else{ /* compter les mots */ char tmp[50]; while ((fscanf(fichier, "%49s", tmp))==1) { printf ("%s\n", tmp); cpt++; } printf("Il y a %d mots dans le fichier \n", cpt); }
Le programme a l'air de s'arretter a ma boucle while(fscanf....) mais je ne comprends pas pourquoi :(

Je pense que clean n'est pas standard, que veux tu faire ?

tu passes un chaîne de 50 caractères a fopen, je ne suis pas si certain que ton fichier s'ouvre.
Il faut que tu donne seulement C:\\..\\fichier à la fonction fopen.

Il devrait t'afficher une erreur. ( sous linux, c'est le cas )

tu peux fair une truc du genre
Code:

1 2 3 4 5 6 7 8 9 10 11 int i=0; char *a; while(nomfichier[i]!='\n') { i++; } a=malloc(i); strcnpy(a,nomfichier,i); fichier=fopen(a,"r"); free(a);
dans ce cas, ajoutes <string.h> en en-tête
Ce n'est peut-être pas la meilleure méthode, mais ça devrait marcher et le reste me semble être correct.

Citation:
Envoyé par je®ome
tu peux fair une truc du genre
Code:

1 2 3 4 5 6 7 8 9 10 11 int i=0; char *a; while(nomfichier[i]!='\n') { i++; } a=malloc(i); strcnpy(a,nomfichier,i); fichier=fopen(a,"r"); free(a);
dans ce cas, ajoutes <string.h> en en-tête
Ce n'est peut-être pas la meilleure méthode, mais ça devrait marcher et le reste me semble être correct.
Sûr, tu as toutes les chances d'avoir un segfault, puisque la chaîne a ne se termine pas par le 0 terminal donc n'est pas une chaîne C valide.
De plus tu ne verifies pas le retour de malloc.
Pour supprimer le '\n' de fin de saisie de fgets on fait en général comme ça :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 char *p; .... p = strchr(nomfichier, '\n'); if (p != NULL) // on a trouvé le '\n', on le vire *p = 0; else / ici, le buffer de saisie de fgets est trop court { // a toi de voir ce que tu veux faire ....... }

Tu peux afficher le nom du fichier que tu tente d'ouvrir et voir s'il y a un probleme. Essaie aussi d'hardcoder le nom du fichier et vois quelle partie ne marche pas (lecture du fichier ou la boucle).

Je ne comprends pas ton "a l'air de s'arreter". Le programme t'affiche quand meme "Il y a 0 mots dans le fichier" ??

Parce qu'avec ça (nom de fichier harcodé), ça marche sous Linux:
Code:

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 #include <stdio.h> int main(void) { unsigned int cpt=0; const char *nomfichier="legio.c"; FILE *fichier = fopen(nomfichier,"r"); printf("Fichier: `%s'\n",nomfichier); if (fichier==NULL) { printf("Erreur sur fichier"); } else { /* compter les mots */ char tmp[50]; while ((fscanf(fichier, "%49s", tmp))==1) { printf ("%s\n", tmp); cpt++; } printf("Il y a %d mots dans le fichier \n", cpt); } return 0; /* commentairesuperlongquidepasseles49caracteresjepensebienenfinjespere */ }
Citation:

Envoyé par ./legio

Fichier: `legio.c'
#include
<stdio.h>
int
main(void)
{
unsigned
int
cpt=0;
const
char
*nomfichier="legio.c";
[...]
\n",
cpt);
}
return
0;
}
/*
commentairesuperlongquidepasseles49caracteresjepe
nsebienenfinjespere
*/
Il y a 61 mots dans le fichier

Citation:

Sûr, tu as toutes les chances d'avoir un segfault, puisque la chaîne a ne se termine pas par le 0 terminal donc n'est pas une chaîne C valide.

Trap D, je suis curieux de savoir comment toi tu arrives a ouvrir un fichier inexistant.
Du moins sous linux ceci ne marche pas

Citation:

fopen("fichier\0","r") pour ouvir fichier

Désolé pour le retour de malloc, j'ai oublié de l'ajouter.
Je ne comprends pas vraiment l'exemple que tu as donné, il y a erreur de segmentation dans *p=0
Code:

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 #include <stdio.h> #include <stdlib.h> #include <unistd.h> #include <string.h> int main() { char *a="fichier\n"; char *p; p=strchr(a,'\n'); if(p!=NULL) { *p=0; } FILE *fd=fopen(a,"r"); if(fd==NULL) { perror("fopen"); exit(1); } fclose(fd); /* tester retour de fclose */ return 0; }

Citation:

Envoyé par je®ome

Je ne comprends pas vraiment l'exemple que tu as donné, il y a erreur de segmentation dans

Normal: "fichier\n" est un const char * et non un char * ! Tu n'as pas le droit de modifier cette chaine de caractere.

Sinon "fichier\0" est équivalent à "fichier".

Citation:
Envoyé par je®ome
Trap D, je suis curieux de savoir comment toi tu arrives a ouvrir un fichier inexistant.
Du moins sous linux ceci ne marche pas

Désolé pour le retour de malloc, j'ai oublié de l'ajouter.
Je ne comprends pas vraiment l'exemple que tu as donné, il y a erreur de segmentation dans *p=0
Code:

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 #include <stdio.h> #include <stdlib.h> #include <unistd.h> #include <string.h> int main() { char *a="fichier\n"; char *p; p=strchr(a,'\n'); if(p!=NULL) { *p=0; } FILE *fd=fopen(a,"r"); if(fd==NULL) { perror("fopen"); exit(1); } fclose(fd); /* tester retour de fclose */ return 0; }
On ne parle pas de la même chose, je te montrais la manière de supprimer le '\n', pas de la manière d'ouvrir un fichier inexistant.
Pour le reste, il t'a déjà été répondu.

Code:

1 2 char *a="fichier\n";
Je ne savias pas que ceci était un const.

et alors ça marche, veuillez m'excuser

et merci

Code:

char *a="fichier\n";

effectivement a est dans une zone mémoire non modifiable.
Par contre

Code:

char a[]="fichier\n";

Ici le tableau est créé dans la pile d'exécution et initialisé avec la chaîne , et là ça aurait fonctionné

Merci pour l' exemple Trap D.