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

2021-03-19 09:25:12 +08:00
 LeroyMooney
public List<IssueStatistics> getIssueStatistics() {
    return issueStatistics == null ? null : issueStatistics;
}
30041 次点击
所在节点    程序员
186 条回复
hejw19970413
2021-03-19 15:21:17 +08:00
没事 编译器会改成 return issueStatistics
ws52001
2021-03-19 15:21:24 +08:00
这也有歧视链??
cgpiao
2021-03-19 15:25:18 +08:00
所以你以后被人鄙视的时候想想自己的所作所为。没大能力,哪来的优越感。
yrj
2021-03-19 15:33:04 +08:00
是不是有绩效?
cd744639718
2021-03-19 15:36:22 +08:00
哈哈哈,学到了
liuzhenyu
2021-03-19 15:44:53 +08:00
你以为你在第二层,他在第一层,实际他在大气层
gwybiaim
2021-03-19 15:46:54 +08:00
@liudaolunhuibl 快别献丑了
supermao
2021-03-19 15:52:56 +08:00
哈哈哈哈哈哈哈哈
rapperx2
2021-03-19 15:56:50 +08:00
楼主能贴下你的代码给我们瞧瞧吗?学习学习
mosliu
2021-03-19 16:03:55 +08:00
这个就不错了。在代码编译的时候应该会优化掉。 扩展性十足。不影响效率。

爆一个前同事,自己号称当年高考数学满分的硕士写的代码

使用 ES 。ES 库每个月也就 1,2T,数据量也不是太大。每条数据有多个现成的相似字段,相似数据条数从没有到几十万都有可能。单条数据量从 1k 到上百 k 都有。每条数据有时间戳字段,这是大环境

需求是,比对一条数据是不是类似数据最早的,返回一个 bool 值。这位的做法是按相似值字段,从 ES 中取出 10000 条不做排序数据的所有的字段到内存中,按照日期字段的字符串直接比较得出结果。

幸亏了他前面程序的一个判断出了 bug (功能故障没被测出来) 这段代码基本上跑不到,不然一次从 ES 拉取取几十 M 的数据做比较。那画面想想真醉了。前两天直接功能全部重写掉才舒心了。
fakeJas0n
2021-03-19 16:25:27 +08:00
写得有什么毛病?
gwybiaim
2021-03-19 16:37:46 +08:00
别幼稚了!评《新来的外包同事,这代码,我???》
https://v2ex.com/t/763248
balckjoker
2021-03-19 16:37:54 +08:00
大家都干一样的活,没必要说出来。
mitu9527
2021-03-19 16:46:41 +08:00
代码写的差是他的问题,嘲讽他就不太合适了,帮他一把不好么。
msg7086
2021-03-19 16:48:40 +08:00
@ho121 #124 对,但是我们这是只返回 boolean 的 Java 代码……
unco020511
2021-03-19 16:52:10 +08:00
高扩展高可用,哈哈
Kilerd
2021-03-19 17:15:58 +08:00
@vicalloy #24 你这个是典型的 deepcopy 的偷懒写法啊。 某些场景里面这么写性能其实也蛮好的。
vicalloy
2021-03-19 17:37:18 +08:00
@Kilerd 问题是没有需要 deepcopy 的东西,返回的对象都是当前函数里构建出来的。当初也怕是 deepcopy 没敢轻易修改。结合整体的”代码风格“,确认就是为了把对象转成`JSON`。
Gladoos
2021-03-19 17:58:20 +08:00
都是打工人 何必呢
chenqh
2021-03-19 18:05:28 +08:00
@mosliu 我干过类似的,不过是 mysql

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

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

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

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

© 2021 V2EX