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

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

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

Vector не до конца понял Конструктор копирования - C++

07.11.2016, 11:07. Просмотров 259. Ответов 5
Метки нет (Все метки)

Доброе утро, господа.
У меня проблема, написал класс вектор, вроде все методы правильные, но проблема в том, что деструктор повторно удаляет, то, что только что удалил, и из-за этого падает программа. Понимаю, что проблема кроется в конструкторе копирования, но все никак не могу вникнуть, что мне сделать. Собственно, сам код программы:


Vector.h
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
#pragma once
#include <iostream>
 
using namespace std;
 
 
class Vector
{
    int *vect; // Вектор
    int size; // Размерность вектора
public:
    Vector()
    {
        size = 0;
        vect = new int[size];
        
 
    }
 
    explicit Vector(int size)
    {
 
        this->size = size;
        
 
    }// Конструктор, позволяющий задавать размерность вектора,
    // но не являющийся конструктором преобразования
    Vector(const int * ptr, int size) // Конструктор, позволяющий проинициализировать вектор
        // с помощью существующего массива
    {
        this->size=size;
        vect = new int[size];
        for (int i = 0; i < size; i++)
            vect[i] = ptr[i];
 
 
    }
    Vector(const Vector& v) // Конструктор копирования
    {
 
        size = v.size;
        vect = new int;
        for (int i = 0; i < size; i++)
        {
            *vect = *v.vect;
            cout << "Copy-Constructor" << endl;
        }
    }
    ~Vector()
    {
        
            cout << "Destructor" << endl;
            delete[]vect;
        
    }
 
 
 
    Vector& operator ++ ();// Увеличение всех компонент вектора на единицу (префикс)
    Vector operator ++ (int k);// Увеличение всех компонент вектора на единицу (постфикс)
    Vector& operator -- (); // Уменьшение всех компонент вектора на единицу (префикс)
    Vector operator -- (int k); // Уменьшение всех компонент вектора на единицу (постфикс)
    Vector& operator = (const Vector& v); // Перегруженный оператор присваивания
    int& operator [] (int index); // Перегруженный оператор индексации
    Vector operator + (const Vector& v); // Сложение двух векторов
    Vector operator + (int n); // Сложение вектора с числом (каждый компонент вектора
    // складывается с числом)
    Vector operator - (const Vector& v);
    Vector operator - (int n); // Вычитание числа из вектора
    Vector& operator -= (const Vector& v); // Перегруженный оператор -= для вычитания двух векторов
    Vector& operator += (const Vector& v);
    Vector operator * (const Vector& v); // Умножение векторов
    Vector operator * (int n); // Умножение вектора на число
    Vector& operator *= (int n); // Перегруженный оператор *= для умножения вектора на число
    friend istream& operator >> (istream& cin, Vector& v); // Ввод вектора с клавиатуры
    friend ostream& operator << (ostream& cout, Vector& v); // Вывод вектора на экран
    int GetSize() const; // Функция возвращает размерность вектора
    void Print(); // Вывод на экран
    void Input(); // Заполнение вектора
    void Clear(); // Очистка вектора
    bool IsEmpty() const; // Возвращает истину, если вектор пустой
    void Add(const int& item); // Добавление элемента в вектор
    void Insert(int index, const int& item); // Вставка элементов в вектор по заданному индексу
    void Remove(int index); // Удаление элемента из вектора по заданному индексу
};
Vector.cpp
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
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
#include <iostream>
#include "Vector.h"
 
using namespace std;
 
 
Vector& Vector::operator ++ ()
{
    for (int i = 0; i < this->size; i++)
    {
        this->vect[i]++;
        
    }
    return *this;
}
Vector Vector::operator ++ (int)
{
    Vector tmp = *this;
    for (int i = 0; i <this->size; i++)
        this->vect[i]++;
    return tmp;
}
 
Vector& Vector::operator -- ()
{
    for (int i = 0; i < this->size; i++)
    {
        this->vect[i]--;
        
    }
    return *this;
}
Vector Vector::operator -- (int )
{
    Vector tmp = *this;
    for (int i = 0; i <this->size; i++)
    {
        
        this->vect[i]--;
    }
    return tmp;
}
Vector& Vector::operator = (const Vector& v)
{
    if (this == &v)
        return *this;
    delete[]this->vect;
    this->vect = new int[v.size];
    this->size = v.size;
    for (int i = 0; i < this->size; i++)
    {
        this->vect[i] = v.vect[i];
    }
    return *this;
 
    /*if (this == &v)
        return *this;
    size = v.size;
        return *this;*/
}
 
 
int& Vector::operator [] (int index)
{
    if (index < 0 || index >= size)
        return vect[0];
    return vect[index];
 
}
Vector Vector::operator + (const Vector& v)
{
    if (this->size > v.size){
        for (int i = 0; i < v.size; i++)
        {
            this->vect[i] += v.vect[i];
        }
        return *this;
    }
    else
    {
        for (int i = 0; i < v.size; i++)
        {
            v.vect[i] += this->vect[i];
        }
        return *this;
    }
 
}
 
