Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57

Не работает е моим деструктором, только с дефолтовым

29.04.2010, 13:12. Показов 1203. Ответов 16
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Что именно делает деструктор который по умолчанию?
Просто есть код, который работает только если деструктор используется именно дефолтовый.
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
119
120
121
122
123
124
125
126
127
128
class mnozh
{
    int n;
    int *mas;
    public:
    mnozh(); // конструктор
    mnozh(int); // конструктор по умолчанию
    mnozh(mnozh&); // конструктор копирования
    ~mnozh(); // Деструктор
    friend const mnozh& operator +(const mnozh& , int) ; // добавка одного элемента 
    mnozh& operator =(const mnozh&); //перегрузка операции присваивания
};
 
 
mnozh::mnozh() // Конструктор
{
    int i,k,j,z;
    cout << "Vvedite kolichestvo elementov \n n=";
    cin >> k;
    n=0;
    mas = new int[k];
    for(i=0;i<k;i++)
    {
        cout << "Vvedite " << i << " element:  ";
        cin >> mas[i];
        z=0;
        for(j=0;j<i;j++)
        {
            if(mas[j]==mas[i])
            {
 //               cout << mas[i] << "==" << mas[j] << "    " << z << "\n";
                z++;
            }
        }
        if(z!=0)
        {
            i--;
            k--;
            cout << "povtor \n";
        }
    }
    n=k;
};
 
mnozh::mnozh(int x) // Конструктор
{
    int i;
    if(x>=0)
    {
        mas = new int[x];
        n=x;
        for(int i=0;i<x;i++)
        {
            mas[i]=i+1;
        }
    }
    else
    {
        cout << "x < 0";
    }
   // cout << "J sozdatel'";
};
 
mnozh::mnozh(mnozh & p) // конструктор копирования
{
    int i;
   // cout << "I am copy " << p;
    //cout << "n="<<n<<"; p.n="<<p.n<<"  ";
    n=p.n;
   // cout << "  posle, n="<<n;
    mas = new int[n];
     for(i=0;i<n;i++)
    {
       // cout << "   mas[" << i << "] = " << mas[i] << " = p.mas[" << i << "] = " << p.mas[i] << "  ";
        mas[i]=p.mas[i];
       // cout << "  posle, mas["<<i<<"]="<<mas[i];
    }
};
 
mnozh::~mnozh() //Деструктор
{
    delete mas;
};
 
const mnozh& operator +(const mnozh & p, int x)// добавка одного элемента
{
    mnozh viv(1);
    viv=p;
    int i;
    if (viv.prov(x))
    {
        viv.n++;
        viv.mas = new int[viv.n];
        for(i=0;i<viv.n-1;i++)
        {
            viv.mas[i] = p.mas[i];
        }
        viv.mas[i] = x;
    }
    else
    {
        cout << "Chislo x=" << x << " uzhe est' v mnozhestve";
    }
    cout << viv;
    return viv;
};
 
 
mnozh& mnozh::operator =(const mnozh & p) //Операция присваивания
{
    int i;
    if(&p != this)
    {
        //cout << "prisvaivaem massivu dlinni " << n << " so znach " << *this << " , massiv dlinni " << p.n << " so znach " << p;
   //     cin >> i;
        n=p.n;
        mas = new int[n];
        for(i=0;i<n;i++)
        {
            mas[i]=p.mas[i];
        }
    return *this;
    }
    else
    {
        cout << "hernj";
    }
};
и есть сам мэйн

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
#include "mnozh.h"
 
