Discussion :

[Legio]

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 :

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
     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®ome]

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 :

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

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.

[Trap D]

tu peux fair une truc du genre

Code :

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

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 :

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

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
  .......
}

[D[r]eadLock]

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 :

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
#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 */
}

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

[je®ome]

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

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 :

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
 #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; 
}

[D[r]eadLock]

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".

[je®ome]

Code :

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

1
2
 char *a="fichier\n";

Je ne savias pas que ceci était un const.

et alors ça marche, veuillez m'excuser

et merci

[Trap D]

Code :

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

char *a="fichier\n";

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

Code :

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

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é

[Trap D]

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 :

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
 #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.