当前位置:首页-关于团队-EasyJF 成员

  • EasyJWeb的验证系统的Bug及使用心得
    作者: 本站会员  来源:easyjf  发布时间:2007-10-12 12:57:00
  • 今天使用EasyJWeb的“人性化”验证功能,发现了存在不少Bug,这里发出来并与大家分享。

        我们系统中的User基类的内容如下:

    @Entity
    @Cache(usage = CacheConcurrencyStrategy.NONSTRICT_READ_WRITE)
    @javax.persistence.Table(name = "UserInfo")
    @Inheritance(strategy = InheritanceType.JOINED)
    @FormPO(inject = "name,email,password,problem,solution")
    public abstract class User implements Principal, UserDetails {

     private static final long serialVersionUID = 2915030721449148555L;

     @Id
     @GeneratedValue(strategy = GenerationType.TABLE)
     private Long id;

     @Column(length = 30, unique = true)
     @Field(validator = @Validator(name = "string", value = "blank;trim;max:30", required = true))
     private String name;

     @Column(length = 100)
     @Field(validator = @Validator(name = "email", required = true, value = "max:100"))
     private String email;

     @Column(length = 64)
     @Field(validator = @Validator(name = "string", value = "blank;trim;max:64", required = true))
     private String password;

     private Date registerTime;

     private Integer status = 1;// -1锁定、0未审核、1通过

     private Integer loginTimes;

     private Date lastLoginTime = new Date();

     private String lastLoginIP;

     private String problem; // 密码提问

     private String solution; // 答案

     private Long imUin;

     @ManyToMany(mappedBy = "users", cascade = CascadeType.ALL)
     @JoinTable(name = "User_Roles")
     private List roles = new java.util.ArrayList();

     @ManyToMany(cascade = CascadeType.ALL)
     @JoinTable(name = "User_Permissions")
     private List permissions = new java.util.ArrayList ();

     @Transient
     private GrantedAuthority[] authorities;

     ...

    }

    下面是一个User的子类,PersonalUser的内容如下:

    @FormPO(label="个人用户信息",disInject="score,activeCode")
    @Entity
    @Cache(usage = CacheConcurrencyStrategy.NONSTRICT_READ_WRITE)
    public class PersonalUser extends User {
     /**
      * 积分
      */
     private Integer score;

     /**
      * 真实姓名
      */
     @Field(name="真实姓名", validators = { @Validator(name = "string", value = "msg:请填写您的真实姓 名。;blank;min:2;max:10", required = true) })
     @Column(length = 20)
     private String trueName;

     /**
      * 身份证号
      */
     @Field(name="身份证号", validators = { @Validator(name="regex",value="msg:请填写有效的证件号码。;expression:\\d{15} |\\d{17}[\\dXx]", required = true) })
     @Column(length = 20)
     private String identifyCard;

     /**
      * 电话
      */
     @Field(name="电话", validators = { @Validator(name = "required", value = "msg:您填写的身份证号码有误,请检查后重新提 交。;blank;min:7;max:15", required = true) })
     @Column(length = 20)
     private String tel;

     /**
      * 手机
      */
     @Field(name="手机", validators = { @Validator(name = "regex", value = "expression:\\d{11}", required = true) })
     @Column(length = 25)
     private String mobile;

    ..
    }

    下面是个人用户注册保存时的源代码:

    /**
      * 保存个人注册信息
      *
      * @param form
      * @return
      */
     public Page savePersonal(WebForm form) {  
      PersonalUser user = form.toPo(PersonalUser.class);
      if(hasErrors())return page("registerStep3Personal");/ 回注册页面
      //执行注册操作
      this.userService.addPersonalUser(user);
      return page("success");
     }


        执行这个程序,针对name及password等属性的验证能正常工作。但PersonalUser中针对电话号码、身份证等属性无论如何都不能正常执行 验证功能,真是郁闷啊。后来仔细发现,不但没有执行验证,尽然连值也没有传入到持久层中。因此,应该是不允许注入这些属性值的问题所 允许的。于是跟踪了一下源代码,在com.easyjf.web.core.FormHandler中发现如下的内容:
        public boolean checkFormPOWriteEnabled(Class clz,
       java.beans.PropertyDescriptor property) {
      boolean ret = true;
      FormPO formPO = (FormPO) clz.getAnnotation(FormPO.class);
      boolean haveFound = false;
      if (formPO != null) {
       String injectEnabled = formPO.inject();
       
       if (StringUtils.hasLength(injectEnabled)) {
        ret = ("," + injectEnabled + ",").indexOf(","
          + property.getName() + ",") >= 0;// 找不到,则不注入
        if (ret)
         haveFound = true;
       } else {
        String disInject = formPO.disInject();    
        if (StringUtils.hasLength(disInject))
         ret = ("," + disInject + ",").indexOf(","
           + property.getName() + ",") < 0;// 找到,则不注入
        if (!ret)
         haveFound = true;
       }  
      }
      if (!haveFound && clz.getSuperclass() != Object.class)
       ret = checkFormPOWriteEnabled(clz.getSuperclass(), property);
      return ret;
     }

     这个方法写得比较有意思,几行代码转了几个弯,真考验人的逻辑思维啊。他是在检查@FormPO标签中的inject或disInject属性情况 。在子类中若找不到标签属性,还会到父类中去找。感觉这里问题就出来了,如果父类跟子类使用的注入指标标签不一致的话,应该就会出现 问题。也就是说如果父类中使用inject来标签注入的属性,而子类中使用disInject来指定不可以注入的属性。则先在子类中发现这个属性没找 到,则haveFound就为false,则到父类中去找,由于父类使用inject标识,而一般他不会把子类的属性也标识进去,所以肯定得到的结果也是 false。因此,就会给我们无法注入的结果。
        这应该是跟子类中想要表达的意思不一致:“也即子类规定的是除了disInject中规定的属性以外(或者是所有属性,即inject及disInject 都没有用的情况下)的所有属性都可以注入”这一功能无法实现。
     
        这不知道算不算上是一个Bug。但想了一下,这个Bug也可以完全避免,只要把父类及子类的注入标识写一致就OK了。于是我把两者的注入 标识都改成了使用disInject属性。如下:

    正常的User基类

    @FormPO(disInject = "id,status,loginTimes,lastLoginTime,lastLoginIP,roles,permissions,authorities")
    public abstract class User implements Principal, UserDetails {

     ...

    }

     

    @FormPO(label="个人用户信息",disInject="score")
    public class PersonalUser extends User {

    ..
    }

        然后执行刚才的用户注册程序,验证系统终于“人性化”地按我要的出来了。呵呵,大功告成,立即走人。
                                                                                                                                                                                                         

  • 评论 】 【收藏】 【 推荐给朋友 】 【字体: 】 【关闭
评论:共0条

发表评论:
评论: 
    

Copyright (C) 2005 EasyJF.com 简易java框架网 渝ICP备06004507号
如有意见请与我们联系