zoukankan      html  css  js  c++  java
  • 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暴露出去
  • 相关阅读:
    设计模式之单例模式实践
    有关集合的foreach循环里的add/remove
    项目中常用的MySQL优化方法--壹拾玖条
    Solr
    Lucene补充
    Lucene
    一千行 MySQL 学习笔记
    Servlet
    CSS未知宽高元素水平垂直居中
    深拷贝和浅拷贝
  • 原文地址:https://www.cnblogs.com/bluejoe/p/5116031.html
Copyright © 2011-2022 走看看