code review手记3

DQR:
  1. getConnection()不应该再调用了,而应该调用JdbcSource的模版方法;
  2. _dbindicator完全不需要做一个成员变量,可以采用临时变量取代;
  3. List getDatasetIndicators 后面的DataSet的S应该大写,风格至少要统一;
DataForge:
  1. 网页文件UpdateHandler等不符合命名规范;
Security:
  1. VdbManager.getUsername()不正确,不应该是单用户; 
  2. VdbSecurityManager.getUsername()不合理,应该getUser()返回User对象; 
  3. VdbManager.getUsername()不合理,应该仅仅返回SecurityManager,类似的方法多达22个!SecurityManager的所有方法都由VdbManager做了包装,完全可以直接将SecurityManager暴露出去

posted on 2010-03-09 12:39  白乔  阅读(106)  评论(0编辑  收藏  举报

导航