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暴露出去。