新入职员工代码问题较多,在这里记录一下这些问题,以后可以作为新员工的入职培训。
- 功能完成后一定要测试;
- 变量命名规范,不要使用a、a1这种;
- 注释一定要写,方法、属性、对象、类,如果不想以后都由自己维护,请增加代码可读性;
- 无特殊情况不要使用system.out.print(),要使用log4j打印到日志中,部署出去后才能进行运维,出问题才好查;
- 资源的释放:druid使用时,没有释放连接,造成数据库连接池用完,影响所有数据库相关操作;IO流、文件、数据库、连接等资源使用完成后必须要及时关闭,将资源释放;
- 循环里面写数据库查询、添加、修改操作,大量占用数据库资源;
- 分页时,数据做了条件筛选,但是总数量没有做筛选;
- 完全不考虑工具类的使用,以前的工具类不是摆设啊;
- 不能任意修改公共方法,修改原有代码前先查询使用该方法的全部代码,看是否会受到影响;
- 远程调用外部接口时,不能直接截取正确数据后反馈,应该将整体返回,如果外部接口出现故障,直接截取部分会出现异常,一定要考虑多种情况;
- 不要使用魔法值,先在常量池定义常量,名称遵守规范;
- 用“==”比较字符串;
- 在原本代码内增加业务时:考虑从线程池使用新的线程执行;一定要try/catch;
- 不能在foreach里修改list的结构,添加或删除;
- 多次使用魔鬼数值,完全不写注释,根本不知道是什么含义,时间长了可能自己也忘了;
- 代码提交时不写提交信息,不知道具体有什么变动,还需要去翻代码才知道;