prob avec constructeur par copie et variable static

Version imprimable

Bonjour!
voila mon prob j'ai crée une classe point avec une donnée static compteur qui s'incrémente ds le constructeur et se décrémente dans le destructeur.
lorque je déclare par exemple point a(2,5) et b(3,6) le compteur affiche 2 càd j'ai deux objets de type point mais si j'ecrit a(2,5) et pointb=a le compteur affiche 1 càd un suel objet je veux comprendre est dans le 2eme cas il y a qu'un seul objet ou deux a et b et merci
voila mon code source
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 #include <iostream.h> class point{ static int compteur; int x ; int y ; public : point (int &, int&) ; void deplace (int, int) ; void affiche () ; ~point () ; } ; int point::compteur=0; point::point (int &abs, int &ord) {x = abs ; y = ord ;compteur++; cout<<"il existe "<<compteur <<" points "<<this<<endl ; } void point::deplace (int dx, int dy) { x = x + dx ; y = y + dy ; } void point::affiche () { cout << "Je suis en " << x << " " << y << "\n" ; } point::~point () {compteur--; cout<<"il existe "<<compteur <<" points"<<endl ; }
Code:

1 2 3 4 5 6 7 /* -------- Utilisation de la classe point -------- */ main() { point a(10,55); //b(4,5) point b=a; }

28/10/2007, 22h31
yan

Bonjour,
Je pense que du au fait que tu n'as pas définit d'operator=
Dans ton code, le compilot va le créer à ta place et faire des = sur tout les membre, dont compteur.
Aprés ca va surement dépendre du compilatur
28/10/2007, 22h58
Bakura
Le compilo te créé automatiquement un opérateur d'affectation et copie juste membre à membre. Quand tu créés deux objets, le constructeur est appelé deux fois, ton compteur est donc incrémenté deux fois. Lorsque tu en créé un, et que tu fais : point b=a; Le compilateur copie membre à membre, mais n'incrémente pas ton compteur. Ce que tu dois faire, comme l'a dit Montgaulois, c'est redéfinir l'opérateur = :
Code:

1 2 3 4 5 6 Point & Point::operator= (const Point & monPoint) { x = monPoint.x; y = monPoint.y; ++compteur; }
28/10/2007, 22h58
must0307

re

Merci Mongaulois
je pense operator = est obligatoire lorsque il y a des pointeurs dans la classe
en plus une donnée static est une donnée de classe càd il y a un compteur partagé par tous les objets
28/10/2007, 23h26
must0307

re

merci Bakura & Mongaulois
j'ai compris
28/10/2007, 23h32
koala01
Salut,

Il y a quatre méthodes qui sont créées d'office par le compilateur - quel qu'il soit - s'il n'existe aucune méthodes créée par l'utilisateur pour en tenir lieu.

Ce sont:
- Le constructeur par défaut (ne prenant aucun paramètre obligatoire) qui ne sera créé que si aucun constructeur "classique" n'est créé par l'utilisateur
- le constructeur par copie, dont il faut garder en tête que, par défaut, il s'agit d'une copie membre à membre (avec les problème connus en ce qui concerne les pointeurs)
- le destructeur
- l'opérateur d'affectation
Ces quatre méthodes, communément connue sous le nom des big four doivent être implémentées - fusse par le compilateur - afin d'assurer la mise en oeuvre de l'idiome RAII(Ressouces Aquisition Is Initialisation)
29/10/2007, 09h37
yan
Citation:
Envoyé par Bakura

Le compilo te créé automatiquement un opérateur d'affectation et copie juste membre à membre. Quand tu créés deux objets, le constructeur est appelé deux fois, ton compteur est donc incrémenté deux fois. Lorsque tu en créé un, et que tu fais : point b=a; Le compilateur copie membre à membre, mais n'incrémente pas ton compteur. Ce que tu dois faire, comme l'a dit Montgaulois, c'est redéfinir l'opérateur = :

Code:

1 2 3 4 5 6 Point & Point::operator= (const Point & monPoint) { x = monPoint.x; y = monPoint.y; ++compteur; }
En y réfléchissant , je croit que l'on as dit une bêtise. Ce n'est pas que l'opérateur = qu'il faut redéfinir , mais la création par recopie aussi. L'operateur = ne devrait pas incrémenté compteur.
sinon
Code:

1 2 3 4 5 6 point a(-1,5); point b =a;//indéfinie même avec un operateur = ??? je sais pas b=a; //compteur++ b=a; //compteur++
et

Code:

point b = a <=> point b(a);//il me semble
29/10/2007, 11h58
must0307
Salut

Code:

point b = a <=> point(const point & a)
Code:

1 2 3 4 point(const point & a) { *this=a; }
voila comment j'ai compris la chose d'abord le compilateur fait un copie du point a sous forme de constant càd avec les valeurs de a.x a.y et compteur=1 puis reserve l'espace pour b (on sait que le constructeur et appelé apres la creation de l'objet) puis il transmet les valeurs qui sont déja copiées aux attributs de b cèd ceux du point a .
il me semble mais ??
29/10/2007, 12h15
screetch

