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

@lFoster · Регистрация: 15.10.2015

Author24 — интернет-сервис помощи студентам

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

Vector.h

C++

#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++

#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++

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();*/
 
 
 
}

Буду благодарен, если поможете указать на ошибку.

Вернуться к обсуждению:
Vector не до конца понял Конструктор копирования C++

Следующий ответ

Другие темы раздела
C++ Error C2039: 'Sth' : is not a member of 'std::vector<_Ty>' https://www.cyberforum.ru/ cpp-beginners/ thread1844377.html #include <iostream> #include <vector> using std::cin; using std::cout; using std::endl; using namespace std; struct MyVectorClientsType { char Letter; vector<int> MyVectorQueue;	SMTP С++ отправка сообщения на почту C++ Ребят, такая задача. Нужно отправить сообщение на почту. программа должна это делать в фотоном режиме, то есть логин и пароль указываются в коде. Я обшарил кучу тем. Толком ничего для решения своей задачи не нашёл. Не прошу написать это за меня, просто укажите направления поиска. Буду очень признателен))))
C++ Для каждого значения A вывести все значения функции попдающие в интервал от C до D Мне дали вот такое условие задачи: Y=f(x; a) X1<=x<=x2, x3 A1<=a<=a2, a3 Для каждого значения A вывести все значения функции попдающие в интервал от C до D. Надо решить в c++ . Пожалуйста помогите https://www.cyberforum.ru/ cpp-beginners/ thread1844368.html	*C++ Правильное объявление переменной с точки зрения стиля [int a vs int* a]** И всё таки, как корректней объявлять переменные? 1. int* a, int& a, int** a или 2. int a, int &a, int *a ? https://www.cyberforum.ru/ cpp-beginners/ thread1844366.html
В заданном массиве целых чисел найти числа, оканчивающиеся на указанные цифры C++ Дан статистический одномерный массив целых чисел. Найти числа оканчивающиеся на 1 и на 0, n-дано(вводим с клавиатуры)	C++ Применение цикла if для определения простых чисел. If внутри цикла for https://www.cyberforum.ru/ cpp-beginners/ thread1844349.html Доброго времени суток, подскажите пожалуйста, в чем тут дело. С кодом все хорошо, все работает, просто я пытаюсь докопаться до сути и почему-то не "обмозговывается". Недавно начал учить с++, учу по книге Лафоре. Работаю в codeblock. И так вот код и далее вопрос: (сначала приведу два примера а сам вопрос будет в третьем примере) #include <iostream> using namespace std; #include <stdlib.h>...
C++ В заданном массиве найти числа, остаток деления на M которых принадлежит заданному интервалу Пожалуйста, помогите решить задачу с одномерным массивом - Даны действительные числа а1, а2, ... , аn. Указать те из них, у которых остаток от деления на M равен L (0<L<M-1) https://www.cyberforum.ru/ cpp-beginners/ thread1844328.html	C++ Записать в предложении символы длинного слова (всех длинных, если их несколько) в обратном порядке Поиогите пожалуйста не получается та часть кода где нужно найти и записать в обратном порядке слова. #include <stdio.h> #include <iostream> #include <cctype> #include <string> #include <cstring> using namespace std; int main() { int max(0),maxd(0);
C++ Принудительная запись значения в ячейку памяти Есть ячейка памяти используемая некой программой . Как внешне изменить значение этой ячейки по адресу ... При попытке запуска программы int main() { int i =(int) 0x0240210C; *i = 99000; return 0; } постоянно выдаёт ошибку https://www.cyberforum.ru/ cpp-beginners/ thread1844288.html	C++ Использование стандартной библиотеки С. Char и опции консольного приложения Всем привет. Сломал уже голову над тем, как можно в С++ из char вытащить число. В char передаются аргументы при запуске консольного приложения. При этом один аргумент (здесь опция) имеет вид: -b=N. Где N -- любое число. В переменную b заносится N. В случае передачи в командной строке не предусмотренных опций программа должна сообщать об ошибке. https://www.cyberforum.ru/ cpp-beginners/ thread1844280.html
Как оформить Direct2D приложение библиотекой? C++ Нужно использовать одну из функций Direct2D (размытие картинок) в wpf c# приложении Как можно оформить все это в качестве dll которую я смогу подключить к шарпу? dll должна делать лишь одну вещь, я отправляю ей картинку - она возвращает мне размытую картинку	C++ Return value; https://www.cyberforum.ru/ cpp-beginners/ thread1844253.html Уважаемые форумчане, подскажите, если объект имеет тип fstream, то при ошибке какое значения я должен передать в return???

	07.11.2016, 11:07