坐标天津
直接看图吧
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
This topic created in 2743 days ago, the information mentioned may be changed or developed.
Supplement 1 · May 5, 2019
更新一下：看了大家的回复，觉得自己做的不对，他代码写的确实不错，值得我这种菜鸡后端学习，而且当时和前端交流时，只是告诉他做个校验，没有告诉他做非空校验，不需要提示出来用户名密码是啥。还有下次一定好好分析接口代码，做个合格的后端😜😜😜
 |
zaimaMay 5, 2019
1
图裂了
|
 |
programHuiMay 5, 2019
2
图炸了
|
 |
yjmhero007May 5, 2019
3
|
 |
4
就你这贴图的本事，让其他观众无言以对
|
2 |
Patrick95May 5, 2019
5
哈哈哈哈哈哈直接提示账号密码是 admin ？屎一样的代码。
PS：图裂是因为新浪图床防盗链了。 |
 |
SankoMay 5, 2019 via Android
6
贵公司还招前端吗？我可以去学
|
 |
8
@fighterlyt 没经验
|
 |
wizardozMay 5, 2019
9
这能说明啥？这明显不是他的最终代码……
也许他只是想先调一下错误提示样式，等业务上确定了提示的内容和提示方式以后，才能真正放入提示逻辑。 我觉得这是一个有自己开发方法的前端 |
 |
yuezhenglingMay 5, 2019 via Android
10
[img]
 [img]不用谢 |
 |
shawshiMay 5, 2019
11
这代码完全没毛病，Demo 代码不都这样么？
|
 |
dingyaguang117May 5, 2019
12
这有点断章取义吧， 也许是因为 lz 后端接口没搞定，前端写些填充代码呢
|
 |
16
@lovescar 不如去掉账号验证算了
|
 |
zzNuckerMay 5, 2019
17
看到这代码我是想先吐槽楼主公司的系统这么屎
|
 |
tianxiaMay 5, 2019 via Android
18
一点毛病都没有
|
 |
TinkMay 5, 2019
19
没毛病，行政一般都希望 admin/admin
|
 |
20
本是同根生
|
 |
docoMay 5, 2019
21
这个代码如果真的这个逻辑的话还可以优化吧...
verifyAccount 和 verifyPwd 改一下再在 submit 里直接调用就可以了吧 |
 |
supuwoercMay 5, 2019
22
都 return 了为啥还要 else 啊。。。
|
 |
sunjourneyMay 5, 2019
23
这不是 PM 的锅吗？ PM 也许就是这么要求的。。如果不是，贵司问题大得老去了
|
|
24
人家还用 vue 做，敢问你后端用的是什么？
|
 |
chinesestudioMay 5, 2019 via Android
26
内部非技术人员用 正常的很 简单粗暴 他们只管用起来舒服 不管怎么实现
|
 |
paloaltoMay 5, 2019
28
说下我的观点，
优点：首先他知道面向对象编程，而且代码逻辑很明白，即使是小白都能懂，以后别人接手修改时就没有难度；其次，代码规范不错，空格、标点、换行、驼峰命名都没毛病，甚至能用 !== 而不是 != ，很不错了。 唯一的缺点，可能就是没有做到代码复用。submit() 里的那俩判断其实可以通过给上面的俩方法参调用来简化。 |
 |
st2udioMay 5, 2019
32
这明显是别人要求这样做的吧，内网部署，不是很重要之类的。
总不能他自己想的这种需求？ |
|
supuwoercMay 5, 2019
33
呜呜呜，我错了，习惯性的不写 else，每次都是 return 解决~ 下次一定审题
|
 |
binuxMay 5, 2019
34
你们难道就没想过这个问题吗？
前端为什么要验证账号密码？ |
 |
daydaydayupMay 5, 2019 via iPhone
39
为何不是后端去验证返回验证错误码？
|
|
40
@daydaydayup 这个逻辑有呀
|
 |
phxsunsMay 5, 2019
41
这个代码是很屎。。。
and 我感觉你们前后端的实现方案也是有问题的。。。 |
 |
123sMay 5, 2019
42
你是看不起 vue 写得辣鸡吗？
|
 |
EulerChenMay 5, 2019
44
歪个楼：
前后端互看不爽，都举得这么简单的事情应该是对方做，以免污染了自己的代码…… 所以前后端还是都自己干吧…… 不要跟我说没精力，这是技术架构的问题，本来前后端就不应该分得太清楚 |
 |
