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

查看 397|回复 48
wanqiangcrack   
更多的是逻辑不够凝练的问题,代码本身没什么大问题。 还有,不要再蹭 Java PHP 这种语言的问题,这类人用什么语言写都是这个逻辑。
Zorro825   
我也是写 PHP 的,代码问题啊,大牛在哪里,请问是什么问题?
fcbwalk   
我劝你不要多管闲事,影响你工资了?😜
guozhiqin   
就那个 123456 有点搞笑。
woodfizky   
我很多次想指出问题的时候都忍住了,然后发现有问题的点都在 123456 那里,然后仔细想想这个同事应该只是比较幽默,写个 123456 就相当于幽默版的标记个 TODO 罢了。
确实其实把那个 123456 那里替换成一个密码长度和复杂度校验的方法就好了,其他确实没啥毛病。
Promtheus   
除了 123456 其他没啥大问题啊
hupeng   
123456 可能是老板要求的
lybcyd   
@MicroG 参考常见成熟框架的设计:
1 、表单验证是分离的 DTO 或者 Request 类来封装负责的,controller 里面只需要调用 validate 方法走页面响应逻辑就行。像这种做法,如果字段多起来会很杂乱,而且规则一条条判断也容易写错。找个验证库,大多支持链式调用或者字符串形式定义验证规则,要清晰很多。
2 、简单密码只查个 123456 这个就不用说了吧。。正常来说应该是规定好密码强度的要求,比如大小写字母、数字、特殊符号必须有几种,通过正则之类的方法实现。再复杂一点还会规定一个常用单词表,不允许里面的单词出现比如 password 等等。
3 、密码加密只有一个 md5 ,现在彩虹表之类的手段很多,最起码也得加个盐,要不分分钟就被人撞了。其实现在成熟框架用 md5 加密的很少了,很多都会使用 bcrypt 、PBKDF2 之类的算法。
gyinbj   
@woodfizky 一般都是敲
您需要登录后才可以回帖 登录 | 立即注册

返回顶部