老哥们，我老大在循环里写 Thread.sleep，到处都这么写，有问题吗

[Aliberter]

他为了实现一个定时任务的功能，因为他说 spring 的定时任务不好用，每次跑着跑着就没了，所以想出来这种写个死循环，然后执行一次就休眠一段时间的方法，但是我看 IDE 都会报警告，说禁止在循环里写 Thread.sleep ，我在 stackoverflow 上看了好多解释的帖子，说很浪费资源，发给他看，他之前说这样写不会浪费资源，看了帖子后又说浪费点资源没事，主要是这样实现的定时任务很稳定，求老哥们的意见，这样到底有没有问题？

[yaphets666]

不管有没有问题，听你领导的就行了，他负总责。

[Leonard]

不要教老大做事🐶

[chir0n]

以后性能优化的时候会有用的。

[simonlu9]

spring 的定时任务不好用，每次跑着跑着就没了，是因为默认配置的线程池只有一个线程，所以有时候执行不了，自己额外配一下线程池就可以了

[qwerthhusn]

主要看 sleep 多久，如果 sleep 1 秒以上。那线程切换占用的一点 CPU 资源都可以忽略不计了

[kop1989]

有问题，但是没准更适合你们的场景和成本。

[securityCoding]

@simonlu9 spring 这个机制的确非常坑

[Aliberter]

@yaphets666 学会了，老哥职场高人

[Aliberter]

@Leonard 学到了学到了

[rsyjjsn]

小心下次分配给你的任务

[Aliberter]

@chir0n 哈哈哈，懂了，不过性能现在已经很难看了

[Aliberter]

@simonlu9 这个我也没具体去排查，定时任务线程池应该是有配，但忘了是在他说问题之前还是之后了，总之后来他就很不愿意用这个了

[Aliberter]

@qwerthhusn sleep 时间挺长的，但我以为这种写到循环里的话，会持续占据资源不会切换呢

[polobug]

你能解决岂不是更好= =当然你不能解决的话。。那总得用把

[shenlanAZ]

我的前老大还写过 System.gc(); 呢。

他写的代码让他自己负责，除非要你来全权维护，你可以改成你认为可行的方法。

[Aliberter]

@polobug 我能解决，我给他提出过 N 多定时任务方案，他看都不看说不能用，就用他写的，属实难顶，所以认了

[sagaxu]

大佬明年一出手，性能提升十倍，对内 PPT 谈 KPI ，对外分享重构案例

[Aliberter]

@shenlanAZ 我老大是代码可以是他写，但问题一般要追究到我们头上，他不负责，即使我们说之前是你让这么做的，那么他也会说是你当时听错了，然后当即给你提出一种新的方法，让你去改，就是死活不承认自己有错误这种，就因为这种风格底下已经被气的离职了两个了，现在项目也是一团糟，唉

[Aliberter]

@sagaxu 哈哈哈 精辟

[WytheHuang]

@Aliberter #18 这还不提桶跑路， 着实不要脸了。

[Aliberter]

@WytheHuang 我着实想跑了，可是才入职半年，而且也年底了，唉，先忍忍吧，明年来了看情况开溜

[gadfly3173]

在循环里写 sleep 也不是不能接受的，比如一些循环调用第三方 api 的，为了防止执行过快碰到 api 的 qps 上限，加个 sleep 简单有效

[Spoter]

@Aliberter 保留聊天证据

[Aliberter]

老哥们，点击量高我有点心慌啊，毕竟吐槽老大了，一会我删帖了哈

[Cihua]

@Aliberter 18# 针对这种情况我后来遇见问题一律文字沟通,事后甩他脸上,文字发过去如果人过来说,那我就下楼抽根烟等回复

[Aliberter]

@Spoter 哈哈 都是口头交流 只能录音了下次

[Aliberter]

@Cihua 学到了，以后多文字沟通，保留证据哈哈

[Aliberter]

@gadfly3173 ok 明白了 谢谢老哥

[0312birdzhang]

@shenlanAZ #15 可以的，有时候 jvm 并不是那么聪明，手动 gc 是有使用场景的。

[dzdh]

@Aliberter v2 不准删帖 2333333

[Aliberter]

@0312birdzhang 说起 gc ，我们现在这个项目，分了 4 个 G 的内存，线程数太多，将近 300 多个，平均每 0.19 秒就要 gc 一次，部署了两天已经 gc 三万多次了，这是不是要炸了啊。。我感觉已经无法直视了，

[Aliberter]

@dzdh 哈哈哈 忘了 那就这样吧 没事

[tabris17]

