这是一个创建于 2063 天前的主题，其中的信息可能已经有所发展或是发生改变。
坐标天津
直接看图吧
第 1 条附言 · 2019-05-05 15:17:41 +08:00
更新一下：看了大家的回复，觉得自己做的不对，他代码写的确实不错，值得我这种菜鸡后端学习，而且当时和前端交流时，只是告诉他做个校验，没有告诉他做非空校验，不需要提示出来用户名密码是啥。还有下次一定好好分析接口代码，做个合格的后端😜😜😜
 |
1
zaima 2019-05-05 14:42:57 +08:00
图裂了
|
 |
2
programHui 2019-05-05 14:45:11 +08:00
图炸了
|
 |
3
yjmhero007 2019-05-05 14:47:08 +08:00
|
 |
4
fighterlyt 2019-05-05 14:47:09 +08:00  2
就你这贴图的本事，让其他观众无言以对
|
 |
5
Patrick95 2019-05-05 14:47:43 +08:00
哈哈哈哈哈哈直接提示账号密码是 admin ？屎一样的代码。
PS：图裂是因为新浪图床防盗链了。 |
 |
6
Sanko 2019-05-05 14:47:45 +08:00 via Android
贵公司还招前端吗？我可以去学
|
 |
8
lovescar OP @fighterlyt 没经验
|
 |
9
wizardoz 2019-05-05 14:51:13 +08:00
这能说明啥？这明显不是他的最终代码……
也许他只是想先调一下错误提示样式，等业务上确定了提示的内容和提示方式以后，才能真正放入提示逻辑。 我觉得这是一个有自己开发方法的前端 |
 |
10
yuezhengling 2019-05-05 14:51:17 +08:00 via Android
[img]
 [img]不用谢 |
 |
11
shawshi 2019-05-05 14:51:20 +08:00
这代码完全没毛病，Demo 代码不都这样么？
|
 |
12
dingyaguang117 2019-05-05 14:51:43 +08:00
这有点断章取义吧， 也许是因为 lz 后端接口没搞定，前端写些填充代码呢
|
 |
16
learnshare 2019-05-05 14:55:13 +08:00 2
@lovescar 不如去掉账号验证算了
|
 |
17
zzNucker 2019-05-05 14:55:47 +08:00
看到这代码我是想先吐槽楼主公司的系统这么屎
|
 |
18
tianxia 2019-05-05 14:55:56 +08:00 via Android
一点毛病都没有
|
 |
19
Tink 2019-05-05 14:57:23 +08:00
没毛病，行政一般都希望 admin/admin
|
 |
20
yalin 2019-05-05 14:57:36 +08:00 1
本是同根生
|
 |
21
doco 2019-05-05 14:59:26 +08:00
这个代码如果真的这个逻辑的话还可以优化吧...
verifyAccount 和 verifyPwd 改一下再在 submit 里直接调用就可以了吧 |
 |
22
supuwoerc 2019-05-05 15:00:11 +08:00
都 return 了为啥还要 else 啊。。。
|
 |
23
sunjourney 2019-05-05 15:00:39 +08:00
这不是 PM 的锅吗？ PM 也许就是这么要求的。。如果不是，贵司问题大得老去了
|
|
24
tianxia 2019-05-05 15:01:04 +08:00 via Android 1
人家还用 vue 做，敢问你后端用的是什么？
|
 |
26
chinesestudio 2019-05-05 15:02:40 +08:00 via Android
内部非技术人员用 正常的很 简单粗暴 他们只管用起来舒服 不管怎么实现
|
 |
28
paloalto 2019-05-05 15:04:03 +08:00
说下我的观点，
优点：首先他知道面向对象编程，而且代码逻辑很明白，即使是小白都能懂，以后别人接手修改时就没有难度；其次，代码规范不错，空格、标点、换行、驼峰命名都没毛病，甚至能用 !== 而不是 != ，很不错了。 唯一的缺点，可能就是没有做到代码复用。submit() 里的那俩判断其实可以通过给上面的俩方法参调用来简化。 |
 |
32
st2udio 2019-05-05 15:07:18 +08:00
这明显是别人要求这样做的吧，内网部署，不是很重要之类的。
总不能他自己想的这种需求？ |
|
33
supuwoerc 2019-05-05 15:07:42 +08:00
呜呜呜，我错了，习惯性的不写 else，每次都是 return 解决~ 下次一定审题
|
 |
34
binux 2019-05-05 15:09:05 +08:00
你们难道就没想过这个问题吗？
前端为什么要验证账号密码？ |
 |
39
daydaydayup 2019-05-05 15:12:34 +08:00 via iPhone
为何不是后端去验证返回验证错误码？
|
|
40
lovescar OP @daydaydayup 这个逻辑有呀
|
 |
41
phxsuns 2019-05-05 15:17:23 +08:00
这个代码是很屎。。。
and 我感觉你们前后端的实现方案也是有问题的。。。 |
 |
42
123s 2019-05-05 15:18:58 +08:00
你是看不起 vue 写得辣鸡吗？
|
 |
