[C++] surcharge operateur d'assignation Résolu/Fermé

Question

Bonjour,

dans ma classe j'ai un tableau dynamique a 2 dimensions de "bool" comme attribut et il semble que j'ai fait une erreur dans le code de cette surcharge car celle ci cesse de fonctionner au bout de quelque appel et et le tableau contient des données incorrectes,
à la l'affichage au lieux d'avoir des 1 et des 0 j'obtiens de plus grand nombre

dans ma classe j'ai les attribut suivant:
	private:
		int largeur;
		int longueur;
		int stock;
		bool **tab;

et voici le code qui pose probleme
en gros je copie les attributs  et je sauvegarde l'adresse du tableau que j'avais au debut pour en créer un autre a la place que j'initialise avec les donnée de l'autre tableau et termine detruire le tableau de depart

MaClasse& MaClasse::operator=(const MaClasse& pp){
	bool **tmp= tab;
	largeur = pp.largeur;
	longueur = pp.longueur;
	stock = pp.stock;
	tab = new bool*[largeur];
	for(int i=0;i<largeur;i++){
		tab[i] = new bool[longueur];
		for(int j=0;j<longueur;j++){
			//tab[i][j] = new bool;
			tab[i][j] = pp.tab[i][j];
		}
	}
	for(int i=0;i< largeur;i++)
			delete[] tmp[i];
	delete[] tmp;
	return *this;
}


si vous pouvez m'aider je vous en serait enormement reconnaissant

azureus33 · Answer

Déjà ta surcharge de l'opérateur = est fait partie de ta class il serait plus logique et juste de mettre la fonction void 

Et comme tu ne te sert pas de tmp autant supprimé directement la matrice c'est inutile d'attendre la fin de la fonction surtout que la largeur a changé et donc tu n'est plus certains de supprimé toutes les colonnes.
Résultat une belle fuite mémoire.

flY · Answer

merci pour ta reponse, ca m'a permis de mettre un eu d'ordre dans ma classe 

mais j'ai un autre probleme qui se pose qui apparait justement a l'utilisation de cette surcharge:

en faite j'utilise un objet de cette classe comme attribut d'une autre classe; ca ne pose aucun probleme jusqu'a ce que j'utilise un assesseur de cette attribut 

ClasseAttribut  MaClasse::getAttribut(){
                return monAttribut;
}

si j'ai le malheur d'executer l'instruction suivant le tableau dynamique de bool crée dans l'attribut devient totalement incoherent 


MaClasse c;
ClasseAttribut a = c.getAttribut();

flY · Answer

un grand merci a toi tout fonctionne comme sur des roulettes

[C++] surcharge operateur d'assignation

3 réponses

Discussions similaires