Results 1 to 13 of 13
 03092013, 07:32 PM #1Member
 Join Date
 Feb 2013
 Posts
 68
 Rep Power
 0
Fractions equals each other problem
Everything else works fine besides this equals code could somebody check it out for me? and point me in the right direction?
Java Code:public boolean equals(Fractions f) { int X = num, X1 = denom, Y = f.num, Y1 = f.denom; if (X/X1 == Y/Y1) return false; else return true; }
Java Code:import java.util.*; public class Fractions { private Scanner scan = new Scanner(System.in); private int num=1; private int denom=1; public Fractions() { } public Fractions(int n, int d) { num = n; denom = d; reduce(); } public void setFraction(int n, int d) { num = n; denom = d; reduce(); } public Fractions add(Fractions op) { int a = num, b = denom, c = op.num, d = op.denom; int num_new = a*d + b*c, denom_new = b*d; Fractions f = new Fractions(num_new, denom_new); return f; } public Fractions subtract(Fractions op) { int a = num, b = denom, c = op.num, d = op.denom; int num_new = a*d  b*c, denom_new = b*d; Fractions f = new Fractions(num_new, denom_new); return f; } public Fractions multiply(Fractions op) { int a = num, b = denom, c = op.num, d = op.denom; int num_new = a*c, denom_new = b*d; Fractions f = new Fractions(num_new, denom_new); return f; } public Fractions divide(Fractions op) { int a = num, b = denom, c = op.num, d = op.denom; int num_new = a*d, denom_new = b*c; Fractions f = new Fractions(num_new, denom_new); return f; } private void reduce() { int smaller = Math.min(Math.abs(num), Math.abs(denom)); for (int divisor = 2; divisor < smaller; divisor++) { while ( (num % divisor == 0) && (denom % divisor == 0)) { num = num / divisor; denom /= divisor; smaller /= divisor; } } } public boolean equals(Fractions f) { int X = num, X1 = denom, Y = f.num, Y1 = f.denom; if (X/X1 == Y/Y1) return false; else return true; } public String toString() { return num + "/" + denom; } public void readin(String label) { while (true) // Keep trying if bad input is received { System.out.println(label); String temp = scan.next(); temp = temp.trim(); // get rid of white space at the beginning and end int index = temp.indexOf('/'); if (index >= 0) { String numStr = temp.substring(0, index); String denomStr = temp.substring(index+1); int n = Integer.parseInt(numStr); int d = Integer.parseInt(denomStr); setFraction(n,d); return; } else System.out.println("Input Fraction missing / "); }//Keep trying until you get it right } public static void main(String[] args) { Fractions f1= new Fractions(); Fractions f2= new Fractions(); Fractions f3=null; Scanner scan = new Scanner(System.in); while(true) { System.out.println("Enter operation: +  * / q (q ==> quit) : "); String input = scan.next(); if (input.charAt(0) == 'q') return; // All done f1.readin("Enter Fraction 1: "); f2.readin("Enter Fraction 2: "); System.out.println("f1 = " + f1); System.out.println("f2 = " + f2); if (f1.equals(f2)) System.out.println("f1 and f2 are equal"); else System.out.println("f1 and f2 are not equal"); switch (input.charAt(0)) { case '+': f3 = f1.add(f2); System.out.println("f1+f2=" + f3); break; case '': f3 = f1.subtract(f2); System.out.println("f1f2=" + f3); break; case '*': f3 = f1.multiply(f2); System.out.println("f1*f2="+f3); break; case '/': f3 = f1.divide(f2); System.out.println("f1/f2="+f3); break; default: System.out.println("Illegal command: " + input ); break; } }// end of while loop } // end of main }
 03092013, 07:36 PM #2Senior Member
 Join Date
 Jan 2013
 Location
 Northern Virginia, United States
 Posts
 5,305
 Rep Power
 9
Re: Fractions equals each other problem
You are using ints instead of doubles for your fractions. Thus anything to the right of the decimal point is discarded.
Regards,
JimThe Java^{TM} Tutorials  SSCCE  Java Naming Conventions
Poor planning on your part does not constitute an emergency on my part
 03092013, 07:42 PM #3Member
 Join Date
 Mar 2013
 Posts
 11
 Rep Power
 0
Re: Fractions equals each other problem
Yes, the int/int division gives int in the result. I suppose, the most convenient solution here would be casting.
Java Code:if ((double) X/X1 == (double) Y/Y1) return false; else return true;
 03092013, 08:13 PM #4Senior Member
 Join Date
 Jan 2013
 Location
 Northern Virginia, United States
 Posts
 5,305
 Rep Power
 9
Re: Fractions equals each other problem
That won't work either because X/X1 is evaluated as an int and then you convert it to a double. The best method would be to use doubles everywhere in place of ints (based on my assumption of what you are trying to do).
A couple more observations. Your equals method is a method of the Fraction class. So a typical approach would be to pass in the num and denom as arguments and use the instance variables for the other num and denom.
And as you get more into Java, you should get into the habit of overriding hashCode if you override equals. Otherwise, the various hashMaps and such will not perform properly.
Regards,
JimThe Java^{TM} Tutorials  SSCCE  Java Naming Conventions
Poor planning on your part does not constitute an emergency on my part
 03092013, 08:46 PM #5
 03092013, 08:56 PM #6
 Join Date
 Sep 2008
 Location
 Voorschoten, the Netherlands
 Posts
 14,293
 Blog Entries
 7
 Rep Power
 24
Re: Fractions equals each other problem
Casting to double is silly: if mathematically a/b == c/d then a*d == c*b and you would need longs at worst.
kind regards,
JosThe only person who got everything done by Friday was Robinson Crusoe.
 03092013, 10:09 PM #7Senior Member
 Join Date
 Jan 2013
 Location
 Northern Virginia, United States
 Posts
 5,305
 Rep Power
 9
Re: Fractions equals each other problem
Hmm, I could have sworn I had done this before and it didn't work. However, I stand humbly corrected!
Regards,
JimThe Java^{TM} Tutorials  SSCCE  Java Naming Conventions
Poor planning on your part does not constitute an emergency on my part
 03102013, 09:23 AM #8
 Join Date
 Sep 2008
 Location
 Voorschoten, the Netherlands
 Posts
 14,293
 Blog Entries
 7
 Rep Power
 24
Re: Fractions equals each other problem
It must've been bit rot; b.t.w. a Fraction implementation with an int numerator and denominator is next to useless; e.g. look at the arithmetic operations above: if any of those multiplications make an int overflow, the result will be incorrect. A solid Fraction class implementation is a nasty beast.
kind regards,
JosThe only person who got everything done by Friday was Robinson Crusoe.
 03102013, 02:17 PM #9Member
 Join Date
 Feb 2013
 Posts
 68
 Rep Power
 0
 03102013, 05:30 PM #10Senior Member
 Join Date
 Jan 2013
 Location
 Northern Virginia, United States
 Posts
 5,305
 Rep Power
 9
Re: Fractions equals each other problem
Well, the first part of my post was incorrect (which has been pointed out). I had gotten my operator and cast precedences bass ackwards. So something like:
double x = (double) n/m is the same as double x = ((double) n)/m
Here is what I meant regarding you're highlighted area (note I will be using Jos' recommendation for comparison)
Java Code:class Fraction { int num; int denom; Fraction(int num, int denom) { this.num = num; this.denom = denom; } public boolean equals(int n, int d) { System.out.println("Comparing " + n + "/" + d + " to " + num + "/" + denom); return num * d == denom * n; } } public class FractionExample { public static void main(String[] args) { Fraction f1 = new Fraction(10, 15); Fraction f2 = new Fraction(20, 40); System.out.println(f1.equals(40, 60)); // true System.out.println(f1.equals(2, 4)); // false System.out.println(f2.equals(50, 100)); // true System.out.println(f2.equals(1, 3)); // false } }
Also, in this case the hashCode recommendation doesn't apply because this signature is different than the one inherited from the Object class.
Note: In an earlier post I discussed overriding equals. Just to clarify, you did *not* override equals because your equals signature takes an object of Fraction and not an object of Object. I hope this doesn't muddy the issue.
Regards,
JimLast edited by jim829; 03112013 at 03:30 PM.
The Java^{TM} Tutorials  SSCCE  Java Naming Conventions
Poor planning on your part does not constitute an emergency on my part
 03122013, 04:04 AM #11Member
 Join Date
 Feb 2013
 Posts
 68
 Rep Power
 0
Re: Fractions equals each other problem
How would you do it with my code though? I have to follow that template for class. It's the only one I don't understand because how would I use the fraction f to make it equal?
 03122013, 04:52 AM #12Member
 Join Date
 Feb 2013
 Posts
 68
 Rep Power
 0
Re: Fractions equals each other problem
figured it out.
 03122013, 09:31 AM #13
 Join Date
 Sep 2008
 Location
 Voorschoten, the Netherlands
 Posts
 14,293
 Blog Entries
 7
 Rep Power
 24
Re: Fractions equals each other problem
I gave it another look: if you properly normalize the numerator and denominator and stick the sign in (say) the numerator, the equals( ... ) method simply has to compare the two numerators and denominators for equality.
kind regards,
JosThe only person who got everything done by Friday was Robinson Crusoe.
Similar Threads

Problem with my equals method
By coasterguy10 in forum New To JavaReplies: 3Last Post: 10242011, 10:45 PM 
trouble using fractions
By jeanOliver in forum New To JavaReplies: 2Last Post: 09062011, 08:48 PM 
Reducing fractions
By SwEeTAcTioN in forum New To JavaReplies: 10Last Post: 01132010, 05:51 PM 
a little problem with (equals)
By gamilah in forum New To JavaReplies: 5Last Post: 10252008, 04:08 AM 
name clash: equals(E) in and equals(java.lang.Object)
By AdRock in forum New To JavaReplies: 0Last Post: 01262008, 12:13 AM
Bookmarks