problème d'affichage d'une liste

Version imprimable

Bonsoir,

quelqu'un peut m'aider ?
ma fonction afficher_liste ne marche pas elle ne m'affiche rien malgré qu'elle est juste.

Code:

1
2
3
4
5
6
7
8
9
void afficher_liste(liste tete)
  { liste p=tete;
    while(p!=NULL)
      { 
        printf("Identifacateur du processus:%d\n",p->id);
        printf("Nom du processus:%s\n",puts(p->nom));
        p=p->svt;
      }
  }

voici le main:

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
void main()
{   int nb_tache,i,choix;
    char cod[maxcar],n[maxcar],etat,nom;
    int id,priorite,t_arrive,t_execution;
    liste tete,l,p,svt;
 
 
 
     liste creer_noeud(); 
     liste creer_liste();
     void  afficher_liste(liste tete);
     void inserer(liste *l, int id);
     void inserer_queue(liste *l,liste *p);
     void supprimer(liste *l);
     void permuter(liste *p1,liste *p2);
     void trier (liste *l);
     void executer(liste *scheduler,liste *actif);
     void entree_sortie(liste *actif, liste *scheduler);
 
 
 
 
 
    printf("******************************** MENU **************************************\n");
    printf("\n");
 
     printf("Les differentes taches possibles sont:\n");
     printf("\n");
     printf("1- Creation de la file des processus\n");
     printf("2- Affichage de la file des processus\n");
     printf("3- Modification de la file\n");
     printf("4- Execution des processus avec FIF0\n");
     printf("5- Executer des processus avec RR\n");
     printf("6- Quitter\n");
     printf("\n");
 
 
 
    printf("Donner le nombre de taches presentes\n");
    scanf("%d", &nb_tache);
 
    if((nb_tache<=0)||(nb_tache>maxtaches))
        {exit(-1);}
 
     for(i=1;i<nb_tache;i++)
{
      printf("Veuillez tapper le numero %d pour continuer l'execution \n", i);
 
      scanf("%d", &choix);
 
     if((choix<1)||(choix>7))
{
            exit(-1);
}
 
     switch(choix)
{
     case 1: printf("**************** Creation de la liste des processus ***************\n");
     printf("\n");
 
     creer_liste(); 
         break;
 
 
 
 
         case 2: printf("**************** Affichage de la liste des processus ***************\n");
     printf("\n");
 
     void afficher_liste(liste tete);
         break;
                  default: exit(1);
 
}
 
}            
     getchar(); getchar();

}

et la console:

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
******************************** MENU ***********
 
Les differentes taches possibles sont:
 
1- Creation de la file des processus
2- Affichage de la file des processus
3- Modification de la file
4- Execution des processus avec FIF0
5- Executer des processus avec RR
6- Quitter
 
Donner le nombre de taches presentes
6
Veuillez tapper le numero 1 pour continuer l'execution
1
**************** Creation de la liste des processus ***************
 
Donnez le nombre de processus
1
********************* Creation du processus numero1 *****************
 
Donnez le nom du processus numero1:
proc1
Donnez l'identificateur du processus numero1:
1
Donnez la priorite du processus numero1:
6
Donnez le temps d'arrivee du processus numero1:
5
Donnez le temps d'execution du processus numero1:
4
La file contient un seul processus
 
Veuillez tapper le numero 2 pour continuer l'execution
2
**************** Affichage de la liste des processus ***************
  ////// ici ça doit m'afficher la liste des processus que j'ai saisie
 
Veuillez tapper le numero 3 pour continuer l'execution

Merci d'avance

05/11/2011, 10h49
Pouet_forever

C'est un peu vague pour qu'on puisse t'aider, il nous faudrait un peu plus de code. Ton problème ne se situe pas dans la fonction afficher_liste, mais ailleurs.
D'ailleurs, la fonction main doit retourner quelque chose, donc c'est 'int main(void)' qu'il faut avoir comme prototype (ou l'autre, mais tu ne te sers pas des argc/v).
C'est toujours une mauvaise idée de cacher un pointeur dans un typedef...
Ton indentation est à revoir, ton code est illisible.
Les prototypes de fonctions sont à déclarer dans un header prévu à cet effet, ou éventuellement en début de fichier.
Les variables globales sont à éviter, ton code est un gruyère.
05/11/2011, 12h45
Bousk

Bonjour,

est-ce normal que la tête que tu passes à afficher_liste ne soit pas un pointeur ? ou liste définit automatiquement un pointeur ?

Citation:

case 2: printf("**************** Affichage de la liste des processus ***************\n");
printf("\n");

void afficher_liste(liste tete);

Il faudrait revoir comment on utilise une fonction, écrire son prototype n'est pas la bonne solution..
Si tenté que afficher_liste soit correcte, ça ne marchera pas avec cet appel.

Citation:
Envoyé par lucciana
Bonsoir,

quelqu'un peut m'aider ?
ma fonction afficher_liste ne marche pas elle ne m'affiche rien malgré qu'elle est juste.
Code:

1 2 3 4 5 6 7 8 9 void afficher_liste(liste tete) { liste p=tete; while(p!=NULL) { printf("Identifacateur du processus:%d\n",p->id); printf("Nom du processus:%s\n",puts(p->nom)); p=p->svt; } }
Bonjour

La majorité des remarques les plus importantes ont été données. On ne sait rien de ta liste et/ou de la façon dont tu la remplis.

Je rajoute ma touche à propos d'un truc que personne n'a remarqué (mais je n'ai aucun mérite, je ne l'ai vu que par hasard tellement ça se fond dans le code): faire un printf(..., puts()) c'est vraiment pas tiptop. Soit tu utilises printf(), soit tu utilises puts() mais pas les 2 à la fois !!!

Citation:

Envoyé par lucciana

malgré qu'elle est juste.

Ne jamais faire ce genre de présomption car preuve en est...

PS: tu peux remplacer ton while() par un for() plus significatif
Code:

1 2 3 4 5 6 7 8 void afficher_liste(liste tete) { liste p; for (p=tete; p!=NULL; p=p->svt) { printf(...); } }