Форум программистов, компьютерный форум CyberForum.ru

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
milvus
0 / 0 / 0
Регистрация: 14.11.2012
Сообщений: 35
#1

Перегрузка operator new - C++

29.10.2013, 14:12. Просмотров 343. Ответов 8
Метки нет (Все метки)

Столькнулся с такой проблемой:
Есть класс Array, в его конструкторе создается массив int
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
    Array::Array(const size_t size):_size(size)
    {
        printf("Array::Array(const size_t size)\n");
        if (_size>0 && _size<UINT_MAX){
            if ( (_data = new int[_size]) == 0)
                printf("error no memory");
        }
    }
    Array::Array( Array const * const src):_size(src->_size)
    {
        printf("Array::Array( Array const * const src)\n");
 
        if ( (_data = new int[_size]) == 0)
            printf("error no memory");
        for (int i=0; i<_size;i++)
            _data[i] = src->_data[i];
    }
 
    Array::~Array(){
        printf("Array::~Array()\n");
        delete[] _data;
    }
Создал класс который автоматом удаляет объекты класса Array созданные как new Arrray
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class Scoped_ptr_arr{
 
private:
    Array * _ptr;
 
    // zapret for BUG fix!!!! call to copy of object or operator=
    Scoped_ptr_arr(const Scoped_ptr_arr & pArray){printf("Scoped constr from class!\n");};
    Scoped_ptr_arr & operator= (const Scoped_ptr_arr&) {};
 
public:
    Scoped_ptr_arr(Array * pArray): _ptr(pArray){printf("Scoped constr from &Array!\n");};
 
    ~Scoped_ptr_arr(){if (_ptr) delete _ptr; printf("Scoped destruct Array!\n");}
 
    Array * ptr(){return  _ptr;}    // return addres then can obj.ptr()->doSomthing();
    bool isNull(){ return _ptr == 0;}
 
    Array * operator-> () {return _ptr;}
    Array & operator*  () {return *_ptr;}
};
в main хочу чтобы все констркуторы вызывалиьс автоматом, но оказывается этого не происходит,
потому что ::new != operator new
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    Array arr(10); // i^2
 
    for (int i = 0; i<10;i++)
        arr.set(i,i*i);
 
    {
        Scoped_ptr_arr pArray = new Array(arr);
 
    }   // automat destruct on end of scope not need delete after new
 
    int t;
    scanf("%d",&t);
 
return 0;
}
а он говорит что вот здесь Scoped_ptr_arr pArray = new Array(arr); не вызывается конструктор по объекту arr, а просто копируется содержимое адресов arr в новый объект, соответственно по выходе из программы возникает ошибка и arr удаляется два раза, пробовал перегрузить operator new но не знаю как из него вызвать конструктор по объекту?? Подскажите!

Добавлено через 9 минут
Консольный вывод:
Array::Array(const size_t size)
Array::operator new(8)
Scoped constr from &Array!
Array::~Array()
Scoped destruct Array!
Array::~Array()
На последнем деструкторе все валится так как массив уже был удален!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.10.2013, 14:12
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Перегрузка operator new (C++):

перегрузка operator<< - C++
помогите отгадать в чет ошибка class MCL{ private: int x,y; public: friend ostream&amp; operator&lt;&lt;(ostream &amp;s, MCL &amp;c); }; ...

Перегрузка operator+ - C++
Как сложить суму годовы продаж всех компаний? Можно использовать перегрузку оператора +? #include &lt;iostream.h&gt; #include &lt;math.h&gt; ...

Перегрузка operator-> () - C++
здравствуйте, объясните пож. как работает operator-&gt; (). Что происходит при этом мне понятно, но непонятно как. template &lt;typename T&gt; ...

Перегрузка operator[]= - C++
Привет народ, вопрос такой - что и как нужно перегрузить в классе, чтобы работал синтаксис вида class ADE {..}; ADE a; ... a |=...

Перегрузка operator[][][] - C++
Здорова господа!!! Как перегрузить operator для класса или operator ??? Что нужно из него возвращать? От что то пытался но не...

Перегрузка operator+ - C++
при сложени myVector2 = myVector0 + myVector1; выодит сообщения &quot;конструктоор 0&quot; ...1,...2,....2(копирования) а потом деструктор 2,2,1,2...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
MrGluck
Модератор
Эксперт CЭксперт С++
7210 / 4376 / 638
Регистрация: 29.11.2010
Сообщений: 11,887
29.10.2013, 14:15 #2
Цитата Сообщение от milvus Посмотреть сообщение
На последнем деструкторе все валится так как массив уже был удален!
Вставьте проверку if-ом

Добавлено через 40 секунд

Не по теме:

Перегрузка оператора new наряду с сишным вводом - что-то новенькое

ForEveR
В астрале
Эксперт С++
7970 / 4732 / 321
Регистрация: 24.06.2010
Сообщений: 10,541
Завершенные тесты: 3
29.10.2013, 14:17 #3
milvus, Данный случай
C++
1
if ( (_data = new int[_size]) == 0)
Никогда не сработает, по той простой причине, что new (без nothrow аргумента) выбрасывает exception, если не может выделить память.
C++
1
_ptr(pArray)
Просто копируются указатели (проблема в том, что _ptr это тоже указатель, поэтому конструктор не вызывается).
milvus
0 / 0 / 0
Регистрация: 14.11.2012
Сообщений: 35
29.10.2013, 14:23  [ТС] #4
Можно конечно, но фишка то в том что должны создаваться два разных объекта класса с разными адресами переменных-полей, а создаются разные объекты класса, но адрес переменной _ptr у них один, из за того что не вызывается конструктор который бы выделил новую память, вот как его вызвать это хотелось бы понять

