erreur d'affichage dans template

**alexorcet** · 03/12/2006, 23h24

Bonsoir a tous
j'ai modifié mon post car je viens d'avoir un nouveau soucis.

j'ai le probleme suivant quand je veux afficher les objet PieceMuseeArt que j'encode et je parcous à l'aide de mon itérateur, à l'affichage il m'affiche tous les parametres par defaut de la PieceMuseeArt sans qu'il prenne en considération ce que je lui ai encodé.

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
 
int main()
{
	int i;
	cout<<"combien delement voulez vous"<<endl;
	cin>>i;
	Personne per;
	PieceMuseeArt p;
	p.Encode();
	TVecteur<PieceMuseeArt>vpm(i);
	cout<<"fini";
	vpm.ajout(&p);
	vpm.affContenu();
 
 
}

la methode Encode

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
 
PieceMusee& PieceMuseeArt::Encode()
{
	PieceMuseeArt pmar;
 
 
     	cout<<"--------------Encodage de la piece-----------"<<endl;
 
	Identifiant i;
	char l;int a;
	cout<<"Inserer la lettre de l'id alphanumerique de la piece: ";
	cin>>l;
	i.setLettre(l);
	this->id.setLettre(l);
	cout<<"Quelle est son annee d acquisition?";
	cin>>a;
	i.setAnnee_dacq(a);
	this->id.setAnnee_dacq(a);
 
	pmar.setIdentifiant(i);
 
	cout<<"l'id avec la surcharge de <<"<<i<<endl;
	cout<<"l'id avec surcharge de = "<<pmar.id<<endl;
	cout<<"l'id avec toString()est: "<<i.toString(l,a)<<endl<<endl;
	cout<<"quelle est la categorie de la piece:  ";
	cin>>category;
	cout<<"decrivez la piece: ";
	cin>>description;
 
	cout<<"Donnez une estimation en euro de la piece:";
	cin>>valeur;
	cout<<"quelle est sa date d'entree au musee: ";
	cin>>dateIn;
	setDateIn(dateIn);
      	cout<<"la piece est-elle exposee au musee O/N: "<<endl;
	cin>>exposed;
	cout<<"encodage de l'auteur"<<endl;
 
	pmar.auteur->Encoder();
 
	this->auteur->nom = pmar.auteur->nom;
	this->auteur->prenom = pmar.auteur->prenom;
	this->auteur->dateDeNaissance = pmar.auteur->dateDeNaissance;
 
 
	return *this;	
}

j'ajoute les element dans le vecteur template avec:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
template <class type>
int TVecteur<type>::ajout( type *n)
{
	type *pNouv = new type;
        *pNouv = *n;// l'operateur = est surchargé correctement 
	data[nbrElem] = *pNouv;	
	nbrElem++;
 
	return 1;	
}

et j'affiche avec la methode

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
template <class type>
void TVecteur<type> :: affContenu(void)
{
	iterTVecteur<type> it(*this);
	while(!it.end())
	{
		type t((type &)it);
		cout<<t<<endl;//surchargé dans la classe type instanciée
		it++;
 
	}

pourquoi à votre avis il m'affiche les parametre par defaut?

**JolyLoic** · 03/12/2006, 23h32

Ton code n'est pas const-correct. *this retourne un objet constant, et le constructeur de ton itérateur indique qu'il va modifier son paramètre.

PS : Inclure le message d'erreur aurait aidé à voir le problème...

**alexorcet** · 04/12/2006, 02h16

desolé j'ai oublié de poster le message d'erreur

Code :

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

1
2
3
main.cxx:28:   instantiated from here
TVecteur.h:39: erreur: ‘it’ has incomplete type
TVecteur.h:39: erreur: storage size of ‘it’ isn't known

Sorry

**JolyLoic** · 04/12/2006, 08h39

Ok, c'est probablement un problème dans des #include et tes forward declarations de classes (en plus du problème de const déjà cité). Il va falloir nous en montrer plus sur ce sujet (les différents fichiers, ce qu'il y a au début...) pour qu'on soit spécifiques.

**alexorcet** · 04/12/2006, 16h36

Merci JolyLoic pour ton aide
voici plus ou moins l'integralité de mon code

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
 
#ifndef TVECTEUR_H
#define TVECTEUR_H
 
//#include "PieceMuseeArt.h"
 
#include <stream.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <ctype.h>
#include <values.h>
 
template <class type>class iterTVecteur;
 
template <class type> class TVecteur
{
	private:
		type *data;
		int nbrElem;
	public:
		TVecteur(int n=10 ){data = new type[n];
	nbrElem = n;}
 
    	        void affContenu (void);
		/*{
			cout << "Nombre d'elements = " << nbrElem << endl<<endl;
 
		}*/
     		int ajout (type *n);
     		int retrait (void);
 
		~TVecteur(void){delete data;}
};
 
 
template <class type>
void TVecteur<type> :: affContenu(void)
{
	iterTVecteur<type> it(*this);
	while(!it.end())
	{
		//PieceMuseeArt *p = new PieceMuseeArt;
		type t((type)it);
		cout<<t<<endl;
		it++;
	}
 
 
 
}

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
 
#ifndef ITERTVECTEUR_H
#define ITERTVECTEUR_H
 
#include "TVecteur.h"
#include <stream.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include <ctype.h>
#include <values.h>
 
template <class type>
class iterTVecteur 
{
        private :
		TVecteur<type>& p;
		type *pCur;
 
	public :
//le parametre est egalement un objet template
 
