大家能看出这段代码的问题么?

本贴最后更新于 3061 天前,其中的信息可能已经物是人非
public <T> Future<T> fireEventAsynchronously(final Event<?> event) {
    final ExecutorService executorService = Executors.newSingleThreadExecutor();

    final FutureTask<T> futureTask = new FutureTask<T>(new Callable<T>() {
        @Override
        public T call() throws Exception {
            synchronizedEventQueue.fireEvent(event);

            return null;
        }
    });

    executorService.execute(futureTask);

    return futureTask;
}
  • Java

    Java 是一种可以撰写跨平台应用软件的面向对象的程序设计语言,是由 Sun Microsystems 公司于 1995 年 5 月推出的。Java 技术具有卓越的通用性、高效性、平台移植性和安全性。

    3187 引用 • 8213 回帖
  • Q&A

    提问之前请先看《提问的智慧》,好的问题比好的答案更有价值。

    8121 引用 • 37033 回帖 • 159 关注
  • 多线程
    11 引用 • 26 回帖
  • Future
    1 引用 • 5 回帖

相关帖子

欢迎来到这里!

我们正在构建一个小众社区,大家在这里相互信任,以平等 • 自由 • 奔放的价值观进行分享交流。最终,希望大家能够找到与自己志同道合的伙伴,共同成长。

注册 关于
请输入回帖内容 ...
  • zonghua

    两个都快看晕了,自己写不出这样的东西

  • 其他回帖
  • 88250

    @hefeng 呃,确实是没啥意义,线程池应该是在方法外部全局的某个地方进行管理的,但这个不是主要问题。

  • hefeng

    这个方法内部创建了线程池,而且是个单线程的线程池意义何在?

  • Bard 1 赞同

    “大人,时代变了”trollface

    这段代码实现了异步执行一个事件的功能。它创建了一个单线程的 ExecutorService,然后使用这个服务执行一个 FutureTask,这个 FutureTask 中包含了真正的事件执行逻辑。虽然这段代码能够工作,但是它有几个潜在的问题和改进的地方:

    1. 资源泄露: 每次调用 fireEventAsynchronously 方法时都会创建一个新的 ExecutorService,但是这个服务并没有被明确地关闭。如果这个方法被频繁调用,那么将会创建大量的线程池,但这些线程池不会被自动回收,最终可能导致资源耗尽。为了避免这个问题,可以考虑使用一个共享的 ExecutorService,并在适当的时候关闭它,或者使用 try-with-resources 确保每个创建的服务都能被正确关闭。
    2. 返回值问题: 这个方法的返回类型是 Future<T>,但在 Callable 的实现中返回的是 null。这意味着,即使调用者期待一个结果,也得不到有意义的返回值。这可能不是设计上的问题,但如果有返回值的期待,应该重新考虑这个实现。
    3. 异常处理: Callable 中的 call 方法抛出了一个异常,但是在异步执行的环境中,异常的处理会比较复杂。调用者需要明确地从返回的 Future 中检查和处理异常,这可能是一个容易被忽视的地方。
    4. 同步问题: 代码中使用了 synchronizedEventQueue.fireEvent(event); 进行事件触发,这表明可能存在对事件队列的同步控制。但是,在异步执行的环境下,仅仅依靠方法级别的同步可能不足以保证线程安全,具体取决于 synchronizedEventQueue 的实现和使用方式。
    5. 效率问题: 如果事件处理通常很快,使用单独的线程池可能是一个过重的解决方案,因为线程的创建和销毁都是昂贵的操作。在这种情况下,可以考虑使用一个已经存在的线程池,或者其他轻量级的并发机制。

    综上所述,虽然这段代码能够实现基本功能,但在实际使用中可能会遇到资源管理、性能和设计上的问题。考虑到这些潜在问题,重构代码以使用共享的 ExecutorService,改善异常处理和同步机制,以及澄清方法的返回值,都是值得考虑的改进方向。

  • 查看全部回帖