Vector Vector:: operator + (int n)
{
    for (int i = 0; i < this->size; i++)
    {
        this->vect[i] += n;
    }
    return *this;
}
 
Vector Vector::operator - (const Vector& v)
{
    if (this->size > v.size)
    {
        for (int i = 0; i < v.size; i++)
        {
            this->vect[i] -= v.vect[i];
        }
        return *this;
    }
    else
    {
        for (int i = 0; i < v.size; i++)
        {
            v.vect[i] -= this->vect[i];
        }
        return *this;
    }
}
Vector& Vector::operator += (const Vector& v)
{
    for (int i = 0; i<this->size; i++)
    {
        this->vect[i] += v.vect[i];
    }
    return *this;
}
Vector Vector::operator - (int n)
{
    for (int i = 0; i < this->size; i++)
    {
        this->vect[i] += n;
    }
    return *this;
}
Vector& Vector::operator -= (const Vector& v)
{
    for (int i = 0; i<this->size; i++)
    {
        this->vect[i] -= v.vect[i];
    }
    return *this;
 
}
Vector Vector::operator * (const Vector& v)
{
    Vector res;
    for (int i = 0; i<this->size; i++)
    {
        res.vect[i] *= v.vect[i];
    }
    return res;
 
}
Vector Vector::operator * (int n)
{
    for (int i = 0; i<this->size; i++)
    {
        this->vect[i] *= n;
    }
    return *this;
 
}
Vector& Vector::operator *= (int n)
{
    
 
    for (int i = 0; i<this->size; i++)
    {
        this->vect[i] *= n;
    }
    return *this;
}
 
istream&  operator >> (istream& cin,  Vector& v)
{
    for (int i = 0; i < v.size;i++)
        cin >> v.vect[i];
    
    return cin;
}
ostream& operator << (ostream& cout, Vector& v)
{
 
    for (int i = 0; i < v.size; i++)
        cout << v.vect[i];
 
    return cout;
 
}
int Vector::GetSize() const
{
    return this->size;
}
void Vector::Print()
{
    for (int i = 0; i < this->size; i++)
        cout << vect[i] << ' ' ;
    cout << endl;
 
 
}
void Vector::Input()
{
    cout << "Enter size  vector: " << endl;
    cin >> size;
    for (int i = 0; i < this->size; i++)
    {
        cout << "Enter element: " << endl;
        cin >> this->vect[i];
    }
}
void Vector::Clear()
{
    
    delete[]vect;
    vect = 0;
    size = 0;
 
}
bool Vector::IsEmpty() const
{
    if (this->size == 0)
        return true;
 
}
void Vector::Add(const int& item)
{
    int * vect2 = new int[size];
    for (int i = 0; i < size; i++)
        vect2[i] = vect[i];
    delete []vect;
    size++;
    vect = new int[size];
    for (int i = 0; i<size - 1; i++)
    {
        vect[i] = vect2[i];
    }
    vect[size - 1] = item;
    delete[] vect2;
 
 
}
void Vector::Insert(int index, const int& item)
{
    int *vect2 = new int[size + 1];
    for (int i = 0; i < index; i++)
        vect2[i] = vect[i];
    vect2[index] = item;
    for (int i = index + 1; i < size + 1; i++)
        vect2[i] = vect[i - 1];
    delete[]vect;
    size++;
    vect = new int[size];
    for (int i = 0; i < size; i++)
        vect[i] = vect2[i];
    delete[]vect2;
 
}
void Vector::Remove(int index)
{
    int *vect2 = new int[size - 1];
    for (int i = 0; i < index; i++)
        vect2[i] = vect[i];
    for (int i = index; i < size - 1; i++)
        vect2[i] = vect[i + 1];
    delete[]vect;
    size--;
    vect = new int[size];
    for (int i = 0; i < size; i++)
        vect[i] = vect2[i];
    delete[]vect2;
 
 
}
Main.cpp
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void main()
{
    setlocale(0, "ru");
    Vector v1, v2(5);
    /*cout << "Выбирете ваши действия: " << endl;
    int a; cin >>a;
    cout << ""*/
    Vector v3 = v1;;
    v3.Input();
    v3.Print();
    /*v2++;
    v2.Print();*/
 
 
 
}

Буду благодарен, если поможете указать на ошибку.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.11.2016, 11:07
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Vector не до конца понял Конструктор копирования (C++):

Не могу сделать чтобы класс содержал основной конструктор и конструктор копирования - C++
Разработать класс ThreeAngle для работы с плоскими треугольниками. В качестве членов-данных задаются длины трех сторон треугольника. Класс...

Не могу правильно сделать конструктор и конструктор копирования и принадлежность точки с заданными координатами треугольнику - C++
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;windows.h&gt; #include &lt;math.h&gt; #include &lt;algorithm&gt; using...

Конструктор инициализации, конструктор копирования, деструктор - C++
Я сделал почти задание по перегрузке операторов. Осталось одно, тоесть три: конструктор инициализации, конструктор копирования, деструктор....

