1
chendy 2021-09-29 22:52:22 +08:00 17
快进到几位不长心的网友因为将公司核心代码截图发在 v 站丢了工作
|
2
coolcfan 2021-09-29 23:19:52 +08:00
Thread.sleep(100)........
|
3
yinusxxxx 2021-09-29 23:30:15 +08:00
为什么要比这个。。。。
|
4
ipwx 2021-09-29 23:41:18 +08:00 1
你给的截图有啥可称道的地方么。。。不过是 annual routine 而已
|
5
BIAOXYZ 2021-09-30 02:20:39 +08:00 1
我以为你要比 commit 数量,哪位热心 V 友补一下地铁墙面那个格子图吧。。。(手动狗头
|
6
msg7086 2021-09-30 03:05:40 +08:00 via Android
我倒是工作的时候写了不少代码,但是发出来要吃官司的还是算了。
|
7
gollwang 2021-09-30 09:08:22 +08:00
这也卷?
|
8
Kasumi20 2021-09-30 09:48:01 +08:00 1
好好的代码,被你改得乱七八糟的
|
9
zhuangzhuang1988 2021-09-30 10:16:28 +08:00
是不是傻?
|
10
kastnerorz 2021-09-30 10:39:15 +08:00 1
- Guava 的 `Lists.partition()` 可以解决你的这个分片的问题
- 不太懂的是,已经 `runAsync`了,为什么要 `Thread.sleep()`,循环里只是添加任务而已,并没有执行任务 - 而且非常不建议在循环里使用 sleep,如果一定要隔一段时间执行一个任务请使用 `ScheduledExecutorService` |
11
jadec0der 2021-09-30 10:40:53 +08:00
你符号前后加不加空格都不统一?就这 IntelliJ 不给你加黄线嘛
|
12
l48x4264l46 2021-09-30 10:53:02 +08:00
兄弟,对于你这个代码,冒昧的提些问题
1 、最好少用反逻辑 2 、代码没有分块,全都挤在一起 3 、该加空格的地方没有加空格 4 、注释写的有点少 |
13
jimliang 2021-09-30 11:09:37 +08:00
你是来添乱的吧
|
14
zhuangyufeng 2021-09-30 11:12:04 +08:00
你是故意找茬是吧(狗头)
|
15
yidinghe 2021-09-30 11:17:10 +08:00
不管出于什么原因,你这么改的目的显然是进行调用限流。然而这不是限流的标准做法,标准做法是先将循环内的部分提取出一个方法,然后使用 sentinel 之类的框架来限流。
|
16
Leviathann 2021-09-30 11:18:18 +08:00 via iPhone
所以就是加了个分块的逻辑?
感觉这种非业务的东西直接弄个声明式的 api 调一下会比较清晰吧 |
17
ChovyChu OP 统一回复一下吧,这个是我刚接手的一个项目,最近经常会做一些 review 和看一些老代码,当时就看到这里。
不知道大家有没看明白这个 commit 到底是在改做什么。 我当时看到这里就很疑惑,想了半天才发现这里他其实就是希望做一个每批任务处理完成之后做一个 sleep 延迟。就是这么简单一个功能尽然写了一大堆,结果还是错的。 另外就是我想了一下确实我这图可能没体现出来重点。。 |
18
AoEiuV020 2021-09-30 14:44:54 +08:00
|
19
astkaasa 2021-09-30 16:15:56 +08:00
我一般这么写
IntStream.range(0, (list.size() + BATCH - 1) / BATCH) .mapToObj(i -> list.subList(i * BATCH, Math.min(list.size(), (i + 1) * BATCH))).forEach(batch -> { }); |
20
uselessVisitor 2021-09-30 21:54:23 +08:00
分片处理? guava 一个 api 就解决了
|
21
qinwangzeng 2021-10-01 00:46:58 +08:00
谢谢,提醒了我写过的代码要多 review 。
|
22
WispZhan 2021-10-01 15:12:14 +08:00 via Android
这么长的代码,点都不想点。直接 close pr
|
23
csfreshman 2021-10-02 10:49:30 +08:00
注释太少,在我司肯定不会 merge,另外公司这种代码尽量不要贴出来,没看出来有啥贴出来的必要。
|