某java web项目代码审查的结果
- 太多的编码、转码代码
String uid = new String(request.getParameter("T1").getBytes("ISO8859-1"),"GBK"); - 使用"abc".equalsIgnoreCase(s)，不要s.equals("abc")
- 不要提供没用的构造函数
- 不要提供main()，请使用Junit
- 文件名带空格？大小写也不规范
- 太多的冗余代码：
report/template1.jsp里面为什么出现上传文件的代码？
sqlView/js/sqlView.js里面出现一些没用的按钮操作代码？ - 尽量采用Action，将大段的java代码封装起来；
- javascript用得太乱！页面里面东一块西一块；
- 权限判定居然用js?