44
EulerChen 2019-05-05 15:21:36 +08:00
歪个楼：
前后端互看不爽，都举得这么简单的事情应该是对方做，以免污染了自己的代码…… 所以前后端还是都自己干吧…… 不要跟我说没精力，这是技术架构的问题，本来前后端就不应该分得太清楚 |
 |
46
mars0prince 2019-05-05 15:27:09 +08:00
我只想知道你为啥知道前端的薪资
|
 |
48
huijiewei 2019-05-05 15:30:47 +08:00
反正这种代码我是看着难受。。。
return else 什么的 ，同样的代码写 2 遍。看着难受 |
|
49
lovescar OP 而且无言以对，并不是觉得他写的不好，真的只是无言以对
|
 |
50
LaplaceTransform 2019-05-05 15:34:06 +08:00
如果不是有人强烈要求这样提示的话，我觉得这代码写的有点置气的意思了。
|
 |
51
janxin 2019-05-05 16:03:17 +08:00
这系统为什么要做登录啊，直接打开就能用不好吗
|
 |
53
WordTian 2019-05-05 16:15:03 +08:00 via Android
说下我的看法🤔
为什么不直接把用户密码填充好呢？ 简单省事，点击登录就行了，还要绕个圈子 |
 |
54
meteor957 2019-05-05 16:15:07 +08:00
内容就不评价了，『公司招的 1w 多的前端』 这什么鬼？
|
 |
55
s112 2019-05-05 16:15:48 +08:00
这个时候就体现出薪资保密的重要性了
|
|
56
meteor957 2019-05-05 16:16:29 +08:00
『公司招的 1w 多的前端』 跟你想表达的东西有什么关联吗
|
 |
57
smiledoll 2019-05-05 16:23:38 +08:00 via Android
把业务实现就好了。为啥还会在意代码质量。这代码是可以继续优化，不过功能就是这么简单实现了不就好了嘛。要不你教一下他怎么写嘛。
|
 |
58
niguyi 2019-05-05 16:31:09 +08:00
文人相轻？
都是领工资的，说的好听点是程序员。 |
 |
59
orqzsf1 2019-05-05 16:35:43 +08:00
楼主怎么知道前端的薪资的，没有保密吗
|
 |
61
jevirs 2019-05-05 16:52:35 +08:00
其实都是需求的锅，不要起内讧，这段代码没有复用验证可以轻喷，其他挺好啦
|
|
62
lovescar OP @jevirs 搞不懂做个非空校验，有必要这么写么，而且当时就交付了，交付给我用户名为 admin ,到你手里，我不知道你会不会觉得写的不错，但是一言难尽呢
|
 |
63
Cloutain 2019-05-05 17:24:12 +08:00
没 JB 说个图ε=ε=ε=┏(゜ロ゜;)┛
|
 |
65
DAMNYOU 2019-05-05 17:40:41 +08:00
这个是不是得先看 是不是需求比较奇葩。。
ps 我看不懂代码 哈哈 |
 |
66
catinsides 2019-05-05 17:52:07 +08:00
所以说不能打听别人工资，比你多比你少心里都不舒服.
|
 |
67
abc612008 2019-05-05 18:00:52 +08:00 1
|
 |
68
shuax 2019-05-05 18:14:15 +08:00
所以为啥要账号密码，看着爽？
|
 |
69
dfjslkjdf 2019-05-05 18:54:20 +08:00
是不是没什么水平的人都喜欢满瓶子不响，半瓶子咣当？
lz 这种口吻我看不像是科班出身的，倒想是培训班出来的，不知道我猜测的对不对？ 给我的感受就是，一个木匠吹嘘自己了不起，为什么了不起？我一分钟能敲个 72 个钉子，他只能敲 60 个。 给人一股没见过世面的感觉。 |
|
70
dfjslkjdf 2019-05-05 18:57:56 +08:00
真的，咱们要说也说点高大上的东西，比点真正能代表程序员水平的东西，
谈点架构，编程技巧，可读性提升之类的， 毕竟自认是工程师， 自嘲是码农，就真把自己当农民了，比起我比你插秧快多少多少， 觉得鄙薄无知，这就是我的感受。 |
 |
71
jimliang 2019-05-05 19:17:26 +08:00
水份很大啊。相同的逻辑在不同人上实现的风格都不同,特别是像 js 这种灵活性很大的语言
|
 |
72
wly19960911 2019-05-05 21:02:11 +08:00
我有个业务问题问问，
this.account !== 'admin' 才报错，等于的时候不报错，这个是什么业务？ 而且这代码没问题，逻辑简单，也没什么冗余或者特别的东西，这样挺好的啊。 |
 |
73
Wolfsin 2019-05-05 21:33:36 +08:00
@wly19960911 #72 登陆提示？登陆不进去就直接告诉你账号和密码吧
|
 |
75
laodao 2019-05-06 13:37:40 +08:00
这是业务逻辑问题，业务逻辑正常情况下是产品去梳理，包括提示内容按理都是产品给的。你没给他详细业务逻辑，每个人对业务逻辑实现有自己的理解。另外可能你俩沟通问题，或者分工问题，有的公司前段只写页面，js 逻辑也是后段写的。
|
 |
76
lingxipaofan 2019-05-07 17:26:19 +08:00
感觉是需求有点乱
|
Select Language