Pointeur de tableau Template

Version imprimable

Bonjour, j'explique mon soucis, j'ai un vecteur de template que je passe a une fonction. je le modifie dans cette fonction mais lorsque je l'affiche a la sortie de la fonction il n'a pas été modifié??

voici le bout de code:

l'appel:

Code:

1
2
addContrat(vectV,vectCl);
	vectCl->affiche();

le code:

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
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
void addContrat(Vecteur<Vehicule> *pVehicule,Vecteur<Client> *pClient)
{
	cout<<" Nouveaux contrats"<<endl;
	cout<<"-------------------"<<endl;
 
	char choix;
	cout<<"Est-ce un contrat d'assurance pour voiture [(o)ui - (n)on]? ";
	cin>>choix;
	system("clear");
	fpurge(stdin);
 
	FichierClient client("/Users/juliengarciagonzalez/client.dat");
	if(choix=='o')
	{
		cout<<" Nouveaux contrats Assurance Vehicule"<<endl;
		cout<<"--------------------------------------"<<endl;
 
		FichierContratVehicule contratVeh("/Users/juliengarciagonzalez/contratVeh.dat");
		ContratAssuranceVehicule CA;
		cin>>CA;
		pClient=addCL(pClient);
		pVehicule=addVehicule(pVehicule);
 
		Client Cl;
		Cl=pClient->getElement(countCl-1);
		CA.setIdClient(Cl.getIdentification());
		client.save(Cl);
 
		Vehicule veh;
		veh=pVehicule->getElement(countV-1);
		char buf[7];
		strcpy(buf,veh.getImmatriculation());
		CA.setCAVehicule(buf);
		contratVeh.save(CA);
 
		return;
	}
	cout<<" Nouveaux contrats Assurance"<<endl;
	cout<<"-----------------------------"<<endl;	
	FichierContrat contrat("/Users/juliengarciagonzalez/contrat.dat");
 
	ContratAssurance CA;
	cin>>CA;
	pClient=addCL(pClient);
 
	Client Cl;
	Cl=pClient->getElement(countCl-1);
	CA.setIdClient(Cl.getIdentification());
	client.save(Cl);	
	contrat.save(CA);	
	return;
}
 
Vecteur<Vehicule> * addVehicule(Vecteur<Vehicule> *p)
{
	Vecteur<Vehicule> *pNew = new Vecteur<Vehicule>(countV+1);
 
	cout<<" Ajout d'un vehicule"<<endl;
	cout<<"-------------------"<<endl;
	*pNew=*p;
	Vehicule mpTemp;
	cin>>mpTemp;
 
	pNew->setElement(countV, mpTemp);
	countV++;
	delete p;
	p=pNew;
	return p;
}
 
Vecteur<Client> * addCL(Vecteur<Client> *p)
{
	Vecteur<Client> *pNew = new Vecteur<Client>(countCl+1);
 
	cout<<" Ajout d'un client"<<endl;
	cout<<"-------------------"<<endl;
	*pNew=*p;
	Client mpTemp;
	cin>>mpTemp;
 
	pNew->setElement(countCl, mpTemp);
	countCl++;
	delete p;
	p=pNew;
	return p;
}

Merci d'avance

julien

15/12/2008, 12h13
JolyLoic
Citation:
Envoyé par garciajulien

Code:

1 2 pVehicule=addVehicule(pVehicule);
Là tu modifie ton pointeur pour qu'il pointe sur autre chose. Pas surprenant que tu ne vois rien de modifié à l'endroit où il pointait initialement.

Par contre, sans connaître ta classe Vecteur, difficile de mieux t'aider.

Pourquoi passer tes vecteurs par pointeurs et non par référence?
Pour ton problème, l'erreur est là:
Code:

1 2 pClient=addCL(pClient); pVehicule=addVehicule(pVehicule);
Lorsque tu passe un pointeur, c'est pour modifier la valeur pointée et non son adresse.
Pourquoi dans addVehicule et addCL réalloué un nouveau vecteur pour ajouter le dernier élément? Ta logique est un peu surprenant: tu rajoutes un élément au vecteur et ensuite tu l'utilises pour renseigner ces valeurs? En restant dans ton optique tu peux faire:
Code:

