Моя программа работает с ошибками перегруженных операторов

Заголовочный файл

#include <iostream>
#include <iomanip>
#include <cstring>
#include <cctype>
using namespace std;

class MyInt
{
friend ostream& operator<< (ostream& out, const MyInt& myint);
friend istream& operator>> (istream& in, MyInt& myint);
friend istream& getline (istream& , MyInt& , char delim = '\n');

friend MyInt operator+ (const MyInt& x, const MyInt& y);
friend MyInt operator* (const MyInt& x, const MyInt& y);

friend MyInt operator++(MyInt& x, int y);     //post-increment ++
friend MyInt operator++(MyInt& x);            //pre-increment ++
friend MyInt operator--(MyInt& x);            //pre-decrement --
friend MyInt operator--(MyInt& x, int y);     //post-decrement --

friend bool operator< (const MyInt& x, const MyInt& y);
friend bool operator> (const MyInt& x, const MyInt& y);
friend bool operator<=(const MyInt& x, const MyInt& y);
friend bool operator>=(const MyInt& x, const MyInt& y);
friend bool operator==(const MyInt& x, const MyInt& y);
friend bool operator!=(const MyInt& x, const MyInt& y);
public:
MyInt(int n=0);                           // default constructor
MyInt(const char* number);               // c-style string with digits

~MyInt();                          // destructor

MyInt(const MyInt& );           // copy constructor
MyInt& operator=(const MyInt& );   // assignment operator
//MyInt& operator+(const MyInt& x);
//MyInt& operator*(const MyInt& x);
int getLength() const;                // return int lengthprivate:

int current_size;
int actual_number;
int max_size;
char * IntList;
};

Декларация класса

    #include <iostream>
#include <iomanip>
#include <cstring>
#include <cctype>
#include "myint.h"using namespace std;

// friend functions (operatior overloads)

ostream& operator << (ostream& out, const MyInt& myint){
for(int i = 0; i < myint.current_size; i ++)
{
out << myint.IntList[i];
}
return out;
}
istream& operator >> (istream& in, MyInt& myint){
int input;
in >> input;
cout << "Input: " << input << endl;
int digits = 10;
myint.current_size = 1;
myint.IntList = new char[myint.current_size];
myint.IntList[0] = '0';

myint.actual_number = input;
if(input >= 0 ){
int length = 0;
for(int x = input; x!=0; x /= 10)
length++;

myint.current_size = length;
for(int i = 0; i < length-2; i++)
digits *= 10;
myint.IntList = new char[myint.current_size];

myint.IntList[0] = '0' + (input/digits);
for(int i = length-1; i > 0; i--){
myint.IntList[i] = '0' + (input % 10);
input/=10;
}}

return in;
}MyInt::MyInt(int n){
actual_number = 0;
int digits = 10;
current_size = 1;
IntList = new char[current_size];
IntList[0] = '0';

if(n >= 0 ){
actual_number = n;
int length = 0;
for(int x = n; x!=0; x /= 10)
length++;

current_size = length;
for(int i = 0; i < length-2; i++)
digits *= 10;
IntList = new char[current_size];

IntList[0] = '0' + (n/digits);
for(int i = length-1; i > 0; i--){
IntList[i] = '0' + (n % 10);
n/=10;
}}}

MyInt::MyInt(const char* num){            // conversion from c-string

current_size = strlen(num);
IntList = new char[current_size];
int count = 0;
int size = 0;
int j = 0;
while(isdigit(num[j])){
count++;
j++;
}
for(int i = 0; i <count; i++)
IntList[i] = num[i];
sscanf(IntList, "%d", actual_number);
}

MyInt::~MyInt(){

delete [] IntList;              //deallocate memory
}

MyInt::MyInt(const MyInt& copy_int){
max_size = copy_int.max_size;
current_size = copy_int.current_size;for(int i = 0; i < current_size; i++)
IntList[i] = copy_int.IntList[i];
}

