Discussion :

[maud4274]

Bonjour,

J'avais ouvert un post qui ressemble à celui-ci dans le titre mais dans ce cas, le problème est tout autre.
Je dispose de :

Code :

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

1
2
3
vector<tuple<int,int,int>> test; 
map<int, pair<MyPoint,MyPoint>> myMap; // La map que je remplie
test = myPipingModel.getvectorIdPipePoint(); // vector de la forme vector<tuple<idPipe, idPoint1, idPoint2>>

Le construction de MyPoint :

Code :

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

1
2
3
MyPoint::MyPoint(int id, double x, double y, double z) : idPoint(id), coordX(x), coordY(y), coordZ(z)
{
}

Je rempli ma map :

Code :

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

1
2
3
4
5
6
int p=0;
for (auto t = test.begin(); t != test.end(); ++t) // TODO : Problème de taille ici (changer j en 2*j)
{
	myMap[get<0>(*t)] = make_pair(MyPoint(get<1>(*t), myPipingModel.getX(p), myPipingModel.getY(p), myPipingModel.getZ(p)), MyPoint(get<2>(*t), myPipingModel.getX(p+1), myPipingModel.getY(p+1), myPipingModel.getZ(p+1)));
	p++;
}

Pour le remplissage ci-dessus, je procède de la manière suivante :
- la clé c'est idPipe que je récupère dans test. Pour remplir la paire, je dispose de trois vectors (pour X, Y et Z) avec les coordonnées. Donc pour le premier passage de boucle, je prend la position 0 de chaque vector et ainsi de suite pour le 2ème pipe et 3ème pipe.

J'affiche ensuite le contenu de ma map :

Code :

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

1
2
3
4
5
6
7
8
for (auto p1 = myMap.begin(); p1 != myMap.end(); ++p1 )
{
	cout << "pipe " << p1->first << endl;
	cout << "IdPoint 1 : " << (p1->second).first.getIdPoint() << " - x : " << (p1->second).first.getCoordX() << " - y : " << (p1->second).first.getCoordY() << " - z : " << (p1->second).first.getCoordZ() << endl;
	cout << "IdPoint 2 : " << (p1->second).second.getIdPoint() << " - x : " << (p1->second).second.getCoordX() << " - y : " << (p1->second).second.getCoordY() << " - z : " << (p1->second).second.getCoordZ() << endl;
             cout << endl;
	getchar();
}

Et là....il se passe quelque chose de très étrange : j'obtiens bien les bons pipe : 1, 2, 3, ... mais pour les affichage des coordonées, cela m'affiche les x, y et z du milieu de mes vectors.
Par exemple, j'ai les coordonées suivantes dans mes vectors de coordonées :
Pour X : 1 2 3 5 6
Pour Y : 5 2 3 6 9
Pour Z : 5 8 9 6 3

et bien cela va m'afficher :

Code :

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

pipe 1  idPoint 1 : 1 - x : 3 - y : 3 - z : 9

Au lieu de :

Code :

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

pipe 1  idPoint 1 : 1 - x : 1 - y : 5 - z : 5

Ca me prend à partir de la position au milieu de mes vectors ! C'est un vrai mystère...
Si vous avez une solution à ce problème

Merci d'avance pour vos réponses

[ternel]

je te propose quelques améliorations pratiques:

écrire une fonction (libre) d'affichage d'un point.

Code :

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

1
2
3
std::ostream& operator<<(std::ostream& os, MyPoint const& p) {
    return os <<"IdPoint :" << p.getIdPoint() << " - x : " << p.getCoordX() << " - y : " << p.getCoordY() << " - z : " << p.getCoordZ() ;
}

et utiliser la range for loop dans ton code d'affichage deviendra:

Code :

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

1
2
3
4
5
for (auto const& entry: myMap) {
	cout << "pipe " << entry.first << endl;
	cout << "point 1 : " << (entry.second).first << "\npoint 2 : " << (entry.second).second << '\n'<<endl;
	getchar();
}

cela posé, tu as probablement déjà ta réponse, sous forme d'un commentaire marqué TODO.
Il y a deux solutions envisageables:

Code :

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

1
2
3
4
5
6
7
8
int p=0;
for (auto const& tuple : test) {
	myMap[get<0>(t)] = make_pair(
		MyPoint(get<1>(t), myPipingModel.getX(p), myPipingModel.getY(p), myPipingModel.getZ(p)),
		MyPoint(get<2>(t), myPipingModel.getX(p+1), myPipingModel.getY(p+1), myPipingModel.getZ(p+1))
	);
	p+=2;
}

et, en évitant une création de temporaire inutile (dans map::operator[]):

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
int p=0;
for (auto tuple : test) {
    myMap.emplace(
		make_pair(
			get<0>(t),
			make_pair(
				MyPoint(get<1>(t), myPipingModel.getX(p), myPipingModel.getY(p), myPipingModel.getZ(p)),
				MyPoint(get<2>(t), myPipingModel.getX(p+1), myPipingModel.getY(p+1), myPipingModel.getZ(p+1))
			)
		)
	);
	p+=2;
}

[maud4274]

Je te remercie pour ta réponse rapide et efficace !
J'utilise Visual C++, je ne peux donc pas utiliser auto de cette manière dans mon code. J'ai deja été confronté à ce problème. J'ai pu modifier le remplissage en :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int p=0;
for (auto tuple = test.begin(); tuple != test.end(); ++tuple) 
{
	myMap.emplace(
		make_pair(
			get<0>(*tuple),
			make_pair(
				MyPoint(get<1>(*tuple), myPipingModel.getX(p), myPipingModel.getY(p), myPipingModel.getZ(p)),
				MyPoint(get<2>(*tuple), myPipingModel.getX(p+1), myPipingModel.getY(p+1), myPipingModel.getZ(p+1))
			)
		)
	);
	p+=2;
}

