Implementing Comparable in Java

Question

So I'm trying to compare two shape's area using Comparable. I implement it in my class and I'm trying to override compare right now in my LineSegment class which extends my own abstract class Shape.

class LineSegment extends Shape implements Comparable{

public int compareTo(Object object1, Object object2)
      {
         LineSegment ls1 = (LineSegment) object1;
    LineSegment ls2 = this;
    return Double.compare(ls1.getArea(), ls2.getArea());
}



}

Before I had an issue with comparing two doubles and I saw a solution to the problem on here with that return statement and Double. getArea() returns a double area for the LineSegment. So I have ran into this error, any help would be appreciated, thanks- LineSegment is not abstract and does not override abstract method compareTo(java.lang.Object) in java.lang.Comparable class LineSegment extends Shape implements Comparable

compareTo should have only one parameter

maskacovnik
– maskacovnik

2014-11-15 17:35:01 +00:00
Commented Nov 15, 2014 at 17:35 — maskacovnik
– maskacovnik, Commented Nov 15, 2014 at 17:35

SMA · Accepted Answer · 2014-11-15 17:31:31Z

1

You need to use Comparator interface instead of Comparable. so your class definition will change to:

class LineSegment extends Shape implements Comparator <LineSegment> {
....//with compare(LineSegment ls1, LineSegment ls2) and you dont need typecasting

Or if you are intending to comparable then you need implementation like:

class LineSegment extends Shape implements Comparable<LineSegment>{
    public int getArea() {...}

    public int compareTo(LineSegment object1)
    {

        return Double.compare(this.getArea(), object1.getArea());
    }
}

answered Nov 15, 2014 at 17:31

SMA

37.2k8 gold badges53 silver badges76 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

SMA Over a year ago

@user4254704 Glad it worked for you. Please accept the answer if indeed it helped and closet his question.

learning_user7 Over a year ago

how do you close a question?

Community · Accepted Answer · 2017-05-23 10:33:41Z

1

In order to implement Comparable, you need to implement compareTo method.

If you want to use Comparator, you should implement compare.

See more here

Use:

class LineSegment extends Shape implements Comparable<LineSegment>{
...

    @Override
    public int compareTo(LineSegment other) {
        return Double.compare(this.getArea(), other.getArea());
    }

}

edited May 23, 2017 at 10:33

CommunityBot

11 silver badge

answered Nov 15, 2014 at 17:31

BobTheBuilder

19.4k6 gold badges45 silver badges62 bronze badges

3 Comments

learning_user7 Over a year ago

Right I need to use Comparable, I changed my code and still getting the same error

SMA Over a year ago

@user4254704 See answer above stackoverflow.com/questions/26948604/…

BobTheBuilder Over a year ago

compareTo should get one argument only. see docs.oracle.com/javase/7/docs/api/java/lang/…

maskacovnik · Accepted Answer · 2014-11-15 17:30:52Z

0

With Comparable, you have to compare one object with this object:

public int compareTo(Object object1){
    LineSegment ls1 = (LineSegment) object1;
    LineSegment ls2 = this;
    return Double.compare(ls1.getArea(), ls2.getArea());
}

this is only rewrited, but you should use <LineSegment>:

class LineSegment extends Shape implements Comparable<LineSegment>{
    public int compareTo(LineSegment ls){
        return Double.compare(ls.getArea(),this.getArea());
    }
}

answered Nov 15, 2014 at 17:30

maskacovnik

3,0845 gold badges23 silver badges26 bronze badges

Collectives™ on Stack Overflow

Implementing Comparable in Java

3 Answers 3

2 Comments

3 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related