Hibernate Bean的Equals方法重载问题

linliangyi2007

浏览: 1012698 次
性别:
来自: 福州

最近访客更多访客>>

anyitzy

pos3721

ymgjava

winco304

博主相关

博客

微博

相册

留言

关于我

文章分类

社区版块

存档分类

博客分类：

程序人生

Bean Hibernate C#C++C

代码如下：

	
	/**
	 * 判断当前流程实例上下文中，是否存在Blocking(阻挡性)的任务
	 * 规则
	 * 1.任务未结束 
	 * 2.任务是必办理的
	 * 3.当前任务实例的token和执行上下文的token一致
	 * @param token
	 * @return
	 */
	public boolean hasBlockingTaskInstances(FlowToken token) {
		boolean hasBlockingTasks = false;

		System.out.println("LOC_1 :"  + token.getId());

		Set<TaskInstance> taskInstances = flowInstance.getTaskInstances();
		if (taskInstances!=null) {
			Iterator<TaskInstance> iter = taskInstances.iterator();
			
			while ( (iter.hasNext()) && (!hasBlockingTasks)) {

System.out.println("LOC_2 : "  + taskInstance.getFlowToken().getId());
System.out.println("LOC_3 : "  + token == taskInstance.getFlowToken());
System.out.println("LOC_4 : "  + token.equals(taskInstance.getFlowToken());

				TaskInstance taskInstance = (TaskInstance) iter.next();
				if ( (! taskInstance.hasEnded()) //任务未结束 
						&& (taskInstance.getIsBlocking()) //任务是必办理的
						&& (token!=null) 
						&& (token.equals(taskInstance.getFlowToken())) ) {
					hasBlockingTasks = true;
				}
			}
		}
		return hasBlockingTasks;
	}

上述的 LOC_1 处和 LOC_2 处打印的ID完全相等， LOC_3处输出true（说明对象地址相同），BUT～～ LOC_4处输出是false；

给出FlowToken的equals实现

	/* (non-Javadoc)
	 * @see java.lang.Object#equals(java.lang.Object)
	 */
	@Override
	public boolean equals(Object obj) {
		if (this == obj)
			return true;
		if (obj == null)
			return false;
		if (!(obj instanceof FlowToken))
			return false;
		final FlowToken other = (FlowToken) obj;
		if (id == null || other.id == null) {
			return false;
		} else if (!id.equals(other.id))
			return false;
		return true;
	}

跟踪发现，实际进入equals的obj参数根本不是外面传入的那个token对象，中间过程被Hinernate的CGLIBLazyInitializer做了代理处理，发生了底层对象的变换。
开始大家都打死不信有这样的BUG，经过对jvm线程和变量编号的反复跟踪，BUG每次都重现。确认CGLIBLazyInitializer确有问题，但目前在国内互联网上没有找到相同的案例。这里提出，供javaeye上的大牛商讨！

如果我们一群人的诊断是对的，hibernate就隐藏了一个相当恐怖的bug咯！！！

分享到：

发布新版IK-Expression开源表达式解析器V2 ... | Graceful Java Programming 优雅Java编程 ...

2009-05-26 17:51
浏览 3818
评论(76)
论坛回复 / 浏览 (76 / 15514)
分类:企业架构
查看更多

76 楼 linliangyi2007 2009-06-02

再说对于帖子标题的看法！

本人一直认为在互联网上的，在BBS上一切都不必太较真。那种拘泥于标题，拘泥于网络形式，而忽视文章内容的做法，不敢苟同。

现在标题改了，算是迎合网络部分较真人士的观念吧！
做人难，现实生活累人，没想到上个bbs现在也很累人！

75 楼 linliangyi2007 2009-06-02

leadyu 写道

linliangyi2007 写道

还有一件事不明白，就是我在跟踪的时候，就像你说的this.id是有值的，但other.getId()依然返回null。
你上面说的，我基本明白了，直接取属性有问题，但通过get方法取id，应该要返回值才是啊？

通过你现在给的一些条件，很难判断为什么other.getId()依然返回null。毕竟token的所有getter,setter,都已经被hibernate代理了，cglib实现了suround级的方法代理，hibernate可以去调super的对应方法，也可以不调，取决于他的实现，现在需要确定几个事情：

1）token和你跟踪到的equels方法里的other是否同一个对象？如果是，没道理other.getId()为null呀，你在外面已经print一次token.getId()了。

2)other.getId()是否调了super.getId();这个通过你在getId设置断点就可以知道，你所设的断点，我前面也说了都是针对super类的。

