Remplissage Vector c++

**chercheur111** · 16/03/2014, 11h39

Bonjour,

Mon code est le suivant:

Code C :

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
#include <iostream>
#include <vector>
#include <string>
using namespace std;

class Auteur{
	string nom;
	bool prime;
	public: Auteur(){
	}
	public: Auteur (string n, bool p=false){
		nom=n;
		prime=p;
	}
	public: string getNom(){
	return nom;
	}
	public:bool getPrix(){
	return prime;
	}
};
class Oeuvre{
	public:string titre;
	Auteur a;
	string langue;
	public: Oeuvre(){
	}
	public: Oeuvre(string tit, Auteur & aut, string lang){
	 titre=tit;
	 a=aut;
	 langue=lang;}
    public: string getTitre(){
	return titre;}
	public: const Auteur getAuteur(){
	return a;}
	public: string getLangue(){
	return langue;}
	public: void affiche(){
    cout<<titre<<", "<<a.getNom()<<", en "<<langue<<endl;}
    public:~Oeuvre(){
    cout<<"L'oeuvre "<<titre<<", "<<a.getNom()<<", en "<<langue<<" n'est plus disponible."<<endl;}
};
class Exemplaire{
	public: Oeuvre  o;
	public: Exemplaire()
	{}
	Exemplaire(Oeuvre & o){
		cout<<"Nouvel exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
	}
	Exemplaire( Exemplaire & e){
    cout<<"Copie d'un exemplaire de : "<<e.o.titre<<", "<<e.o.a.getNom()<<", en "<<o.langue;}
    public:~Exemplaire(){
	cout<<"Un exemplaire de "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<" a été jeté !"<<endl;
	}
	public: const Oeuvre getOeuvre(){
	return o;}
	public: void affiche(){
    cout<<"Exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
    }
};
class Bibliotheque{
	public: string nom;
	public:vector<Exemplaire *> exps;
	public:Bibliotheque(string n){
		nom=n;
		cout<<"La bibliothèque "<<nom<<" est ouverte !"<<endl;
		
		}
	public: string getNom(){
	return nom;}
	public: void stocker(Oeuvre  &oo, int n=1){
		int count=1;
		while (count<=n)
		{exps.push_back(new Exemplaire( oo));
		count++;}
		
	}
	public: void lister_exemplaires(string langue=""){
		int taille=exps.size();
		if(langue.empty()==true)
		{ 
			for (auto & ex : exps) {
			(*ex).affiche();}

			for(int i(0);i<taille;i++)
			{
			Exemplaire e=(*(exps[i]));
			Oeuvre o=e.getOeuvre();
		    cout<<endl;
			}
		}
		else {
			for(int i(0);i<taille;i++)
			{if(exps[i]->o.langue==langue){
		     exps[i]->affiche();
		     cout<<endl;
		     cout<<"empty False"<<endl;}
		    }
		}
	}
	
};

int main()
{
  Auteur a1("Victor Hugo"),
         a2("Alexandre Dumas"),
         a3("Raymond Queneau", true);

  Oeuvre o1("Les Misérables"           , a1, "français" ),
         o2("L'Homme qui rit"          , a1, "français" ),
         o3("Le Comte de Monte-Cristo" , a2, "français" ),
         o4("Zazie dans le métro"      , a3, "français" ),
         o5("The Count of Monte-Cristo", a2, "anglais" );

  Bibliotheque biblio("municipale");
  biblio.stocker(o1, 2);
  biblio.stocker(o2);
  biblio.stocker(o3, 3);
  biblio.stocker(o4);
  biblio.stocker(o5);
  

  cout << "La bibliothèque " << biblio.getNom()
       << " offre les exemplaires suivants :" << endl;
  biblio.lister_exemplaires();

  return 0;
}

J'ai un problème au niveau de la fonction "stocker", elle ne fonctionne pas correctement et les objets ne sont pas stockés!
Lorsque j'appelle " lister_exemplaires", rien ne s'affiche!
Au lieu d'avoir:

Code :

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

1
2
3
4
5
6
7
8
9
 
Exemplaire de : Les Misérables, Victor Hugo, en français
Exemplaire de : Les Misérables, Victor Hugo, en français
Exemplaire de : L'Homme qui rit, Victor Hugo, en français
Exemplaire de : Le Comte de Monte-Cristo, Alexandre Dumas, en français
Exemplaire de : Le Comte de Monte-Cristo, Alexandre Dumas, en français
Exemplaire de : Le Comte de Monte-Cristo, Alexandre Dumas, en français
Exemplaire de : Zazie dans le métro, Raymond Queneau, en français
Exemplaire de : The Count of Monte-Cristo, Alexandre Dumas, en anglais

J'ai eu:

Code :

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

1
2
3
4
5
6
7
8
9
 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en 
Exemplaire de : , , en

Avez-vous une idée ?
Merci d'avance pour votre aide.