Создать класс "Вектор" и реализовать конструктор по умолчанию, конструктор копирования и деструктор - C++
Всем доброго времени суток! нужна ваша помощь! нужно создать класс вектор и реализовать конструктор по умолчанию, копирования и...

Конструктор копирования - C++
Привет Подскажите пожалуйста, как правильно сделать конструктор копирования. Заголовки: class String { public: char *Data; ...

Конструктор копирования - C++
Ответьте пожалуйста какой необходим конструктор копирования для объекта класса который передается в функцию?

5
Кудаив
329 / 406 / 24
Регистрация: 27.05.2012
Сообщений: 1,168
Завершенные тесты: 2
07.11.2016, 12:20 #2
C++
1
2
3
4
5
6
7
8
9
10
11
Vector(const Vector& v) // Конструктор копирования
    {
 
        size = v.size; // перекопировали размер
        vect = new int; // создаем по указателю один единственный int
        for (int i = 0; i < size; i++)
        {
            *vect = *v.vect; // size - 1 раз присваиваем одному int  элемент *v.vect не смещая указатели
            cout << "Copy-Constructor" << endl;
        }
    }
а потом
C++
1
2
3
4
5
6
7
~Vector()
    {
        
            cout << "Destructor" << endl;
            delete[]vect; // удаляем один элемент int с помощью delete[] - undefined behaviour
        
    }
сомневаюсь, что приведенный код способен вывести, что то внятное
0
zss
Модератор
Эксперт С++
6483 / 6046 / 1983
Регистрация: 18.12.2011
Сообщений: 15,676
Завершенные тесты: 1
07.11.2016, 12:25 #3
Цитата Сообщение от lFoster Посмотреть сообщение
explicit Vector(int size) { this->size = size; }// Конструктор, позволяющий задавать размерность вектора,
А где тогда выделяется память под vector?
C++
1
2
3
4
5
6
7
8
9
10
Vector(const Vector& v) // Конструктор копирования
    {
 
        size = v.size;
        vect = new int[size]; 
        for (int i = 0; i < size; i++)
        {
            vect[i] = v.vect[i];
        }
    }
0
lFoster
0 / 0 / 0
Регистрация: 15.10.2015
Сообщений: 49
07.11.2016, 19:25  [ТС] #4
переписал конструктор копирования, но ошибка все равно вылетает...
0
Kant
33 / 33 / 9
Регистрация: 15.05.2013
Сообщений: 236
07.11.2016, 22:07 #5
Тебе тут надо многое переписать.

1. Прочитай идиому copy-and-swap
Надо переписать конструктор копирования, оператор присвоения.

2. Удели внимание Add, Insert, Remove. Ты там совершаешь одну и туже ошибку.

Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
void Vector::Add(const int& item)
{
    int * vect2 = new int[size];
    for (int i = 0; i < size; i++)
        vect2[i] = vect[i];
    delete []vect;
    size++;
 
    // вот это лишние
    //vect = new int[size];
    //for (int i = 0; i<size - 1; i++)
    //{
    //    vect[i] = vect2[i];
    //}
    //vect[size - 1] = item;
    //delete[] vect2;
 
   // просто переставь указатель
   vect2[size] = item;
   vect = vect2;
}


3. Перегруженные операторы с векторами с ошибками.
Кликните здесь для просмотра всего текста

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Vector Vector::operator+(const Vector &v) {
  int passSize = 0;
  if (this->size > v.size) {
    passSize = v.size;
  } else if (this->size < v.size) {
    passSize = this->size;
  } else {
    passSize = this->size;
  }
 
  for (int i = 0; i < passSize; i++) {
    this->vect[i] += v[i];
  }
 
  return *this;
}


4. using namesace std в хедерах не исп.


Еще много разных мелочей. Допустим оператор индексирования:

C++
1
2
3
    // v.vect[i]
   // а надо
   v[i]

Это не все, но начало хорошее Напиши тесты под свой Вектор, так тебе будет проще вносить изменения.
0
lFoster
0 / 0 / 0
Регистрация: 15.10.2015
Сообщений: 49
07.11.2016, 22:29  [ТС] #6
Еще есть над чем работать. Спасибо)
0
07.11.2016, 22:29
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.11.2016, 22:29
Привет! Вот еще темы с ответами:

Конструктор копирования в c++ - C++
Добрый день, такая задача по с++ Какая ошибка в следующей реализации конструктора копирования по умолчанию и деструктора?? Какой еще...

конструктор копирования - C++
не могу понять, почему не срабатывает конструктор копирования. #include &lt;iostream&gt; using namespace std; class Shape { ...

Конструктор копирования - C++
В общем есть вот такой код....МЕНЯТЬ В НЕМ НИЧЕГО НЕЛЬЗЯ - можно только дописывать методы... #include &lt;iostream&gt; #include...

Конструктор копирования - C++
Всем привет. У меня такая проблема: есть некий класс, допустим Test: class Test { protected: int value; ...


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

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

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