Ca me parait correct (dis-moi sinon)

Mais pour l'affichage, je ne sais pas comment modifier la boucle FOR, j'ai essayer :

Code :

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

1
2
3
4
5
6
for (auto const& entry = myMap.begin(); entry != myMap.end(); ++entry) 
{
	cout << "pipe " << entry.first << endl;
	cout << "point 1 : " << (entry.second).first << "\npoint 2 : " << (entry.second).second << '\n'<<endl;
	getchar();
}

Mais j'ai l'erreur suivante :

Code :

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

error C2678: '++' binaire*: aucun opérateur trouvé qui accepte un opérande de partie gauche de type 'const std::_Tree_iterator<_Mytree>' (ou il n'existe pas de conversion acceptable)

Je ne sais pas comment modifier !

Merci d'avance pour ta réponse

[maud4274]

Je précise, j'utilise Visual C++ 2010 !

[maud4274]

J'ai trouvé !!
Pour l'affichage j'ai tout simplement mis :

Code :

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

1
2
3
4
5
6
7
8
for (auto entry = myMap.begin(); entry != myMap.end(); ++entry) 
{
	cout << "pipe " << entry->first << endl;
	cout << "IdPoint 1 : " << (entry->second).first.getIdPoint() << " - x : " << (entry->second).first.getCoordX() << " - y : " << (entry->second).first.getCoordY() << " - z : " << (entry->second).first.getCoordZ() << endl;
	cout << "IdPoint 2 : " << (entry->second).second.getIdPoint() << " - x : " << (entry->second).second.getCoordX() << " - y : " << (entry->second).second.getCoordY() << " - z : " << (entry->second).second.getCoordZ() << endl;
 
	getchar();
}

Merci beaucoup pour ton aide
Je ne regrète vraiment pas de m'être inscrit sur ce forum !

[ternel]

Pour information, les .begin() et .end() de map/vector/set... retourne des itérateurs, qu'il faut manipuler comme des pointeurs pour accéder à la valeur.
Un itérateur est une classe (contenant un pointeur ou un index) et définissant deux opérations:

• avancer (operator++())
• comparaison (operator!=())
• valeur courante (operator->() et operator*() )

for (TYPE const& item : collection) permet d'accéder directement par référence constante aux éléments de la collection.
for (Collection_type<Truc>::const_iterator it = collection.begin(); it!=collection.end(); ++it) utilise l'itérateur (constant si possible) de la collection.

Comme c'est parfois très long à écrire, pour cause de collection template de classe template, C++11 a introduit le "auto" qui signifie littéralement:
"Cher compilateur, puisque tu sais quel est le seul type utilisable, sers-t'en."
ce qui autorise la notation raccourcie: for (auto it = collection.begin(); it!=collection.end(); ++it)Par contre, n'utilise auto que lorsque tu sais exactement quel type se cache derrière, et que ce sera aussi le cas du moindre lecteur.

Comme tu l'as vu dans mes fragments de code, auto accepte certains modificateurs, tels que const et & (référence).
D'une manière général, utilise autant que possible la référence constante quand tu manipules des classes (d'où mesauto const&)

[maud4274]

Cette syntaxe :

Code :

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

for (TYPE const& item : collection) permet d'accéder directement par référence constante aux éléments de la collection.

n'existe malheureusement pas sous Visual C++ 2010 ...

Merci pour toutes ces informations !

[ternel]

Au plaisir, et bonne continuation

Au passage, essaie de passer à un version plus récente, comme la 2013.

[maud4274]

Merci

Malheureusement, je suis en stage et on m'impose cet environnement de développement ...

[ternel]

Tu peux toujours essayer le "avec la dernière version, les performances sont meilleurs, grace à des améliorations du langage". C'est presque vrai, et ca peut les motiver.

Sinon, tant pis, tu as déjà une partie de C++11

[Médinoc]

Il faut aussi avoir Windows 7 ou supérieur pour faire tourner VS2013 dessus. Il n'accepte même pas Vista, ce que je considère comme une connerie (ou malveillance) monumentale de la part de Microsoft puisque Vista était encore supposé être supporté, contrairement à XP dont le support a expiré.

[maud4274]

Je suis sous XP, donc il ne supporterai même pas la dernière version !
Ils ont même pas acheter le pack Visual Studio, j'ai seulement la version Express alors imagine si je leur demande d'acheter...moi petite stagiaire que je suis ^^

[r0d]

Il faut aussi avoir Windows 7 ou supérieur pour faire tourner VS2013 dessus. Il n'accepte même pas Vista, ce que je considère comme une connerie (ou malveillance) monumentale de la part de Microsoft puisque Vista était encore supposé être supporté, contrairement à XP dont le support a expiré.

C'est une conséquence de la politique de Microsoft: le testeur est le client. Ils sont donc contraints de forcer les clients à passer à des versions plus récentes de leurs OS.
De plus, Microsoft a toujours fait en sorte que les développeurs soient un peu des pionniers (versions gratuites de visual studio, facilités pour les VM, etc.), de façon à servir de "locomotive", pour le dire comme ça.
C'est nul, mais c'est comme ça.