Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.67/6: Рейтинг темы: голосов - 6, средняя оценка - 4.67
0 / 0 / 0
Регистрация: 08.09.2014
Сообщений: 88
1

Конструкторы и деструктор

23.10.2015, 14:39. Показов 1138. Ответов 6
Метки нет (Все метки)

Дорогие форумчане! Need help!

Создаю класс array.
Создаю различные конструкторы.
Плюс, есть исключения, но они сейчас не важны.
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
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
#include <iostream>
#include <conio.h>
#include <algorithm>
#include <iterator>
 
using namespace std;
//--------------Класс array------------
class array
{
 public:
 typedef unsigned int UINT;
 array(UINT size = 10, double k = 0.0);
 array(int l, int r, double k = 0.0);
 array(const array &a);
 array(const array &a, int l, int r);
 array(const double * const begin, const double * const end);
 ~array();
 
 double& operator[] (int index);
 double& operator[] (int index) const;
 array& operator=(const array &a);
 array& assign(const array &a, int left, int right);
 array& assign(const double *begin, const double * end);
 array operator+=(const double &a);
 array operator-=(const double &a);
 array operator*=(const double &a);
 array operator/=(const double &a);
 
 int Left() const;
 int Right() const;
 UINT size() const
 {return size_array;}
 
 double& max_value();
 double& min_value();
 double summa();
 double product();
 void operator~();
 double* find_value(double a);
 
 double* get_data()
 {
 double* p = data;
 return p;
 }
 
 void Swap(array &other)
{ swap(data, other.data);
  swap(left, other.left);
  swap(right, other.right);
  swap(size_array, other.size_array);
}
 friend array operator+(const array&a, const array &b);
 friend array operator*(const array&a, const array &b);
 friend double product_inner(const array&a, const array &b);
 friend ostream& operator << (ostream& t, const array&a);
 friend istream& operator >> (istream& t, array& a);
 
 
 struct bad_Range
 { const char *message;
   bad_Range(const char *s): message(s) {}
 };
 
 struct bad_Size
 { const char *message;
   bad_Size(const char *s): message(s) {}
 };
 
 struct bad_Index
 { const char *message;
   bad_Index(const char *s): message(s) {}
 };
 
 private:
 double* data;
 double dummy;
 int left, right;
 UINT size_array;
 
};
 
//-----------------Конструкторы----------
array::array(UINT size, double k)
{ if (size > 0)
  { left = 0;
    right = size - 1;
    size_array = size;
    data = new double[size_array];
    fill_n(data, size_array, k);
    cout << "constr (UINT size, double k) " << *this << endl;
  }
  else throw new bad_Size("Size is wrong!");
}
 
array::array(int l, int r, double k)
{ if (l < r)
  { size_array = (r - l) + 1;
    left = l;
    right = r;
    data = new double[size_array];
    fill_n(data, size_array, k);
    cout << "constr (int l, int r, double k)" << *this << endl;
  }
  else throw new bad_Range("Range is wrong!");
}
 
array::array(const array &a)
{ data = a.data;
  left = a.left;
  right = a.right;
  size_array = a.size_array;
  cout << "constr (const array &a) " << *this << endl;
}
 
array::array(const array &a, int l, int r)
{ if ((l < r) && (a.left <= l) && (r <= a.right))
   {
    left = l;
    right = r;
    size_array = (right - left) + 1;
    data = new double[size_array];
    copy(a.data + (l - a.left),
         a.data + (l - a.left) + size_array, data);
   cout << "constr (const array &a, int l, int r) " << *this << endl;
   }
   else throw new bad_Range("Range is wrong!");
}
 
array::array(const double * const begin, const double * const end)
{ if (begin < end)
   {
    size_array = (end - begin);
    left = 0; right = size_array - 1;
    data = new double[size_array];
    copy(begin, end, data);
    cout << "constr (const double *begin, const double * end) " << *this << endl;
   }
   else throw new bad_Range("Range is wrong!");
}
 
//-----------------Деструктор----------
array::~array()
{cout << "destr " << *this << endl;
 delete[] data;
 data = 0;
}
//-----------------Перегрузка операций----------
double& array::operator[] (int index)
{ if (left <= index && index <= right) return data[index];
  else throw new bad_Index("Index is wrong!");
}
 
array& array:: assign(const array &a, int left, int right)
{ array temp(a, left, right);
  Swap(temp);
  return *this;
}
 
 
array& array:: assign(const double *begin, const double * end)
{ array temp(begin, end);
  Swap(temp);
  return *this;
}
     
array& array::operator=(const array &a)
{ cout << "= beginning " << a << endl;
  if (this != &a)
  { double *new_data = new double[a.size_array];
    copy(a.data, a.data + a.size_array, new_data);
    delete[] data;
    data = new_data;
    left = a.left;
    right = a.right;
    size_array = a.size_array;
    cout << "= end " << *this << endl;
  }
  return *this;
}
 
double& array::operator[] (int index) const
{ if (left <= index && index <= right)
   return data[index];
   else throw new bad_Index("Index is wrong!");
}
 
 
void array::operator~()
{ sort(data, data + size_array); }
 
double& array::min_value()
{ return *min_element(data, data + size_array); }
 
double *array::find_value(double v)
{ double *i = find(data, data + size_array, v);
  if (i != data + size_array) return i;
  else return 0;
}
 
