code review手记1

某java web项目代码审查的结果

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