坐标天津
直接看图吧

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