各位大佬大佬,帮忙看一段 Java 线程池的代码

2017-08-18 11:18:20 +08:00
 hqtc

// 初始化执行器

    threadPool = Executors.newFixedThreadPool(PROCESSOR_NUMBER, new ThreadFactory()
    {
        private AtomicInteger seqNo = new AtomicInteger(1);
        
        public Thread newThread(Runnable r)
        {
            return new Thread(r, "Message-Thread-" + seqNo.getAndIncrement());
        }
    });

Fortify 扫描了之后报出如下警告, J2EE Bad Practices:Threads (FORTIFY.J2EE_Bad_Practices--Threads) 0. InCall: Thread()

请问该怎么改啊?

5263 次点击
所在节点    Java
6 条回复
hustlike
2017-08-18 11:35:19 +08:00
感觉这个是 Fortify 的问题啊,没有识别出这里是线程池,估计是觉得自己创建线程不是 best practice 吧。
palmers
2017-08-18 11:41:11 +08:00
好像是说你创建线程的时候方式不太好 可能是因为没有指定 Group 这方面区别我也不是很清楚 你自己看下
hqtc
2017-08-18 11:43:01 +08:00
@hustlike 哎对啊,我看这段代码意思只是给后面要执行的线程绑定一个带编号的名字而已,又没有 start() 和 run()... 感觉要自定义一个 Thread 子类了,不知道能不能消掉警告,哎,就很烦
hqtc
2017-08-18 11:46:02 +08:00
@palmers 不是很理解 Group ,没用过
SoloCompany
2017-08-18 22:53:12 +08:00
没办法,j2ee 的规范之一貌似是 webapp 不能自己管理线程,即使非禁止

如果你写的不是 webapp,那就是 fo   rtify 没配置好啰
hustlike
2017-08-19 10:53:14 +08:00
@hqtc 可以绕过这个问题,通过第三方的包去做这个事情,不过意义不大……
guava 就有 ThreadFactoryBuilder

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

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

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

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

© 2021 V2EX