Ошибка при очистке памяти

[alexey_kip]

Всё просто: в классе создаю конструктор, деструктор, указатель на тип char и перегружаю операцию "больше".
В main'e создаю 2 переменных типа "клас" и сравниваю их между собой.
При сравнении вызывается деструктор и уничтожает вторую переменную, т.е. после сравнения пользоваться ей уже не получится, т.к. там будет мусор. Ну и соответственно на выходе из программы деструктор вызовется два раза, из-за чего вылетит ошибка (при выполнении) _BLOCK_TYPE_IS_VALID...
Кто знает как от этой ошибки избавиться?

\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
"2.h"
class c
{
public:
char *t;
c();
bool operator >(c);
~c();
};

c::c()
{
t=new char;
*t='t';
}

bool c:perator >(c v)
{
if (*t>*v.t) return 1;
else return 0;
}

c::~c()
{
delete t;
}


\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
"1.cpp"
#include "2.h"
void main()
{
c t,u;
bool b=t>u;
}

[alexey_kip]

Тоже самое не только при перегрузке оператора, а вообще при вызове любого метода (созданного в классе), которое возвращает значение не void

[Сыроежка]

Сообщение от alexey_kip

Всё просто: в классе создаю конструктор, деструктор, указатель на тип char и перегружаю операцию "больше".
В main'e создаю 2 переменных типа "клас" и сравниваю их между собой.
При сравнении вызывается деструктор и уничтожает вторую переменную, т.е. после сравнения пользоваться ей уже не получится, т.к. там будет мусор. Ну и соответственно на выходе из программы деструктор вызовется два раза, из-за чего вылетит ошибка (при выполнении) _BLOCK_TYPE_IS_VALID...
Кто знает как от этой ошибки избавиться?

\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
"2.h"
class c
{
public:
char *t;
c();
bool operator >(c);
~c();
};

c::c()
{
t=new char;
*t='t';
}

bool c:perator >(c v)
{
if (*t>*v.t) return 1;
else return 0;
}

c::~c()
{
delete t;
}


\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
"1.cpp"
#include "2.h"
void main()
{
c t,u;
bool b=t>u;
}

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

Вы объявили оператор сравнения bool operator >(c);, в который объект передается по значению. Это означает, что когда вы пишите выражение bool b=t>u;, то создается временный объект с помощью конструктора копирования из объекта u. В этом случае член c::t в обоих объектах указывает на один и тот же адрес. При завершении работы оператора сравнения временный объект удаляется и вызывается его деструктор, который в свою очередь удаляет память, на которую указывает член временного объекта c::t , Затем когда программа завершается, удаляется также объект u, и его деструктор уже по второму разу пытается удалить память, на которую указывает его u.t. Естественно возникает аварийное завершение.

То есть вам помимо конструктора по умолчанию нужно было еще явно определить конструктор копирования

c::c( const c &rhs ) : t( new char( rhs.t ) ) {}

Кроме того ваш оператор сравнения лучше было объявить так, чтобы он принимал ссылку на объект. В этом случае временный объект создаваться не будет.

class c
{
   // другие объявления опущены
   bool operator >( const c &rhs ) const;
};

bool c::operator >( const c &rhs ) const
{
   return ( t > rhs.t );
}

Вам также надо объявить свой оператор присваивания, так как в противном случае может возникнуть такая же ошибка, если вы будете примваивать объякты вашего класса друг друггу.
Функция main в С++ объявляется как int main()

[alexey_kip]

Спасибо, учту