Discussion :

[skyangel]

Bonjour,

J'ai un problème avec mon ceode. L'objectif est de saisir des noms de personne et un numéro attribué.
Je n'arrive pas à les enregistré dans une liste chainée.
Le programme me demande de saisir mais a la fin de la saisie , il quitte automatiquement le programme. Je n'arrive pas a mettre la main sur le bug. Je pense que l'erreur vient de ajout_releve.

je vous donne mon code

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
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
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
# include <iostream.h>
 
struct releve {
       char nom[50];
       float num;
};
 
struct cellule {
       releve rel;     
       cellule *suivant;
};
typedef cellule *carte;
//initialisation
int initialiser(carte &C){
    C = NULL;
}
//saisie releve
int saisir_releve (releve &R){
 
     cout << "Saisissez le nom:" << '\t' ;
     cin >> R.nom;
     cout << "Saisissez le num:" << '\t';
     cin   >> R.num;  
}
//saisi ajout
int ajout_releve (carte &C){
     cellule *pc=C ,*pred=NULL, *q;
     releve R;
     saisir_releve(R);
     while (pc != NULL && pc->rel.num < R.num ){
           pred=pc;
           pc=pc->suivant;    
     }
     q= new cellule;
    q->rel=R;
     q->suivant=pc;
     if (pred = NULL){
        C=q;           
     }
     else {
          pred->suivant=q;     
     }
 
}
//supprimer
int supprimer (carte &C){
          cellule *P,*pred;
          P=C;
         char nom[20];
         cout << "Saisissez le nom " << '\t';
         cin >> nom;
         while (C != NULL && strcmp( P->rel.nom,nom)){
               pred=P;
               P=P->suivant;      
         }
         if (P == NULL){
               cout << nom << " est absent " << '\t';
 
         }
         else {
               if (pred == NULL){
                  pred->suivant=P->suivant;         
               }
               else 
               {
                    C->suivant=P->suivant;     
               }     
         }
 
}
//afiche 
 
int afficher(carte C){
    cellule *P;
    P=C;
    while (P != NULL){
          cout << "Le nom est: "<< P->rel.nom << '\t';
          cout << "Le num est: "<<P->rel.num << '\t';
          P=P->suivant;
    }      
 
}                  
int tempmin(carte C){
    cellule *P;
    P=C;
    float min=100;
    while (P != NULL){
          if ( P->rel.num < min){
          min = P->rel.num;
          P=P->suivant;      
          }
    }
        cout << "La temperature minimal est :" << min;
 
}
int tempmax(carte C){
    cellule *P;
    P=C;
    float max=0;
    while (P != NULL){
          if ( P->rel.num > max){
          max = P->rel.num;
          P=P->suivant;      
          }
    }
    cout << "La temperature maximale est :" << max;
 
}
int tempmoy(carte C){
    cellule *P;
    P=C;
    float moy=0;
    int nb=0;
    while (P != NULL){
          moy=moy+P->rel.num;
          nb++;
    }
    moy=moy/nb;
    cout << "La temperature moyenne est :" << moy;
 
}
int menu(carte &C){
 
    int a;
    cout << "1. Initialiser "<<endl;
    cout << "2. Afficher "<<endl;
    cout << "3. Ajouter "<<endl;
    cout << "4. Supprimer" << endl;
    cout << "5. Num min" << endl;
    cout << "6. Num max" << endl;
    cout << "7. Num moy" << endl;
    cin >>a;
    switch (a)
    {
           case 1 : initialiser(C); break;     
           case 2 : afficher(C);break;
           case 3 : ajout_releve(C);break;
           case 4 : supprimer(C);break;
           case 5 : tempmin(C);break;
           case 6 : tempmax(C);break;
           case 7 : tempmoy(C);break;
    }
}
int main (){
    carte C;
    initialiser(C);
    ajout_releve(C);
    ajout_releve(C);
    ajout_releve(C);
   // afficher(C);
    system("pause");
    return 0;       
}

Merci de penser aux balises code...

Merci d'avance

[Matthieu Brucher]

Tu es obligé de recoder la liste chaînée ?

