自定义比较器时，相同返回0，不等返回一个正整数或者负整数，但是要做健壮性判断的下面这种情况该怎么在判断值为假的时候返回什么样的值？

1. class CompareScore implements Comparator<Object>{
   
2. 
   
3.         @Override
   
4.         public int compare(Object o1, Object o2) {
   
5.                 // TODO Auto-generated method stub
   
6.                 if (o1 instanceof Student&&o2 instanceof Student) {//健壮性判断
   
7.                         Student student1=(Student)o1;
   
8.                     Student student2=(Student)o2;
   
9.                                    //比较过程
   
10.                     if (student1.getScore()>student2.getScore()) {
    
11.                                 return -1;
    
12.                         }else if (student1.getScore()<student2.getScore()) {
    
13.                                 return 1;
    
14.                         } else{
    
15.                                 return 0;
    
16.                         }
    
17.                 }
    
18.                 //return null;//此处应该返回什么样的值？
    
19.         }
    
20.         
    
21. }

复制代码

随便给一个0,1，-1以外的数表示他们不满足同时为Student对象

孙茜茜 发表于 2013-6-14 23:02
随便给一个0,1，-1以外的数表示他们不满足同时为Student对象

万一传递进来的参数不是Student类型呢？那不就全部返回一样的结果了？

peerless2012 发表于 2013-6-15 06:51
万一传递进来的参数不是Student类型呢？那不就全部返回一样的结果了？

反正就设一个数字表示传送的对象不符合要求。
难道还要表达出传错的对象到底是谁吗？

我把代码修改成这样了

1. //自定义一个比较器，实现Comparator接口
   
2. class CompareScore implements Comparator<Object>{
   
3. 
   
4.         //重写compare方法
   
5.         @Override
   
6.         public int compare(Object o1, Object o2) {
   
7.                 // TODO Auto-generated method stub
   
8.                 //进行健壮性判断，判断传递来的对象是否是Student类型
   
9.                 if (o1 instanceof Student&&o2 instanceof Student) {
   
10.                         //如果是Student类型，则对其进行强制转换
    
11.                         Student student1=(Student)o1;
    
12.                     Student student2=(Student)o2;
    
13.                     //第一个对象的分数大于第二个对象的分数，返回-1；小于的话返回1
    
14.                     if (student1.getScore()>student2.getScore()) {
    
15.                                 return -1;
    
16.                         }else if (student1.getScore()<student2.getScore()) {
    
17.                                 return 1;
    
18.                         } else{
    
19.                                 //如果成绩相等，则调用String拥有的比较函数，按字典顺序对名字进行排序，
    
20.                                 //并且compareTo 只在方法 equals(Object) 返回 true 时才返回 0。
    
21.                                 return student1.getName().compareTo(student2.getName());
    
22.                         }
    
23.                 }
    
24.                 return 0;
    
25.         }
    
26.        
    
27. }

复制代码

孙茜茜 发表于 2013-6-15 09:20
反正就设一个数字表示传送的对象不符合要求。
难道还要表达出传错的对象到底是谁吗？ ...

我把代码又修改了
顺便说一下，你的名字跟我的一个美女同学的一样。。。。。。。。。。。。。

比较器，首选你会定义泛型吧，那么就是定义你需要比较类的比较器
定义了泛型，你还能传入对象错误，程序编译都通过不了

同楼上，定义泛型，可以强制传入对象为Student，把运行过程中的问题转移到编译上。

如果不定义泛型， if (o1 instanceof Student&&o2 instanceof Student)这句建议改成if (!(o1 instanceof Student&&o2 instanceof Student)){}。函数内抛出RuntimeException异常。硬性判断传入参数是否符合学生类，基本作用跟定义泛型差不多。总之就是从主观限制，变成系统的硬性限制。

还是感觉定义泛型比较好。