java - 这段代码中的 AtomicBoolean 是否矫枉过正？

Question

这是我编写的一些代码，用于以通用方式处理一个事件处理程序，该事件处理程序对于任何 JavaFX 事件都应该只触发一次:

public final class OneShotEvent<T extends Event>
    implements EventHandler<T>
{
    private final AtomicBoolean triggered = new AtomicBoolean(false);
    private final EventHandler<T> delegate;

    public static <E extends Event> EventHandler<E> of(final EventHandler<E> delegate)
    {
        return new OneShotEvent<>(delegate);
    }

    private OneShotEvent(final EventHandler<T> delegate)
    {
        this.delegate = Objects.requireNonNull(delegate);
    }

    @Override
    public void handle(final T event)
    {
        if (!triggered.getAndSet(true))
            delegate.handle(event);
    }
}

出于某种原因，我将 triggered 设为 AtomicBoolean 而不是普通的 boolean。

在考虑之后，我认为这是矫枉过正，因为事件处理程序将在 JavaFX 的“平台线程”上运行……或者它们不会？因为如果是这种情况，那么一个简单的 boolean 值就足够了……对吧？

Answer 1

如果您知道调用是单线程或线程安全的，那么一个简单的 boolean 值就足够了，尽管您可以放弃它。

private EventHandler<T> delegate;

public void handle(T event) {
    if (delegate != null)
        delegate.handle(event);
    delegate = null; // don't retain a delegate we don't need
}