array operator*(const array &a, const array &b)
{
  if (a.size() == b.size())
  { array z(a.size());
    for (int i = 0; i < a.size(); ++i)
    z[i] = a[i]*b[i];
    cout << "*" << z << endl;
    return z;
  }
 
  else throw new array::bad_Size("Size is wrong!");
}
 
ostream& operator <<(ostream& to, const array &a)
{ ostream_iterator <double, char> out(to, " ");
  copy(a.data, a.data + a.size_array, out);
  return to;
}
 
//-----------------Функция main----------
int main(void)
{
double a[] = { 2, 2, 2, 2, 2, 2, 2, 2, 2 };
double a1[] = { 1, 2, 3, 4, 5, 6, 7, 8, 9 };
array B(a, a + (sizeof(a)/sizeof(double)));
array G(B);
array C(a1, a1 + (sizeof(a1)/sizeof(double)));
 
cout << "This is B: " << B << endl;
cout << "This is G: " << G << endl;
cout << "This is C: " << C << endl;
 
cout << B*G << endl;
 
cout << "This is C: " << C << endl;
 
C = B*G;
cout << "This is C: " << C << endl;
 
getch();
return 0;
}
Всё идёт хорошо до 237 строки (если запустить программу без операции умножения).
Если же хочу выполнить операцию умножения, то, вроде бы, элементы перемножаются, но на месте первого возникает какая левая цифра. Плюс, в момент вызова деструктора возникает какая-то ошибка - "...exeption...Access violation at adress..."
__________________
Помощь в написании контрольных, курсовых и дипломных работ здесь
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
23.10.2015, 14:39
Ответы с готовыми решениями:

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

Конструкторы и деструктор
Здравствуйте. Мне нужно написать маленькую и простую программу, имеющую конструкторы:...

Определить конструкторы, деструктор
Добрый день. Времени в обрез, нужна помощь доработать код. Задание: Создать класс...

Дописать в класс конструкторы и деструктор
\Дописать в класс: конструктор, считывающий массив и его размерность с клавиатуры, конструктор,...

6
Don't worry, be happy
17208 / 10085 / 1946
Регистрация: 27.09.2012
Сообщений: 25,165
Записей в блоге: 1
23.10.2015, 14:45 2
Лучший ответ Сообщение было отмечено avovana как решение

Решение

Конструктор копирования сделайте нормально.
Сейчас у Вас нет "глубокого" копирования.
1
0 / 0 / 0
Регистрация: 08.09.2014
Сообщений: 88
23.10.2015, 16:11  [ТС] 3
Цитата Сообщение от Croessmah Посмотреть сообщение
Конструктор копирования сделайте нормально.
Сейчас у Вас нет "глубокого" копирования.
Сейчас создаются какие-то дополнительные объекты при присваивание. Не пойму в чем суть
0
В астрале
Эксперт С++
8035 / 4792 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
23.10.2015, 16:18 4
Лучший ответ Сообщение было отмечено avovana как решение

Решение

Конструктор копии.
C++
1
2
3
4
5
array::array(const array &a) : 
data(new double[a.size_array]), left(a.left), right(a.right), size_array(a.size_array)
{
   copy(a.data, a.data + a.size_array, data);
}
Операция swap
C++
1
2
3
4
5
6
7
void array::swap(array& a)
{
   std::swap(a.data, data);
   std::swap(a.left, left);
   std::swap(a.right, right);
   std::swap(a.size_array, size_array);
}
Оператор присваивания
C++
1
2
3
4
5
6
array& operator =(const array& a)
{
    array tmp(a);
    swap(tmp);
    return *this;
}
1
0 / 0 / 0
Регистрация: 08.09.2014
Сообщений: 88
23.10.2015, 17:08  [ТС] 5
Цитата Сообщение от Croessmah Посмотреть сообщение
Конструктор копирования сделайте нормально.
Сейчас у Вас нет "глубокого" копирования.
Начал думать в это направление...
ForEveR, спасибо! всё работает!
Получается при конструирование нового объекта нужно явно выделить участок памяти для data?
Только пока не пойму, почему до момента использования умножения всё и так работало.
0
В астрале
Эксперт С++
8035 / 4792 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
23.10.2015, 17:11 6
avovana, Суть не в умножении, суть в использовании конструктора копии и оператора присваивания. У вас они работали неверно.
0
0 / 0 / 0
Регистрация: 08.09.2014
Сообщений: 88
23.10.2015, 17:30  [ТС] 7
Теперь я вижу.
Интересный момент.
При вхождение в операцию умножения, там создаём array (вызывается конструктор) для вычисления произведения. Ок.
При return этот array - z - уничтожается (вызывается деструктор).

После этого компилятор переходит к операции присваивания. Т.е. здесь он должен взять тот самый z - то, что вернула операция умножения. Но я же видел как вызвался деструктор для его уничтожения!

По идее return, но и тут же деструктор! КАК ТАК!
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
23.10.2015, 17:30

Конструкторы и деструктор для списка на C++
Помогите, пожалуйста, не понимаю почему конструктор с параметрами и деструктор не работают. В...

Конструкторы и деструктор класса BigInteger
Задание выглядит так. Определить класс BigInteger в файле BigInteger.h, который может быть...

Добавить конструкторы и деструктор struct student
При решении задачи требуется реализовать тип данных struct, соответствующий варианту...

Разработать конструкторы и деструктор для заданного класса
разработать конструкторы и деструктор для заданного класса. Осуществить инициализацию объектов...


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

Или воспользуйтесь поиском по форуму:
7
Ответ Создать тему
Опции темы

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