                iterTVecteur (TVecteur<type>& s) : p(s)
                        { pCur = s.data; }
 
		int end() 
                {
                        return pCur==0;
                }
 
		void reset () { pCur = p.data; }
 
                int operator++ ()
                {
                        if (!end())
                        {
                                pCur++;
                                return 1;
                        }
                        else return 0;
                }
                int operator++ (int) { return operator++(); }
 
                operator type ()const
		{
 
		  return *pCur;
 
		  }
                void operator= (type *v)
		{ pCur = v;}
 
 
 
};
 
 
#endif

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
 
#include "TVecteur.h"
 
 
 
 
int main()
{
	int i;
	cout<<"combien delement voulez vous"<<endl;
	cin>>i;
	PieceMuseeArt p;
	TVecteur<PieceMuseeArt>vpm(i);
	cout<<"fini";
	vpm.ajout(&p);
	vpm.affContenu();
 
 
}

à l'aide...

**Jean-Marc.Bourguet** · 04/12/2006, 17h11

J'ai pas le temps de regarder en detail, mais qu'est-ce que vient faire #include "PieceMuseeArt.h" (qui est visiblement quelque chose de specifique a l'application) dans un entete d'une classe "technique" comme un vecteur?

**alexorcet** · 04/12/2006, 17h19

l'include de PieceMuseeArt est present car à un certain moment je voulais instancier cette classe (plus loin).
je vais essayer de l'enlever et te dire ce que ça donne.
a tout de suite.

**alexorcet** · 04/12/2006, 17h45

rien de nouveau jean marc toujours le meme probleme

**r0d** · 04/12/2006, 18h27

Je ne comprends pas, j'ai copié ton code et je l'ai compilé (sous visual 2005) et ça fonctionne. Je me mange juste une seg fault sur la dernière itération de la boucle for de affContenu, je n'ai pas regardé puisque ce n'est pas le sujet.

A mon avis, tu dois avoir oublié un ';' quelque part ou une bêtise dans le genre.

En fait, l'erreur "has incomplete type", si j'ai bien compris, viens du fait que tu fais appel à une fonctionnalité de la classe qui n'est pas implémentée. Mais je ne comprends pas pourquoi il ne donne pas plus de détails dans le message d'erreur.

**Jan Rendek** · 04/12/2006, 18h45

Dans ton main, le fichier iterVecteur.h n'est jamais inclus. Tu inclues uniquement Vecteur.h
et Vecteur.h n'inclue pas iterVecteur.h

A priori ton compilo ne valide pas les définitions de templates, mais uniquement leur instanciation.
A l'instanciation du template dans ton main.c le type TVecteur est connu, mais celui-ci fait appel au type iterVect qui lui est inconnu.
(en fait pas tout à fait inconnu puisqu'il y a une déclaration par avance dans TVecteur.h, d'où le type incomplet).

**alexorcet** · 04/12/2006, 21h03

Jan Rendek Bien vu j'ai bien oublié l'include

ça a resoulu partiellement mon erreur car je vois apparaitre de nouvelle erreurs

Code :

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

1
2
3
4
5
6
TVecteur.h: In member function â€˜void TVecteur<type>::affContenu() [with type = PieceMuseeArt]â€™:
main.cxx:30:   instantiated from here
TVecteur.h:44: erreur: no matching function for call to â€˜PieceMuseeArt::PieceMuseeArt(iterTVecteur<PieceMuseeArt>&)â€™
PieceMuseeArt.h:34: note: candidats sont: PieceMuseeArt::PieceMuseeArt(PieceMuseeArt&)
PieceMuseeArt.h:25: note:                 PieceMuseeArt::PieceMuseeArt(Identifiant, char*, char*, int, char*, char, char, char*, Personne*, char*)
PieceMuseeArt.h:17: note:                 PieceMuseeArt::PieceMuseeArt()

ça me rend fou ces template ,pk il arrive pas a instancier mon itérateur en tant que pieceMusee?

**alexorcet** · 04/12/2006, 21h19

c bon g trouvé l'erreur

Merci pour l'aide

**JolyLoic** · 04/12/2006, 21h30

J'ai l'impression que tu n'as pas corrigé les problèmes de const que j'ai signalés dans mon premier post...

En outre, quand tu écris :

Code :

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

type t((type)it);

Tu essaye de caster ton itérateur en objet, ce qui à la base semble pas très propre. Mais passons.

Ce cast va te créer un objet temporaire, que tu passes ensuite en paramètre à ton constructeur par copie. Ceci demande l'existance d'un constructeur de copie prenant en paramètre une référence constante vers un objet. Or, tu as visiblement défini le constructeur par copie suivant :

Code :

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

PieceMuseeArt::PieceMuseeArt(PieceMuseeArt&)

Qui prend son paramètre par référence non constante, et empêche la génération automatique du constructeur par copie classique.

Il va vraiment falloir que tu fasses plus attention à tes const...

Quant au message d'erreur que tu as mis, est-tu certain qu'il correspond à la version du code montré ? Tel qu'est le code, j'aurais plutôt vu :

Code :

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

TVecteur.h:44: erreur: no matching function for call to â€˜PieceMuseeArt::PieceMuseeArt(PieceMuseeArt const &)â€™

Ou alors le message que tu as indiqué pour le code :

Code :

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

type t(it);

**alexorcet** · 04/12/2006, 22h27

Merci JolyLoic c'est exactement ça

je te remercie pour ton aide.

à bientot

erreur d'affichage dans template

Langage C++

Discussions similaires

Partager

Partager