Discussion :

[game44]

Bonjour

J'ai un petit probleme;
J'ai une fichier main.cpp qui appelle la classe Sal_info ( et Sal_info est une sous classe de Sal )
Dans un prmier temps, j'avais
" Sal_info(string nom,double prix,int place,string logiciel) "
et dans ce cas tout marchait

Maintenant au lieu de rentrer logiciel en string, je veux le rentrer en tant que vecteur de string et la ca me pose probleme. j'ai donc modifié seulement le main,le Sal_info.cpp et Sal_info.h

voici mes codes

main.cpp

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
 
#include <iostream>
#include <string>
#include <fstream>
#include <vector>
#include "Sal_info.h"
 
using namespace std;
 
int main (){
 
	string nom_salle="rfij";
	int nbreplace=5;
	double taxesalle=4;
 
	vector<string> log;
	log.push_back("contenu1");
	log.push_back("contenu2");
 
	Sal_info sal2(nom_salle,nbreplace,taxesalle, log);
	cout<<endl<<endl<<sal2.get_logiciel()<<" logiciel  "<<endl;
 
return 0;
}

Sal_info.cpp

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
 
#include <iostream>
#include <string>
#include "Sal_info.h"
 
using namespace std;
 
Sal_info::Sal_info(string nom,double prix,int place,vector<string> logiciel):Sal(nom,prix,place){
	this->logiciel = logiciel;
}
 
Sal_info::~Sal_info(void)
{
}
 
string Sal_info::get_logiciel() {
	return logiciel;
}

Sal_info.h

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
 
#include <string>
#include <fstream>
#include <vector>
#include "Sal.h"
 
class Sal_info:
	public Sal
{
 
private :
 
	std::string logiciel;
 
public:
 
	Sal_info(std::string,double,int,std::vector<std::string>);
	~Sal_info(void);
	std::string get_logiciel();
	void afficher();
 
};

Donc si quelqu'un pouvais me donner un petit coup de pouce

Merci

[dragonjoker59]

Dans ta classe 'Sal_info', il faut déclarer 'logiciel' en tant que vecteur de string.
Si c'était une classe à moi, j'aurais de plus passé mes paramètres par référence constante, afin d'éviter les copies intempestives et inutiles :

Code :

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

1
2
 
Sal_info(const std::string &, double, int, const std::vector<std::string> &)

Code :

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

1
2
3
4
5
6
 
Sal_info::Sal_info(const string & p_nom, double p_prix, int p_place, const vector<string> & p_logiciel)
	:	Sal(p_nom,p_prix,p_place)
{
	logiciel = p_logiciel;
}

[3DArchi]

