CompareTo method

[Beckie Wirefly]

Hello,

This is my first post, so I hope I am doing this correctly. I am studying Programming at University (College) and am a little stuck on the CompareTo method in the Comparable interface.


What I need to do is:
If the price is equal (return 0), then compare the area's.

I hope this is clear enough, here is my code:

   
public interface Imperial {
	
	public String toStringImperial ();

}

public class House implements Imperial, Comparable {
	
	private int price;
	private double landArea;
	private int bedrooms;
	
	public House (int price, double landArea, int bedrooms) {
		this.price = price;
		this.landArea = landArea;
		this.bedrooms = bedrooms;
	}

	// hectares* 2.471 = acres
	public String toStringImperial() {
		return landArea * 2.471 + "";
	}

	public int getPrice() {
		return price;
	}

	public double getLandArea() {
		return landArea;
	}

	public int compareTo(Object o) {
		// TODO Auto-generated method stub
		return 0;
	}	
}

public class testHouse extends House implements Comparable, Imperial {

	public testHouse(int price, double landArea, int bedrooms) {
		super(price, landArea, bedrooms);
	}

	public static void main(String[] args) {
		
		House house1 = new House (370000, 0.2, 3);
		House house2 = new House (370000, 4.0, 2);

	}
	
	public int compareTo(Object o) {
		
		if (this.getPrice() > ((House)o).getPrice())
			return 1;
		else if (this.getPrice() < ((House)o).getPrice())
			return -1;
		else
			return 0;
		
		boolean b = o.equals(this.getPrice());
		if (b == true)
			return   //WHERE I'M STUCK...I THINK LOL.  Something like:  this.getlandArea ().compareTo (House)o).getLandArea());
		
	}
}



 

I believe I am doing the correct thing, but not too sure. I think the problem may lay in comparing two objects???

Thank you for your insight.

Beckie

[Henry Wong]

Line 61 and after, of the compareTo() method, should be unreachable -- as it is not possible to get pass the if-then statement without returning from the method.

Henry

[Beckie Wirefly]

Henry Wong wrote:
Line 61 and after, of the compareTo() method, should be unreachable -- as it is not possible to get pass the if-then statement without returning from the method.

Henry

Of course, thank you for correcting that for me!!

Beckie

[Rob Spoor]

o.equals(this.getPrice())

That only returns true if o is an Integer object, but at that point (should it be reached by removing a previous return statement) you already know that o is a House object - or the cast on line 54 would already have failed.