tout ce qui est CONSTRUCTEUR doit incrementer le compteur
tout ce qui est DESTRUCTEUR doit decrementer
tout ce qui est OPERATEUR DE RECOPIE ne doit pas y toucher, ce n'est pas un nouvel objet.

Dans la liste que t'a donné koala, il a parlé :

-d'un constructeur par defaut, vide : celui ci est generé par le compilateur si tu n'en fournis aucun autre. Si tu fournis un constructeur alors il ne sera pas generé, mais tu peux en creer un. Si tu en crées un, c'est un constructeur donc incremente le compteur.
- d'un constructeur par recopie : celui ci est generé par le compilateur si tu n'en donnes pas. Celui generé par le compilateur ne fera pas l'incrementation, il faut que tu lui en fournisses un qui fait l'incrementation.
- d'un destructeur : si tu n'en fournis pas, il sera generé automatiquement, mais tu en as un dans ta classe qui decremente le compteur
- d'un operateur de recopie : celui ci n'est pas un constructeur. Soit tu en fournis un soit le compilateur le genere pour toi.
29/10/2007, 12h38
Bakura
Citation:
Envoyé par Mongaulois

En y réfléchissant , je croit que l'on as dit une bêtise. Ce n'est pas que l'opérateur = qu'il faut redéfinir , mais la création par recopie aussi. L'operateur = ne devrait pas incrémenté compteur.
sinon

Code:

1 2 3 4 5 6 point a(-1,5); point b =a;//indéfinie même avec un operateur = ??? je sais pas b=a; //compteur++ b=a; //compteur++

et

Code:

point b = a <=> point b(a);//il me semble
En effet, j'ai dis une grosse connerie :aie:

je pense que c'est Resolu !!

merci a tous

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
 
#include <iostream.h>
class point{
        int x ;
        int y ;
public :
        static int compteur;
        point (int , int) ;
        point(const point &p);	
        ~point () ;
 
        } ;
int point::compteur=0;
point::point (int abs, int ord) //: x(abs), y(ord),nom(p)
{
        x = abs ; y = ord ;
        compteur++;
        cout<<"il existe "<<compteur <<" points "<<endl ;
}
 
point::point(const point &p)
{
        this->x = p.x ; this->y = p.y ;
        compteur++;
        cout<<"il existe "<<compteur <<" points "<<endl ;
}
point::~point ()
{
        compteur--;        
        cout<<"il existe "<<compteur <<" points"<<endl ;
}
main()
{
        point a(10,55);  //incrémente le compteur
        point b=a;//incrémente le compteur
        point c(a);    //incrémente le compteur
        a=b;  //n'incréménte pas le compteur 
        cout<<"compteur ="<<point::compteur<<endl<<endl; // affiche le nombre d'ibjets
}

ça marche trés bien !