Une Fonction en C

Version imprimable

Une Fonction en C

Bonjour,
je suis un débutant en Programmation C (ANSI) sous Ubuntu Linux et je Compile avec le GCC de GNU. je voudrai vous demande un petit coup de pouce pour rectifier mon petit programme pour qu'il puisse lire un fichier texte le stocker mot par mot dans un tableau de chaînes de caractères . et afficher le contenu de ce tableau par une deuxième fonction.
Merci d'avance.

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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
 
/*
 * Lecture simplifié d'un fichier texte 
 * Auteur :
 * Date :
 */
 
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#define LGMAX 30
#define MAXMOTS 100
 
 
/*
 * Structure simple de la fonction lireDico
 * Lecture du fichier et affichage sur l'ecran
 * sans utilisation de structures de données
void lireDico(char *nomFichier)
{
  FILE *f;
  char tmp[LGMAX];
  f=fopen(nomFichier,"r");
  if(f==NULL)
    {
      fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
      exit(1);
    }
  else
    fprintf(stdout,"Ouverture avec Succes !\n\n");
  while(fscanf(f,"%s",tmp)!=EOF)
    printf("%s\n",tmp);
  fclose(f);
}
 
 
 */
 
 
 
/* Declaration de la variable Globale
 * Structure de donnée tableau de
 * chaines de caractères 
 */
void lireDicotab(char *nomFichier,char *dictionnaire[MAXMOTS])
{
  FILE *f;
  char tmp[LGMAX];
  int i=0;
  f=fopen(nomFichier,"r");
  if(f==NULL)
    {
      fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
      exit(1);
    }
  else
    fprintf(stdout,"Ouverture avec Succes !\n\n");
  while((fscanf(f,"%s",tmp)!=EOF) && i<MAXMOTS)
    {
      dictionnaire[i]=tmp;
     // printf("%s\n",dictionnaire[i]);
      i++;
    }
 
 
  fclose(f);
 
}
 
/* Fonction pour affichage des mots du dictionnaire */
void afficher(char *dictionnaire[MAXMOTS])
{
  int j;
  for(j=0;j<=MAXMOTS;j++)
    {
      printf("%s\n",dictionnaire[j]);
      j++;
    }
  printf("Fonction executée avec succes !\n");
}
 
 
 
 
int main(int argc,char *argv[])
{
  char *filename;
  char *dictionnaire[MAXMOTS] = (char *)malloc(MAXMOTS*sizeof(char)*LGMAX);
  // int ** a = malloc(nrows * sizeof *a );
 
  filename=argv[1];
  // lireDico(filename);
  lireDicotab(filename,dictionnaire);
  afficher(dictionnaire);
 
  fprintf(stdout,"\nOn atteint la fin - Fichier Close !\n");
  return EXIT_SUCCESS;
}

Bonjour

Citation:

Envoyé par ubu1707

Bonjour,
je suis un débutant en Programmation C (ANSI)

Bonjour

avant commencé STP tu click sur le bouton Editer en bas et ajoute la balise pour ton code et supprime les instructions inutiles merci :king:
regarde ton code comment il est jolie :mrgreen:

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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
/*
* Lecture simplifié d'un fichier texte
* Auteur :
* Date :
*/
 
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#define LGMAX 30
#define MAXMOTS 100
 
 
/*
* Structure simple de la fonction lireDico
* Lecture du fichier et affichage sur l'ecran
* sans utilisation de structures de données
void lireDico(char *nomFichier)
{
FILE *f;
char tmp[LGMAX];
f=fopen(nomFichier,"r");
if(f==NULL)
{
fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
exit(1);
}
else
fprintf(stdout,"Ouverture avec Succes !\n\n");
while(fscanf(f,"%s",tmp)!=EOF)
printf("%s\n",tmp);
fclose(f);
}
 
 
*/
 
 
 
/* Declaration de la variable Globale
* Structure de donnée tableau de
* chaines de caractères
*/
void lireDicotab(char *nomFichier,char *dictionnaire[MAXMOTS])
{
FILE *f;
char tmp[LGMAX];
int i=0;
f=fopen(nomFichier,"r");
if(f==NULL)
{
fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
exit(1);
}
else
fprintf(stdout,"Ouverture avec Succes !\n\n");
while((fscanf(f,"%s",tmp)!=EOF) && i<MAXMOTS)
{
dictionnaire[i]=tmp;
// printf("%s\n",dictionnaire[i]);
i++;
}
 
 
fclose(f);
 
}
 
