在自定义方法equals中发现的问题，慎用对象的私有属性。

下面的类是所有实体（jpa entity）的父类，所有实体的比较都有该类实现。

当使用延迟加载时会出现问题，延迟加载时比较的并不是实体本身，而是实体的拦截实例，他是实体的子类，

other.id并没有值，正常情况下是通过 other.getId（）来访问id的，拦截实例会调用他包装的实体类的 getId（）。

 

    @ManyToOne(fetch=FetchType.LAZY)
    @JoinColumn(name = "SERIESTYPE_SEQ")
    private SeriesType seriesType;

public abstract class BaseEntity implements Serializable

 

    private Long id;

    public Long getId() {
        return id;
    }

    public void setId(Long id) {
        this.id = id;
    }

有bug的方法

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

修改过后的方法

@Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (!(obj instanceof BaseEntity))
            return false;
        final BaseEntity other = (BaseEntity) obj;
        if (getId() == null) {
            if (other.getId() != null){
                return false;               
            }
        } else{
            if (!getId().equals(other.getId（） )){
                return false;
            }
        }
           
        return true;
    }

要避免直接使用对象的私有属性