[skyangel]

c'est à dire??

C'est un exercice dans le but d'apprendre les listes chainées. Car c'est nouveau pour moi

[Matthieu Brucher]

OK, donc tu dois la recoder, c'était pour savoir -> std::list.
Bon déjà, utilise des string dans ta liste à la place des char*, ça sera plus cohérent, d'autant que tu ne vérifies jamais la taille des données que tu rentres.

[skyangel]

ok je comprends pour les string .

qu'est ce que entends par le fait que je dois le recoder??? et quoi?? Je iens de taper le code mais il ne me laisse pas le temps de voir la fenetre de l'aplplication alors que j'ai mis une pause dans le logiciel
Je débute en C et les listes ne sont pas évidentes.
Merci en tout cas.

[Matthieu Brucher]

*J'ai dit que tu es obligé de recoder la liste chaînée, ce que tu fais, c'est tout. Si tu n'étais pas obligé, tu aurais pu utiliser la STL.

Tu as passé ton programme au débuggeur pour voir exactement où ça plante ?

[skyangel]

oui je l'ai passé au debuggeur.
Il bloque au while de la fonction ajout_releve.

[Matthieu Brucher]

Je me demande comment ton code peut marcher. Tu mélanges partout des références et des pointeurs.
Donc avant de continuer :
- changer les références en pointeurs
- changer les char* en string.

[Bebert71]

Tu utilise quel logiciel pour developper??
Si tu es sous builder, dans ton main avant return 0 tappe ceci:

Code :

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

1
2
3
 
cin.sync();
cin.get();

[skyangel]

J'ai exactement le meme code dans un autre fichier qui est celui fait en classe . On m'a appris a coder comme ca. :/

Maintenant il marche dans l'autre fichier .
Le but n'étant pas de copier ce fichier mais de le comprendre .

[skyangel]

Tu utilise quel logiciel pour developper??
Si tu es sous builder, dans ton main avant return 0 tappe ceci:

Code :

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

1
2
3
 
cin.sync();
cin.get();

j'utilise devcpp

[Matthieu Brucher]

Non, ton prof ne te donnera jamais un code tel que celui-ci. D'ailleurs, le compilateur doit t'afficher une tonne de warnings, non ? Tu assignes des références à des pointeurs et inversement. Ca ne PEUT pas marcher.

En regardant de plus prêt, ça va passer par moment, et puis à d'autres pas du tout. Pourquoi fais-tu un typedef de cellule et de carte ?

[skyangel]

j'ai trouve. C'est la fonction initialiser qui met le bazar

[Matthieu Brucher]

j'ai trouve. C'est la fonction initialiser qui met le bazar

D'où le pb référence/pointeur que je demandais d'expliciter un peu mieux

Content que tu aies trouvé

Au fait "iostream.h" -> "iostream"

[skyangel]

Non, ton prof ne te donnera jamais un code tel que celui-ci. D'ailleurs, le compilateur doit t'afficher une tonne de warnings, non ? Tu assignes des références à des pointeurs et inversement. Ca ne PEUT pas marcher.

En regardant de plus prêt, ça va passer par moment, et puis à d'autres pas du tout. Pourquoi fais-tu un typedef de cellule et de carte ?

Je ne te remets pas en cause. Mais je soutiens que c'est comme ca qu'on a appris . Les fonctions ajout et supprimer ont été codé comme ca. J'ai calqué dessus.

De plus ce programme marche. Maintenant. sans la fonction initialiser

[Matthieu Brucher]

Le principal est que ça marche
En revanche, je te conseille, si tu as du mal à comprendre ce qui se passe, de supprimer carte et de le remplacer par cellule* afin que tu puisses te rendre compte exactement de ce qui se passe et de ce que tu manipules effectivement - pointeur ou objet -

[skyangel]

Le principal est que ça marche
En revanche, je te conseille, si tu as du mal à comprendre ce qui se passe, de supprimer carte et de le remplacer par cellule* afin que tu puisses te rendre compte exactement de ce qui se passe et de ce que tu manipules effectivement - pointeur ou objet -

D'accord . A tester