一切水落石出了。
在equals方法中，this对象的类是FlowToken，而other对象的类是proxy。this.id是有值的，other.id为null，但使用other.getId()是会取到lazy的id值的（之前估计这里测试的时候有问题，居然没取到值。）
一切如leadyu所预测的那样。

74 楼 icewubin 2009-05-31

jianfeng008cn 写道

到底原因是什么呢，既然搞清楚了，为何不写写清楚？

楼主只是说思路清楚了，估计还没开始调试呢。

73 楼 jianfeng008cn 2009-05-31

到底原因是什么呢，既然搞清楚了，为何不写写清楚？

72 楼 linliangyi2007 2009-05-31

leadyu 写道

linliangyi2007 写道

十分感谢，思路已经清楚了。后面就是沿着思路把问题摸清楚了

71 楼 leadyu 2009-05-31

linliangyi2007 写道

70 楼 linliangyi2007 2009-05-31

非常赞同使用getClass()==getClass()做比较，看了你推荐的帖子后，正有此意！！
简单且相对了instanceof更合理，但又不用反射每个属性那么耗性能

69 楼 icewubin 2009-05-31

linliangyi2007 写道

icewubin 写道

要说equals方法逻辑问题，不知道楼主说的哪个部分。

下面这个短路运算就是有问题的，虽然和这贴讨论的东西没有任何关系。

if (!(obj instanceof FlowToken))      
        return false;

深入研究Java equals方法

感谢提供了这么有用的思路。

确实有这个问题。但就性能而言，尤其是在有了类型约束的Set和Map的容器比较中，我还是偏向于旧的，“带有问题”但高性能的实现。当然，会慎重的考虑到文章中提到的bug（如果出现了逻辑上的不正确，特别会去思考一下，是不是这个问题造成的，呵呵）

用getClass()==getClass()做比较就可以常规短路，这是Eclipse代码生成equals()的实现方法。

如果要支持Hibernate代理对象，可以这样：

        if (getClass().getPackage() != other.getClass().getPackage()) {   
            return false;   
        }

至于性能高不高，楼主有空可以自己测一下，我也没细测，但是看过JDK源代码，没什么问题，而且都是引用（内存地址）比较，应该不慢。

68 楼 linliangyi2007 2009-05-31

icewubin 写道

要说equals方法逻辑问题，不知道楼主说的哪个部分。

下面这个短路运算就是有问题的，虽然和这贴讨论的东西没有任何关系。

if (!(obj instanceof FlowToken))      
        return false;

深入研究Java equals方法

感谢提供了这么有用的思路。

67 楼 linliangyi2007 2009-05-31

leadyu 写道

首先，我们看看这句:

System.out.println("LOC_3 : " + token == taskInstance.getFlowToken());

这说明token 和taskInstance.getFlowToken() 得到的是同一个对象，都是Cglib的代理类。

那么在这过程中有几个类呢？

第一有FlowToken 和 FlowToken$Proxy类（Cglib创建的），而token是FlowToken$Proxy的实例。

那么Cglib怎么实现代理的呢？它并不会修改原类的任何部分，只会新创建一个类，复写所有父类的public方法

那么在FlowToken$Proxy里面equels就变成：

public  boolean  equels(Object obj){

	doCglibHandle...

	super.equels(obj);


	doCglibHandle...

}

而，这里最要注意的是在调试工具里面设置的断点一直都是FlowToken该方法的某一行，而不是FlowToken$Proxy类里面的，（这点你可以在debug时注意看stack，可以看到先被FlowToken$Proxy.equels调了以后才到你的断点），所以，你监控的是父类

再看父类的方法：

public boolean equals(Object obj) {   
    if (this == obj)   
        return true;   
    if (obj == null)   
        return false;   
    if (!(obj instanceof FlowToken))   
        return false;   
    final FlowToken other = (FlowToken) obj;   
    if (id == null || other.id == null) {   
        return false;   
    } else if (!id.equals(other.id))   
        return false;   
    return true;   
}

