Я должен реализовать 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 ;
}
Ваша функция добавления модифицирует переменные-члены *this
, который вызывает странные вещи, чтобы происходить.
Более подходящим прототипом будет
Rational addition(const Rational &) const;
так как это позволит компилятору сказать вам, что вы делаете что-то странное.
Вы можете либо использовать локальные переменные вместо назначения членам, либо полностью обходиться без промежуточных переменных:
Rational Rational::addition(const Rational &a)
{
return Rational(numerator * a.denominator + a.numerator * denominator,
denominator * a.denominator);
}
Просто создайте новые переменные с разными именами для новых значений числителя и знаменателя в дополнение (), примерно так …
int n, d; //you could use better names
n = numerator * a.denominator + a.numerator * denominator;
d = denominator * a.denominator;
return Rational(n, d);
Я проверил, это работает Вот.
В общем, не используйте одинаковые имена для двух переменных в одной и той же области видимости.