Discussion :

[dauphin11]

Bonjour

J'ai besoin d'utiliser une map de map mais j'ai bcp de soucis à la compilation; avec l'itérateur de map< string, map<string, vector<float> > >, et aussi pour accéder aux éléments du vector<float>>...

Voici 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
#include <fstream>
#include <string> 
#include <iostream>
#include <map>
#include <vector>
#include <stdio.h>
#include <algorithm>
#include <iterator>
using namespace std;

vector<float> notes;
vector<string> noms;
vector<string> matieres;
typedef  map < string, vector<float> >  mapstrvect;

typedef map < string, mapstrvect > Nom_Notes_mat;

string nom;
string matiere;
float note;

int s = sizeof(matieres);
int t = sizeof(noms);
vector< vector<float> > synthese((s+1)*(t+1));

void afficher_synthese() {

  map<string,mapstrvect>::iterator itn=Nom_Notes_mat.begin();
  for(;itn!= Nom_Notes_mat.end();++itn){
    if ( (*itn).first == ((*(itn++)).first) && ((*itn).second).first == (((*(itn++)).second))).first))
    ((*itn).second).push_back( ( (*itn++).second ).second );
    Nom_Notes_mat.erase(itn++);}

for(int j=0;j<t+1;++j){
  for(int i=0;i<s;++i){
    vector<float> v1 = (Nom_Notes_mat[matieres[j]])[noms[i]];
    int size = s*t;
    string mat = matieres[j];
    string name = noms[i];
    int l = 0;
    float sum;
	
    for (int k=0; k<size; k++){
      sum = sum+ ((Nom_Notes_mat[mat])[name])[k];
      l++;}
    float moy = sum/l;
  }
 }
}

J'obtiens les erreurs suivantes:

