JavaFX MediaPlayer属性读取线程安全吗？能否直接获取CurrentTime？

首先直接给结论：你直接在非FX应用线程调用player.getCurrentTime()是不安全的，而且你的代码里确实存在线程安全隐患，尤其是getCurrentTime()和deleteMe()的交互部分。

为什么直接调用不安全？ #

JavaFX的整个UI生态（包括MediaPlayer）都是基于单线程模型设计的——所有与组件状态相关的操作（不管是修改还是大部分读取）都必须在FX应用线程上执行。虽然有些属性内部可能做了同步处理，但官方文档明确不推荐跨线程访问这些组件，否则可能导致竞态条件、数据不一致，甚至直接抛出异常。

你的代码里的具体风险 #

看你的代码：

• playMe()和deleteMe()都是通过Platform.runLater()在FX线程里操作player，这部分是对的；
• 但getCurrentTime()是直接在调用线程（大概率是非FX线程）里访问player，这里有两个问题：
  1. 跨线程读取MediaPlayer的属性，可能拿到错误的过时值，或者触发线程冲突；
  2. 存在竞态条件：你刚检查完player != null，下一秒FX线程可能已经执行了deleteMe()里的player.dispose()并把player设为null，这时候调用player.getCurrentTime()就会抛出NullPointerException，或者因为播放器已被释放引发其他错误。

怎么解决？ #

所有涉及MediaPlayer的操作（包括读取属性）都要放到FX应用线程上执行。如果需要立即获取结果，可以用Platform.call()（它会同步等待FX线程执行完任务并返回结果），而不是异步的Platform.runLater()。

修改后的getCurrentTime()示例：

public Double getCurrentTime() {
    // 先做一次快速检查，避免不必要的FX线程调用
    if (player == null) {
        return null;
    }
    try {
        // 用Platform.call()在FX线程内执行读取操作
        return Platform.call(() -> {
            // 这里再次检查player，防止在等待FX线程执行期间，player被dispose
            if (player != null) {
                return player.getCurrentTime().toSeconds();
            }
            return null;
        });
    } catch (InterruptedException | ExecutionException e) {
        e.printStackTrace();
        return null;
    }
}

另外，建议把所有player的访问逻辑都统一封装到FX线程操作里，比如读取、修改、释放，这样能从根源上避免线程安全问题。比如你可以把player的引用访问都限制在FX线程内，外部只通过Platform相关方法来交互。

额外提醒 #

• 永远不要在非FX线程直接调用MediaPlayer的任何方法（包括getter、setter、play/dispose等）；
• 如果你的getCurrentTime()调用频率很高，频繁用Platform.call()可能会影响FX线程的性能，这时候可以考虑在FX线程里定期更新一个线程安全的变量（比如AtomicDouble）来缓存当前时间，外部线程直接读取这个缓存变量，这样既能保证线程安全，又能减少FX线程的压力。

内容的提问来源于stack exchange，提问作者Guillaume F.