重新审阅一年前自己写的代码.

这一周在维护自己一年前才进公司写的代码.经过领导,同事,同学审视提出了各种建议.零零散散,总结如下:
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 日志输出
避免有歧义的输出,宁可少写,不要误导。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值