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

[totoro52]

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

[rrZ2C]

这是传说中的参数封装吧

[kerb15]

也许槽点在于，写了一星期

[totoro52]

@nonoyang 自信点，怀疑两个字去掉

[murmur]

@auto 这个没看出问题来。。。

[hccsoul]

我看懂了,但我还是大受震撼

[liangch]

不大懂。map 参数不判 null 么？

[totoro52]

@xinyang5302677 那倒不至于

[jiyingze]

第一行 if 可能抛两个 npe
哈哈哈

[philchang1995]

@auto 颇有异曲同工之妙

[zpxshl]

@glfpes 他们会说你是精神资本家

[yazinnnn]

槽点是挺多的。。。

[majiaxxyy]

入参 map 一时爽，后期维护惨。 前面的 return null 前端怎么知道要干嘛。 后面的 return 都知道回个 success 。

[glfpes]

@zpxshl 那就让他们和这种贵物一块干活，给他擦屁股吧。

[mango88]

项目中有类似的代码（不是他提交的） ？

[philchang1995]

@totoro52 方法名不太合适、参数用 map 也不太合适、equals 按照上面那样写容易空指针、除特殊情况下方法最好不要返回 null 这些是我看出来的 不知道别的还有没有

[totoro52]

@mango88 第一次见到这种，代码已经全部打回重写了

[philchang1995]

@totoro52 回复错人了 不好意思

[ZField]

槽点很多，map 传参、返回值不规范、校验可以用注解……

[Zeffon]

工资估计高于楼主吧

[cxshun]

参数用 map ，后续估计想死的心都有了。
CRUD 一个星期倒不是啥问题，有可能刚入职，很多不清楚，要熟悉一下。只是上面的 map 槽点就真是没法理解了。

PS：我厂，某养猪的，定义的接口就是一个 dict （ python 语言），作为一个 javaer ，真是受不了。虽然前期方便，但后续真的是坑爹啊。