Problème d'affichage d'une multimap

Version imprimable

bonjour,

Voici mon problème :
j'ai une multimap std::multimap<int, std::map<std::string, std::string> >

j'ai crée un itérateur sur cette multimap et j'ai fait une boucle pour afficher son contenu :
Code:

1 2 3 4 5 6 for(it = laMapCalendrier.begin() ; it != laMapCalendrier.end() ; it++) { std::cout << it->first << "|"; std::cout << it->second.first << "|"; std::cout << it->second.second<<std::endl; }
mais le compilateur me met une erreur pour

Code:

std::cout << it->second.first << "|";

et je ne sais pas comment faire pour que ça marche :?

Merci d'avance pour votre aide

20/05/2008, 20h36
Mat007

Salut,

it->second est de type std::map<std::string, std::string> et n'a donc pas de membre first.
Tu vas devoir faire une 2e boucle imbriquée...

MAT.

J'ai essayé

Code:

1
2
3
4
5
6
7
8
    for(it = laMapCalendrier.begin() ; it != laMapCalendrier.end() ; it++)
    {
        std::cout << it->first;
        for(it3 = mapMatch.begin(); it3 != mapMatch.end(); it3++)
        {
            std::cout << it3->first << "," << it3->second;
        }
    }

avec

Code:

1
2
3
4
std::multimap<int, std::map<std::string, std::string> > laMapCalendrier;
std::map<std::string, std::string> mapMatch;
std::multimap<int, std::map<std::string, std::string> >::const_iterator it;
std::map<std::string, std::string>::const_iterator it3;

mais quand j'essaye de compiler, le fichier stl_pair.h s'ouvre pour me dire qu'il y a une erreur dans ce fichier :?

20/05/2008, 22h30
Mat007

Peux-tu prendre la bonne habitude de donner un code minimal qui reproduit le problème, le but étant qu'on ait juste à le copier-coller dans un fichier pour pouvoir le compiler ?
Et puis donne aussi un copier-coller de la sortie du compilateur pour avoir les erreurs exactes (ou au moins la première), en indiquant à quelle ligne précise ça correspond dans le code que tu donnes.
Parce que "mais quand j'essaye de compiler, le fichier stl_pair.h s'ouvre pour me dire qu'il y a une erreur dans ce fichier", c'est à la fois vague et pas très réaliste... :roll:

Sinon en général on écrit ++it et non it++ dans une boucle.

MAT.
20/05/2008, 23h55
pikmin

ok je donnerais ce qui est nécessaire demain ;)

pour le it++, j'ai toujours appris comme ça, mais c'est bon à savoir, j'essaierai de faire gaffe dorénavant

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
#include "Equipe.hpp"
#include <vector>
#include <string>
 
class Championnat
{
    std::vector <std::string> lesEquipes;
    public:
    void statistique();
    void genererCalendrier();
};

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
    std::vector<std::string>::const_iterator it1, it2;
    std::multimap<int, std::map<std::string, std::string> > laMapCalendrier;
    std::map<std::string, std::string> mapMatch;
 
    int leNumeroJournee=1;
    for(it1 = lesEquipes.begin() ; it1 != lesEquipes.end() ; it1++)
    {
        it2=it1;
        for( it2++ ; it2 != lesEquipes.end() ; it2++)
        {
            std::pair<std::string, std::string> p(*it1, *it2);
            leNumeroJournee=(leNumeroJournee+1)%(sonNombreEquipe-1)+1;
            std::pair<int, std::pair<std::string, std::string> >p2(leNumeroJournee, p);
            laMapCalendrier.insert(p2);
 
        }
    }
 
    int leNombreJourneeAller = sonNombreEquipe-1;
    std::multimap<int, std::map<std::string, std::string> >::const_iterator it;
    std::map<std::string, std::string>::const_iterator it3;
    std::cout<<"Matchs Aller"<<std::endl;
    for(it = laMapCalendrier.begin() ; it != laMapCalendrier.end() ; it++)
    {
        std::cout << it->first;
        for(it3 = mapMatch.begin(); it3 != mapMatch.end(); it3++)
        {
            std::cout << it3->first << "," << it3->second;
        }
    }

Code:

1
2
3
4
5
6
7
8
9
10
||=== gestion_championnat, Debug ===|
E:\IUT\projet_prog2\gestion_championnat\championnat.cpp||In member function `void Championnat::genererCalendrier()':|
E:\IUT\projet_prog2\gestion_championnat\championnat.cpp|84|warning: unused variable 'leNombreJourneeAller'|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\3.4.5\..\..\..\..\include\c++\3.4.5\bits\stl_pair.h||In constructor `std::pair<_T1, _T2>::pair(const std::pair<_U1, _U2>&) [with _U1 = int, _U2 = std::pair<std::string, std::string>, _T1 = const int, _T2 = std::map<std::string, std::string, std::less<std::string>, std::allocator<std::pair<const std::string, std::string> > >]':|
E:\IUT\projet_prog2\gestion_championnat\championnat.cpp|79|instantiated from here|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\3.4.5\..\..\..\..\include\c++\3.4.5\bits\stl_pair.h|90|error: no matching function for call to `std::map<std::string, std::string, std::less<std::string>, std::allocator<std::pair<const std::string, std::string> > >::map(const std::pair<std::string, std::string>&)'|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\3.4.5\..\..\..\..\include\c++\3.4.5\bits\stl_map.h|166|note: candidates are: std::map<_Key, _Tp, _Compare, _Alloc>::map(const std::map<_Key, _Tp, _Compare, _Alloc>&) [with _Key = std::string, _Tp = std::string, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, std::string> >]|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\3.4.5\..\..\..\..\include\c++\3.4.5\bits\stl_map.h|156|note:                 std::map<_Key, _Tp, _Compare, _Alloc>::map(const _Compare&, const typename std::_Rb_tree<_Key, std::pair<const _Key, _Tp>, std::_Select1st<std::pair<const _Key, _Tp> >, _Compare, _Alloc>::allocator_type&) [with _Key = std::string, _Tp = std::string, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, std::string> >]|
C:\Program Files\CodeBlocks\MinGW\bin\..\lib\gcc\mingw32\3.4.5\..\..\..\..\include\c++\3.4.5\bits\stl_map.h|148|note:                 std::map<_Key, _Tp, _Compare, _Alloc>::map() [with _Key = std::string, _Tp = std::string, _Compare = std::less<std::string>, _Alloc = std::allocator<std::pair<const std::string, std::string> >]|
||=== Build finished: 1 errors, 1 warnings ===|

21/05/2008, 19h28
Mat007
Ok, mais c'est loin d'être un code minimal et tu n'indiques pas vraiment quelle est la fameuse ligne 79 de championnat.cpp qui génère l'erreur...

Cela dit déjà tu ne dois pas déclarer les variables à l'avance en C++ (c'est en C qu'on fait ça) mais uniquement au moment où tu en as besoin, par ex :
Code:

1 2 for(std::vector<std::string>::const_iterator it1 = lesEquipes.begin() ; it1 != lesEquipes.end() ; ++it1) ...
(ce qui t'éviterait d'oublier d'initialiser correctement mapMatch notamment)

Sinon globalement tu devrais plutôt écrire le code petit bout par petit en ne rajoutant que quelques lignes à chaque fois après t'être assuré que tout est correct et fonctionne, plutôt que d'essayer de tout écrire d'un coup parce que tu ne t'en sortiras pas de cette manière...

MAT.
21/05/2008, 22h35
pikmin

c'est bon j'ai réussi à m'en sortir, l'erreur venait de l'insert pour la multimap. J'avais changé une partie du code, mais pas ça