конструктор копирования — Как написать Rational Fractions с использованием классов в переполнении стека

Я должен реализовать Rational класс для того, чтобы получить рациональные дроби. header.h Файл предоставлен моим инструктором, поэтому я должен продолжить. Я также должен написать конструктор копирования в Rational::Rational(const Rational& cRational) функция так object можно скопировать. Я написал свой код, но добавление дробей в вывод неверно. Кто-нибудь может помочь мне понять это? Что не так с моим кодированием, особенно в Rational::addition(const Rational &a) или как я могу это исправить?

выход :

 Begin Rational Class Tests

Testing default constructor: 1/1

Testing std constructor: 1/2

Testing copy constructor: 1/2

Testing addition: 4/4 + 1/2 = 4/4    // it should be 1/2 + 1/2 = 4/4

основная функция:

int main()
{
cout << "Begin Rational Class Tests\n\n";

cout<<"Testing default constructor: ";
Rational n1;
n1.printRational();
cout << endl << endl;

cout<<"Testing std constructor: ";
Rational n2(1,2);
n2.printRational();
cout << endl << endl;

cout<<"Testing copy constructor: ";
Rational n3(n2);
n3.printRational();
cout << endl << endl;

cout<<"Testing addition: ";
n1 = n2.addition(n3);
n2.printRational();
cout <<" + ";
n3.printRational();
cout <<" = ";
n1.printRational();
cout << endl << endl;
}

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

class Rational {
public:
Rational();  // default constructor
Rational(int, int); //std (initialisation) constructor
Rational(const Rational&); //copy constructor
Rational addition(const Rational &);
void printRational();
private:
int numerator;
int denominator;
};

моя программа:

//default constructor
Rational::Rational()
{
numerator = 1;
denominator = 1;
}
//initialize constructor
Rational::Rational(int n, int d)
{
numerator = n;
if (d==0)
{
cout << "ERROR: ATTEMPTING TO DIVIDE BY ZERO" << endl;
exit(0); // will terminate the program if division by 0 is attempted
}
else
denominator = d;
}
//copy constructor
Rational::Rational(const Rational& cRational)
{
numerator = cRational.numerator;
denominator = cRational.denominator;
}
//addition
Rational Rational::addition(const Rational &a)
{
numerator = numerator * a.denominator + a.numerator * denominator;
denominator = denominator * a.denominator;
return Rational(numerator,denominator);
}

void Rational::printRational()
{
cout << numerator << "/" << denominator ;
}

-2

Решение

Ваша функция добавления модифицирует переменные-члены *this, который вызывает странные вещи, чтобы происходить.
Более подходящим прототипом будет

Rational addition(const Rational &) const;

так как это позволит компилятору сказать вам, что вы делаете что-то странное.

Вы можете либо использовать локальные переменные вместо назначения членам, либо полностью обходиться без промежуточных переменных:

Rational Rational::addition(const Rational &a)
{
return Rational(numerator * a.denominator + a.numerator * denominator,
denominator * a.denominator);
}
1

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

Просто создайте новые переменные с разными именами для новых значений числителя и знаменателя в дополнение (), примерно так …

int n, d;  //you could use better names
n = numerator * a.denominator + a.numerator * denominator;
d = denominator * a.denominator;
return Rational(n, d);

Я проверил, это работает Вот.

В общем, не используйте одинаковые имена для двух переменных в одной и той же области видимости.

1

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