@wr410 也不算是纯调用吧，因为写的是接口，好多参数在前端判断了之后我后台也会做一下判断，这段代码之所以让我改是因为前端传过来的时间是字符串类型，之后还可能传“开始时间”的中文，我需要对他重新做一下处理。所以让我来改这段代码。
如果按我的逻辑来讲的话 ：
先给 dto 赋值，之后再拿着 dto 去查询数据库，最后将查到的数据赋值给 OutVoGlobal。我要是进行修改的话就应该改只在第一步加验证就行了。。但是他 dto 赋值是放在了方法的最后。。流程和我的惯性思维正好是相反的。
确实是在他的 return 之前加上我的代码就好了，但是我在没读他代码之前确实是不知道他是不是做了其他的操作，所以说我还是读了一遍他的代码。

@DingSoung 有的时候我也想呢。是不是真的是我太菜了？（昨天晚上突然想到这事，2 点多才睡。）
但是仔细想想我还是读的懂他的代码呢。就是读起来非常别扭。然后刚刚有人帮我指出我为什么别扭了。。


return new OutVoGlobal(EnumRetCode.SUCCESS).setData(orderMapper.list(dto.setBelong(user.getUserNo())));
写的问题在哪里，我们分析一下：
执行顺序：创建 OutVoGlobal 返回值——>setData 设置数据——>orderMapper.list 执行——>dto.setBelong 封装 userNo，同时要获取 userNo
在我们眼里的顺序是怎样的，获取 userNo ——>封装——>执行 list ——>封装返回 OutVoGlobal。可以看出它实际上需要逆着思维的顺序，user.getUserNo()这部分如果嵌套过多甚至你需要不断找。所以这种风格有点像用但是没用好的感觉。

@leafre 整个帖子看下来就你阴阳怪气，我是昨天和人家当面讨论的，还鸡蛋里挑骨头？哪轮到你这小丑说话了？
回你侮辱我的智商，浪费我的时间，看你回帖瞎了我的眼睛，在这找存在感来了，也不想想老子为什么怼你

@whp1473 非常感谢，受益匪浅。

@freebird1994 确实是，controller 层做了 try catch..

@leafre 又没说看不懂，团队协作提提意见怎么了？而且我也是抱着讨论的心思和他谈的
再说了，我和别人讨论，关你什么事？你有资格评论我的帖子？

@ccpp132 也不是说读起来困难吧，只不过是觉得不能一目了然。在改的时候还要从头读。
不过这应该还是和读代码能力有关系的。

@guokeke
就是这个问题。。。
我认为写一行可读性差，他认为写很多可读性差
我也在想到底哪种方式写好呢，

@wysnylc 小作坊公司。。。刚工作一年多。。技术确实是不到位。。大佬喜怒。。

@IceChen
我是觉得这句话干的事情有点多。不过确实是没问题。。刚用阿里编码规范扫描了一下，都没问题。。。但是我觉得第一次读起来不算太习惯。。

new OutVoGlobal(EnumRetCode.SUCCESS).setData(orderMapper.list(dto.setBelong(user.getUserNo()))
1：新建了一个 OutVoGlobal
2：给 OutVoGlobal 赋了一个 EnumRetCode.SUCCESS 的值
3：给 OutVoGlobal 的 Date 赋值 A
4：A 是用 orderMapper.list(dto)查的
5：给 dto 设置了一个 belong,belong 的值是从 user 中拿的。。

我改他代码是因为他 dto 中的东西需要重新判断一下。
dto 里面有个开始时间和结束时间，前端有可能传过来的是汉字，我需要 try catch 一下，如果发生异常我把这个字段改为空字符串。确实是可以在他这句话之前加判断。
但是按照正常逻辑来讲应该是在他代码的第 5 步来判断吧。。

其实我还是坚持他这么写可读性不好。。但是确实是可以在他这句话之前加判断。（也不清楚是不是自己水平不行。。不过我总觉得再让我重新来一遍我还是得看到第五步。。）

@0x11901
其实这是个 98 年的小同事写的。。（我 94。。）

@shawnbluce 他是觉得这么写比写好多行看起来方便。。
而我是觉得看很多行方便容易理解，看他那一行很麻烦，还得挨个分析。。

大学的时候一直用 DW，结果当我第一次使用 HB 的时候。。。。。
（我是 JAVA。。）