Посчитать объекты? - C/C++

Есть класс:

class clsTime
{

    private:

        short siHH, siSS, siMM;
                
    public:
        
        clsTime() : siHH ( 0 ), siMM ( 0 ), siSS ( 0 )
        {

        }

        clsTime ( short h, short m, short s ) : siHH ( h ), siMM ( m ), siSS ( s )
        {

        }

        void <...>

};

Я хочу посчитать сколько объектов создано под этим классом. Вроде бы всё просто — добавляем свойства:

class clsTime
{

    private:

        <...>
        static unsigned int uiCounter;
        unsigned int uiNum;

А в конструкторах пишем:

    <...>
    public:
        
        clsTime() : siHH ( 0 ), siMM ( 0 ), siSS ( 0 )
        {
            uiCounter++;
            uiNum = uiCounter;
        }

        clsTime ( short h, short m, short s ) : siHH ( h ), siMM ( m ), siSS ( s )
        {
            uiCounter++;
            uiNum = uiCounter;
        }

    <...>

Получается, что uiNum и будет показывать порядковый номер вновь созданного объекта, однако при записи:

TimeObj3 = TimeObj1 + TimeObj2;

Номер TimeObj3 инкрементируется и становится равным не 3, а 4.

Как сделать правильно?

DND>Как сделать правильно?

сделать конструктор и копировать счетчик оттуда

clsTime(const clsTime & r) {
   this.uiNum = r.uiNum;
   // other stuff
}

Здравствуйте, Ovl, Вы писали:

DND>>Как сделать правильно?

Ovl>сделать конструктор и копировать счетчик оттуда

Ovl>

Ovl>clsTime(const clsTime & r) {
Ovl>   this.uiNum = r.uiNum;
Ovl>   // other stuff
Ovl>}
Ovl>

но все равно это не спасет от ситуация типа. хотя и уберет проблему с временными объектами.

clsTime a;
clsTime b = a;

чем не устраивает текущий вариант (с 4-кой вместо 3-ки)?

Здравствуйте, DukeNukem3D, Вы писали:

DND>Я хочу посчитать сколько объектов создано под этим классом. Вроде бы всё просто — добавляем свойства:
...
DND>А в конструкторах пишем:
...

DND>Получается, что uiNum и будет показывать порядковый номер вновь созданного объекта, однако при записи:

DND>

DND>TimeObj3 = TimeObj1 + TimeObj2;
DND>

DND>Номер TimeObj3 инкрементируется и становится равным не 3, а 4.

Не упомянут конструктор копирования, хотя судя по всему он реализован, и счетчик в нем инкрементируется...

DND>Как сделать правильно?
Никак — ты не сможешь контролировать создание временных объектов.

Расскажи лучше, что именно тебе нужно.

Здравствуйте, Ovl, Вы писали:

Ovl>Здравствуйте, Ovl, Вы писали:

DND>>>Как сделать правильно?

Ovl>>сделать конструктор и копировать счетчик оттуда

Ovl>>

Ovl>>clsTime(const clsTime & r) {
Ovl>>   this.uiNum = r.uiNum;
Ovl>>   // other stuff
Ovl>>}
Ovl>>

Ovl>но все равно это не спасет от ситуация типа. хотя и уберет проблему с временными объектами.

Ovl>

Ovl>clsTime a;
Ovl>clsTime b = a;
Ovl>

Ovl>чем не устраивает текущий вариант (с 4-кой вместо 3-ки)?

вот этим:

for ( int i=1; i<60; i++)
{
      clsTime b = a;
}

Каким будет номер объекта в этом случае?

Здравствуйте, Bell, Вы писали:

B>Не упомянут конструктор копирования, хотя судя по всему он реализован, и счетчик в нем инкрементируется...

Конструктор копирования я не писал, пользуюсь конструктором по умолчанию.

B>Расскажи лучше, что именно тебе нужно.

Хочу просто чтобы у каждого объекта был свой порядковый номер, хранящийся в одном из полей и присваивался во время создания объекта, еще красивее было бы чтобы его нельзя было изменить.

Здравствуйте, DukeNukem3D, Вы писали:

B>>Не упомянут конструктор копирования, хотя судя по всему он реализован, и счетчик в нем инкрементируется...

DND>Конструктор копирования я не писал, пользуюсь конструктором по умолчанию.

Значит при конструировании копии не будет инкрементирован счетчик, и будет использовано значение из исходного объекта, т.е. в результате получатся объекты с дублирующимися номерами.

B>>Расскажи лучше, что именно тебе нужно.

DND>Хочу просто чтобы у каждого объекта был свой порядковый номер, хранящийся в одном из полей и присваивался во время создания объекта...

Значит строгая последовательность номеров не важна?

DND>...еще красивее было бы чтобы его нельзя было изменить.

class Test
{
   const int id_;
   static int counter_;

public:
   Test() : id_(counter_++) {}
   Test(const Test& t) : id_(counter_++) {}
};

int Test::counter_ = 0;

есть вариант со смарт поинтерами.
в этом случае временных объектов не будет.
примерно так:

class A;
typedef boost::shared_ptr<A> APtr;

class A
{
    A() { ++ Count; }
    A( const A& );
    A& operator=( const A& );
    static unsigned Count;
public:
    static APtr create() { return APtr( new A() ); }
};

unsigned A::Count = 0;

...

APtr p1( A::create() );
APtr p2 = p1;
APtr p3( A::create() );
p1 = p3;
assert( 2 == A::Count );

DND>Я хочу посчитать сколько объектов создано под этим классом. Вроде бы всё просто — добавляем свойства:
Ну ты и считаешь, все правильно. Просто временный объект — это тоже объект.

Здравствуйте, DukeNukem3D, Вы писали:

DND>Есть класс:
DND>Как сделать правильно?
Воспользуйся фабричной функцией для создания объектов. В ней и считай их количество.

Здравствуйте, DukeNukem3D, Вы писали:

[Skip]

DND>

DND>TimeObj3 = TimeObj1 + TimeObj2;
DND>

DND>Номер TimeObj3 инкрементируется и становится равным не 3, а 4.

DND>Как сделать правильно?

Может перекрыть операторы new и delete и считать в них?
Тогда стековые ребята в счет не попадут.

	От:	DukeNukem3D
	Дата:	17.07.08 05:37
	Оценка:

	От:	Ovl
	Дата:	17.07.08 05:41
	Оценка:

	От:	Ovl
	Дата:	17.07.08 05:44
	Оценка:

	От:	Bell
	Дата:	17.07.08 06:00
	Оценка:

	От:	Аноним
	Дата:	17.07.08 06:01
	Оценка:

	От:	Greg Zubankov
	Дата:	18.07.08 06:59
	Оценка:	+3