新来的外包同事,这代码,我???

2021-03-19 09:25:12 +08:00
 LeroyMooney
public List<IssueStatistics> getIssueStatistics() {
    return issueStatistics == null ? null : issueStatistics;
}
30024 次点击
所在节点    程序员
186 条回复
mahone3297
2021-03-19 10:17:05 +08:00
好简洁的写法。。。应该这样写

```
public List<IssueStatistics> getIssueStatistics() {
// 如果为 null,则返回 null
if (issueStatistics == null) {
return null;
} else {
// 如果不为 null,则返回本来的值
return issueStatistics;
}
}
```
ljzxloaf
2021-03-19 10:20:46 +08:00
pojo 不都这样吗?如果不是 pojo,他之前可能是写 c/c++的
webcoder
2021-03-19 10:21:48 +08:00
@SyncWorld 回去看半年前的代码,这是哪个傻 x 写的垃圾代码,哦是我啊,那没事了。
dengji85
2021-03-19 10:23:58 +08:00
真的高深莫测。你们这些凡人看不懂吧?还笑别人
q9OxQg
2021-03-19 10:24:57 +08:00
我不懂编程,不知道大家笑什么。但是我也还是笑了
celeron533
2021-03-19 10:27:45 +08:00
可能是为了未来的扩展性(逃
daimen
2021-03-19 10:27:57 +08:00
不出 bug 的代码就是好代码,再说了,编译器会优化的
bleepbloop
2021-03-19 10:28:55 +08:00
也许是被资本家剥削得太狠,精神状态不大好呢?
thtznet
2021-03-19 10:29:02 +08:00
从经济学角度看,这个社会大部分的问题其本质都是经济问题
tinyuu
2021-03-19 10:29:04 +08:00
挺好的 逻辑清晰,封装良好,设计充分,重构方便,拥抱未来。
liuxey
2021-03-19 10:29:49 +08:00
@anonydmer #23
@liudaolunhuibl #18

感谢提醒,写惯了 immutable 语言,Java 都快成伪代码了
LiubaiQ
2021-03-19 10:30:58 +08:00
@daimen 代码不规范,用不了半年就难以维护了。。。
daimen
2021-03-19 10:33:03 +08:00
@LiubaiQ 楼主就是来找相对于外包的优越感的,一两行代码能看出来个啥,我也是搞不懂了。
iceneet
2021-03-19 10:33:04 +08:00
???迷惑主义
cat007
2021-03-19 10:33:43 +08:00
没有 bug 的代码就是好代码
jianpeng957
2021-03-19 10:37:36 +08:00
有种为了炫技而炫技的感觉
QRan
2021-03-19 10:39:13 +08:00
@lagoon 我也是这样想的,我自己菜我也就不发言了 ^_^哈哈哈
UBcai
2021-03-19 10:39:40 +08:00
贴的第一个代码,就是按照我这里需求来说,有些 list 是要返回 null,有些需要 new arraylist<>()(看了上面大佬的写法,我准备改为 Collections.emptyList()),这样写就需要看前端需要什么了,然后好改。 贴的第二个代码估计我太菜才一年多 java,我还是没看到哪里有问题?各位大佬有知道的给我分析分析么。
wennew
2021-03-19 10:39:46 +08:00
这么闲吗,我从来不看别人代码,如果你是他上司就让他 checkstyle 下呗
VictorJing94
2021-03-19 10:40:31 +08:00
字母数计 kpi?

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

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

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

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

© 2021 V2EX