Manipulation des vecteurs 3D

Version imprimable

Bonsoir,

Mon code est le suivant:

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
78
79
 
#include <iostream>
#include <string>
#include <vector>
using namespace std;
 
typedef string Forme   ;
typedef string Couleur ;
 
class Brique
{
private:
  Forme   forme   ;
  Couleur couleur ;
 
public:
Brique(Forme f, Couleur c)
{forme=f;
 couleur=c;
};
ostream& afficher(ostream& sortie) const {
	if(couleur.empty()==false){
	sortie<<"("<<forme<<", "<<couleur<<")";}
	else{
	sortie<<forme;}
	return sortie;
}
 friend std::ostream& operator <<(std::ostream&, const Brique&); 
};
std::ostream& operator <<(std::ostream& Stream, const Brique& Obj){
    Obj.afficher(Stream);
    return Stream; 
}
class Construction
{
  friend class Grader;
  public: vector<vector<vector<Brique>>> contenu;
  public:Construction(Brique b):contenu (1, vector<vector<Brique>>(1,vector<Brique> (1, b))){
  }
  ostream& afficher(ostream& sortie) const {
		for(int i(0); i < (int)contenu.size(); i++) {
      for(int j(0); j < (int)contenu[i].size(); j++) {
          for(int k(0); k < (int)contenu[i][j].size(); k++) {
                   contenu[i][j][k].afficher(sortie);}}}
    return sortie;
 
}
void operator^=(Construction const& b)
  {   
	  for(int i(0); i < (int)contenu.size(); i++) {
		  contenu.push_back(b.contenu[i]);
		  }
 
  }     
 
};
 std::ostream& operator <<(std::ostream& Stream, const Construction& Obj){
    Obj.afficher(Stream);
    return Stream; 
}
 
 
int main()
{
  // Modèles de briques
  Brique toitD("obliqueD", "rouge");
  Brique toitG("obliqueG", "rouge");
  Brique toitM(" pleine ", "rouge");
  Brique mur  (" pleine ", "blanc");
  Brique vide ("                 ", "");
 
cout << toitD;
//Tester Construction 
Construction c(mur);
Construction c1(toitD);
c^= c1;
cout<< c;
  return 0;
}

Mon programme compile bien, mais il se plante au moment de l'exécution..
J'ai un doute concernant:

Code:

1
2
3
4
5
6
7
8
 
void operator^=(Construction const& b)
  {   
	  for(int i(0); i < (int)contenu.size(); i++) {
		  contenu.push_back(b.contenu[i]);
		  }
 
  }

Avez-vous une idée ? Est-ce je dois modifier la taille de "contenu" ?

Merci d'avance pour votre aide.

Citation:
Envoyé par chercheur111
Bonsoir,

Mon code est le suivant:
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 78 79 #include <iostream> #include <string> #include <vector> using namespace std; typedef string Forme ; typedef string Couleur ; class Brique { private: Forme forme ; Couleur couleur ; public: Brique(Forme f, Couleur c) {forme=f; couleur=c; }; ostream& afficher(ostream& sortie) const { if(couleur.empty()==false){ sortie<<"("<<forme<<", "<<couleur<<")";} else{ sortie<<forme;} return sortie; } friend std::ostream& operator <<(std::ostream&, const Brique&); }; std::ostream& operator <<(std::ostream& Stream, const Brique& Obj){ Obj.afficher(Stream); return Stream; } class Construction { friend class Grader; public: vector<vector<vector<Brique>>> contenu; public:Construction(Brique b):contenu (1, vector<vector<Brique>>(1,vector<Brique> (1, b))){ } ostream& afficher(ostream& sortie) const { for(int i(0); i < (int)contenu.size(); i++) { for(int j(0); j < (int)contenu[i].size(); j++) { for(int k(0); k < (int)contenu[i][j].size(); k++) { contenu[i][j][k].afficher(sortie);}}} return sortie; } void operator^=(Construction const& b) { for(int i(0); i < (int)contenu.size(); i++) { contenu.push_back(b.contenu[i]); } } }; std::ostream& operator <<(std::ostream& Stream, const Construction& Obj){ Obj.afficher(Stream); return Stream; } int main() { // Modèles de briques Brique toitD("obliqueD", "rouge"); Brique toitG("obliqueG", "rouge"); Brique toitM(" pleine ", "rouge"); Brique mur (" pleine ", "blanc"); Brique vide (" ", ""); cout << toitD; //Tester Construction Construction c(mur); Construction c1(toitD); c^= c1; cout<< c; return 0; }
Mon programme compile bien, mais il se plante au moment de l'exécution..
J'ai un doute concernant:
Code:

