Удалить объект в конструкторе

[Perchik71]

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

[JUDAS]

можно в виде кода, а то , в отличии от Делфи, С++ на слух не воспринимаю

[Perchik71]

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

можно в виде кода, а то , в отличии от Делфи, С++ на слух не воспринимаю

Пост.
справочник

class Myclass
{
public:
        void* operator new(size_t);
        void operator delete(void*);
};

void* Myclass::operator new(size_t size)
{
    void *storage = malloc(size);
    if(NULL == storage) {
            throw "allocation fail : no free memory";
    }
    return storage;
}

Как пример определения, надеюсь вы поняли, что если проверив, что объект плохо создался нужно сделать free.. и вернуть NULL

ЗЫ: вы можете описать функцию getError у класса, и если он отличен от 0, то объект не создался успешно.

[JUDAS]

Понятно, нужно "обернуть" конструктор в операторы. Буду пробовать =)

по поводу з.ы.
Я конечно не специалист в С++ и не шарю как смотрится с точки зрения гуру С++ во такой код:

class TIntellectDrawer
  {
     private:
        TDrawCircle * Circle;
     public:
        bool created;

     // конструктор
     TIntellectDrawer 
     {
        created = true;
        try
          Circle = new TDrawCircle;
        catch ()
        {
          Circle = 0;
          created =  false;
        }   
     }   

     ~TIntellectDrawer(void) 
     {
       created =  false;
     }   
   }

TIntellectDrawer drawer = new TIntellectDrawer;
if (!drawer->created) 
{
   ваш объект не создан
   drawer = 0;
}

в Паскале (Делфи) считается одним из подвидов говнокода
Не хочется начинать изучения с++ с этого диалекта

[Perchik71]

Ну я бы защитил эту переменную, что бы её не было возможности изменить и сделал бы функцию, которая только возвращает. Так безопаснее на мой взгляд.

ЗЫ: можно и не логическое хранить а и целое, например если вы хотите за этими цифрами спрятать текст.
Наапример: создать массив строк, а индекс ошибки - индекс в этом массиве (с вычитом -1, ибо с с++ массивы начинаются с 0 индекса)

З.Ы.Ы:
Если вобще продолжать тему з.ы. то можно сделать так...

static std::string MyClass::getErrorMsg() const
{
    if(nerror)
         return g_array_error_msg[nerror-1];
    else return "No Error";
}

Поскольку функция статик, то доступ к членам класса запрещён, но можно это исправить

static std::string MyClass::getErrorMsg(const MyClass* Self) const
{
    if(Self->getError())
         return g_array_error_msg[Self->getError() - 1];
    else return "No Error";
}

Тем самым функцию можно вызвать минуя создание класса,

MyClass::getErrorMsg(my_class);

[waleri]

Не надо делать delete this
Надо либо НЕ ловить там exception, либо перебросить его дальше.
Компилятор сам вызовет нужные деструкторы.

Вам первым же ответом указали как надо делать.
http://programmersforum.ru/showpost....68&postcount=2

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

Как пример определения, надеюсь вы поняли, что если проверив, что объект плохо создался нужно сделать free.. и вернуть NULL

Ага, а еще надо будет в этом new угадывать *тип* объекта, который мы создаем, поскольку переопределив new для класс мы переопределяем его и для его *наследников*. Каждый из этих наследников тоже может выдать ошибки, которые будет отлавливать new.

[Perchik71]

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

Ага, а еще надо будет в этом new угадывать *тип* объекта, который мы создаем, поскольку переопределив new для класс мы переопределяем его и для его *наследников*. Каждый из этих наследников тоже может выдать ошибки, которые будет отлавливать new.

Ничего там угадывать не нужно, удаляем то что инициализировалось и освобождаем указатель... вы большой любитель усложнять.
ЗЫ тот пост ему поможет лишь с членами объекта а не с самим объектом.
человек спросил как сделать так, что бы new реагировал на конструктор.. ему сказали.

И Да, если вы не в курсе, тело функции остаётся не изменой, даже при наследовании, если наследник не изменил её... и то можно вызвать путём родитель::функция. и результат будет такой же, кк его и описали. Вообщем не усложняйте.

[waleri]

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

Ничего там угадывать не нужно, удаляем то что инициализировалось и освобождаем указатель... вы большой любитель усложнять.

Т.е. вы предлагаете заменить механизм, встроенный в язык, своим костылем, который будет работать только для строго определенных условий и никак не будет предупреждать, если вы эти условия не соблюдаете и это я усложняю?!

[Perchik71]

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

Т.е. вы предлагаете заменить механизм, встроенный в язык, своим костылем, который будет работать только для строго определенных условий и никак не будет предупреждать, если вы эти условия не соблюдаете и это я усложняю?!

Я предлагаю дополнить механизм new для конкретного класса, а не изменить его вовсе, тем более new возвращает ошибку только в одном случаи, если не удалось выделить память. А это мало, если требуется следить за инициализацией.

[JUDAS]

Сделал вставку быдлокода с п.с. =)
Более сложные механизмы внутренней кухни С++ мне как то "не хотят" вклиниваться в черепушку. Ну не привык я заниматься байтодро...есвтом и изучениме таинсв устройства ОС и менеджера памяти в программах, которые предназначены совершенно для других целей. В Делфи это делает за меня разработчик среды.
Извините господа гуру, что я выбрал такой топорный метод

class TIntellectDrawer
  {
     private:
        bool valid;
        TDrawCircle * Circle;
     public:
       
     // конструктор
     TIntellectDrawer 
     {
        valid = true;
        try
          Circle = new TDrawCircle;
        catch ()
        {
          Circle = 0;
          valid =  false;
        }   
     }   

     ~TIntellectDrawer(void) 
     {
       valid =  false;
     }   
     
     bool assigned(){return valid;}

   }

TIntellectDrawer drawer = new TIntellectDrawer;
if (!drawer->assigned) 
{
   ваш объект не создан
   drawer = 0;
}