c++ probleme d'affichage d'une variable membre d'un objet

Version imprimable

Bonsoir à tous
mon programme consiste à creer des pieces(classe Piece) chaque piece a un identifiant(classe Identifiant).

class PieceMusee
{
private:

Identifiant id;
char category[SIZE];
char description[SIZE];

etc........

j'ai une methode Encode() qui retourne une Piece
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 Piece Piece::Encode() { PieceMusee p; cout<<"--------------Encodage de la piece-----------"<<endl; Identifiant i; char l;int a; cout<<"Inserer la lettre de l'id alphanumerique de la piece"<<endl; cin>>l; i.setLettre(l); cout<<"Quelle est son annee d acquisition?"<<endl; cin>>a; i.setAnnee_dacq(a); p.setIdentifiant(i); etc..... return p;
Le probleme se pose qd je veux afficher l'id de la piece avec ailleur de la methode encode().

Code:

cout<<"---l'id de la Piece est"<<p.id<<endl;

il me donne comme resultat l'id que j'ai mis dans mon constructeur par defaut.

d'ou pourrai venir le probleme???

Ma 2éme question comment je peux faire un cin>>sur une chaine de caracteres qui contient un espace car avec cin>>machaine il me coupe tout apres l'espace.

24/10/2006, 23h36
Luc Hermitte

1- Lis les valeurs nécessaire à la construction de ton objet. Et ensuite construit directement ton objet.

2- laisse tomber les les tableaux statiques de caractères. C++ a un type standard pour les chaînes de caractères -> std::string. Utilise std::getline pour lire des lignes en ignorant les espaces.

Salut,

Si une méthode appartient à une classe donnée (comme c'est le cas de ta méthode Encode()) tu dispose d'un pointeur sur l'objet réel du type de ta classe...

Ce pointeur est this et peut etre sous-entendu quand tu veux simplement modifier la valeur d'un membre de l'objet en cours.

Donc, tu n'a nullement besoin de créer un nouvel objet dans la méthode, et qui plus est, surement pas un objet statique ainsi que tu le fait...

La définition de ta classe aurait fortement avantage à se transformer en quelque chose du genre de

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
class PieceMusee
{
    private:
        Identifiant id;
        std::string category;
        std::string description;
    public:
        //constructeur par défaut
        PieceMusee(){}
        //constructeur avec les informations necessaires
        PieceMusee(Identifiant i,const std::string& cat, 
                   const std::string& desc): id(i), category(cat), 
                   description(desc){}
        //le destructeur
        ~PieceMusee(){}
        //la methode d'encodage (renvoie une référence sur l'objet en cours)
        PieceMusee& Encode();
    //le reste des déclarations

(j'ai gardé sensiblement le meme prototype pour la méthode, mais, comme elle s'applique à un objet, il pourrait ne rien renvoyer et modifier directement les membres qu'il contient ;)
La méhtode Encode serait codée sous la forme de (j'ai pris ton code, à la base ;)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
PieceMusee& Encode()
{
    char l;
    cout<<"Inserer la lettre de l'id alphanumerique de la piece"<<endl;
    cin>>l;
    //on a acces au membre id... donc si les identifiant ont une méthode
    //setLetre, on peut faire
    id.setLettre(l);
    //cela revient exactement au meme si on fait
    this->id.setLettr(l);
    int annee;
    cout<<"Quelle est son annee d acquisition?"<<endl;
    cin>>a;
    //meme remarque que plus haut ;)
    id.setAnnee_dacq(a);
    //on renvoie ce qui est pointé par this
//...
    return *this;
}

Une autre solution serait, dans la fonction qui a besoin d'une variable PieceMusee, de travailler sous la forme de

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
 
void MaFonct_qui_cree_un_objet_PieceMusee()
{
    //ici, on a besoin d'un identifiant
    char l;
    cout<<"Inserer la lettre de l'id alphanumerique de la piece"<<endl;
    cin>>l;
    Identifiant i;
    i.setLettre(l);
    int annee;
    cout<<"Quelle est son annee d acquisition?"<<endl;
    cin>>a;
    i.setAnnee_dacq(a);
    //Tant qu'a faire, on demande la categorie et la description
    std::cout<<"Veuillez introduire la catégorie de la piece ";
    std::string cat;
    std::cin>>cat;
    std::cout<<"Veuillez introduire la description de la piece ";
    //typiquement, une description fait plusieurs mots, on est mieux 
    //d'utiliser getline() ;)
    std::string desc;
    std::getline(std::cin,desc);
    //on appelle maintenant le constructeur avec toutes les données 
    //qu'on a récupérées
    PieceMusee lapiece(i,cat,desc);
    //ou, s'il faut une allocation dynamique (n'oublie pas qu'il faudra 
    //le detruire avec delete ;) ):
    PieceMusee *PLaPiece=new PieceMusee(i,cat, desc);
//on continue à travailler sur lapiece ;)
}

Ceci dit, d'apres le code que tu fournis, l'identifiant n'est constitué que d'une lettre et d'un entier...

Deux choses sont à prendre en compte à son sujet:

-une lettre, ca peut etre un peu court pour servir d'identifiant (à moins que l'identifiant final soit "l'union" de la lettre et de la date (A20061230 B20061230 A20070116...), car, tu ne dispose que de relativement peu de caractère imprimables certains, et que je serais surpris que le musée n'aie "que", meme en criant fort, 255 pieces ;)

-De prime abord, ni la lettre, ni le nombre qui sert de date n'est susceptible d'etre modifié apres sa création... Tout ce dont tu risque d'avoir besoin, au niveau de l'identifiant, c'est de récupérer la lettre (ou la chaine) et le nombre (voire, une chaine unissant la lettre et le nombre)...

Il parraitrait donc intéressant d'envisager de réétudier un peu la question de ta conception de l'identifiant...

La classe qui me *semble* (sous réserve que j'aie bien compris le role et le contenu de ton identifiant) le plus adéquat serait quelque chose du genre de
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 class Identifiant { private: //les membres je remplace la lettre (unique) par une chaine, pour //le cas où de nombreuses pieces arriveraient le meme jour ;) std::string lettres; int date_aquis; //on interdit le constructeur par defaut (sans valeur) Identifiant(){} public: //Le constructeur qu'il est obligatoire d'utiliser Identifiant(const std::string& ch, int aquis): lettres(ch), date_aquis(aquis){} //récupération de la chaine seule std::string GetLettre(){return lettres;} //récupération du nombre seul int GetDate(return date_aquis;} //récupération d'une chaine qui contient lettres et date_aquis std::string GetAll() { std::string retour; retour=lettres; //il faut passer par un ostringstream pour convertir le nombre // en chaine std::ostringstream oss; oss<<lettres; retour+=oss.str(); return retour; } }
Evidemment, tu es libre de choisir parmis ces trois méthodes lesquelles te sont utiles, ainsi que leur nom (et, d'ailleurs, le nom des membres aussi ;))...

Elles ne sont là que pour te servir d'exemple ;)

Certains operateurs de comparaisons pourraient d'ailleurs aussi s'avérer utiles, pour le tri et/ou pour s'assurer de l'unicité de l'identifiant ;)
La création d'un identifiant se fera d'office, apres avoir récupéré les informations nécessaires, en utilisant le constructeur fournit sous la forme de
Code:

1 2 3 4 5 6 7 8 9 10 11 std::string l; cout<<"Inserer la chaine alphanumerique servant d'identifiant à la piece"<<endl; cin>>l; int n; cout<<"Quelle est son annee d acquisition?"<<endl; cin>>a; //on a tout, on peut créer l'identifiant Identifiant id(l,a); //pour l'utiliser, par exemple plus tard, en appelant le constructeur pour //la piece recue ;)

Bonsoir ,

koala01, je me demande ce que ce code signifie

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
std::string GetAll()
        {
                std::string retour;
                retour=lettres;
                //il faut passer par un ostringstream pour convertir le nombre
                // en chaine
                std::ostringstream oss;
                oss<<lettres;
                retour+=oss.str();
                return retour;
        }

Il va y avoir deux fois lettres dans retour qui ne sert à rien, de plus return oss.str() suffit, et le nombre n' est pas injecté dans le flux.

Cordialement.

Citation:

Envoyé par dj.motte

Bonsoir ,

koala01, je me demande ce que ce code signifie

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
std::string GetAll()
        {
                std::string retour;
                retour=lettres;
                //il faut passer par un ostringstream pour convertir le nombre
                // en chaine
                std::ostringstream oss;
                oss<<lettres;
                retour+=oss.str();
                return retour;
        }

Il va y avoir deux fois lettres dans retour qui ne sert à rien, de plus return oss.str() suffit, et le nombre n' est pas injecté dans le flux.

Cordialement.

De fait... j'étais distrait....
L'ostringstream, il était destiné à recevoir date_aquis :oops:
Le code modifié:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
std::string GetAll()
        {
                std::string retour;
                retour=lettres;
                //il faut passer par un ostringstream pour convertir le nombre
                // en chaine
                std::ostringstream oss;
                oss<<date_aquis;
                retour+=oss.str();
                return retour;
        }

voire

Code:

1
2
3
4
5
6
7
8
9
10
11
 
std::string GetAll()
        {
                std::string retour;
                //il faut passer par un ostringstream pour convertir le nombre
                // en chaine
                std::ostringstream oss;
                oss<<lettres<<date_aquis;
                retour+=oss.str();
                return retour;
        }

27/10/2006, 00h15
dj.motte
Bonsoir,

koala01, il me semble que votre code pourait être simplifié .
Code:

1 2 3 4 5 6 std::string GetAll() { std::ostringstream oss; oss<< lettres<< date_aquis; return oss.str(); }
On peut faire l' inverse en mettant la date avant 'lettres'.
Mais dans une situation ou une paire ( T, V ) ou ( V , T ) d' autres choix auraient pu être possibles, mais je ne tiens pas à chambouler le choix du demandeur.

Cordialement.
28/10/2006, 19h07
alexorcet

Merci bcp les garrs.
j'ai utilisé vos solutions et tout fonctionne super bien.