第一行，为什么不等呢？很简单，这里的this已经是父类的实例，而不是你的token ,所以和传进来的实例自然不等。

那么为什么你看到的传进来的Obj属性全是null，也很简单，因为这个obj也是hibernate的代理出来的一个延时加载类，所有属性都没有的，都是通过get从数据库获得。

不知，这样解释，LZ明白否？

66 楼 icewubin 2009-05-31

要说equals方法逻辑问题，不知道楼主说的哪个部分。

下面这个短路运算就是有问题的，虽然和这贴讨论的东西没有任何关系。

if (!(obj instanceof FlowToken))      
        return false;

深入研究Java equals方法

65 楼 linliangyi2007 2009-05-31

melode11 写道

你自己的equals逻辑错了。只要你对象的id == null,你就不管两个实际上相不相等，直接return false。
if (id == null || other.id == null)

应该改成

if ((id == null || other.id == null) &&!(id ==null&&other.id==null))

你这种逻辑根本不是我要的，对我而言要的业务逻辑是就当id为空时，对象相等比较是无意义的，应设置为不相等。
而且你的写法根本解决不了我提的问题。

不要动不动就说人家的逻辑错了，你根本不知道我要的equals逻辑是啥，又凭什么说我错呢！！

64 楼 linliangyi2007 2009-05-31

leadyu 写道

public  boolean  equels(Object obj){

	doCglibHandle...

	super.equels(obj);


	doCglibHandle...

}

public boolean equals(Object obj) {   
    if (this == obj)   
        return true;   
    if (obj == null)   
        return false;   
    if (!(obj instanceof FlowToken))   
        return false;   
    final FlowToken other = (FlowToken) obj;   
    if (id == null || other.id == null) {   
        return false;   
    } else if (!id.equals(other.id))   
        return false;   
    return true;   
}

难得有人把事情说清楚了，这样即使被评新手，也认了。

JE上浅薄的人不少啊，还有不少人吧思路放在equals方法的逻辑上，又或者简单的说明proxy中那些谁都明白的概念。
如果问题都那么简单，还用发贴？！
这种问题别不见的人人都碰到并及时发现的。发贴更多是想提醒大家，碰到同样问题的时候，可以很快的醒悟，但很多人在BBS上只是想显摆自己很牛...
动不动就是隐藏和新手，我看je这样走不远（一直希望它能超越CSDN，看来是很困难了）

63 楼 icewubin 2009-05-30

icewubin 写道

给所有人建议，hibernate中取ID有更好的通用方法，明天我来贴代码，可以放在BaseModel中的。

http://www.iteye.com/topic/397854

62 楼 leadyu 2009-05-30

public  boolean  equels(Object obj){

	doCglibHandle...

	super.equels(obj);


	doCglibHandle...

}

public boolean equals(Object obj) {   
    if (this == obj)   
        return true;   
    if (obj == null)   
        return false;   
    if (!(obj instanceof FlowToken))   
        return false;   
    final FlowToken other = (FlowToken) obj;   
    if (id == null || other.id == null) {   
        return false;   
    } else if (!id.equals(other.id))   
        return false;   
    return true;   
}

61 楼 leadyu 2009-05-30

这个问题有意思，不应该被投新手帖的呀，这个问题并不是大家所想的那样，还要从Cglib的内部机制和java调试JDA两方面造成的。我来说说真正的原因吧。

60 楼 icewubin 2009-05-29

啊，楼主至少还要再次修改两个id的用法，id都换成getId()。

59 楼 melode11 2009-05-29

58 楼 KimShen 2009-05-28

被拼新手了啊，可惜了

57 楼 icewubin 2009-05-28

Sunteya 写道

可以看到虽然tokenB是通过getter 访问了, 但 tokenA 还是使用id 属性. 这样结果当然还是不对.

即使this.id不对，但是楼主先质疑为什么other.getId()为什么是null。（个人觉得other.getId()因该不为空，楼主真的确认了么？）

发表评论

您还没有登录,请您登录后再发表评论

最近访客更多访客>>

博主相关

文章分类

社区版块

存档分类

最新评论