推荐标签 标签

  • Google

    Google(Google Inc.,NASDAQ:GOOG)是一家美国上市公司(公有股份公司),于 1998 年 9 月 7 日以私有股份公司的形式创立,设计并管理一个互联网搜索引擎。Google 公司的总部称作“Googleplex”,它位于加利福尼亚山景城。Google 目前被公认为是全球规模最大的搜索引擎,它提供了简单易用的免费服务。不作恶(Don't be evil)是谷歌公司的一项非正式的公司口号。

    49 引用 • 192 回帖 • 1 关注
  • jsDelivr

    jsDelivr 是一个开源的 CDN 服务,可为 npm 包、GitHub 仓库提供免费、快速并且可靠的全球 CDN 加速服务。

    5 引用 • 31 回帖 • 58 关注
  • 以太坊

    以太坊(Ethereum)并不是一个机构,而是一款能够在区块链上实现智能合约、开源的底层系统。以太坊是一个平台和一种编程语言 Solidity,使开发人员能够建立和发布下一代去中心化应用。 以太坊可以用来编程、分散、担保和交易任何事物:投票、域名、金融交易所、众筹、公司管理、合同和知识产权等等。

    34 引用 • 367 回帖
  • V2Ray
    1 引用 • 15 回帖 • 2 关注
  • SSL

    SSL(Secure Sockets Layer 安全套接层),及其继任者传输层安全(Transport Layer Security,TLS)是为网络通信提供安全及数据完整性的一种安全协议。TLS 与 SSL 在传输层对网络连接进行加密。

    70 引用 • 193 回帖 • 431 关注
  • LeetCode

    LeetCode(力扣)是一个全球极客挚爱的高质量技术成长平台,想要学习和提升专业能力从这里开始,充足技术干货等你来啃,轻松拿下 Dream Offer!

    209 引用 • 72 回帖
  • 钉钉

    钉钉,专为中国企业打造的免费沟通协同多端平台, 阿里巴巴出品。

    15 引用 • 67 回帖 • 339 关注
  • IPFS

    IPFS(InterPlanetary File System,星际文件系统)是永久的、去中心化保存和共享文件的方法,这是一种内容可寻址、版本化、点对点超媒体的分布式协议。请浏览 IPFS 入门笔记了解更多细节。

    21 引用 • 245 回帖 • 241 关注
  • 导航

    各种网址链接、内容导航。

    40 引用 • 173 回帖
  • Chrome

    Chrome 又称 Google 浏览器,是一个由谷歌公司开发的网页浏览器。该浏览器是基于其他开源软件所编写,包括 WebKit,目标是提升稳定性、速度和安全性,并创造出简单且有效率的使用者界面。

    62 引用 • 289 回帖
  • 运维

    互联网运维工作,以服务为中心,以稳定、安全、高效为三个基本点,确保公司的互联网业务能够 7×24 小时为用户提供高质量的服务。

    149 引用 • 257 回帖
  • Logseq

    Logseq 是一个隐私优先、开源的知识库工具。

    Logseq is a joyful, open-source outliner that works on top of local plain-text Markdown and Org-mode files. Use it to write, organize and share your thoughts, keep your to-do list, and build your own digital garden.

    6 引用 • 63 回帖 • 1 关注
  • 又拍云

    又拍云是国内领先的 CDN 服务提供商,国家工信部认证通过的“可信云”,乌云众测平台认证的“安全云”,为移动时代的创业者提供新一代的 CDN 加速服务。

    21 引用 • 37 回帖 • 547 关注
  • 服务器

    服务器,也称伺服器,是提供计算服务的设备。由于服务器需要响应服务请求,并进行处理,因此一般来说服务器应具备承担服务并且保障服务的能力。

    125 引用 • 588 回帖
  • 倾城之链
    23 引用 • 66 回帖 • 137 关注
  • Lute

    Lute 是一款结构化的 Markdown 引擎,支持 Go 和 JavaScript。

    25 引用 • 191 回帖 • 16 关注
  • GAE

    Google App Engine(GAE)是 Google 管理的数据中心中用于 WEB 应用程序的开发和托管的平台。2008 年 4 月 发布第一个测试版本。目前支持 Python、Java 和 Go 开发部署。全球已有数十万的开发者在其上开发了众多的应用。

    14 引用 • 42 回帖 • 764 关注
  • Tomcat

    Tomcat 最早是由 Sun Microsystems 开发的一个 Servlet 容器,在 1999 年被捐献给 ASF(Apache Software Foundation),隶属于 Jakarta 项目,现在已经独立为一个顶级项目。Tomcat 主要实现了 JavaEE 中的 Servlet、JSP 规范,同时也提供 HTTP 服务,是市场上非常流行的 Java Web 容器。

    162 引用 • 529 回帖
  • gRpc
    11 引用 • 9 回帖 • 73 关注
  • 小薇

    小薇是一个用 Java 写的 QQ 聊天机器人 Web 服务,可以用于社群互动。

    由于 Smart QQ 从 2019 年 1 月 1 日起停止服务,所以该项目也已经停止维护了!

    34 引用 • 467 回帖 • 743 关注
  • wolai

    我来 wolai:不仅仅是未来的云端笔记!

    2 引用 • 14 回帖
  • 音乐

    你听到信仰的声音了么?

    60 引用 • 511 回帖
  • AngularJS

    AngularJS 诞生于 2009 年,由 Misko Hevery 等人创建,后为 Google 所收购。是一款优秀的前端 JS 框架,已经被用于 Google 的多款产品当中。AngularJS 有着诸多特性,最为核心的是:MVC、模块化、自动化双向数据绑定、语义化标签、依赖注入等。2.0 版本后已经改名为 Angular。

    12 引用 • 50 回帖 • 474 关注
  • uTools

    uTools 是一个极简、插件化、跨平台的现代桌面软件。通过自由选配丰富的插件,打造你得心应手的工具集合。

    6 引用 • 14 回帖 • 2 关注
  • Maven

    Maven 是基于项目对象模型(POM)、通过一小段描述信息来管理项目的构建、报告和文档的软件项目管理工具。

    186 引用 • 318 回帖 • 303 关注
  • 锤子科技

    锤子科技(Smartisan)成立于 2012 年 5 月,是一家制造移动互联网终端设备的公司,公司的使命是用完美主义的工匠精神,打造用户体验一流的数码消费类产品(智能手机为主),改善人们的生活质量。

    4 引用 • 31 回帖 • 2 关注
  • Firefox

    Mozilla Firefox 中文俗称“火狐”(正式缩写为 Fx 或 fx,非正式缩写为 FF),是一个开源的网页浏览器,使用 Gecko 排版引擎,支持多种操作系统,如 Windows、OSX 及 Linux 等。

    8 引用 • 30 回帖 • 407 关注