Discussion :

[lia20]

j'ai un tableau de caracteres ou je dois saisir les donnees moi meme. Je dois utiliser ctrl d (compilation avec la console sous Unix) pour finir et afficher mes mots. Mais ca ne marche pas ou je ne sais pas comment le controler. J'utilise fgets pour lire les donnees.

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
57
58
59
60
61
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
 
/* Fct: read an array of string  */
 
int lecture(char *tab[], int max)
{
	int nbchaine=0;    /* The number of word */
	char chaine[100];  /* Word to read */
	puts("Enter words and Ctrl -d to finish ");
 
	/* lesen und speichern woerter in Felder (saving word in the array) */
 
	while(nbchaine<=max && fgets(chaine,101,stdin))
	{
	        if(nbchaine==max)
		{
			max=max*2;
			tab=realloc(tab,max*sizeof *tab);
		}
		if(strlen(chaine)<100 && strlen(chaine)>0)
		{
			tab[nbchaine]=(char*)malloc(strlen(chaine)+1);
			strcpy(tab[nbchaine],chaine);
			nbchaine++;
		}
		else
			if(strlen(chaine)>=100 || strlen(chaine)<=0)
				perror("Error");
	}
	return nbchaine;   /* Return tne number of word*/
}
 
/* compare 2 words with strcmp */
 
int sort(const void *a, const void*b)
{
	return strcmp(*(char * const *)a, *(char * const *)b);
}
 
/*  Printing words*/
 
void ecrire(char *tab[], int max)
{
	int i;
	/*  sortieren mit qsort */
	qsort((void *)tab, max, sizeof(tab[0]), sort);
	for(i=0;i<max;i++)
		puts(tab[i]);
}
 
int main()
{
	int nblues=0;     /* number of words in the array */
	char **tab=malloc(100*sizeof(char *));   /* Ferder von Zeigern*/
	nblues=lecture(tab,100);
	printf("\n");
	ecrire(tab,nblues);
	return 0;
}

[gl]

Je dois utiliser ctrl d (compilation avec la console sous Unix) pour finir et afficher mes mots. Mais ca ne marche pas ou je ne sais pas comment le controler. J'utilise fgets pour lire les donnees.

Je viens de faire le test chez moi, sans souci (sous Cygwin n'ayant pas d'Unix sous la main).

Je me permets quelques remarques sur ton code:
* fgets(chaine,101,stdin) pourquoi mentir ainsi a fgets() ? chaine ne fait que 100 byte pas 101. D'une maniere generale, utilises sizeof chaine plutot qu'une valeur en dur.
* Il faut tester le retour des fonctions d'allocation, et en cas d'echec de ces fonctions, ne pas utiliser le pointeur.
* Il manque les liberations memoires des zones allouees.

[diogene]

En complément des remarques de gl :
La fonction lecture est fausse

Code :

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

1
2
3
4
5
if(nbchaine==max)
	{
		max=max*2;
		tab=realloc(tab,max*sizeof *tab);
	}

tab et max sont des variables locales. Leur modification ne sera pas répercutée dans le code appelant cette fonction. Grave conséquence: tab dans main pointe maintenant sur une zone qui peut être désallouée.

[lia20]

j'ai mis 10 parceque le nombre de caracteres de dois pas depasser 100 (par mot)
merci pr les remarques jai modifier mon code et ca marche.

[dj.motte]

Salut,

Bah, c'est pas terrible cette fonction appelée comme

Code :

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

1
2
 
int lecture(char *tab[], int max)

qui fait des allocations mémoires pour tenter réajuster le tableau.

La fonction appelante doit avoir une idée du traitement à réaliser, afin de parer aux bidouilles dans l'appelée.

Ainsi soit-il en C ...

[souviron34]

Salut,

Bah, c'est pas terrible cette fonction appelée comme

Code :

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

1
2
 
int lecture(char *tab[], int max)

qui fait des allocations mémoires pour tenter réajuster le tableau.

La fonction appelante doit avoir une idée du traitement à réaliser, afin de parer aux bidouilles dans l'appelée.

Ainsi soit-il en C ...

Je vais recommencer ce que j'ai dit dans l'autre post, pusique tu ne sembles pas avoir compris :

c'est LE CONTRAIRE en C ...

C'est la fonction qui est APPELEE qui doit se débrouiller, pusique c''est ELLE qui sait ce qui se passe, sur la base "philosophique" du principe des blocs de programmation.

Et en général, pour faire ce que veut faire le P.O., on passerait sans doute par un triple pointeur..

char * : une chaîne
char ** : un pointeur sur DES chaînes
char *** : un pointeur sur (un pointeur sur DES chaines), signifiant que la valeur modifiée par le realloc ressort..

Code :

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

1
2
3
4
5
6
7
8
9
10
 
int lecture(char ***tab, int max)
{
char **tmp = NULL ;
......
tmp = realloc ( *tab, NOUVELLE_TAILLE );
if ( tmp != NULL )
    *tab = tmp ;
....
}

[dj.motte]

Salut,

Bien d'accord mon ami. Mais la fonction est :

Code :

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

1
2
 
int lecture(char *tab[], int max)

Pas une invention de la dernière minute.