In function ‘void afficher_synthese()’:
forum.cpp:32:53: error: expected primary-expression before ‘.’ token
forum.cpp:33:27: error: expected primary-expression before ‘.’ token
forum.cpp:34:64: error: ‘class std::map<std::basic_string<char>, std::vector<float> >’ has no member named ‘first’
forum.cpp:34:95: error: expected primary-expression before ‘.' token
forum.cpp:36:18: error: expected unqualified-id before ‘.’ token
forum.cpp:40:50: error: size of array has non-integral type ‘std::basic_string<char>’
forum.cpp:40:50: error: size of array has non-integral type ‘std::basic_string<char>’
forum.cpp:40:38: error: expected primary-expression before ‘[’ token
forum.cpp:48:37: error: size of array has non-integral type ‘std::string’
forum.cpp:48:37: error: size of array has non-integral type ‘std::string’
forum.cpp:48:33: error: expected primary-expression before ‘[’ token

pourtant la syntaxe de déclaration de l'itérateur itn est bonne!!!!
et je ne comprend pas la signification du message d'erreur size of array has non-integral type ‘std::basic_string<char>’bref si quelqu'un pourrait m'aider, parce que là je vois pas du tout !

merci beaucoup

[Bousk]

Bonjour,

pourtant la syntaxe de déclaration de l'itérateur itn est bonne!!!!

Faux !

Code :

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

map<string,mapstrvect>::iterator itn=Nom_Notes_mat.begin();

Nom_Notes_mat est un type, et pas une variable.
begin est une fonction sur un objet map.

[Davidbrcz]

Code :

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

map< string, map<string, vector<float> > >

Smell code !!!!!!


Il faut comprendre que lorsqu'on voit ca, il faut tiquer et se dire que quelque chose ne va pas.

Il manque dans ton programme des abstractions nécéssaires pour rendre le tout plus lisible et manipulable. Ton programme est sensé faire quoi ?

[dauphin11]

merci !!

maintenant j'ai un autre problème...

Le code suivant doit lire un fichier texte donné contenant sur chaque ligne une matière, un nom d'élève, et une note
et remplir une map de map à partir de celui-ci, sauf que l'affichage provoque une segfault dont je ne vois pas l'origine!!

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
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
#include <fstream>
 
#include <string> 
 
#include <iostream>
 
#include <map>
 
#include <vector>
 
#include <stdio.h>
 
#include <algorithm>
 
#include <iterator>
 
#include <cstring>
 
using namespace std;
 
 
 
 
 
bool test_lecture()
 
{
 
    ifstream file( "notes100.txt" );
 
    if ( !file )
 
    {
 
        cerr << "Erreur d'ouverture\n";
 
        return false;
 
    }
 
    else return true;
 
    string line;
 
    if ( ! ( file >> line ) )
 
    {
 
        cerr << "Erreur de lecture\n";
 
        return false;
 
    }
 
    else return true;
 
}
 
 
 
 
 
int main(){
 
 
 
 
 
vector<float> notes;
 
vector<string> noms;
 
vector<string> matieres;
 
map < string, vector<float> >  mapstrvect;
 
 map < string, vector<float> > Mat_Notes;
 
 map < string, vector<float> > Nom_Notes;
 
map < string, map < string, vector<float> > > Nom_Notes_mat;
 
string nom;
 
string matiere;
 
float note;
 
 
 
 
 
int s = sizeof(matieres);
 
    int t = sizeof(noms);
 
    vector< vector<float> > synthese((s+1)*(t+1));
 
 
 
 
 
ifstream fichier( "elevesNotes.txt"); 
 
 if (!test_lecture()) cout<<"pb"<<endl;
 
 else{
 
   while (!fichier.eof( ))   {
 
 
 
     fichier>>nom;
 
     fichier>>matiere;
 
     fichier>>note;
 
 
 
     matieres.push_back(matiere);
 
     notes.push_back(note);
 
     noms.push_back(nom);
 
 
 
     sort (noms.begin(), noms.end());
 
     Nom_Notes[nom].push_back(note);
 
     Mat_Notes[matiere].push_back(note);
 
     ((Nom_Notes_mat[matiere])[nom]).push_back(note);
 
   }
 
 
 
map< string, map < string, vector <float> > >::iterator itn=Nom_Notes_mat.begin();
 
map< string, vector <float> >::iterator it=(itn->second).begin(); 
 
 
 
for(;itn!= Nom_Notes_mat.end();++itn){
 
   for(;it!= (itn->second).end();++it){
 
     //for(int k=0;k<((*it).second).size();++k){
 
     cout << (it->first) << endl;
 
       //cout << (it->second) [k] <<endl;
 
       // cout << (*itn).first << endl;
 
   }
 
	 }
 
 
 
vector<string>::iterator itv;
 
itv=noms.begin();
 
for (;itv< noms.end();itv++){
 
  /*if ( itv++ != noms.end() )
 
    {cout << *itv << *(itv++) << endl;}
 
    else cout << *itv << endl;*/
 
    const char* c = (itv->c_str());
 
    const char* cc = ( (itv++)->c_str());
 
   if ( strcmp( c,cc ) == 0)
 
	 {  
 
         vector<string>::iterator it=itv;
 
         noms.erase(it);}
 
 
 
	}
 
 
}
}

Le message donné par valgrind ne m'inspire pas..
vous arrivez à voir où est le problème????

merciiiiiiiiiiiiiiiiiiiiii

[dauphin11]

j'oubliais: le but est d'afficher un tableau répertoriant toutes les matières, tous les élèves et leurs notes respectives avec la moyenne s'ils ont plusieurs notes...

la suite de mon prog s'en occupe mais pour l'instant je n'arrive meme pas à afficher le contenu de ma map histoire de voir ce qu'elle contient!!!

[regis.portalez]

(*itn).first : itn->first
C'est plus lisible et ça fait moins peur

sizeof(matieres) - > matieres.size()
tu fais du C++

Nom_Notes_mat.begin()
comme dit l'autre, le Nom_Notes_mat est un type et pas une instance...

Dans la suite, faire une boucle sur une map dans laquelle on erase certains iterators est dangereux...

Sinon t'es sur que t'as besoin de map de map pour faire ça?
Ya pas plus simple? (toujours se poser cette question...)