Discussion :

[sihamnet]

Bonjour ,

j'aimerais bien lire un fichier qui contient des mots séparés par des espaces
et placer ces mots dans un tableau alloué dynamiquement

pour un fichier d'entiers séparés par des espaces , ça marche très bien

j'ai procédé de la même manière en implémentant le code ci dessous , mais ça donne rien lors de la compilation

j'attend vos conseils ,ils seront les bienvenue

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
45
46
47
48
49
50
51
52
53
54
55
56
 
#include <stdlib.h>
#include <stdio.h>
#include <conio.h>
 
int main()
{
 
    int cpt; int i; char mot[25]={0};
    char *tab;
 
    FILE *fichier = NULL;
 
    fichier = fopen("fich_mot.txt", "r+");
 
 
    cpt=0;
    while(!feof(fichier))
    {
        fscanf(fichier,"%s",mot);
        cpt++;
    }
    tab =(char*)malloc(cpt*sizeof(char));
 
    //Tester l'ouverture du fichier
    if(fichier != NULL)
    {
        rewind(fichier);//renvoyer à la position 0 dans le fichier
        i = 0;
        while (!feof(fichier))
        {             
             mot = "";
             fscanf(fichier,"%s",mot);
             tab[i]=mot;
             i++;
        }
 
 
        printf("les elements du tableau construits sont : \n");
        for(i=1 ; i<cpt  ;i++)
        {  
             printf ("tab[%d] = %s ",i,tab[i]);
             printf("\n"); 
        }
 
        free(tab);  
        fclose(fichier);
     }
    else
    {
        printf("Impossible d'ouvrir le fichier fich_entier.txt");
    }
 
  getch();	
  return 0;
}

[Invité]

Bonsoir,

Quelques remarques préliminaires :

• tab =(char*)malloc(cpt*sizeof(char)); : le cast de malloc() est inutile en C. Vérifier cette allocation serait mieux
• ligne 26 : vérifier if(fichier != NULL), c'est bien, mais avant la première utilisation (ligne 18), c'est mieux
• Plutôt que d'utiliser getch() et <conio.h>, qui ne sont absolument pas portables, utilise getchar() inclu dans <stdio.h>.
• Ton utilisation de feof() n'est pas correcte. Cf FAQ A quoi sert la "fonction" feof ?
• mot étant un tableau, tu ne peux pas faire mot = ""; (qui au passage est inutile vu la suite). mot[0] = '\0';.

• Code :

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

  1
  2
  printf ("tab[%d] = %s ",i,tab[i]);
  printf("\n");

  Tu peux directement inclure \n dans la première ligne : printf ("tab[%d] = %s\n",i,tab[i]);

Venons-en maintenant au cœur de ton problème

Pour commencer, tu te sers d'un simple pointeur sur char char *tab; pour stocker plusieurs chaînes de caractères.
La ligne

Code :

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

tab = (char*)malloc(cpt*sizeof(char));

ne fait qu'allouer de l'espace pour cpt caractères. Et chaque tab[i] ne peut stocker en fait qu'un caractère.
D'où l'erreur que tu as à la ligne printf ("tab[%d] = %s ",i,tab[i]); qui te dit que %s voudrait bien un char * et non char.

Une autre erreur qui découle en partie de la précédente : tab[i]=mot;.
L'erreur que tu as vient du fait que tu mets l'adresse d'un tableau à la place d'un caractère char donc.

Mettons que tu aies corrigé l'erreur précédente et que tab[i] représente bien un char *. Tu n'auras alors plus d'erreur lors de la compilation... Mais qu'en est-il lors de l'exécution ? Je te laisse le découvrir
Un indice : mot n'est qu'un tableau dont l'adresse est fixe, et dont tu remplaces le contenu à chaque lecture.

Bonne continuation

[imperio]

Hum... Je ne m'y serais pas du tout pris comme ça. J'aurais plutôt utiliser la fonction getline et j'aurais ensuite parser le buffer qu'elle m'aurait retourné. Ca me semble plus simple.

[moins1]

getline() n'est pas standard. Comme le PO utilise conio.h y'a de grosse chance qu'il utilise Windows. Ça risque donc de ne pas fonctionner. Du moins avec mingw.

[Bktero]

Code :

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

1
2
char mot[25]={0};
mot = "";

Et bim !

Code :

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

D:\Users\pgradot\Documents\C\main.c|32|error: incompatible types when assigning to type 'char[25]' from type 'const char *'|