1 2 3 4 5 void addVehicule(Vecteur<Vehicule> &_vect_vehicule) { _vect_vehicule.push_back(Vehicule()); }
Mais, pourquoi ne pas faire (j'ai repris qu'un bout, à transposer partout):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 /* pClient=addCL(pClient); pVehicule=addVehicule(pVehicule); Client Cl; Cl=pClient->getElement(countCl-1); CA.setIdClient(Cl.getIdentification()); client.save(Cl); */ Client Cl; CA.setIdClient(Cl.getIdentification()); client.save(Cl); pClient->push_back(Cl); ....
[EDIT]: effectivement, j'ai supposé que ta classe Vecteur suivait std::vector. D'ailleurs, pourquoi ne pas utiliser directement celle-ci?

car je dois créer moi meme une classe vecteur pour mon cours.

(quest-ce que ce push.back?)

voici ma classe vecteur

header:

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
/*
 *  Vecteur.h
 *  ISL1Q
 *
 *  Created by Julien Garcia Gonzalez on 6/11/08.
 *  Copyright 2008 __MyCompanyName__. All rights reserved.
 *
 */
#ifndef _VECTEUR
#define _VECTEUR
#include <iostream>
#include "Personne.h"
#include "ContratAssurance.h"
using namespace std;
 
template<class T> class Vecteur
{
	private:
		T *data;
		int nbrElem;
		int *occupation;
 
	public:
		//Constructeur
		Vecteur(int n=10);
		Vecteur(const Vecteur<T>& vect);
 
		//Destructeur
		~Vecteur();
 
		//Fontions
		void affiche();
		int indiceValide(int empl);
		int estOccupe(int empl);
		T& retireElement(int empl);
		Vecteur& operator=(const Vecteur& vect);
 
		//Setter
		void setElement(int empl,T dNew);
 
		//Getter
		T& getElement(int empl);
		int getNbrElement(){return nbrElem;}
};
#endif

Cpp:

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
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
/*
 *  Vecteur.cpp
 *  ISL1Q
 *
 *  Created by Julien Garcia Gonzalez on 6/11/08.
 *  Copyright 2008 __MyCompanyName__. All rights reserved.
 *
 */
 
#include <iostream>
#include "Vecteur.h"
using namespace std;
 
template<class T>
Vecteur<T>::Vecteur(int n)
{
 
	data=new T[n];
	occupation=new int[n];
 
	nbrElem=n;
	for(int i=0;i<nbrElem;i++)
	*(occupation+i)=0;
}
 
template<class T>
Vecteur<T>::Vecteur(const Vecteur<T>& vect)
{
	int i;
	for(i=0;i<nbrElem;i++)
		{
			*(data+i)=*(vect.data+i);
			*(occupation+i)=*(vect.occupation+i);
		}
}
template<class T>
Vecteur<T>& Vecteur<T>::operator=(const Vecteur& vect)
{
	int i,nbrTemp;
 
	if(nbrElem>=vect.nbrElem)
		nbrTemp=vect.nbrElem;
	else
		nbrTemp=nbrElem;
 
	for(i=0;i<nbrTemp;i++)
		{
			*(data+i)=*(vect.data+i);
			*(occupation+i)=*(vect.occupation+i);
		}
	return *this;
}
 
template<class T>
Vecteur<T>::~Vecteur()
{
	delete []data;
	delete []occupation;
}
 
template<class T>
void Vecteur<T>::affiche()
{
	for(int i=0;i<nbrElem;i++)
	{
		if(*(occupation+i)==1)
			cout<<*(data+i)<<" ";
		else
			cout<<"-- ";
	}
}
 
template<class T>
int Vecteur<T>::indiceValide(int empl)
{
	if(empl<=nbrElem)
		return 1;
 
	return 0;
}	
 
template<class T>
int Vecteur<T>::estOccupe(int empl)
{
	return *(occupation+empl);
}
 
template<class T>
void Vecteur<T>::setElement(int empl,T dNew)
{
 
	*(occupation+empl)=1;
	*(data+empl)=dNew;
}
 
template<class T>
T& Vecteur<T>::getElement(int empl)
{
	return *(data+empl);
}	
 
template<class T>
T& Vecteur<T>::retireElement(int empl)
{
	*(occupation+empl)=0;
 
	return *(data+empl);
}
 
template class Vecteur<int>;
template class Vecteur<Client>;
template class Vecteur<MembrePersonnel>;
template class Vecteur<ContratAssurance>;
template class Vecteur<ContratAssuranceVehicule>;
template class Vecteur<Vehicule>;