/* Fonction pour affichage des mots du dictionnaire */
void afficher(char *dictionnaire[MAXMOTS])
{
int j;
for(j=0;j<=MAXMOTS;j++)
{
printf("%s\n",dictionnaire[j]);
j++;
}
printf("Fonction executée avec succes !\n");
}
 
 
 
 
int main(int argc,char *argv[])
{
char *filename;
char *dictionnaire[MAXMOTS] = (char *)malloc(MAXMOTS*sizeof(char)*LGMAX);
// int ** a = malloc(nrows * sizeof *a );
 
filename=argv[1];
// lireDico(filename);
lireDicotab(filename,dictionnaire);
afficher(dictionnaire);
 
fprintf(stdout,"\nOn atteint la fin - Fichier Close !\n");
return EXIT_SUCCESS;
}

et après la coloration syntaxique je constate qu'il y a une fonction inutile alors STP modifier ton code :( puis dis nous exactement c'est quoi ton problème ;)
Juste une petite remarque fait attention à l'instruction j++; car il a y des affichages ratés :aie:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
/* Fonction pour affichage des mots du dictionnaire */
void afficher(char *dictionnaire[MAXMOTS])
{
int j;
for(j=0;j<=MAXMOTS;j++)
{
printf("%s\n",dictionnaire[j]);
j++;// mot raté pour chaque incrémentation
}
printf("Fonction executée avec succes !\n");
}

Bonne journée :P

15/10/2008, 09h10
Invité(e)
Bonjour,

Déjà, gcc n'est pas d'accord pour compiler avec ce code :
Code:

