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

图裂了

图炸了

就你这贴图的本事，让其他观众无言以对

哈哈哈哈哈哈直接提示账号密码是 admin ？屎一样的代码。

PS：图裂是因为新浪图床防盗链了。

贵公司还招前端吗？我可以去学

@zaima 我错了，下次换个图床

@fighterlyt 没经验

这能说明啥？这明显不是他的最终代码……
也许他只是想先调一下错误提示样式，等业务上确定了提示的内容和提示方式以后，才能真正放入提示逻辑。
我觉得这是一个有自己开发方法的前端

[img] [img]


不用谢

这代码完全没毛病，Demo 代码不都这样么？

这有点断章取义吧， 也许是因为 lz 后端接口没搞定，前端写些填充代码呢

@shawshi emmm 给公司行政用的系统，行政说了，这系统好，不怕忘记密码

@lovescar 这个账号和密码感到很悲伤，因为自己完全没有存在的价值。

@wizardoz 给我的就是他的最终版了

@lovescar 不如去掉账号验证算了

看到这代码我是想先吐槽楼主公司的系统这么屎

一点毛病都没有

没毛病，行政一般都希望 admin/admin

本是同根生

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

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

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

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

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

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

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

说下我的观点，

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

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

@supuwoerc 好好审题，认真看图

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

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

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

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

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

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

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

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

@binux 因为后端要求，哈哈

@tianxia .netcore

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

@daydaydayup 这个逻辑有呀

这个代码是很屎。。。
and
我感觉你们前后端的实现方案也是有问题的。。。

你是看不起 vue 写得辣鸡吗？

@lovescar 你看着功能都得到别人的一直好评了，说明这个代码很优秀。

歪个楼：

前后端互看不爽，都举得这么简单的事情应该是对方做，以免污染了自己的代码……

所以前后端还是都自己干吧……

不要跟我说没精力，这是技术架构的问题，本来前后端就不应该分得太清楚

@shawshi 也就是说，没有开发业务经验，只要代码优秀就可以咯。了解了

我只想知道你为啥知道前端的薪资

@lovescar 前端 感觉用户体验优先。后端注意自己的参数校验和业务逻辑。

反正这种代码我是看着难受。。。

return else 什么的 ，同样的代码写 2 遍。看着难受

而且无言以对，并不是觉得他写的不好，真的只是无言以对

如果不是有人强烈要求这样提示的话，我觉得这代码写的有点置气的意思了。

这系统为什么要做登录啊，直接打开就能用不好吗

@lovescar
你不应该怪开发，应该怪奇葩的需求

说下我的看法
为什么不直接把用户密码填充好呢？
简单省事，点击登录就行了，还要绕个圈子

内容就不评价了，『公司招的 1w 多的前端』 这什么鬼？

这个时候就体现出薪资保密的重要性了

『公司招的 1w 多的前端』 跟你想表达的东西有什么关联吗

把业务实现就好了。为啥还会在意代码质量。这代码是可以继续优化，不过功能就是这么简单实现了不就好了嘛。要不你教一下他怎么写嘛。

文人相轻？
都是领工资的，说的好听点是程序员。

楼主怎么知道前端的薪资的，没有保密吗

@orqzsf1 难道你就不知道公司其他人的薪资么？

其实都是需求的锅，不要起内讧，这段代码没有复用验证可以轻喷，其他挺好啦

@jevirs 搞不懂做个非空校验，有必要这么写么，而且当时就交付了，交付给我用户名为 admin ,到你手里，我不知道你会不会觉得写的不错，但是一言难尽呢

没 JB 说个图ε=ε=ε=┏(ロ;)┛

@Cloutain 没 jb 不能说么

这个是不是得先看 是不是需求比较奇葩。。

ps 我看不懂代码 哈哈

所以说不能打听别人工资，比你多比你少心里都不舒服.

@tianxia
@stiekel
@axwz88
@ThomasZ
可能问的是为什么有 return 了还写 else。
比如
```
if(something()){
// do A
return;
}else{
// do B
}
```
和
```
if(something()){
// do A
return;
}
// do B
```
的区别。不少 IDE 都会提示使用下面的写法的吧。

所以为啥要账号密码，看着爽？

是不是没什么水平的人都喜欢满瓶子不响，半瓶子咣当？

lz 这种口吻我看不像是科班出身的，倒想是培训班出来的，不知道我猜测的对不对？
给我的感受就是，一个木匠吹嘘自己了不起，为什么了不起？我一分钟能敲个 72 个钉子，他只能敲 60 个。
给人一股没见过世面的感觉。

真的，咱们要说也说点高大上的东西，比点真正能代表程序员水平的东西，
谈点架构，编程技巧，可读性提升之类的，
毕竟自认是工程师，
自嘲是码农，就真把自己当农民了，比起我比你插秧快多少多少，
觉得鄙薄无知，这就是我的感受。

水份很大啊。相同的逻辑在不同人上实现的风格都不同,特别是像 js 这种灵活性很大的语言

我有个业务问题问问，

this.account !== 'admin' 才报错，等于的时候不报错，这个是什么业务？

而且这代码没问题，逻辑简单，也没什么冗余或者特别的东西，这样挺好的啊。

@wly19960911 #72 登陆提示？登陆不进去就直接告诉你账号和密码吧

@abc612008 只是体现出了 v2 中一部分人的水平而已，看过笑过就行了，我都不想去提……

这是业务逻辑问题，业务逻辑正常情况下是产品去梳理，包括提示内容按理都是产品给的。你没给他详细业务逻辑，每个人对业务逻辑实现有自己的理解。另外可能你俩沟通问题，或者分工问题，有的公司前段只写页面，js 逻辑也是后段写的。

感觉是需求有点乱