Merci d'avance

15/12/2008, 15h00
3DArchi

Formellement, push_back rajoute un élément à la fin du conteneur. Avec ta classe, il peut être remplacer par ton setElement.
Quelques remarques sur ta classe:
-> Elle ne gère pas correctement l'echec d'allocation dans le constructeur (cf ici).
-> Ton constructeur par copie est incorrect (data et occupation ne sont pas contruits)
-> Ton opérateur = n'est pas celui qu'on attendrait.
-> il existe bool pour les valeurs true/false. Pourquoi utiliser un int pour occupation?
-> Réminiscence du C : partout tu fais *(ptr+indice) au lieu de ptr[indice]?
-> Tu ne teste jamais la validité des indices :aie:
-> Attention à la séparation entre .h et .cpp pour les templates
16/12/2008, 11h48
garciajulien

Et si j'utilise realloc? ca peut marcher?
16/12/2008, 12h09
garciajulien

Citation:

-> Elle ne gère pas correctement l'echec d'allocation dans le constructeur (cf ici).
-> Ton constructeur par copie est incorrect (data et occupation ne sont pas contruits)
-> Ton opérateur = n'est pas celui qu'on attendrait.

Je ne vois pas ce que vous dites là?

et aussi je viens de voir que ce qui serait cool pour ma classe vecteur c'est la fonction resize de vector mais moi je dois utiliser ma classe pas la classe vector donc est-ce qu'il existe autre chose pour resize?

merci
Ju
16/12/2008, 12h50
loufoque
Citation:

Je ne vois pas ce que vous dites là?

Un constructeur par copie et un opérateur d'affectation sont toujours définis par défaut en faisant des copies ou des affectations membre à membre.
Or, dans ton cas, ça risque de tout casser. Il faut soit que tu redéfinisses ces choses comme il faut, soit que tu les désactives.
Citation:
Code:

1 2 data=new T[n]; occupation=new int[n];
Ce code produira une fuite mémoire si new int[n] lève une exception, puisque data ne sera pas libéré.
16/12/2008, 13h04
3DArchi

Citation:

Envoyé par garciajulien

Et si j'utilise realloc? ca peut marcher?

Euh, non. Pourquoi faire????

Citation:

Envoyé par loufoque

Un constructeur par copie et un opérateur d'affectation sont toujours définis par défaut en faisant des copies ou des affectations membre à membre.
Or, dans ton cas, ça risque de tout casser. Il faut soit que tu redéfinisses ces choses comme il faut, soit que tu les désactives.

Relire le code: ils sont redéfinis mais incorrect !

Citation:

Envoyé par garciajulien

Je ne vois pas ce que vous dites là?
-> Elle ne gère pas correctement l'echec d'allocation dans le constructeur (cf ici).

Cf la réponse de loufoque. Si la seconde allocation échoue, le premier pointeur est perdu. C'est le mécanisme des exceptions.

Citation:

Envoyé par garciajulien

Je ne vois pas ce que vous dites là?
-> Ton constructeur par copie est incorrect (data et occupation ne sont pas contruits)

L'appel du constructeur par copie n'est pas précédé de l'appel du constructeur par défaut. Donc this->data et this->occupation sont invalides, quant tu va essayer de copier des données dedans :aie:

Citation:

Envoyé par garciajulien

Je ne vois pas ce que vous dites là?
-> Ton opérateur = n'est pas celui qu'on attendrait.

Pour l'opérateur =, on s'attend à ce que la partie de gauche soit 'recopié' sur la partie de droite. Or, tu ne recopie qu'une partie des valeurs. C'est la sémantique (le sens) de l'opérateur qui est surprenant (pour ne pas dire erroné!).

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
32
33
34
35
36
37
38
39
40
41
template<class T>
Vecteur<T>::Vecteur(const Vecteur<T>& vect)
{
	int i;
 
	if(!this->data)
		data=new T[vect.nbrElem];
	if(!this->occupation&&this->data)
		occupation=new int[vect.nbrElem];
 
	for(i=0;i<nbrElem;i++)
		{
			*(data+i)=*(vect.data+i);
			*(occupation+i)=*(vect.occupation+i);
		}
}
template<class T>
Vecteur<T>& Vecteur<T>::operator=(const Vecteur& vect)
{
	int i,nbrTemp;
 
	if(nbrElem>=vect.nbrElem)
		nbrTemp=vect.nbrElem;
	else
	{
		nbrTemp=nbrElem;
		this->nbrElem=nbrElem;
	}
 
	if(!this->data)
		data=new T[this->nbrElem];
	if(!this->occupation&&this->data)
		occupation=new int[this->nbrElem];
 
	for(i=0;i<nbrTemp;i++)
		{
			*(data+i)=*(vect.data+i);
			*(occupation+i)=*(vect.occupation+i);
		}
	return *this;
}

