Добавление двух векторов с использованием перегрузки оператора

Question

У меня есть класс Vectors, который имеет частный динамический массив. Все, что я хотел сделать, это добавить два объекта Vectors, таких как A = A + B, но программа продолжает сбой.

Это заявление моего класса:

class Vectors 
    { 
    private: 
     int* vector; 
    public: 
     Vectors(int); 
     Vectors(Vectors&); 
     ~Vectors(); 

     Vectors operator+(Vectors&); 

    }; 

Это моя реализация:

#include "Vectors.h" 
#include "iostream" 
using namespace std; 

Vectors::Vectors(int value) 
{ 
    this->vector = new int[3]; 
    for (auto i = 0; i < 3; i++) 
    { 
     vector[i] = 3; 
    } 
} 

Vectors::Vectors(Vectorsy& copy) 
{ 
    this->vector = new int[3]; 
    for (auto i = 0; i < 3; i++) 
    { 
     vector[i] = copy.vector[i]; 
    } 
} 

Vectors::~Vectors() 
{ 
    delete[] vector; 
} 

Vectors Vectors::operator+(Vectors& obj) // There is sth wrong here. 
{ 
    for (auto i = 0; i < 3; i++) 
     this->vector[i] += obj.vector[i]; 

    return *this; 
} 

Это ошибка, я получаю:

Answer 1

Я считаю, что вам нужно функция operator=. Вы не реализовали его, поэтому компилятор записывает файл по умолчанию, который делает неправильную вещь (из-за того, что класс имеет указатель). Скорее всего, сбой происходит при одновременном удалении одной и той же памяти.

См What is The Rule of Three?

Answer 2

Проблема была копия Конструктор (Vectors :: Векторы (Const Векторы & копия) Услышать рабочий код.

#include "iostream" 
using namespace std; 

class Vectors 
    { 
    private: 
     int* vector; 
    public: 
     Vectors(int); 
     Vectors(const Vectors&); 
     ~Vectors(); 

     Vectors operator+(Vectors&); 

    }; 


Vectors::Vectors(int value) 
{ 
    this->vector = new int[3]; 
    for (auto i = 0; i < 3; i++) 
    { 
     vector[i] = 3; 
    } 
} 

Vectors::Vectors(const Vectors& copy) 
{ 
    this->vector = new int[3]; 
    for (auto i = 0; i < 3; i++) 
    { 
     vector[i] = copy.vector[i]; 
    } 
} 

Vectors::~Vectors() 
{ 
    delete[] vector; 
} 

Vectors Vectors::operator+(Vectors& obj) // There is sth wrong here. 
{ 
    for (auto i = 0; i < 3; i++) 
     this->vector[i] += obj.vector[i]; 

    return *this; 
} 

int main() 
{ 
     Vectors A(3), B(3); 
     Vectors C = A+B; 
}