Il faut utiliser strcpy() pour copier une chaine dans une autre.

Code :

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

tab[i]=mot;

Pourquoi essayer de copier une chaine dans un caractère ?

Code :

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

1
2
#include <conio.h>
getch();

Enlève conio.h et utilise la fonction getchar(). Tu gagneras en portabilité en utilisant une fonction standard. Ce qui se trouve dans conio.h est spécifique à Windows et ne se comporte pas toujours pareil.

[imperio]

getline() n'est pas standard. Comme le PO utilise conio.h y'a de grosse chance qu'il utilise Windows. Ça risque donc de ne pas fonctionner. Du moins avec mingw.

Je m'en suis toujours servi sans aucun soucis sous linux. Mais si c'est pas standard il vaut mieux éviter dans ce cas. Dommage, elle est si pratique cette fonction...

[Bktero]

http://linux.die.net/man/3/getline

Conforming To

Both getline() and getdelim() were originally GNU extensions. They were standardized in POSIX.1-2008.

Je viens de tenter de m'en servir avec MinGW sous Windows 7, sans succès...

[Metalman]

Exact, j'ai dû la recoder récemment pour la faire fonctionner sur MinGW32...
...et c'était moyen-moyen comme expérience !

[imperio]

Exact, j'ai dû la recoder récemment pour la faire fonctionner sur MinGW32...
...et c'était moyen-moyen comme expérience !

Boarf ! Tu peux trouver des codes tout fait de moins 50 lignes de 2 fonctions qui le font. C'est pas la mort non plus.

[Metalman]

Je sais que certains codes utilisent eux-même des fonctions pas toujours dispos....
Et pour faire de l'ultra-portable c'est pas génial...

Du coup je me trimballe un "ansi.c" (qui sert juste à compiler strictement en ansi) avec pleins de fonctions pompées un peu partout et/ou recodées... et j'ai toujours un peu peur de l'utiliser

Bref...

[fab_hunter1100]

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
void filecopy(FILE* inFile, FILE* outFile )
{
    int c; 
    while((c = getc(inFile)) != EOF)
    putc(c, outFile)
}
 
main()
{
    //Tu fais ce que tu veux !!!! ;) 
    FILE *fichier = NULL;
 
    fichier = fopen("fich_mot.txt", "r+"); 
 
   filecopy(fichier, stdout); //Ecran 
 
}

Essaye de modifier ce code pour avoir ce que tu veux.

Les gars pourquoi vous vous cassez la tête.
stdin et stdout font largement l'affaire.

pour le standard (clavier . ecran )

Code :

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

1
2
#define lire_du_clavier() getc(stdin)
#define afficher_a_l_ecran(c) putc((c), stdout)

et c'est tout .

Bon l'avantage avec les stdin et stdout justement c'est qu'on peut modifier son code pour avoir les I/O formatés, et on peut les redirectionner où on veut buffer(char*) et fichier pour être dans le sujet (j'ai donné la solution juste mais a côté pour que notre ami cherche un peu ) sans oublier de faire des flush. putc() et getc() étant portable, pas de problème de portabilité .

[Bktero]

Participer c'est bien. Répondre à la question, c'est mieux.

[fab_hunter1100]

Je pensais qu'il était proscrit de donner des réponses toutes faites sur le forum. J'ai juste profité pour donner mon avis sur le débat qui avait suivi après (ce n'étais pas non plus des réponses ou je me trompe).
En ce qui me concerne, j'ai bel et bien donné une réponse. Si elle n'est pas claire, le mieux c'est de demander au lieu de dire que je n'ai pas donné de réponse.

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
void filecopy(FILE* inFile, FILE* outFile )
{
    int c; 
 
    while(c = getc(inFile)){
        if (c == " "){
            break;
        }
        else
        {
            putc(c, outFile);
        }
    }
}
 
 
main()
{
    FILE *fichier = NULL;
 
    fichier = fopen("fich_mot.txt", "r+"); 
 
   filecopy(fichier, stdin); //Ecran  
 
}

Je laisse juste la possibilité de chercher comment mettre le deuxième argument dans son buffer (à l'aide des fonctions évoquées dans certains postes - ), et ensuite , de raffiné la fonction filecopy() (Il manque par exemple la fin du document EOF ). Je pense avoir signaler mon intension , répondre à coté mais laisser des pistes pour résoudre le problème.

Pour mon orthographe : Desolé je ne suis germanique .

[imperio]

C'est le cas ici ou encore là.