main()
{
    int k;
    mnozh ed;
    mnozh cd(5);
    cout << "\n" << ed;
    cout << "\n" << cd;
    cd = (ed + 1);
    cout << "\n";
    cout << "\n" << ed;
    cout << "\n" << cd;
    return 0;
}
Прога создает класс множество, прибавление элемента соответственно прибавляет элемент, если его еще нет. Ну вот хоть убей не могу понять почему работает только с дефолтовым деструктором =(
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
29.04.2010, 13:12
Ответы с готовыми решениями:

Не работает с деструктором
#define st system(&quot;pause&quot;); #include &lt;iostream&gt; using namespace std; class temp{ public: int a; unsigned int b; void...

Что с моим кодом не так?Он работает
Из несколько примеров собрал свой) Я в JS ноль)! Я чувствую что-то с ним не так) $(document).ready(function(){ ...

Что не так с моим фреймворком? Не работает консольный ввод
Всем привет! У меня такая проблема: не могу организовать нормальный консольный ввод на c#. Я точно не знаю, что произошло, но буквально...

16
221 / 135 / 48
Регистрация: 12.04.2010
Сообщений: 248
29.04.2010, 13:37
Что такое дефолтовый деструктор?
И с каким деструктором не работает?

Кстати...
C++
1
2
3
4
mnozh::~mnozh() //Деструктор
{
    delete[] mas;
};
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
29.04.2010, 20:27  [ТС]
Не работает именно с тем что я написал, и работает если не написать никакого, т.е. с тем что существует по умолчанию.
Даже если мой деструктор не будет делать ничего, т.е. {}
все равно не работает.

И да, я понимаю что в функции сложения я возвращаю адрес локальной переменной, я знаю что это не правильно, но почему тогда работает если я не пишу никакого деструктора? И как можно сделать без этого что бы запись
a=b+3 работала корректно.
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
29.04.2010, 22:13
Цитата Сообщение от IIGoshaII Посмотреть сообщение
Не работает именно с тем что я написал, и работает если не написать никакого, т.е. с тем что существует по умолчанию.
Даже если мой деструктор не будет делать ничего, т.е. {}
все равно не работает.
"работает" - "не работает" - ...
Как проверялась "работоспособность" деструктора?
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
29.04.2010, 22:49  [ТС]
Не рабочесть заключается в том что прога компилируется, но при выполнении когда доходит до места со сложением выдает виндовую ошибку. Вся прога заканчивается с кодом ошибки 0xC0000005. Это ошибка с памятью, почему она не возникает, если деструктор дефолтовый?
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
29.04.2010, 22:56
Цитата Сообщение от IIGoshaII Посмотреть сообщение
Не рабочесть заключается в том что прога компилируется, но при выполнении когда доходит до места со сложением выдает виндовую ошибку. Вся прога заканчивается с кодом ошибки 0xC0000005. Это ошибка с памятью,
"не работать" деструктор не может, если он описан
Проблема в общем алгоритме

Цитата Сообщение от IIGoshaII Посмотреть сообщение
почему она не возникает, если деструктор дефолтовый?
"дефолтовый" не освобождает память выделенную оператором new
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
29.04.2010, 23:08  [ТС]
Цитата Сообщение от Roma_F Посмотреть сообщение
"дефолтовый" не освобождает память выделенную оператором new
Но эта ошибка возникает даже если я описываю пустой деструктор, который не делает ничего.
0
 Аватар для and
22 / 22 / 3
Регистрация: 12.05.2009
Сообщений: 43
30.04.2010, 00:10
Ты в перегруженных операторах не очищаешь указатели...
C++
1
2
3
4
5
6
7
оператор =
        n=p.n;
///Здесь
        delete this->mas;
///
        mas = new int[n];
        for(i=0;i<n;i++)
И все же в динструкторе надо...
C++
1
delete[] mas;
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
30.04.2010, 00:41  [ТС]
Так правильнее, но от этого ничего не меняется.
Работает нормально ed=qd и ed+1, но ed=qd+1 , т.е. проблемы не в каком либо операторе, а в передаче значения от одного к другому.
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
30.04.2010, 09:54
Нужны все исходники. Что за метод prov, где переопределение "<<"
Зачем "cout << viv" в операторе "+"?
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
30.04.2010, 20:02  [ТС]
Цитата Сообщение от Roma_F Посмотреть сообщение
Нужны все исходники. Что за метод prov, где переопределение "<<"
Зачем "cout << viv" в операторе "+"?
prov проверяет принадлежность элемента множеству, вывод viv исключительно для проверки.


Прилагается полный хедер класса , мэйн в заголовке есть.
Вложения
Тип файла: txt mnozh.txt (6.9 Кб, 20 просмотров)
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
30.04.2010, 22:29
уффф..
Чем ты это компилировал
И что это за загадочный оператор "if (viv.prov(x) or viv.null(x))"

Оператор "+" у тебя возвращает ссылку на удалённый объект - viv. С остальными та же история.

Такой вариант у меня работает:
C++
1
mnozh operator +(int x,mnozh & p)
И ещё - функция возвращающая значение должна всегда его возвращать
Вот это не есть хорошо:
C++
1
2
3
4
    else
    {
        cout << "hernj";
    }
нужно вынести "return *this;"

Добавлено через 5 минут
хм, насчёт or вопрос отменяется - в Code::blocks работает
0
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
30.04.2010, 22:45  [ТС]
Я в Код::Блокс и компилю =)

Я понимаю что я возвращаю адреса локальных переменных. Но я не понимаю двух вещей.
1) Как сделать по другому
2) Почему если не прописывать деструктор, то все работает, а если прописать, даже пустой, то нет.

Цитата Сообщение от Roma_F Посмотреть сообщение
уффф..
Чем ты это компилировал
И что это за загадочный оператор "if (viv.prov(x) or viv.null(x))"
Оператор сначала проверяет что бы элемент x присутствовал в множестве viv и проверяет множество viv на пустоту, хотя это наверно лишнее =)
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
30.04.2010, 23:23
В общем написал простенький клаcс с похожими операторами - http://codepad.org/3gv0yQaF
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
#include <iostream>
#include <vector>
#include <map>
 
