有没来比拼 commit 的，重在真实

commit

href

重在

真实

23 条回复 • 2021-10-02 10:49:30 +08:00

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

2021-09-30 11:40:46 +08:00

统一回复一下吧，这个是我刚接手的一个项目，最近经常会做一些 review 和看一些老代码，当时就看到这里。

不知道大家有没看明白这个 commit 到底是在改做什么。

我当时看到这里就很疑惑，想了半天才发现这里他其实就是希望做一个每批任务处理完成之后做一个 sleep 延迟。就是这么简单一个功能尽然写了一大堆，结果还是错的。

另外就是我想了一下确实我这图可能没体现出来重点。。

18

AoEiuV020

2021-09-30 14:44:54 +08:00

是时间祭出我印象最深的两个极大幅优化了，
一个是把字符串拆成字符遍历，再把字符转成字符串用正则判断是否中文，而且正则还不预编译，

?1
一个是数据库查个 count 直接查出所有数据再遍历，还在每个循环节调用三次 queryForAll 完整读取 3n 次数据库，

?1

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 -> {
});