1 2 3 4 5 >gcc -Wall -Wextra 08-10-15.c 08-10-15.c: In function `main': 08-10-15.c:87: error: invalid initializer 08-10-15.c: At top level: 08-10-15.c:84: warning: unused parameter 'argc'
La ligne suivante pose problème :
Code:

1 2 char *dictionnaire[MAXMOTS] = (char*) malloc(MAXMOTS*sizeof(char)*LGMAX);
On veut allouer de la mémoire pour un tableau de pointeur de taille fixe.
La bonne écriture aurai été :
Code:

1 2 3 4 5 char *dictionnaire[MAXMOTS]; for(i = 0; i < MAXMOTS; ++i) { dictionnaire[i] = malloc(LGMAX); }
Mais allouer dynamiquement des tableaux de taille fixe est rarement utile et contraignant puisqu'il faut tester que la fonction malloc a bien fonctionné, autant écrire :

Code:

char dictionnaire[MAXMOTS][LGMAX];

gcc lève aussi un warning sur la non utilisation de argc. Il a raison, si le programme est lancé sans argument, argv[1] n'est pas défini, le comportement du programme non plus...
15/10/2008, 09h24
Invité(e)

Autre chose
une copie de chaine de caractères ne se fait pas comme ça :

Code:

dictionnaire[i]=tmp;

Mais à l'aide de la fonction str[n]cpy :

Code:

strncpy(dictionnaire[i], tmp, LGMAX);

Sachant que strncpy() ne termine pas la chaîne en cas de dépassement de buffer.
Il est plus sûr d'utiliser strncat() comme ceci :
Code:

1 2 3 4 5 6 7 8 char* mystrncpy(char *dest, char const *src, size_t cchDestSize) { size_t cchDestLength = cchDestSize - 1; assert(dest != NULL); assert(src != NULL); dest[0] = '\0'; return strncat(dest, src, cchDestLength); }

2eme version mais ne marche pas

Merci beaucoup de votre aide je suis tujours entrain de voir purquoi ça marche pas! j'ai changer très peut de truc et ça s'execute mais en voyant le resultats (erreur de segmentation ) du probablement à l'allocation du tableau dictionnaire. merci

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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
/*
 * Lecture simplifié d'un fichier texte 
 * Auteur :
 * Date :
 */
 
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#define LGMAX 30
#define MAXMOTS 100
 
 
/*
 * Structure simple de la fonction lireDico
 * Lecture du fichier et affichage sur l'ecran
 * sans utilisation de structures de données
void lireDico(char *nomFichier)
{
  FILE *f;
  char tmp[LGMAX];
  f=fopen(nomFichier,"r");
  if(f==NULL)
    {
      fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
      exit(1);
    }
  else
    fprintf(stdout,"Ouverture avec Succes !\n\n");
  while(fscanf(f,"%s",tmp)!=EOF)
    printf("%s\n",tmp);
  fclose(f);
}
 
 
 */
 
 
 
/* Declaration de la variable Globale
 * Structure de donnée tableau de
 * chaines de caractères 
 */
void lireDicotab(char *nomFichier,char *dictionnaire[MAXMOTS])
{
  FILE *f;
  char tmp[LGMAX];
  int i=0;
  f=fopen(nomFichier,"r");
  if(f==NULL){
    fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
    exit(1);
  }
  else
    fprintf(stdout,"Ouverture avec Succes !\n\n");
  while((fscanf(f,"%s",tmp)!=EOF) && i<MAXMOTS){
    dictionnaire[i]=tmp;
    i++;
  }
  fclose(f);
}
 
/* Fonction pour affichage des mots du dictionnaire */
void afficher(char *dictionnaire[MAXMOTS])
{
  int j;
  for(j=0;j<=MAXMOTS;j++){
    printf("%s\n",dictionnaire[j]);
  }
  printf("Fonction executée avec succes !\n");
}
 
 
 
 
int main(int argc,char *argv[])
{
  char *filename;
  int i;
  // char *dictionnaire;// = (char*)malloc(MAXMOTS);
   char *dictionnaire[MAXMOTS];
   //char dictionnaire[MAXMOTS][LGMAX];
  for(i = 0; i < MAXMOTS; ++i) {
    dictionnaire[i] = (char*)malloc(LGMAX);
  }
  filename=argv[1];
  // lireDico(filename);
  lireDicotab(filename,dictionnaire); /* Lecture à partir du fichier */
  afficher(dictionnaire); /* afficher le dictionnaire */
  fprintf(stdout,"\nOn atteint la fin - Fichier Close !\n");
  return EXIT_SUCCESS;
}

Bonjour,

Citation:

Envoyé par ubu1707

Merci beaucoup de votre aide

Visiblement, tu n'as pas écouté les remarques de mes camarades. Donc reprenons:

Citation:

Envoyé par mabu

Autre chose
une copie de chaine de caractères ne se fait pas comme ça :

Code:

dictionnaire[i]=tmp;

Mais à l'aide de la fonction str[n]cpy :

Code:

strncpy(dictionnaire[i], tmp, LGMAX);
Citation:
Envoyé par Médinoc
Sachant que strncpy() ne termine pas la chaîne en cas de dépassement de buffer.
Il est plus sûr d'utiliser strncat() comme ceci :
Code:

1 2 3 4 5 6 7 8 char* mystrncpy(char *dest, char const *src, size_t cchDestSize) { size_t cchDestLength = cchDestSize - 1; assert(dest != NULL); assert(src != NULL); dest[0] = '\0'; return strncat(dest, src, cchDestLength); }
Pourquoi as-tu encore ce petit bout de code:
Code:

1 2 3 4 while((fscanf(f,"%s",tmp)!=EOF) && i<MAXMOTS){ dictionnaire[i]=tmp; i++; }
----------------------------------------

Citation:

Envoyé par mabu

Mais allouer dynamiquement des tableaux de taille fixe est rarement utile et contraignant puisqu'il faut tester que la fonction malloc a bien fonctionné,

Donc pourquoi, il n'y a pas de tests d'échec ici:
Code:

1 2 3 for(i = 0; i < MAXMOTS; ++i) { dictionnaire[i] = (char*)malloc(LGMAX); }
----------------------------------------

Citation:

Envoyé par mabu

autant écrire :

Code:

char dictionnaire[MAXMOTS][LGMAX];

Je le penses aussi.
----------------------------------------

Citation:

Envoyé par mabu

gcc lève aussi un warning sur la non utilisation de argc. Il a raison, si le programme est lancé sans argument, argv[1] n'est pas défini, le comportement du programme non plus...

La remarque est toujours d'actualité.

15/10/2008, 16h11
3DArchi
Ici aussi tu risques d'avoir un problème:
Citation:
Code:

1 2 3 4 5 6 7 8 void afficher(char *dictionnaire[MAXMOTS]) { int j; for(j=0;j<=MAXMOTS;j++){ printf("%s\n",dictionnaire[j]); } printf("Fonction executée avec succes !\n"); }
Explications: ta boucle for va balayer j de 0 à MAXMOTS inclus. Or ton tableau est de taille MAXMOTS. Donc, quand tu adresse dictionnaire[MAXMOTS] tu pars vers l'aventure d'un point de vue mémoire.;)
Comme dictionnaire est de taille MAXMOTS, alors tu peux prendre les items de 0 à MAXMOTS-1. Donc ta boucle for devrait faire varier l'indice j de 0 à MAXMOTS-1. Pour cela, remplace '<=' par '<'.
15/10/2008, 17h21
Invité(e)

Citation:

Envoyé par 3DArchi

Citation:

Envoyé par mabu

autant écrire :

Code:

char dictionnaire[MAXMOTS][LGMAX];

Je le penses aussi.

Attention, si on change de type pour dictionnaire, il faut aussi faire attention aux prototypes de afficher et lireDicotab. Si on connait par avance la taille du tableau, on peut la donner aux fonctions.

la touche finale

tout qui marche sauf ! le tri se fait à l'envers :s merci d'avnace

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
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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
 
/*
 * Lecture est tri d'un fichier texte 
 * Auteur : moi
 * Date : 22 octobre 2008
 */
#include<stdio.h>
#include<string.h>
#include<stdlib.h>
#define LGMAX 30
#define MAXMOTS 100
/* Les fonction pour trier le tableau de chaines */
/*Fonction permutaion de deux mots */
void echanger(char *dictionnaire[],int j)// A adapter
{
  char *tmp=(char *)malloc(LGMAX);
  tmp=dictionnaire[j];
  dictionnaire[j]=dictionnaire[j-1];
  dictionnaire[j-1]=tmp;
}
 
/*Fonction de tri parinsertion*/
void tri_par_insertion(char *dictionnaire[], int L) //A adapter
{
  int i,j;
  for(i=0;i<L-1;i++){
      for(j=i;j>0;j--)
	if(strcmp(dictionnaire[j],dictionnaire[j-1])>0)
	  echanger(dictionnaire,j);
    }
} 
 
/* Structure de donnée tableau de chaines de caractères */
void lireDicotab(char *nomFichier,char *dictionnaire[MAXMOTS])
{
  FILE *f;
  char tmp[LGMAX];
  int i=0;
  f=fopen(nomFichier,"r");
  if(f==NULL){
    fprintf(stderr,"Erreur d'ouverture du fichier ( %s )!\n",nomFichier);
    exit(1);
  }
  else {
    fprintf(stdout,"Ouverture avec Succes !\n\n");
    while((fscanf(f,"%s",tmp)!=EOF) && i<MAXMOTS){
      strncpy(dictionnaire[i], tmp, LGMAX);
      i++;
    }
    fclose(f);
  }
}
 
/* Fonction pour affichage des mots du dictionnaire */
void afficher(char *dictionnaire[MAXMOTS])
{
  int j=0;
  while(strlen(dictionnaire[j])>0) {
    printf("%s\n",dictionnaire[j]);
    j++;
  }
  printf("Fonction executée!\n");
}
/* fonction main */
int main(int argc,char *argv[])
{
  char *filename;
  char *dictionnaire[MAXMOTS];
  int i=0;
  for(i=0;i<MAXMOTS-1;i++) {
    dictionnaire[i]=(char*)malloc(LGMAX);
  }
  filename=argv[1];
  /* Lire le fichier texte et stocker dans un tableau de chaines de caractères*/
  lireDicotab(filename,dictionnaire);
  printf("***Les mots du dictionnaire avant le Tri:***\n\n");
  afficher(dictionnaire);
  tri_par_insertion(dictionnaire,MAXMOTS);
  printf("*********************\n\tApres le tri\n*********************\n");
  afficher(dictionnaire);
  fprintf(stdout,"\nOn atteint la fin - Fichier Close !\n");
  return EXIT_SUCCESS;
}

Bonjour
A momn vis, il suffit d'inverser la comparaison de tri non ?
Code:

1 2 3 4 5 6 7 8 9 10 /*Fonction de tri parinsertion*/ void tri_par_insertion(char *dictionnaire[], int L) //A adapter { int i,j; for(i=0;i<L-1;i++){ for(j=i;j>0;j--) if(strcmp(dictionnaire[j],dictionnaire[j-1])<0) echanger(dictionnaire,j); } }
Par contre, il y a encore quelques choses qui ne vont pas : si le nombre de mots dans le fichier n'est celui attendu, cela des problèmes...
Voici une correction qui évite pas mal de problèmes :
Code:

1 2 3 for(i=0; i < MAXMOTS; i++) { dictionnaire[i] = calloc(1, LGMAX); }