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

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

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

Shared_str реализация - На выходе из деструктора падает программа - C++

12.08.2014, 21:07. Просмотров 745. Ответов 21
Метки нет (Все метки)

Решил реализовать shared_str

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
25
26
27
28
29
30
class shared_ptrMy
{
    private:
        Storage *st;
 
 
    public:
        shared_ptrMy(){};
        shared_ptrMy(Object*);
        ~shared_ptrMy();
        shared_ptrMy& operator=(shared_ptrMy&);
};
 
shared_ptrMy::shared_ptrMy(Object *o){
    st = new Storage(o);
}
 
shared_ptrMy& shared_ptrMy::operator=(shared_ptrMy &o){
    st->Copy(o.st);
    st = o.st;
    Storage *a = st;
    return *this;
}
 
 
 
shared_ptrMy::~shared_ptrMy(){
        delete st;
        st = NULL;
}

для этого я создал класс контейнер store
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
25
26
27
28
class Storage
{
    public:
        Storage(Object *o){
            pObj = o;
            counter = 1;
        }
        ~Storage(){
 
            int &k = counter;
            --counter;
 
            if(counter == 0)
                delete pObj;
        }
 
       Object& getObject(){
            return *pObj;
        }
 
        void Copy(Storage *s){
            ++(s->counter);
        }
 
 
   // private:
        Object *pObj;
        int counter ;
при этом counter работает нормально. Но когда counter == 0 вызывается деструктор для object все удаляется и на выходе из деструктора падает программа. Подскажите, что это может быть
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
12.08.2014, 21:07
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Shared_str реализация - На выходе из деструктора падает программа (C++):

Очередное деструктивное: при вызове деструктора все падает - C++
Доброго времени суток! Есть некий класс H, с конструктором, деструктором: H(int size, int top){ this->size = size; this->top =...

Реализация конструкторов и деструктора - C++
Дорогие форумчане! В своем обучение дошел до создания класса. В этом примере должны были заработать конструкторы и деструктор: ...

Программа выдаёт ошибку при вызове деструктора - C++
Здравствуйте! Есть два шаблона: 1) шаблон двухсвязного списка 2) шаблон файла, для записи двусвязного списка в файл, и для чтения...

После объявления деструктора программа перестает выводить данные на экран - C++
Помогите, плиз! Без деструктора программа работает (правда, компилятор возмущается, что их нет). После того, как я его добавляю, данные...

Падает программа - C++
ошибки нет. предупреждения такого вида: warning C4996: 'strcat': This function or variable may be unsafe. Consider using strcat_s instead....

Падает программа с opencv - C++
Код #include <cv.h> #include <highgui.h> int main() { // задаём высоту и ширину картинки int height = 620; ...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
DrOffset
7090 / 4231 / 950
Регистрация: 30.01.2014
Сообщений: 7,006
14.08.2014, 18:37 #16
Сообщение было отмечено автором темы, экспертом или модератором как ответ
Цитата Сообщение от MishaMoix Посмотреть сообщение
НО и еще в объекте st удаляется int counter (т.к. это не указатель и удаляется автоматом при удалении объекта)?
После отработки деструктора Storage, данные которые он хранит перестают быть доступными для обращения (т.к. память под объект освобождена). Т.е. ты конечно сможешь к ним обратиться, и даже возможно получишь адекватный результат, но это неопределенное поведение (т.е. может быть что угодно, например падение или вернутся какие-то левые данные) и делать так нельзя. Т.е. да, счетчик будет уничтожен при первом вызове ~Storage().
Цитата Сообщение от Avazart Посмотреть сообщение
Проверяй счетчик потом только освобождай память.
Ему еще нужно что-то сделать с неинициализированными указателями на Storage, т.к. конструктор копирования и оператор присваивания неадекватно себя ведут при том коде, который сейчас есть.
Avazart
14.08.2014, 18:40
  #17

Не по теме:

И опять же возникает вопрос почему класс не шаблонный ?
Что за название такое "shared_ptrMy" ? что нельзя было обвернуть shared_ptr в свое пространство имен "My"?

MishaMoix
0 / 0 / 0
Регистрация: 03.04.2012
Сообщений: 99
14.08.2014, 18:58  [ТС] #18
Все спасибо. Т.е. при вызове деструктора мы говорим "окей комп, вот у меня объект, все переменные объекта мне не нужны забирай себе, а так же вызови деструкторы у др. объектов (которые объявлены через указатели)"

