这一周在维护自己一年前才进公司写的代码.经过领导,同事,同学审视提出了各种建议.零零散散,总结如下:
1 变量命名
我原先直接用 FTPUtil test = new FTPUtil();
修改成ftpUtil…
2 系统输出
我原先直接用 System.out.println(“xxx”);部署到生产环境上面.发现在catalina.out中一直有输出…但不知道是哪一行什么时间输出的…
删掉!或者改成log.info
3 日志输出
我原先直接 log.info(“开始删除数据”);
修改成英文log.info(“Delete data start”);
4 对象判空或者null
我原先直接不管Object object = new Object();
修改成if(object.isEmpty()){…}else {…}
5 异常处理
我原先没考虑这个,对于那种工具类的方法抛异常,外面怎么做的情况
如果调用某个方法,里面就有异常那么就在调用语句外加上try{}catch(Exception e){…}
修改成 try{}catch(Exception e){e.print…};
6 日志级别
我原先只用log.info,对于不用的打印数据,方便后期从日志上面查找数据
修改成log.debug
7 计算次数
我原先从不考虑日期转换需要进行多少次,想到什么就直接写了。直到大佬说只用计算一次,后面直接用就好了。修改为变量保存计算的值,后面使用直接调用
8 日志输出
避免有歧义的输出,宁可少写,不要误导。
重新审阅一年前自己写的代码.
最新推荐文章于 2021-04-06 10:40:33 发布