MyInt operator+ (const MyInt& x, const MyInt& y){
cout << "Adding Ints: " << endl;
MyInt r;
r.actual_number = y.actual_number + x.actual_number;
return r;
}
MyInt operator* (const MyInt& x, const MyInt& y){
cout << "Multiplying Ints: " << endl;
MyInt r;
r.actual_number = y.actual_number * x.actual_number;
return r;

}

bool operator< (const MyInt& x, const MyInt& y){
if(x.actual_number < y.actual_number)
return true;
else
return false;
}
bool operator> (const MyInt& x, const MyInt& y){
cout << "x: " << x.actual_number << "y:" << y.actual_number << endl;
if(x.actual_number > y.actual_number)
return true;
else
return false;
}
bool operator<= (const MyInt& x, const MyInt& y){
if(x.actual_number <= y.actual_number)
return true;
else
return false;
}
bool operator>= (const MyInt& x, const MyInt& y){
if(x.actual_number >= y.actual_number)
return true;
else
return false;
}
bool operator==(const MyInt& x, const MyInt& y){
if(x.actual_number == y.actual_number)
return true;
else
return false;
}
bool operator!=(const MyInt& x, const MyInt& y){
if(x.actual_number != y.actual_number)
return true;
else
return false;
}MyInt& MyInt::operator=(const MyInt& assign_int ){
if(this != &assign_int)
{
delete [] IntList;
this->actual_number = assign_int.actual_number;
cout << "actual num: " << this->actual_number << endl;

int digits = 10;
current_size = 1;
IntList = new char[current_size];
IntList[0] = '0';

if(actual_number >= 0 ){
int length = 0;
for(int x = actual_number; x!=0; x /= 10)
length++;

current_size = length;
for(int i = 0; i < length-2; i++)
digits *= 10;
IntList = new char[current_size];

IntList[0] = '0' + (actual_number/digits);
for(int i = length-1; i > 0; i--){
IntList[i] = '0' + (actual_number % 10);
actual_number/=10;
}
}

}
return *this;

}

Проблема в моих перегруженных операторах + и *. Я перепробовал все, что мог придумать. Я получаю сообщение об ошибке «Ошибка сегментации (ядро сброшено)» или «Ошибка шины» (ядро сброшено). Я знаю, что это просто то, чего я не вижу, и любая помощь приветствуется !! Спасибо!

1

Решение

Единственная причина, по которой вы видите проблемы с вашим перегруженным operator+() а также operator*() является то, что они возвращаются по значению.

Возврат по значению означает создание копии того, что возвращается. Эта копия будет возвращена вызывающей стороне и (как правило) уничтожена в конце концов.

Фактическая проблема в вашем конструкторе копирования

MyInt::MyInt(const MyInt& copy_int){
max_size = copy_int.max_size;
current_size = copy_int.current_size;for(int i = 0; i < current_size; i++)
IntList[i] = copy_int.IntList[i];
}

который не инициализирует IntList указать на что-либо, прежде чем назначить IntList[i] в петле. Результатом этого является неопределенное поведение.

Так как значение, возвращаемое вашими функциями, в конце концов перестанет существовать, будет вызван его деструктор.

MyInt::~MyInt(){

delete [] IntList;              //deallocate memory
}

который также потому, что IntList не был инициализирован, вызывает неопределенное поведение.

Большой совет: оба ваших других конструктора имеют утверждение IntList = new char[current_size];, Логика может предположить, что конструктору копирования нужно нечто подобное.

0

Другие решения

Выделите память для IntList в вашем конструкторе копирования.

MyInt::MyInt(const MyInt& copy_int){
max_size = copy_int.max_size;
current_size = copy_int.current_size;

for(int i = 0; i < current_size; i++)
IntList[i] = copy_int.IntList[i];
}

Ваш код в operator+ а также operator* создает новый объект r чьи члены не инициализированы должным образом, а также в вашем конструкторе копирования члены не копируются должным образом.

0

По вопросам рекламы [email protected]