Преждевременный вызов деструктора

[Даниил]

Приветствую всех.
Появилась внезапная трабла - хочу запихать в вектор свой класс. Только вот незадача, как только я push_back'аю объект в вектор, тот вызывает деструктор, при чем имеется какая-то странная зависимость от количества "запинаных" объектов, посмотрите, подскажите - где тут засада.

Код

C++ (Qt)
#include <iostream>
#include <cstdio>
#include <vector>
using namespace std;
 
class Coord
{
private:
    int x, y;
public:
    Coord(){x = 0; y = 0;}
    Coord(int _x, int _y){x = _x; y = _y;}
    Coord(const Coord &c){x = c.x;}
    ~Coord(){cout << "Delete Coord[" << x << ',' << y << "]" << endl;}
    Coord &operator=(const Coord &c);
    bool operator <(const Coord &c) const;
    friend ostream &operator <<(ostream& s, const Coord &c);
};
 
Coord &Coord::operator =(const Coord &c)
{
    if(this == &c)return *this;
    else
    {
        this->x = c.x;
        this->y = c.y;
        return *this;
    }
}
 
bool Coord::operator <(const Coord &c)const
{
    if(this->x < c.x)return true;
    else if(this->x == c.x)
    {
        if(this->y < c.y)return true;
    }
    return false;
}
 
ostream &operator <<(ostream& s, const Coord &c)
{
    cout << "Coord[" << c.x << "," << c.y << "]" << endl;
    return s;
}
 
int main()
{
    vector<Coord> v;
    v.push_back(Coord());
    v.push_back(Coord());
    v.push_back(Coord());
    cout << v[0];
    cout << v[1];
    cout << v[2];
 
    getchar();
    return 0;
}
 

Вывод:

Код:

Delete Coord[0,0]
Delete Coord[0,11665604]
Delete Coord[0,0]
Delete Coord[0,11665604]
Delete Coord[0,825111086]
Delete Coord[0,0]
Coord[0,11669608]
Coord[0,2037603443]
Coord[0,1127952947]

[Пантер]

Все правильно - вектор хранит копию объекта. При изменении вектора происходит перераспределение объектов копированием.

[Даниил]

м-м-м, поставлю вопрос конкретней - что нужно сделать, чтобы можно было в векторе хранить самописные объекты.

[mutineer]

м-м-м, поставлю вопрос конкретней - что нужно сделать, чтобы можно было в векторе хранить самописные объекты.

Они уже сейчас там хранятся. В чем проблема-то?

[Даниил]

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

Код:

Coord[0,11669608]
Coord[0,2037603443]
Coord[0,1127952947]

[ddrtn]

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

[mutineer]

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

Код:

Coord[0,11669608]
Coord[0,2037603443]
Coord[0,1127952947]

Вектор копирует объекты. Причем не один раз (при перераспределении памяти)
А у тебя неполный конструктор копирования

Код

C++ (Qt)
Coord(const Coord &c){x = c.x;}

[Пантер]

Coord(const Coord &c){x = c.x;} - а y игноришь, вот и получаешь мусор.

[Даниил]

Спасибо. Прозрел.

[Пантер]

В данном случае свой копирующий конструктор и опрератор присваивания не нужны, так как класс тривиален.

[Даниил]

Да я класс обрезал, чтобы сюда не кидать 300 строк кода, а так да, только в теории нужно еще оператор == перегрузить, а то вектор может начать ругаться.

[Igors]

Конструктор лучше оформить так

Код

C++ (Qt)
Coord( int _x = 0, int _y = 0 )  { x = _x; y = _y; }
 

Тогда он годится и по умолчанию

А оператор < написать проще, (а то у Вас голову можно сломать)

Код

C++ (Qt)
bool Coord::operator < ( const Coord & c ) const
{
    if (x < c.x) return true;
    if (x > c.x) return false;
    return y < c.y;
}
 

Оператор == вектору не требуется, но в такого класса нужен все равно

[m_ax]

Конструктор лучше оформить так

Код

C++ (Qt)
Coord( int _x = 0, int _y = 0 )  { x = _x; y = _y; }
 

Тогда он годится и по умолчанию

А оператор < написать проще, (а то у Вас голову можно сломать)

Код

C++ (Qt)
bool Coord::operator < ( const Coord & c ) const
{
    if (x < c.x) return true;
    if (x > c.x) return false;
    return y < c.y;
}
 

Оператор == вектору не требуется, но в такого класса нужен все равно

А оператор < вообще бредово определять для данного класса)
Какая точка будет больше p1(10, 1) или p2(1, 10) ?

Согласно вашей реализации p1 окажется больше.. Но с чего это вы нарушаете равноправие между x и y компонентами?
И как конечный пользователь должен досмысливать поведение такого оператора для объекта точки? Это не логично.

[m_ax]

Да и ещё:
Обычно так не пишут:

Код

C++ (Qt)
Coord &Coord::operator =(const Coord &c)
{
    if(this == &c)return *this;
    else
    {
        this->x = c.x;
        this->y = c.y;
        return *this;
    }
}
 

Лучше так:

Код

C++ (Qt)
Coord &Coord::operator =(const Coord &c)
{
    if (this != &c) 
    {
        x = c.x;
        y = c.y;
    }
    return *this;
}
 

[Igors]

А оператор < вообще бредово определять для данного класса)
Какая точка будет больше p1(10, 1) или p2(1, 10) ?

Согласно вашей реализации p1 окажется больше.. Но с чего это вы нарушаете равноправие между x и y компонентами?
И как конечный пользователь должен досмысливать поведение такого оператора для объекта точки? Это не логично.

Не всегда концептуальность идет в такт с конкретикой/реализацией. Есть вектор точек, надо удалить повторяющиеся. Что будете делать без оператора < ? Или у Вас есть др. предложения как его определить? Так что потише там с "бредово" и.т.п.