嗨,我在实现compareTo方法时遇到了麻烦。我找到了答案但没有任何帮助。我正在尝试用各种大小的圆填充TreeSet。我需要在我的圈子类中进行compareTo才能以这种方式存储它们。
import java.util.*;
import java.lang.*;
abstract class Shape
{
private String name; //e.g."circlel", "rectangle3"
Shape(String name0)
{
name = name0;
}
abstract double area (); // area of shape
abstract double perim(); // length of perimeter of shape
void put()
{ // display shape details
System.out.println(name + " with area " + area()
+ " and perimeter " + perim() );
}
}
class Circle extends Shape implements Comparable
{
private static String name;
private int radius;
Circle(String n, int r)
{
super(n);
radius = r;
}
public double area()
{
return Math.PI * radius * radius;
}
public double perim()
{
return 2 * Math.PI * radius;
}
public int compareTo(Circle c)
{
if(c.name == name && c.radius == radius)
{
return 0;
}
else
{
return 1;
}
}
}
编辑:谢谢我忘记了什么: Circle不是抽象的,并且不会覆盖Comparable中的抽象方法compareTo(Object)
感谢您的帮助,现在我已经开始测试该类,当尝试向树集添加一个圆圈时,此异常会弹出任何想法,
Exception in thread "main" java.lang.NullPointerException
at Circle.compareTo(Shape.java:47)
at Circle.compareTo(Shape.java:23)
at java.util.TreeMap.compare(Unknown Source)
at java.util.TreeMap.put(Unknown Source)
at java.util.TreeSet.add(Unknown Source)
at CircleTreeSet.main(CircleTreeSet.java:24)
答案 0 :(得分:3)
此方法永远不会返回-1。如果其中一个圆圈与另一个圆圈相比“更大”,则该圆圈应返回1,而另一个圆圈与较大圆圈相比应返回-1。你必须确保你的圆圈遵循传递属性和其他一些guildelines。
答案 1 :(得分:0)
如果当前实例小于c
,则必须返回-1;如果当前实例大于c
,则返回1;如果实例相等,则返回0。
这就是compareTo
的工作方式。现在你正在把它当作平等检查。 comparTo
做的不止于此。它应该确定一个项目是否等于,小于或大于另一个项目。
此代码将按排序时的大小顺序对具有相同名称的对象进行分组。对象也将按名称按字母顺序排序。
public int compareTo(Circle c)
{
if(c.name.equals(name)){
if(c.radius < radius)
{
return 1;
}
else if(c.radius>radius)
{
return -1;
}
return 0;
}
//names aren't the same compare alphabetically.
return this.name.compareTo(c.name);
}