具体问题具体分析，就一个线程，也不是频繁切换，这么搞完全没问题

[Aliberter]

@tabris17 ok 明白了 谢谢老哥

[Aliberter]

老哥们，暂停回复吧，我要搬砖了，谢谢大家哈~

[luozhiyun]

以我的经验来看，遇到这种事情，最多提醒一次，后面就不要再说了，老大说什么都是对的。你要是觉得在他手上学不到东西，可以跳槽

[JoJoStark]

@Aliberter 这种甩锅领导太多了，能当领导脸皮都不是一般的厚

[ykk]

想了下我也在循环里写过 sleep ，为了协调两个进程的时间，不然一个队列灌太快会被冲掉，至今不理解原因

[bxb100]

while true + sleep 做定时任务没问题啊

[ligiggy]

非 java ，我的 死循环里面好像都加了 thread.sleep(). 就算在线程里面跑，我也加了

[0xZhangKe]

性能问题要有数据对比，你能拿出数据才能证明真的有问题，否则都是空谈。

[xiaoyang7545]

很简单，你能不能有更好的方式来解决现有问题。如果有，就调整一下给他看看是否可以用。如果没有，在那边说这样不行那样不行意义不大。

[ch2]

又不是不能用.jpg

[liudaolunhuibl]

sleep 的问题应该是线程状态切换，唤醒和进入休眠都要耗费 CPU 资源的，但是不是太频繁的话理论上还好，虽然算是野路子但是这也是一种解决办法吧，不过更好的是配置 spring 定时任务线程池或者换成其他的框架比如 XxlJob

[easing]

@Aliberter #13 并不是，sleep 会切换出去的（最后走 futex_wait ），如果 sleep 时间挺长，其实没啥问题。而且确实稳定。

[meeop]

代码工作正常,没有性能问题爱怎么写怎么写,并无不妥

你觉得高级的做法也并不是最佳实践,就算改了,重构后效果和重构前差不多,没啥用

除非,你趁机牵头搞一个公司内部的服务调度框架 /中台 /平台,那还不错

[ForkNMB]

不是 。定时任务为啥要用 Spring 自带的，贼不好用，为啥不用 XXLJOB

[BBCCBB]

4L 正解.

[jackzhengjbs]

我是你领导，下班之后来我办公室，🐶头

[C603H6r18Q1mSP9N]

正解：写个 内部请求地址，linux crontab 定时调用这个链接

[code4you]

性能优化

以前 Thread.sleep(10000)

优化 Thread.sleep(1000)

再次优化 Thread.sleep(100)

极度优化 Thread.sleep(10)

😁

[corningsun]

@shanghai1998 自己用着爽，运维火葬场。

[Jooooooooo]

sleep 当然不会消耗资源

这当然也是也是一种定时任务的办法

你领导说的不靠谱的现象确实存在, 要是异常没处理好, 定时任务可能就会没了

[Lemeng]

出问题，不用自己负责的事，不操心，呵呵

[zhuangzhuang1988]

@shenlanAZ 强如 scala 老爹的项目 里面也有 System.gc()的, 有时候直接重启
https://github1s.com/lampepfl/dotty/blob/HEAD/language-server/src/dotty/tools/languageserver/DottyLanguageServer.scala#L102

[ligiggy]

@code4you 致富之路

[fangcan]

可以发信息再找他确认下，留下证据

[yousabuk]

犟啥呢，已经出现对你的所有建议看都不看直接否定的情况了。

小心了，对你有意见了已经。

[AlexRoot]

@Aliberter 这种老大，建议跑路，感觉以后要背大锅。

[kaneg]

sleep 是会一直占用当前线程，应该用 object 的 wait 方法。
如果只是用定时，可以用 ScheduledExecutorService

[railgun]

符合业务需求就没有问题

[anonymous1024]

如果当前操作系统是抢占式内核，在死循环中写 sleep 是可以放弃 cpu 占用时间的，提高其他程序的命中率。

[C603H6r18Q1mSP9N]

正解：写个 内部请求地址，linux crontab 定时调用这个链接

@corningsun 不会的，crontab 有运维管理的平台统一调度处理的

[goalidea]

首先不考虑性能的话，能实现就行，其次 sleep 要考虑多线程的问题，毕竟是要抱着锁去睡觉

[olaloong]

看具体用法吧....我有个 10 秒一次的定时任务就是循环+sleep ，无锁无竞争也不会有啥问题

[oOoOoOoOoOo]

@code4you

等 sleep 语句删掉的时候：

重新优化了代码结构，缩减了代码行数，提高了运行效率。