2014-12-07 96 views
0

我想我的程序正在跳过我的while循环,但我实在不确定发生了什么。该函数应该通过找到GCD然后再除以分子和分母来减少分数。为什么我的while循环被跳过?

class Rational { 

private int numerator, denominator; 

//Constructor 
public Rational (int num, int den) { 
    numerator = num; 
    denominator = den; 
} 

//Method for multiplying fractions 
public Rational times (Rational that) { 
    Rational x = new Rational (this.numerator*that.numerator, this.denominator*that.denominator); 
    x = x.reduce(); 
    return x; 
} 

//Method for displaying fractions as strings 
public String toString() { 
    return new String(numerator+"/"+denominator); 
} 

//Method for adding fractions 
public Rational plus(Rational that) { 
    Rational x = new Rational ((this.numerator*that.denominator)+(that.numerator*this.denominator), 
      this.denominator*that.denominator); 
    //x = x.reduce(); 
    return x; 
} 

//Method for subtracting fractions 
public Rational minus(Rational that) { 
    Rational x = new Rational ((this.numerator*that.denominator)-(that.numerator*this.denominator), 
      this.denominator*that.denominator); 
    //x = x.reduce(); 
    return x; 
} 

//Method for dividing fractions 
public Rational divideBy(Rational that) { 
    Rational x = new Rational (this.numerator*that.denominator, this.denominator*that.numerator); 
    //x = x.reduce(); 
    return x; 
} 

public Rational reduce() { 
    int a = Math.abs(this.numerator); 
    int b = Math.abs(this.denominator); 
    int c = Math.min(a, b); 
    System.out.println(c); 
    System.out.println(a%c); 
    System.out.println(b%c); 
    if (a==0) { 
     return new Rational (0,1); 
    } 
    else { 
     while (((a%c)!= 0) && ((b%c)!= 0)) { 
      c = c-1; 
      System.out.println(c); 
     } 
     System.out.println(c); 
     return new Rational (this.numerator/c,this.denominator/c); 
    } 
} 
} 

public class RationalTester { 

public static void main(String[] args) { 
    Rational x = new Rational (6,4); //The fraction 6/4 
    Rational y = new Rational (5,2); //The fraction 5/2 
    Rational z = x.times(y); //Their product 
    Rational w = x.plus(y); //Their sum 
    Rational v = x.minus(y); //Their difference 
    Rational u = x.divideBy(y); //Their quotient 
    JOptionPane.showMessageDialog(null, x.toString()+" * "+y.toString()+" = "+z.toString()); 
    JOptionPane.showMessageDialog(null, x.toString()+" + "+y.toString()+" = "+w.toString()); 
    JOptionPane.showMessageDialog(null, x.toString()+" - "+y.toString()+" = "+v.toString()); 
    JOptionPane.showMessageDialog(null, x.toString()+"/"+y.toString()+" = "+u.toString()); 

} 

} 

我得到的分子和分母的绝对值,以确保如果分数是负的,我会保持,在年底。如果分子是0,我被要求返回(0,1)。问题是关于while循环......它似乎被完全跳过了。有什么建议么?

+1

提供一些样本输入和期望的输出用于测试 – 2014-12-07 22:33:47

+1

请给出'a','b'和'c'的示例值...注意,比较浮点数的平等性很少是一个好主意。他们*需要*是'double'而不是'int'吗?请注意,如果您提供简短但完整的*程序来证明问题,这将有助于... – 2014-12-07 22:33:50

+0

http://en.wikipedia.org/wiki/Modulo_operation#Common_pitfalls – specializt 2014-12-07 22:35:55

回答

6

因为它的条件总是错误的。

在第一行中设置c等于ab。所以有两种可能性:

  • 如果c == a,则a%c将为零。所以这个条件是错误的。
  • 如果c == b,则b%c将为零。所以这个条件是错误的。
+0

这就是原因。你说c = min(a,b)所以a%c == 0或b%c == 0总是! – simopopov 2014-12-07 22:36:41

+0

哈哈哈,非常感谢你!这是那些长时间在墙上敲打我的头的人之一。男人我觉得愚蠢。 – Gil 2014-12-07 22:41:38