无效减少（）；会员功能减少分数

Question

这是我需要编写的成员函数。该功能旨在减少分数。 无效减少（）；

这就是我所拥有的

void reduce()

{
    num /= gcd();
    den /= gcd();
}

提供了gcd函数，所以没有错。

它不允许我为类定义的其余部分包含我的代码，因为它太长了

我的问题是； 为什么我收到 reduce() 错误

我也试过了

void reduce()
// reduce this fraction to simplest form. For instance,
// 2/4 will be reduced to 1/2
{
    int a = gcd();
    den /= a;
    num /= a;
}

我的代码正在测试它：

// Test reduce

f1.reduce(); // f1 is -4/5
f2.reduce(); // f2 is 2/3
if(f1.get_numerator() != -4 || f1.get_denominator() != 5 || f2. 
get_numerator() != 2 || f2.get_denominator() != 3)
{
    cout <<"The reduce function was wrong.\n";
    result -= 0.5;
}

Answer 1

假设gcd()返回的值依赖于num和den，那么第一行gcd()返回的值将不同于第二行gcd()返回的值。

将gcd()的返回值存储在一个局部变量中并使用它。

void reduce()
{
    auto g = gcd();
    num /= g;
    den /= g;
}

---

在你的程序中添加一些调试输出：

if ( f1.get_numerator() != -4 ||
     f1.get_denominator() != 5 ||
     f2.get_numerator() != 2 ||
     f2.get_denominator() != 3)
{
   cout <<"The reduce function was wrong.\n";

   if ( f1.get_numerator() != -4 )
   {
      cout << "Expected value of f1.get_numerator(): " << -4 << "\n";
      cout << "Actual value of f1.get_numerator(): " << f1.get_numerator() << "\n";
   }
   else
   {
      cout << "Got the expected value of f1.get_numerator()\n";
   }

   if ( f1.get_denominator() != 5 )
   {
      cout << "Expected value of f1.get_denominator(): " << 5 << "\n";
      cout << "Actual value of f1.get_denominator(): " << f1.get_denominator() << "\n";
   }
   else
   {
      cout << "Got the expected value of f1.get_denominator()\n";
   }


   if ( f2.get_numerator() != 2 )
   {
      cout << "Expected value of f2.get_numerator(): " << 2 << "\n";
      cout << "Actual value of f2.get_numerator(): " << f2.get_numerator() << "\n";
   else
   {
      cout << "Got the expected value of f2.get_numerator()\n";
   }

   if ( f2.get_denominator() != 3 )
   {
      cout << "Expected value of f2.get_denominator(): " << 3 << "\n";
      cout << "Actual value of f2.get_denominator(): " << f2.get_denominator() << "\n";
   }
   else
   {
      cout << "Got the expected value of f2.get_denominator()\n";
   }

    result -= 0.5;
}