公司招的 1w 多的前端，写出的代码，让我这个后端无言以对

[lovescar]

坐标天津 直接看图吧

[doco]

这个代码如果真的这个逻辑的话还可以优化吧...
verifyAccount 和 verifyPwd 改一下再在 submit 里直接调用就可以了吧

[supuwoerc]

都 return 了为啥还要 else 啊。。。

[sunjourney]

这不是 PM 的锅吗？ PM 也许就是这么要求的。。如果不是，贵司问题大得老去了

[tianxia]

人家还用 vue 做，敢问你后端用的是什么？

[zzNucker]

@doco 我也这么想的，不过我感觉这个系统反正也没啥逻辑，开发估计就是直接拷贝粘贴拉倒了。

[chinesestudio]

内部非技术人员用 正常的很 简单粗暴 他们只管用起来舒服 不管怎么实现

[tianxia]

@supuwoerc 看清楚，不 return 就继续往下走了

[paloalto]

说下我的观点，

优点：首先他知道面向对象编程，而且代码逻辑很明白，即使是小白都能懂，以后别人接手修改时就没有难度；其次，代码规范不错，空格、标点、换行、驼峰命名都没毛病，甚至能用 !== 而不是 != ，很不错了。

唯一的缺点，可能就是没有做到代码复用。submit() 里的那俩判断其实可以通过给上面的俩方法参调用来简化。

[ThomasZ]

@supuwoerc 好好审题，认真看图

[stiekel]

@supuwoerc submit() 中第一个 return 是没有问题的，用户名错了直接就不检查密码了。

[axwz88]

@supuwoerc 满足 if 条件才 return，不满足就 else

[st2udio]

这明显是别人要求这样做的吧，内网部署，不是很重要之类的。

总不能他自己想的这种需求？

[supuwoerc]

呜呜呜，我错了，习惯性的不写 else，每次都是 return 解决~ 下次一定审题

[binux]

你们难道就没想过这个问题吗？
前端为什么要验证账号密码？

[lovescar]

@st2udio 我就是哪个后端，我告诉他做个校验，没了

[lovescar]

@binux 前端为什么不能校验非空

[tianxia]

@binux 因为后端要求，哈哈

[lovescar]

@tianxia .netcore

[daydaydayup]

为何不是后端去验证返回验证错误码？

[lovescar]

@daydaydayup 这个逻辑有呀