Discussion :

[chercheur111]

Bonsoir,

Mon code est le suivant:

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
 
#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 :

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

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.

[Emmanuel Deloget]

Bonsoir,

Mon code est le suivant:

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
 
#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 :

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

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 :

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

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 :

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

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.

[Bousk]

BOnsoir,

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

Code :

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

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)

[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 :

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

contenu.insert( contenu.end(), b.contenu.begin(), b.contenu.end() );

[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 ..