Добавлено через 3 минуты
Цитата Сообщение от ForEveR Посмотреть сообщение
milvus, Данный случай

Просто копируются указатели (проблема в том, что _ptr это тоже указатель, поэтому конструктор не вызывается).
О! Спасибо! А как можно исправить сохранив логику работы?

Добавлено через 1 минуту
Я то жду если мы в new вызываем Array(arr) то он вызовет конструктор, создаст копию, выделит для нее память и вернет указатель уже на новую копию в другой памяти которую мы и приравняем к _ptr
MrGluck
Модератор
Эксперт CЭксперт С++
7210 / 4376 / 638
Регистрация: 29.11.2010
Сообщений: 11,887
29.10.2013, 14:28 #5
Цитата Сообщение от ForEveR Посмотреть сообщение
Никогда не сработает, по той простой причине, что new (без nothrow аргумента) выбрасывает exception, если не может выделить память.

Не по теме:

А вот в былые времена...



Добавлено через 57 секунд
Цитата Сообщение от milvus Посмотреть сообщение
// zapret for BUG fix
.
milvus
0 / 0 / 0
Регистрация: 14.11.2012
Сообщений: 35
29.10.2013, 14:40  [ТС] #6
ну люблю я комментировать, а то потом забуду зачем все нагородил

Добавлено через 8 минут
C++
1
2
3
4
5
6
7
    
void* Array::operator new (size_t size) throw (bad_alloc) {
        printf("Array::operator new(%ld)\n",size  );
        Array* bar = (Array*)::operator new(size);
//      bar->Array(??);
        return bar;
    }
Перегрузку делал так, из класса Array? а вот как там конструктор вызвать?
MrGluck
Модератор
Эксперт CЭксперт С++
7210 / 4376 / 638
Регистрация: 29.11.2010
Сообщений: 11,887
29.10.2013, 14:46 #7

Не по теме:

Цитата Сообщение от milvus Посмотреть сообщение
ну люблю я комментировать, а то потом забуду зачем все нагородил
это хорошо, транслит только не ок. Понимаю, что для себя пока что делаете, но ведь это пока что. А то будет как тут: Комментарии на английском от российских разработчиков



Советую почитать Скотта Майерса, "Эффективное использование C++. 50 рекомендаций по улучшению ваших программ и проектов". В самом начале очень подробно описывается перегрузка оператора new, создание собственного new handler и есть несколько примеров.
milvus
0 / 0 / 0
Регистрация: 14.11.2012
Сообщений: 35
29.10.2013, 16:30  [ТС] #8
Хотя такой код в main вызывает конструктор правильно и программа выходит без ошибок
C++
1
        Scoped_ptr_arr pArray = new Array(1);
чем отличается от
C++
1
        Scoped_ptr_arr pArray = new Array(arr);
??? Почему в одном случае вызывается конструктор с параметром int а в другом не вызвыается конструктор с пар - объектом класса Array?
ForEveR
В астрале
Эксперт С++
7970 / 4732 / 321
Регистрация: 24.06.2010
Сообщений: 10,541
Завершенные тесты: 3
29.10.2013, 16:45 #9
milvus, Я ведь уже объяснил почему. Давайте возьмем более простой пример.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
#include <iostream>
 
struct A
{
   A() {}
   A(A* ptr)
   {
      std::cout << "from pointer c-tor called" << std::endl;
   }
};
 
struct C
{
   C(A* ptr) : ptr_(ptr)
   {
   }
private:
   A* ptr_;
};
 
int main()
{
   C c = new A();
}
Конструктор не вызван, все логично.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
#include <iostream>
 
struct A
{
   A() {}
   A(A* ptr)
   {
      std::cout << "from pointer c-tor called" << std::endl;
   }
};
 
struct C
{
   C(A* ptr) : ptr_(ptr)
   {
   }
private:
   A ptr_;
};
 
int main()
{
   C c = new A();
}
Все вызывается.

Суть в том, что вы несколько неверно понимаете что происходит в написанной вами программе. Вы написали в классе Array конструктор, который принимает указатель. Вы написали в классе Scoped_Array конструктор который принимает указатель на Array. В этом конструкторе вы пытаетесь инициализировать указатель на Array, хранящийся в классе Scoped_Array, переданным указателем - указатель просто копируется, т.к. для того чтобы был вызван конструктор, должно быть хотя бы

C++
1
_ptr(new Array(ptr))
Тогда да, вызовется конструктор.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
29.10.2013, 16:45
Привет! Вот еще темы с ответами:

Перегрузка operator= - C++
Возможно ли перегрузить оператор= так, что бы можно было написать? val = {10, 20, 15};

Перегрузка operator<() - C++
#include &lt;QCoreApplication&gt; #include &lt;QDebug&gt; #include &lt;QVector&gt; class My { public: int a; void operator=(const...

Перегрузка operator= - C++
const Point&amp; operator= (const Point &amp;p) { if (this ==&amp;p) return *this; x = p.x; y = p.y; color = p.color; ...

Перегрузка operator+ - C++
Нужно сложить число с каждым елементом вектора, вот мой код: MyVectorClass.h namespace program { class MyVectorClass { ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
29.10.2013, 16:45
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru