关于代码的一些感想

最近从公司接手了一个14年的项目,该项目主要是一个活动报名系统，活动每年举办一次，每次从各个学校推荐的人才中进行筛选并进行培养。

今天该活动也即将开始报名且客户需要有一些功能上的更改，因此公司让我负责此项目， 但是在我看了代码后整个人感觉都不好了。

 

下面是报名时的面试时间选择项,看了下源文件竟然是写死的

 

更无语的还在后面,下面是后台管理员导出报名信息功能，半天才看明白代码中switch的逻辑(如果用户没有选择某个日期则以空字符串填充,目的应该是在excel里进行过滤、统计等)

 

 

重构后的代码

1. 报名日期更改为动态保存到数据库

2.删除switch逻辑代码,在输出sheet时再检查用户的报名日期是否存在于全局报名日期数组中,如果不存在则以空字符串填充并继续(多执行了N*N次in_array会带来一定的性能开销)

 

 

接手别人的代码其实是痛苦的,但我们应该以一种学习的心态去看待,如果别人写的好我们应该学习

如果写的不好我们也应该学习(如何避免自己犯类似的错误)

来自为知笔记(Wiz)