最近从公司接手了一个14年的项目,该项目主要是一个活动报名系统，活动每年举办一次，每次从各个学校推荐的人才中进行筛选并进行培养。
今天该活动也即将开始报名且客户需要有一些功能上的更改，因此公司让我负责此项目，但是在我看了代码后整个人感觉都不好了。
下面是报名时的面试时间选择项,看了下源文件竟然是写死的
更无语的还在后面,下面是后台管理员导出报名信息功能，半天才看明白代码中switch的逻辑(如果用户没有选择某个日期则以空字符串填充,目的应该是在excel里进行过滤、统计等)
重构后的代码
1.报名日期更改为动态保存到数据库
2.删除switch逻辑代码,在输出sheet时再检查用户的报名日期是否存在于全局报名日期数组中,如果不存在则以空字符串填充并继续(多执行了N*N次in_array会带来一定的性能开销)
接手别人的代码其实是痛苦的,但我们应该以一种学习的心态去看待,如果别人写的好我们应该学习
如果写的不好我们也应该学习(如何避免自己犯类似的错误)