ArrayList类中，indexOf方法的返回值问题

[simonqsm]

class Router{
   private String ip;
   public boolean equals(Router r){
      return ip.equals(r.ip);
   }
   public Router(String ip){
      this.ip=ip;
   }
   public static void main(String[] args){
      ArrayList<Router> rlist=new ArrayList<Router>();
      Router r1=new Router("192.168.0.1");
      rlist.add(r1);
      Router r2=new Router("192.168.0.1");
      System.out.println(rlist.indexOf(r2));//此处返回值为-1       
   }
}

我重写了equals方法，希望是如果Router类的两个对象r1,r2中的ip属性值相同，就认为两个对象是一样的，在rlist中判断出r2已经存在，但返回值总是-1，怎么做才能返回值不为-1呢？



谢谢

[fsn011362]

了解

[tfsict2008]

错误很经典，2楼正解

[freezgw1985]

呵呵，看看加学习学习！！

[gukuitian]

你的equals方法是自己的，不是重写Object类的，

[gukuitian]

public boolean equals(Object obj) {

	return this.ip.equals(((Router)obj).ip);
}

[叶涛网站推广优化]

赞一个

[gefengxztg]

public boolean equals(Object r){
if(r == null || !(r instanceof Router))
return false;
return ip.equals(((Router)r).ip);
}

[swandragon]

public boolean equals(Object obj){
		if(this == obj){
			return true;
		}
		if(obj instanceof Router){
			Router anotherRouter = (Router) obj;
			return ip.equals(anotherRouter.ip);
		}
		return false;
	}

[铁匠梁老师]

路过，不错，2楼正解

[fengfengye]

是不是应该先rlist.add(r2)这样啊？？

[修电视的攻城狮]

重写方法只能是重写子类从父类中继承来的方法；
重写必须和原来的方法有相同的方法名、相同的参数列表和相同的返回值类型；
重写方法不能比被重写方法具有更严的访问控制权限。

[zhengmj777]

equals方法错了

@Override
	  public boolean equals(Object obj) {
		  	// TODO Auto-generated method stub
		  	if(obj == this){
		  		return true;
		  	}
		  	if(obj == null || obj.getClass() != this.getClass()){
		  		return false;
		  	}
		  	Router router = (Router)obj;
		  	return this.ip.equals(router.ip);
	  }

[bobo364]

[Quote=引用 10 楼 mingjian_zheng 的回复:]
equals方法错了

Java code
@Overridepublicboolean equals(Object obj) {// TODO Auto-generated method stubif(obj==this){returntrue;
}if(obj==null|| obj.getClass()!=this.getClass()){returnfalse;
}
Router router= (Router)obj;returnthis.ip.equals(router.ip);
}
[/Quote]

正解重写方法必须是同样的方法名，类型，及输入的数据的类型