**JolyLoic** · 16/03/2014, 12h53

Il est faux de dire que rien ne s'affiche : Le bon nombre d’exemplaires s'affiche, mais ces exemplaires sont "vides".
Regardons donc comment un exemplaire est crée :

Code :

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

1
2
3
	Exemplaire(Oeuvre & o){
		cout<<"Nouvel exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
	}

On passe en argument au constructeur d'un exemplaire une œuvre, mais même si on affiche des choses à son sujet, on ne stocke pas sa valeur dans l'exemplaire (le fait que le paramètre du constructeur ait le même nom que la variable membre n'empêche pas qu'il s'agit de deux objets différents).

Code :

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

1
2
3
4
	Exemplaire(Oeuvre & o) : o(o)
        {
		cout<<"Nouvel exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
	}

Devrait commencer à aller mieux. Mais ce n'est pas très satisfaisant : Chaque exemplaire va créer une copie de l’œuvre, alors que l’œuvre est probablement censée être unique. Et il y a d'autres soucis dans le code (par exemple un constructeur de copie qui affiche des choses, mais ne copie rien). En terme de style, répéter public: devant chaque fonction est peu courant en C++ : On préfère mettre une section pulbique avec tout un ensemble de fonctions à l'intérieur.

**chercheur111** · 16/03/2014, 15h40

D'accord. Merci beaucoup pour votre réponse

S'il vous plaît, j'ai juste une deuxième question:

J'ai ajouté le destructeur :

Code :

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

1
2
3
4
5
6
7
8
 
          ~Bibliotheque(){
	    cout<<"La bibliothèque "<<nom<<" ferme ses portes,"<<endl;
	    cout<<"et détruit ses exemplaires :"<<endl;
	    int taille=exps.size();
		for(int i(0);i<taille;i++)
		delete exps[i];
   }

à la fin de l'exécution, il s'exécute automatiquement et m'affiche:

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
 
La bibliothèque municipale ferme ses portes,
et détruit ses exemplaires :
Un exemplaire de Les Misérables, Victor Hugo, en français a été jeté !
L'oeuvre Les Misérables, Victor Hugo, en français n'est plus disponible.
Un exemplaire de Les Misérables, Victor Hugo, en français a été jeté !
L'oeuvre Les Misérables, Victor Hugo, en français n'est plus disponible.
Un exemplaire de L'Homme qui rit, Victor Hugo, en français a été jeté !
L'oeuvre L'Homme qui rit, Victor Hugo, en français n'est plus disponible.
Un exemplaire de Le Comte de Monte-Cristo, Alexandre Dumas, en français a été jeté !
L'oeuvre Le Comte de Monte-Cristo, Alexandre Dumas, en français n'est plus disponible.
Un exemplaire de Le Comte de Monte-Cristo, Alexandre Dumas, en français a été jeté !
L'oeuvre Le Comte de Monte-Cristo, Alexandre Dumas, en français n'est plus disponible.
Un exemplaire de Le Comte de Monte-Cristo, Alexandre Dumas, en français a été jeté !
L'oeuvre Le Comte de Monte-Cristo, Alexandre Dumas, en français n'est plus disponible.
Un exemplaire de Zazie dans le métro, Raymond Queneau, en français a été jeté !
L'oeuvre Zazie dans le métro, Raymond Queneau, en français n'est plus disponible.
Un exemplaire de The Count of Monte-Cristo, Alexandre Dumas, en anglais a été jeté !
L'oeuvre The Count of Monte-Cristo, Alexandre Dumas, en anglais n'est plus disponible.
L'oeuvre The Count of Monte-Cristo, Alexandre Dumas, en anglais n'est plus disponible.
L'oeuvre Zazie dans le métro, Raymond Queneau, en français n'est plus disponible.
L'oeuvre Le Comte de Monte-Cristo, Alexandre Dumas, en français n'est plus disponible.
L'oeuvre L'Homme qui rit, Victor Hugo, en français n'est plus disponible.
L'oeuvre Les Misérables, Victor Hugo, en français n'est plus disponible.

Alors que je voudrais avoir ceci:

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
 
La bibliothèque municipale ferme ses portes,
et détruit ses exemplaires : 
Un exemplaire de "Les Misérables, Victor Hugo, en français" a été jeté !
Un exemplaire de "Les Misérables, Victor Hugo, en français" a été jeté !
Un exemplaire de "L'Homme qui rit, Victor Hugo, en français" a été jeté !
Un exemplaire de "Le Comte de Monte-Cristo, Alexandre Dumas, en français" a été jeté !
Un exemplaire de "Le Comte de Monte-Cristo, Alexandre Dumas, en français" a été jeté !
Un exemplaire de "Le Comte de Monte-Cristo, Alexandre Dumas, en français" a été jeté !
Un exemplaire de "Zazie dans le métro, Raymond Queneau, en français" a été jeté !
Un exemplaire de "The Count of Monte-Cristo, Alexandre Dumas, en anglais" a été jeté !
L'oeuvre "The Count of Monte-Cristo, Alexandre Dumas, en anglais" n'est plus disponible.
L'oeuvre "Zazie dans le métro, Raymond Queneau, en français" n'est plus disponible.
L'oeuvre "Le Comte de Monte-Cristo, Alexandre Dumas, en français" n'est plus disponible.
L'oeuvre "L'Homme qui rit, Victor Hugo, en français" n'est plus disponible.
L'oeuvre "Les Misérables, Victor Hugo, en français" n'est plus disponible.

Avez-vous une explication, pourquoi j'ai eu des redondances ?
Merci d'avance.
Cordialement.

**JolyLoic** · 16/03/2014, 20h19

C'est lié à ma remarque précédente : Chaque exemplaire stocke une COPIE de l’œuvre, alors même que l’œuvre n'est pas prévue pour être copiable. Je conseillerais par exemple de rendre l’œuvre non copiable (http://cpp.developpez.com/faq/cpp/?p...les_definition), puis en regardant les erreurs de compilation apparaître (c'est instructif pour voir où des copies étaient effectuées).

Ensuite, il faudra modifier la classe exemplaire, pour qu'elle contienne une référence vers l’œuvre, au lieu d'en contenir une copie.

**chercheur111** · 17/03/2014, 21h12

Bonsoir,
Merci beaucoup pour votre réponse.

J'ai rendu l'auteur et l’œuvre non copiables, comme suit:

Code :

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

1
2
3
4
 
 
Auteur( const Auteur& other ); // non construction-copyable
Auteur& operator=( const Auteur& ); // non copyable

Code :

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

1
2
3
 
Oeuvre( const Oeuvre& other ); // non construction-copyable
Oeuvre& operator=( const Oeuvre& ); // non copyable

J'ai changé la classe Exemplaire comme vous m'avez dit:

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
 
class Exemplaire{
	public: Oeuvre & o;
	/*public: Exemplaire()
	{}*/
	Exemplaire(Oeuvre & o):o(o){
		cout<<"Nouvel exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
	}
	Exemplaire( Exemplaire & e){
    cout<<"Copie d'un exemplaire de : "<<e.o.titre<<", "<<e.o.a.getNom()<<", en "<<o.langue;}
    public:~Exemplaire(){
	cout<<"Un exemplaire de \""<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<"\" a été jeté !"<<endl;
	}
	public: const Oeuvre & getOeuvre(){
	return o;}
	public: void affiche(){
    cout<<"Exemplaire de : "<<o.titre<<", "<<o.a.getNom()<<", en "<<o.langue<<endl;
    }
};

J'ai eu l'erreur suivante:

Code :

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

1
2
 
biblio.cc:57:2: erreur: uninitialized reference member 'Exemplaire::o' [-fpermissive]

au niveau du constructeur de copie:

Code :

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

1
2
 
Exemplaire( Exemplaire & e)

J'ai pas su comment l'initialiser ..
Merci bien pour votre aide

**JolyLoic** · 17/03/2014, 22h02

http://cpp.developpez.com/faq/cpp/?p...EUR_liste_init

Code :

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

1
2
3
 
Exemplaire( Exemplaire & e) : o(e.o) {
}

**chercheur111** · 17/03/2014, 22h43

D'accord. Merci.
Le faite de rendre Auteur non copiable m'a généré une erreur au niveau:

Code :

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

1
2
3
4
5
public: Oeuvre(string tit, Auteur & aut, string lang){
	 titre=tit;
	 a=aut;
	 langue=lang;}

L'erreur est:

Code :

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

1
2
3
4
5
 
 
In function `Oeuvre::Oeuvre(std::string, Auteur&, std::string)':
prog.cpp:(.text._ZN6OeuvreC2ESsR6AuteurSs[_ZN6OeuvreC5ESsR6AuteurSs]+0x3d): undefined reference to `Auteur::operator=(Auteur const&)'
collect2: error: ld returned 1 exit status

Je l'ai changé comme suit et ça marche:

Code :

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

1
2
3
4
 
public: Oeuvre(string tit, Auteur & aut, string lang): a(aut){
	 titre=tit;
	 langue=lang;}

**ternel** · 18/03/2014, 10h40

En fait, tu devrais faire de même avec tous les membres.
Dans ce que tu faisais, il y avait trois initialisations par défaut (constructeur sans argument, ou seulement les valeurs par défaut), et trois affectations.

Ton correctif pour l'auteur est d'utiliser le constructeur de copie, ici auteur(const auteur&)

**chercheur111** · 18/03/2014, 11h18

Merci beaucoup pour votre aide et pour vos remarques.
Cordialement.

Remplissage Vector c++

C++

Discussions similaires

Partager

Partager