mars0princeMay 5, 2019
46
我只想知道你为啥知道前端的薪资
|
 |
huijieweiMay 5, 2019
48
反正这种代码我是看着难受。。。
return else 什么的 ，同样的代码写 2 遍。看着难受 |
|
49
而且无言以对，并不是觉得他写的不好，真的只是无言以对
|
 |
LaplaceTransformMay 5, 2019
50
如果不是有人强烈要求这样提示的话，我觉得这代码写的有点置气的意思了。
|
 |
janxinMay 5, 2019
51
这系统为什么要做登录啊，直接打开就能用不好吗
|
 |
WordTianMay 5, 2019 via Android
53
说下我的看法🤔
为什么不直接把用户密码填充好呢？ 简单省事，点击登录就行了，还要绕个圈子 |
 |
meteor957May 5, 2019
54
内容就不评价了，『公司招的 1w 多的前端』 这什么鬼？
|
 |
s112May 5, 2019
55
这个时候就体现出薪资保密的重要性了
|
|
meteor957May 5, 2019
56
『公司招的 1w 多的前端』 跟你想表达的东西有什么关联吗
|
 |
smiledollMay 5, 2019 via Android
57
把业务实现就好了。为啥还会在意代码质量。这代码是可以继续优化，不过功能就是这么简单实现了不就好了嘛。要不你教一下他怎么写嘛。
|
 |
niguyiMay 5, 2019
58
文人相轻？
都是领工资的，说的好听点是程序员。 |
 |
orqzsf1May 5, 2019
59
楼主怎么知道前端的薪资的，没有保密吗
|
 |
jevirsMay 5, 2019
61
其实都是需求的锅，不要起内讧，这段代码没有复用验证可以轻喷，其他挺好啦
|
|
62
@jevirs 搞不懂做个非空校验，有必要这么写么，而且当时就交付了，交付给我用户名为 admin ,到你手里，我不知道你会不会觉得写的不错，但是一言难尽呢
|
 |
CloutainMay 5, 2019
63
没 JB 说个图ε=ε=ε=┏(゜ロ゜;)┛
|
 |
DAMNYOUMay 5, 2019
65
这个是不是得先看 是不是需求比较奇葩。。
ps 我看不懂代码 哈哈 |
 |
catinsidesMay 5, 2019
66
所以说不能打听别人工资，比你多比你少心里都不舒服.
|
 |
67
|
 |
momo1999May 5, 2019
68
所以为啥要账号密码，看着爽？
|
 |
dfjslkjdfMay 5, 2019
69
是不是没什么水平的人都喜欢满瓶子不响，半瓶子咣当？
lz 这种口吻我看不像是科班出身的，倒想是培训班出来的，不知道我猜测的对不对？ 给我的感受就是，一个木匠吹嘘自己了不起，为什么了不起？我一分钟能敲个 72 个钉子，他只能敲 60 个。 给人一股没见过世面的感觉。 |
|
dfjslkjdfMay 5, 2019
70
真的，咱们要说也说点高大上的东西，比点真正能代表程序员水平的东西，
谈点架构，编程技巧，可读性提升之类的， 毕竟自认是工程师， 自嘲是码农，就真把自己当农民了，比起我比你插秧快多少多少， 觉得鄙薄无知，这就是我的感受。 |
 |
jimliangMay 5, 2019
71
水份很大啊。相同的逻辑在不同人上实现的风格都不同,特别是像 js 这种灵活性很大的语言
|
 |
wly19960911May 5, 2019
72
我有个业务问题问问，
this.account !== 'admin' 才报错，等于的时候不报错，这个是什么业务？ 而且这代码没问题，逻辑简单，也没什么冗余或者特别的东西，这样挺好的啊。 |
 |
WolfsinMay 5, 2019
73
@wly19960911 #72 登陆提示？登陆不进去就直接告诉你账号和密码吧
|
 |
laodaoMay 6, 2019
75
这是业务逻辑问题，业务逻辑正常情况下是产品去梳理，包括提示内容按理都是产品给的。你没给他详细业务逻辑，每个人对业务逻辑实现有自己的理解。另外可能你俩沟通问题，或者分工问题，有的公司前段只写页面，js 逻辑也是后段写的。
|
 |
lingxipaofanMay 7, 2019
76
感觉是需求有点乱
|
Select Language