1 2 3 4 5 6 7 8 void operator^=(Construction const& b) { for(int i(0); i < (int)contenu.size(); i++) { contenu.push_back(b.contenu[i]); } }
Avez-vous une idée ? Est-ce je dois modifier la taille de "contenu" ?
Non, push_back() va le faire pour toi.

Ceci dit, ce que tu fait est une copie. Je ne vois pas le rapport avec l'opérateur ^=. Si tu souhaite faire une copie, alors un simple contenu = b.contenu suffira (c'est peu élégant, mais ça marchera et tu n'a pas besoin de te préocupper des question de parcours ou de la taille).

Accessoirement, la boucle devrait plutôt ressembler à :
Code:

1 2 3 4 for (std::size_t i = 0; i < contenu.size(); ++i) { ... }
* type de i == std::size_t, pas int. sizeof(std::size_t) n'est pas garanti d'être sizeof(int) et std::size_t n'est pas signé, contrairement à int.
* ++i au lieu de i++. Bien que ça n'ait que très peu d'impact dans ce cas précis, il est toujours préférable d'utiliser la notation préfixée lorsque c'est possible (on évite une copie dans de très nombreux cas)

Pendant que j'y suis : tu viens de récupérer un gage absolument horrible (genre "faire 3 fois le tour de monde sur un pied les yeux bandés") pour cette ligne de code, absolument affreuse (et qui va plus que probablement te poser de très, très nombreux problèmes :
Code:

1 2 public: vector<vector<vector<Brique>>> contenu;
C'est immonde. Horrible. Impensable. Indicible.

La punition sera allégée (2 tours du monde) si tu es capable de justifier un tel choix technique.

27/03/2014, 01h22
Bousk
BOnsoir,
Citation:
Envoyé par chercheur111

Mon programme compile bien, mais il se plante au moment de l'exécution..
J'ai un doute concernant:

Code:

1 2 3 4 5 6 7 8 void operator^=(Construction const& b) { for(int i(0); i < (int)contenu.size(); i++) { contenu.push_back(b.contenu[i]); } }
TU as un bon feeling, mais es-tu seulement conscient de ce que tu fais là ?
Tant que tu as un élément dans contenu, tu ajoutes à contenu l'élement i de b.contenu
... encore faut-il que b.contenu[i] existe, vu que ça crash, on dirait que non
remercie le crash qui t'empêche de partir en boucle infini : i < size, j'ajoute un élement (ie: incrément de size), incrément de i; repeat...

Classiquement l'opérateur ^ est surchargé pour des vector3 pour réaliser le cross-product (produit vectoriel)
27/03/2014, 10h01
dragonjoker59

Ta boucle est fausse, tu teste la taille de contenu au lieu de la taille de b.contenu.

Sinon, pour faire ça sans boucle dans ton code :

Code:

contenu.insert( contenu.end(), b.contenu.begin(), b.contenu.end() );
31/03/2014, 23h36
chercheur111

Merci Tous pour vos réponses.
Je suis encore "débutant" avec les vecteurs 3D.. J'essaye de prendre vos remarques en considération ..