Re: My set fails
On Sun, 23 Mar 2008 00:49:21 -0700 (PDT), -Rick-
<rick.softly@gmail.com> wrote:
Could anyone tell me why? How do I fix it?
How does it fail? What is it doing that you do not want it to do?
What is it not doing that you want it to do?
package setTest;
import java.util.*;
public class SetTest implements Comparable<SetTest> {
public String toString(){
return this.s1 + " " + this.x1;
}
public static void display(){
for(SetTest s : set){
System.out.println(s);
}
}
public static void populate(){
for (int i = 1; i < 3; i++){
set.add(new SetTest(1, "One"));
set.add(new SetTest(2, "One"));
set.add(new SetTest(1, "Two"));
}
}
public String getX1(){
return x1;
}
public int getS1(){
return s1;
}
public boolean equals(Object o) {
if (!(o instanceof SetTest))
return false;
SetTest s = (SetTest)o;
if(s.getS1() == this.getS1() && s.getX1() == this.getX1())
return true;
else
return false;
}
public int compareTo(SetTest s){
if(this == s)
return 1;
else
return -1;
}
This implementation of compareTo is almost certainly wrong. compareTo
returns a negative, zero or positive. Your version here can never
return zero. You need to check the documentation first.
public SetTest(int s, String x){
s1 = s;
x1 = x;
}
public static void main(String[] args){
populate();
System.out.println("The size of set is: " + set.size());
What is set.size()? You have not declared anything called "set" and
you have not defined a method called"size()".
rossum
display();
}
private int s1;
private String x1;
static protected Set<SetTest> set = new TreeSet<SetTest>();
}