Voilà mes constructeur de copie et mon opérateur d'affection
est-ce correct?

de plus, il me faudrai pouvoir faire un push_back la dessus hors je ne peux pas utiliser la classe std::vector?

comment puis-je faire pour augmenter la taille de mon tableau passé en paramètre?

Merci

ju

16/12/2008, 13h58
garciajulien

Pour modifier l'adresse pointée, j'ai trouvé dans ma fonction j'envoie un pointeur de référence:
objet *& monvect

et ca marche nickel!!!
16/12/2008, 13h59
Médinoc

Ceci n'est pas un pointeur de référence, mais une référence vers un pointeur.

Citation:

Envoyé par garciajulien

Pour modifier l'adresse pointée, j'ai trouvé dans ma fonction j'envoie un pointeur de référence:
objet *& monvect

et ca marche nickel!!!

Tu as surtout de la chance que ça tombe en marche !
Citation:
Envoyé par garciajulien
Code:

1 2 3 4 5 6 7 8 9 template<class T> Vecteur<T>::Vecteur(const Vecteur<T>& vect) { int i; if(!this->data) data=new T[vect.nbrElem]; if(!this->occupation&&this->data) occupation=new int[vect.nbrElem];
100% de chance qu'en release data et occupation ne valent pas NULL et que ça plante ! Tu devrais tout simplement les allouer! Mais comme pour ton autre constructeur, tu devrais gérer l'échec du new (exceptions levées).

Citation:

Envoyé par garciajulien

template<class T>
Vecteur<T>& Vecteur<T>::operator=(const Vecteur& vect)
{
int i,nbrTemp;

if(nbrElem>=vect.nbrElem)
nbrTemp=vect.nbrElem;
else
{
nbrTemp=nbrElem;
this->nbrElem=nbrElem;
}

if(!this->data)
data=new T[this->nbrElem];
if(!this->occupation&&this->data)
occupation=new int[this->nbrElem];

for(i=0;i<nbrTemp;i++)
{
*(data+i)=*(vect.data+i);
*(occupation+i)=*(vect.occupation+i);
}
return *this;
}[/CODE]

Voilà mes constructeur de copie et mon opérateur d'affection
est-ce correct?

Non. Tu ne fais pas une vrai copie puisque this peut avoir moins d'element que l'objet passer en paramètre !
Une solution souvent mise en place ressemble à ceci
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 template<class T> Vecteur<T>& Vecteur<T>::operator=(const Vecteur& vect) { Vecteur<T> temp(vect); Swap(this,temp); } avec Swap(Vecteur<T>&rhs,Vecteur<T>lhs) { std::swap(rhs.data,lhs.data); std::swap(rhs.occupation,lhs.occupation); std::swap(rhs.nbrElem,lhs.nbrElem); }
Citation:

Envoyé par garciajulien

de plus, il me faudrai pouvoir faire un push_back la dessus hors je ne peux pas utiliser la classe std::vector?

En fait, ta méthode setElement devrait te suffire.

Citation:

Envoyé par garciajulien

comment puis-je faire pour augmenter la taille de mon tableau passé en paramètre?

A partir du moment où tu as décidé d'utiliser un tableau pour ton vecteur, il ne te reste plus qu'a en réalloué un de taille un peu plus grande à chaque fois.

17/12/2008, 16h15
Goten

tiens j'avais posté ça sur un topic similaire y'a pas longtemps :

http://www.research.att.com/~bs/except.pdf

c'est un papier de BS qui traite de l'exception safety au travers d'un exemple concret, une classe vector basique mais STL-like.

Tu verras notamment qu'il préfère utilisait les uninitialized_fill pour éviter les new et surtout qu'au lieu d'utiliser des exceptions il dérive d'une classe de base qui gére uniquement l'allocation dynamique et qui implémente le RAII.