Добавлено через 38 секунд
Ну я не ставил себе цель делать это шаблонным или с правильным названием
DrOffset
7090 / 4231 / 950
Регистрация: 30.01.2014
Сообщений: 7,006
14.08.2014, 19:20 #19
MishaMoix, Если интересно, вот тебе небольшой пример:
Кликните здесь для просмотра всего текста
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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
#include <iostream>
 
template <typename T>
class storage
{
public:
    explicit storage(T * p)
        : counter_(0), data_(p)
    { }
    ~storage()
    {
        delete data_;
    }
    void inc_ref()
    {
        ++counter_;
    }
    void dec_ref()
    {
        if(--counter_ == 0)
        {
            delete this;
        }
    }
 
private:
    storage & operator=(storage const &);
    storage(storage const &);
 
    size_t counter_;
    T *    data_;
};
 
namespace my
{
 
template <typename T>
class shared_ptr
{
public:
    shared_ptr()
        : st_(0)
    { }
 
    shared_ptr(T * p)
        : st_(0)
    {
        if(p != 0)
        {
            st_ = new(std::nothrow) storage<T>(p);
            if(st_ != 0)
            {
                st_->inc_ref();
            }
            else
            {
                delete p;
                throw std::bad_alloc();
            }
        }
    }
    shared_ptr(shared_ptr const & x)
        : st_(x.st_)
    {
        if(st_ != 0)
        {
            st_->inc_ref();
        }
    }
 
    ~shared_ptr()
    {
        if(st_ != 0)
        {
            st_->dec_ref();
        }
    }
 
    shared_ptr & operator=(shared_ptr const & x)
    {
        shared_ptr(x).swap(*this);
        return *this;
    }
 
    shared_ptr & operator=(T * x)
    {
        shared_ptr(x).swap(*this);
        return *this;
    }
 
    void swap(shared_ptr & x)
    {
        storage<T> * tmp = st_;
        st_ = x.st_;
        x.st_ = tmp;
    }
 
private:
    storage<T> * st_;
};
 
}
 
class Object
{
public:
    Object()  { std::cout << "Object()\n";  }
    ~Object() { std::cout << "~Object()\n";}
};
 
 
int main()
{
    my::shared_ptr<Object> a(new Object());
    my::shared_ptr<Object> b = a;
    my::shared_ptr<Object> c;
    c = b;
}

http://codepad.org/fXiO0PHo
MishaMoix
0 / 0 / 0
Регистрация: 03.04.2012
Сообщений: 99
14.08.2014, 19:41  [ТС] #20
Я только не понял как и для чего swap?
DrOffset
7090 / 4231 / 950
Регистрация: 30.01.2014
Сообщений: 7,006
14.08.2014, 20:12 #21
Цитата Сообщение от MishaMoix Посмотреть сообщение
Я только не понял как и для чего swap?
Copy-And-Swap
Tulosba
:)
Эксперт С++
4393 / 3236 / 297
Регистрация: 19.02.2013
Сообщений: 9,045
14.08.2014, 22:34 #22
DrOffset, ещё можно провести микрооптимизацию:
C++
1
2
3
4
5
shared_ptr & operator=(shared_ptr const & x)
{
   shared_ptr(x).swap(*this);
   return *this;
}
Заменить на:
C++
1
2
3
4
5
shared_ptr & operator=(shared_ptr x)
{
   x.swap(*this);
   return *this;
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.08.2014, 22:34
Привет! Вот еще темы с ответами:

Почему падает программа? - C++
Почему падает программа? Компилируется без ошибок, но при запуске аварийное завершение работы. #include &lt;iostream&gt; #include &lt;stdlib.h&gt; ...

Реализация класса (конструктора и деструктора) - Visual C++
Нужна помощь в исправлении программы (в Dev c++ компилируется и работает, а в Visual Studio - нет) #include &lt;iostream.h&gt; #include...

При выходе из функции падает приложение SIGSEGV Segmentation fault - C++ Qt
Работаю в QtCreator. При выходе из функции падает приложение с ошибкой SIGSEGV Segmentation fault, Я понимаю, что это значит что с памятью...

Реализация сохранения при выходе из программы - Delphi
Как реализовать при закрытие программы чтобы она запрашивала сохранить изменения в файле или нет?!(нужно именно чтобы запрашивала только...


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

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

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