using namespace std;
 
class MyPoint
{
    public:
        int x;
        int y;
        int *mas;
 
        MyPoint (void);
        MyPoint (int S);
        MyPoint (const MyPoint &);
        ~MyPoint (void);
        MyPoint& operator= (const MyPoint&);
        friend MyPoint operator+(const MyPoint &P1, const MyPoint &P2);
        friend MyPoint operator+(const MyPoint &P1, int C);
        friend ostream& operator <<(ostream&, MyPoint&);
};
 
MyPoint::MyPoint (void)
{
    mas = new int[10];
    for (int i = 0; i < 10; i++) mas[i] = 0;
}
 
MyPoint::MyPoint (int S)
{
    mas = new int[10];
    for (int i = 0; i < 10; i++) mas[i] = S*i;
}
 
MyPoint::MyPoint (const MyPoint &P)
{
    mas = new int[10];
    for (int i = 0; i < 10; i++) mas[i] = P.mas[i];
}
 
MyPoint::~MyPoint (void)
{
    delete[] mas;
}
 
MyPoint & MyPoint::operator= (const MyPoint &P)
{
    for (int i = 0; i < 10; i++) mas[i] = P.mas[i];
    return *this;
}
 
MyPoint operator+ (const MyPoint &P1, const MyPoint &P2)
{
    MyPoint p(0);
    for (int i = 0; i < 10; i++) p.mas[i] = P1.mas[i] + P2.mas[i];
 
    return p;
}
 
MyPoint operator+ (const MyPoint &P1, int C)
{
    MyPoint p1(0);
    for (int i = 0; i < 10; i++) p1.mas[i] = P1.mas[i] + C;
 
    return p1;
}
 
ostream& operator <<(ostream& Stream, MyPoint& P)
{
    for (int i = 0; i < 10; i++) Stream << P.mas[i] << "\t";
    Stream << endl;
    return Stream;
}
 
int main()
{
    MyPoint p1, p2, p3(10), p4(100);
 
    cout << "p1 " << p1 << "p2 " << p2 << "p3 " << p3 << "p4 " << p4;
 
    p1 = p3 + p4;
    p2 = p1 + 1000;
 
    cout << endl << "p1 " << p1 << "p2 " << p2;
 
    return 0;
}
1
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
03.05.2010, 20:39  [ТС]
Малость не то. Тут динамики нету =)
0
332 / 247 / 32
Регистрация: 13.12.2009
Сообщений: 589
03.05.2010, 20:49
Цитата Сообщение от IIGoshaII Посмотреть сообщение
Тут динамики
какой такой динамики
1
8 / 8 / 1
Регистрация: 20.03.2010
Сообщений: 57
03.05.2010, 21:23  [ТС]
На самом деле уже не важно, я все сделал на основе выше написаного и все пока работает =)

Динамики работы с массивами, прибавление элемента это именно прибавление элемента, а не к элементу.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
03.05.2010, 21:23
Помогаю со студенческими работами здесь

СМА Whirlpool AWE 7515 работает только слив, работает только слив
перестал закрываться замок, хозяева поменяли сами и ничего у них не вышло, вызвали мастера. замок не закрывается, мало врямя подачи...

Проблема с деструктором
Помогите реализовать алгоритм: Пусть имеется класс сотрудников примерно следующего вида: Unit { char name; Unit *Master;...

Ошибка с Деструктором
Проблема заключается в следующем: Имеется класс Вектор, в котором есть поля два поля (для координат и их количества). Есть конструктор по...

Проблемы с деструктором
Вот код : #include&lt;iostream.h&gt; #include&lt;string.h&gt; struct Value { char val_1; int val_2; bool val_3; };

Ноутбук работает только с Wi Fi роутера дома, в кафе не работает и везде. Почему?
Подскажите пожалуйста, почему у меня ноутбук с виндой 7, работает вай фай только дома с роутера. А когда я прихожу в кафе или в парк, у...


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

Или воспользуйтесь поиском по форуму:
17
Ответ Создать тему
Новые блоги и статьи
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора
ФедосеевПавел 14.03.2026
Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .
делаю науч статью по влиянию грибов на сукцессию
anaschu 13.03.2026
прикрепляю статью
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++
8Observer8 10.03.2026
Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:
Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE
8Observer8 10.03.2026
Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .
Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки
Neotwalker 08.03.2026
Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .
Модульная разработка через nuget packages
DevAlt 07.03.2026
Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F#
DevAlt 06.03.2026
В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .
Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование
8Observer8 05.03.2026
Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru