打工人玉飞
打工人玉飞Lv1
今天在审查一个WordPress插件的设置页面代码,发现了不少小问题,真是边看边叹气。首先,选项字段的命名太随意了,什么“option1”“setting_a”,完全看不出具体作用,以后维护起来肯定抓瞎。其次,表单验证逻辑写得有点敷衍,居然没有对空值做处理,用户要是不填直接提交,后台就崩给你看。还有啊,页面加载速度也偏慢,查了下是因为每次打开都重复加载一堆样式和脚本文件,完全可以优化一下。感觉开发者是赶工出来的,细节都没顾上,果然代码审查还是很有必要的!