练习两年半的同事写的代码

2021-12-30 10:19:02 +08:00
 totoro52

新来的同事写的代码,两年半工作经验,一个 CURD 功能写了一星期,今天看了 git commit ,我不做评论,各位看官看吧

26046 次点击
所在节点    Java
206 条回复
ytll21
2021-12-30 10:22:40 +08:00
槽点太多,无从下手
qiany
2021-12-30 10:23:21 +08:00
你们公司还招人吗
66beta
2021-12-30 10:25:05 +08:00
不懂 java ,但我觉得注释不行
glfpes
2021-12-30 10:26:39 +08:00
试用期就是给这种贵物准备的。

面试的人呢?拖出去枪毙十分钟。
aragakiyuii
2021-12-30 10:29:05 +08:00
这咋面试过的
IamCyborg
2021-12-30 10:29:59 +08:00
前端表示看不懂,没学过 java ,有人解释一下吗
xiao109
2021-12-30 10:30:01 +08:00
这不写的挺好的嘛,map 入参减少新建一个类的麻烦。
现在两年半经验约等于零经验。培训班出来都是两年经验起步
murmur
2021-12-30 10:31:32 +08:00
update = 获取,我英语是跟体育老师学的么
k9982874
2021-12-30 10:31:52 +08:00
用 map 充分考虑到了 api 的扩展性,没毛病,建议全司推广🐶️
zhoudaiyu
2021-12-30 10:32:46 +08:00
你们公司还招人吗
totoro52
2021-12-30 10:33:42 +08:00
@k9982874 硬是把面对对象概念按在地上打
nonoyang
2021-12-30 10:34:03 +08:00
我怀疑除了方法里面的内容,其他的都是从别的地方拷贝过来的。。。
dejavuwind
2021-12-30 10:34:13 +08:00
map 一把梭 要啥 get 啥
gbw1992
2021-12-30 10:34:29 +08:00
前端看到接口名会不会心里会不会**
mineralsalt
2021-12-30 10:34:36 +08:00
1.map 做参数是不太好, 但是也有实际应用场景
2.return null 问题比较大, 这里应该抛异常然后全局处理异常, 或者用 ApiResult.error()
3.equal 写的不规范容易空指针异常
4.id 没有判空, 也容易空指针
night98
2021-12-30 10:34:44 +08:00
http method -》 post
func name -》 update
service func name -》 get

好家伙,差点就集齐神龙了
xinyang5302677
2021-12-30 10:35:02 +08:00
他是不是你工资比你高?
auto
2021-12-30 10:35:16 +08:00
fkdtz
2021-12-30 10:35:26 +08:00
这是高手,要写让人一看就懂的代码,无论是初学者还是架构师,而不是写炫技的代码。
timothyye
2021-12-30 10:35:39 +08:00
@murmur 估计是复制粘贴忘了改

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/825212

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX