Discussion :

[Smanyx]

Salut !

Je voudrais écrire un programme qui accepte des noms saisis au clavier et les amasse dans un tableau. Comme les noms sont entrés dans une boucle, la taille du tableau n'est pas connue en avance et doit donc être gérée dynamiquement.

voici le code que j'ai à l' instant et qui ne tourne pas. Votre aide est vraiment la bienvenue.
Merci.

Smanyx

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
#include <stdio.h>
#include <stdlib.h>
 
int main ()
{
	int i = 0;
	char Noms[][25];
 
	puts("\nPour terminer, tapez EOF (Ctrl + Z) ");
             Noms = (char *) malloc(1 * sizeof(char));
	do
	{
		printf("\nVeuillez entrer un nom svp: ");
		gets(Noms[i]);
		i++;
		Noms = (char *) realloc (Noms, i * sizeof (char));
		printf("\nVeuillez entrer un nom svp: ");
	}while(!feof(stdin));
 
	printf ("Liste de noms saisis: ");
    for (int n = 0; n<i; n++) 
	{
		printf ("%s ",Noms[n]);
	}
    free (Noms);
 
	getchar();
	return 0;
}

[Melem]

Il faut que Noms soit un pointeur, qui recevra l'adresse d'un tableau que tu vas créer (et agrandir, etc.), et non un tableau figé. D'ailleurs, même si Noms devait être un tableau figé, ta déclaration est incorrecte car tu ne l'initialises pas. La bonne déclaration est :

Code :

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

char (*Noms)[25];

Pour le malloc, c'est malloc(... * sizeof(char [25])) et non sizeof(char). Et on n'a pas besoin de caster malloc.

Pareil pour realloc.

On peut aussi faire d'autres remarques sur ton code mais essaie déjà d'arranger (et de comprendre) ça.