J'aurais même tendance à passer une paire d'itérateur plutôt qu'un vecteur. A la volée :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Sal_info:
	public Sal
{
 
private :
 
	std::vector<std::string> logiciel;
 
public:
        template<class iterator_t_>
	Sal_info(std::string const&,double,int,iterator_t_ begin_, iterator_t_ last_)
        :logiciel(begin_,last_)
       {}
// etc...

Autre chose : quel est l'intérêt de définir un destructeur vide ? Cf F.A.Q. Quand dois-je définir un destructeur ?

[Steph_ng8]

Il ne faut pas oublier que l'accesseur doit retourner un vecteur et non plus un string.
Enfin, un vecteur constant…
http://cpp.developpez.com/faq/cpp/?p...SSE_accesseurs

J'aurais même tendance à passer une paire d'itérateur plutôt qu'un vecteur.

Je suppose que c'est pour permettre une instanciation à partir de n'importe quel type de conteneur ou de tableau.
Mais il vaut maîtriser un minimum les template avant d'utiliser cette méthode, non ?

[koala01]

Salut,

Il ne faut pas oublier que l'accesseur doit retourner un vecteur et non plus un string.
Enfin, un vecteur constant…
http://cpp.developpez.com/faq/cpp/?p...SSE_accesseurs

Oui, mais non...

En fait, Demeter nous conseille de n'exposer que ce qui est absolument nécessaire.

Si tu renvoie carrément la collection de chaines de caractères, tu expose une structure entière, dont tu deviendra dépendant partout où tu l'utilisera, ce qui pourra poser problème le jour où, pour une raison ou une autre, tu souhaiterais changer ton vecteur en list ou en set pour des raisons d'efficacité.

3DArchi donne ce qui pourrait être la meilleure méthode, à savoir utiliser une paire d'itérateur, l'un renvoyant le "premier" élément de la collection, et l'autre renvoyant "ce qui suit le dernier élément, de manière à pouvoir travailler sur le "range" semi ouvert [begin,end[.

Si, en plus, tu imbrique dans ta classe un alias de type sur l'itérateur, et que tu conseille d'utiliser cet itérateur de préférence à tout autre, tu gagne énormément de souplesse et de modification de code lorsque tu prendra la décision dont j'ai parlé plus haut.

En plus, tu remarquera assez facilement que cela se marie plutôt très bien avec les différents algorithmes fournis par le standard

au final, ta classe sal_info pourrait ressembler à quelque chose comme

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
class sal_info : public sal
{
    /* ce typedef n'est là que pour satisfaire ma flegme légendaire :D
     * mais il suffira de changer vector en n'importe quelle collection
     * compatible pour appliquer le changement, sans devoir modifier quoi
     * que ce soit d'autre ;)
     */
    typedef std::vector<std::string> log_collection;
    public:
        typedef typename log_collection::const_iterator log_const_iterator;
        log_const_iterator beginLogiciel() const{return log.begin();}
        log_const_iterator endLogiciel() const{return log.end();}
        /* on peut décider à tout moment d'ajouter le nom d'un logiciel */
        void addLogName(std::string const &  n){log.push_back(n);}
        /* ou de vérifier si la classe dispose d'un logiciel donné */
        bool logExists(std::string const & n)
        {
            struct cmp
            {
                cmp(std::string const & n):n(n){}
                std::string const & n;
                bool operator()(std::string const & s2)
                {
                    return n==s2;
                }
            };
            return std::find_if(log.begin(),log.end(),cmp(n)) != log.end();
        }
    private:
        log_collection log;
};

Je suppose que c'est pour permettre une instanciation à partir de n'importe quel type de conteneur ou de tableau.

Entre autres, mais c'est surtout parce qu'il est préférable de travailler sur des écarts plutôt que sur des collections complètes

Mais il vaut maîtriser un minimum les template avant d'utiliser cette méthode, non ?

PAs forcément, dans le sens où il "suffit" de savoir qu'on peut le faire, et savoir comment le faire, sans forcément comprendre le principe sous-jasent

Bien sur, si on comprend ce qu'on fait, c'est encore mieux

[game44]

OHLA OHLA OHLA....

J'ai pas tout compris.... (et oui je debute)

Alors a part la reponse de dragon joker j'ai rien compris (et pourtnt je reflechi)

Ce qu'a proposé dragon joker m'interesse et j'ai essayé de le faire, mais ca ne marche toujours pas

J'ai donc reduit mon code pour simplifier les choses

main.cpp

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
 
#include <iostream>
 
#include <string>
#include <fstream>
#include <vector>
#include "Sal_info.h"
 
 
 
using namespace std;
 
 
int main (){
 
	vector<string> log;
	log.push_back("contenu1");
	log.push_back("contenu2");
	log.push_back("contenu3");
 
 
 
	Sal_info sal2(nom_salle,nbreplace,taxesalle, log);
	cout<<sal2.get_logiciel()<<" logiciel  "<<endl;
 
return 0;
}

Sal_info.cpp

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
#include <iostream>
#include <string>
 
#include "Sal_info.h"
 
 
 
using namespace std;
 
Sal_info::Sal_info(const vector<string> & p_logiciel){
	logiciel = p_logiciel;
}
 
 
 
Sal_info::~Sal_info(void)
 
{
 
}
 
string Sal_info::get_logiciel() {
 
	return logiciel;
 
}

Sal_info.h

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
 
#include <string>
#include <fstream>
#include <vector>
 
 
class Sal_info
{
 
private :
 
	std::<vector<std::string logiciel>>;
 
public:
 
 
	Sal_info(const std::vector<std::string> &);
	~Sal_info(void);
	std::string get_logiciel();
 
};

Merci

[3DArchi]

Tu l'as un peu trop réduit car cela ne correspond à rien.

Dans ton constructeur initialise le membre vecteur comme les autres : par la liste d'initialisation :

Code :

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

Sal_info::Sal_info(string const & nom,double prix,int place,vector<string> const & logiciel_):Sal(nom,prix,place),logiciel (logiciel_){

[Steph_ng8]

@koala01 Tout ce que je voulais dire, c'est que laisser l'accesseur retourner un string ne plaira pas au compilateur.
Après, ce que tu dis est tout à fait juste, et je reconnais que je n'y avais pas du tout pensé…
Mais c'est peut-être aller un peu loin quand on débute.
(Oui, je sais… Il faut prendre des bons réflexes le plus tôt possible…)

En passant, game44, on ne peut pas afficher un conteneur directement avec l'opérateur « << ».

il est préférable de travailler sur des écarts plutôt que sur des collections complètes

Ah ouais ?
J'avais remarqué que la STL fonctionnait pas mal avec des paires d'itérateurs, mais je ne me suis jamais vraiment demandé pourquoi.
C'est juste pour pouvoir travailler indépendemment avec n'importe quel type de conteneur, ou il y a d'autres raisons ?

[Steph_ng8]

Code :

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

	std::<vector<std::string logiciel>>;

Euh… j'espère que tu t'es trompé en recopiant, parce que ça ne veut rien dire…

Code :

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

std::vector<std::string> logiciels;

[3DArchi]

J'avais remarqué que la STL fonctionnait pas mal avec des paires d'itérateurs, mais je ne me suis jamais vraiment demandé pourquoi.
C'est juste pour pouvoir travailler indépendemment avec n'importe quel type de conteneur, ou il y a d'autres raisons ?

Ce qui t'intéresse c'est de parcourir les éléments (itérateur) pas comment ils sont rangés (vecteur, list, map, etc.). Donc, oui cela permet de travailler indépendamment du conteneur parce que le concept attendu est celui d'itérateur (parcours d'éléments).

[game44]

OUi, c'est ce que je veux, c'est juste recuperer les elements (peu importe l'ordre car c'est une liste

Merci pour la remarque de steph, ca fait une erreur de compilation en moins, Mais il en reste encore beaucoup

[Steph_ng8]

Ok, je vois.

[3DArchi]

Merci pour la remarque de steph, ca fait une erreur de compilation en moins, Mais il en reste encore beaucoup

Euhh. J'avais réellement pensé qu'il s'agissait d'une erreur de copier/coller.
Normalement, avec un code comme celui que tu as proposé, les erreurs sont assez explicites et triviales à corriger pour peu que tu révises tes cours de C++ . Si vraiment tu bloques sur certaines, alors n'hésites pas à nous les proposer ici. Mais pour les plus simples, fais quand même l'effort de chercher