如何评价我旁边同事写的代码,给我气笑了

查看 399|回复 48
StubbornHuang   
笑点?
oneKnow   
除了判断密码太简单那里不严谨,这么写有什么问题吗,大佬们教一下
zuixinwenyue   
请贴一个你认为不笑的代码
EsWann   
简单清晰,一目了然
评价:好代码
tool2d   
123456 本来就是占行用的,以后改成查询本地弱密码字典,就没问题了。
把程序写复杂不难的,github 上一堆你看不懂的代码。能让任何人都能看懂的代码,却很少。
woodfizky   
多嘴问一句,PHP 的 IDE 会自动补充那个$符号吗还是每次输入变量都得自己手敲?。。
sanwang566123   
@StubbornHuang 123456 、1234567 、12345678
fpure   
笑点是什么?
SeaRecluse   
楼上都是认真的吗...
首先这段代码命名问题,true_pwd 应该是 pwd 二次校验,写成 true_pwd 意义不明,因为后面 md5 取得是 pwd 的值
然后逻辑冗余,除非是项目要求,不然空字符也属于两次属于密码不一致的范畴
字符串判别勉强可以接受,最好还是使用 md5 进行比较
最后才是简单正则的校验问题,这个在我看来倒是问题不大,可以预先输入一些简单密码进行匹配测试,因为前文也没有看到密码组合复杂度的要求,如果有要求,则有必要加强这部分
SilentOrFight   
少了 trim( ) ?
您需要登录后才可以回帖 登录 | 立即注册

返回顶部