Bonjour je chercher a faire une liste chainee d une structure mais je n arrive pas a afficher a la fin les elements de la liste :

1. include <stdio.h>
2. include <stdlib.h>

int main (int argc, char* argv[])
{ typedef struct liste liste;
struct liste
{
char nom [20];
struct liste *nxt;
} *tete, *courant, *nouveau;
int i;
tete = NULL;
for (;;)
{
nouveau = (struct liste*) malloc( sizeof (struct liste) );
if (nouveau == NULL)
{
perror("Erreur d'allocation memoire");
exit(EXIT_FAILURE);
}
nouveau -> nxt = NULL;
printf("Nom 'FIN pour quitter' :"); scanf("%s", nouveau -> nom);
printf("%s\n",nouveau -> nom);
if(strcmp(nouveau -> nom,"FIN") == 0)
{
courant = nouveau;
while(courant != NULL)
{
printf("%s\n", courant->nom);
courant = courant -> nxt;
}
break;
}
}
free(nouveau);
return 0;
}

merci

je ne sais pas si c'est bien reglo de coller des typedef dans le main, de créer à la volée des types de variable au moment de leur déclaration et de ne rien mettre dans les conditions d'un for()

et puis, tu devrais mettre ton code sur pastebin, là c'est pas très lisible sans indendation

Il fait peur ce code dîtes.

" et de ne rien mettre dans les conditions d'un for() "

Non ça c'est tout a fait normal "for(;;)" est ce qui est communément utilisé pour une boucle infinie, on peut aussi mettre un truc comme "while(1)" bien sur.

merci pour ton lien , j espere que sa sera plus comprehensible et merci pour votre aide
http://pastebin.com/y02zYjJm

"Non ça c'est tout a fait normal "for(;;)" est ce qui est communément utilisé pour une boucle infinie"

va pour le normal, à la limite... mais communément utilisé, ça fait bizarre, je crois que le while(true) colle plus aux standards de programmation (si tant est qu'une boucle infinie aie sa place dans de tels standards )

je pense que c est a la ligne 24 : courant = nouveau .
Qu en pensez vous ? , je bloque depuis 1h

" va pour le normal, à la limite... mais communément utilisé, ça fait bizarre "

Je maintiens, c'est assez commun.

oui je vois for (;;) souvent.

NdM: La frequence d'utilisation de differente forme de boucle infinie est relativement HS sur ce topic. Donc continuer la sur le blabla si vous voulez rajouter qqch.

je reviendrai voir demain , voir si quelqu un a trouve une solution car la je seche
merci a ceux qui m aide

Desole, mais je n'arrive pas a voir la ligne ou tu lies les maillons entre eux.

Parce que pour le moment ce que je vois, c'est que tu crees des structures, et que tu leurs mets TOUTES NULL en next, ce qui fait que tu perds tes maillons les uns a la suite.

merci de votre aide et en particulier a Nightmarez la correction
http://pastebin.com/88y5pT4r

Je ne vois nulle part un endroit ou tu fais genre
"courrant->next = tete;
tete = courrant;"

donc au final tu ne links pas les maillons de ta liste chainee, qui n'en est donc pas une.

hum , se n est pas bon ?
if ( head == NULL) {
head = news;
courant = news;
}
else {
courant->next = news;
courant = news;
}

desole du double post , mais j avais mis en ligne la mauvaise correction
http://pastebin.com/pqwJSYK9

Tu n'as besoin que de deux pointeurs sur structure ps pour cette fonction, un qui represente la racine de la liste chainee, l'autre qui servira a allouer un maillon.

Je te conseille de garder "head" et "nouveau". Tu initialises head a NULL avant la boucle.

A la fin de ta boucle for, tu devrais avoir
"nouveau->next = head;
head = nouveau;"

Ce qui devrait bien lier ta liste chainee et eviter de perdre des maillons.