组里面下任务,下发了几个老系统里面的js文件来做重构设计。
抽了其中一小段出来搞了一下,感觉还是有收获的,先贴上自己重构的代码,后续还会有讨论的分享。
主要的业务逻辑是在表单提交之前做很多的逻辑验证,金额以及日期的选择,由于是后台系统,所以交互上面不是特别看重,错误提示都直接alert了。
这段代码主要问题是if..else的嵌套太多,此外感觉有一些地方原来写的不严谨。
我的第一反应是将过多的嵌套去掉,增强代码的可读性, 所以这是第一个重构迭代的主要目标;
第二轮,就会想想是不是可以把一些东西抽象一下。
优化的代码也可以直接看这里。



- Most commented