20年老鸟告诉你，这是公司找理由胡扯，或者你还是中级水平，理由一，判空？一般中级都会这么做，但是真正的判定参数合法化不是空那么简单，真正函数里面对参数做了太多处理反而掩盖了调用者的问题，按你这个逻辑，Windows API 不会有异常只有提示了，抛出异常是最正确的处理。其二，关于if ，做的的确有点幼稚，但是根本不影响时空效率，不要小看编译系统的优化能力了。有比如有人总觉得行内if很高效，这其实很搞笑，编译系统是吃素的？其实做系统不是写程序，代码可读性永远是第一位的，非关键代码那点效率，呵呵

没毛病啊，强行降低程序效率，等老板需要你优化代码再把if删了又能水一波

这代码影响程序性能。1-非空没做判断，2-逻辑本身可在sql环境实现。

这公司是有多闲？还纠结这种代码。。。。

我反而很痛恨那种return一个表达式的写法，因为这样断点不好跟，你在这个方法里面看不到最终将返回什么。还得把表达式丢到watch看

这样也被扣钱，那我还不如把前面几行也合称一行表达式来搞，方法里面第一句就直接return表达式

如果 真 返回 真
直接返回就行了 不用加if

这种也要扣500块？什么公司啊？你们老大也是打工的，都是打工的何必为难大家？

作为工具类为空抛出异常才是正常的处理，这里没有做抛出的处理显得偷懒

就一个if而已就扣了500？？是我就辞职了，此地不留爷自有留爷处，爱咋滴咋滴

我觉得这样可读性好，一眼就看明白。代码优化不是应该在其他地方吗？比如算法啥的

这都能扣500我感觉这公司没法呆了除非年薪百万不在乎

渣渣，谁能告诉我这个渣渣出了什么毛病！