为什么matlabcontrol在代理创建时中断调用线程？

Question

我让我的Matlab控制线程可以中断,并发现它在第一次运行时一直被中断.

这是因为GetProxyRequestCallback内部有中断代码:

private static class GetProxyRequestCallback implements RequestCallback
{
    private final Thread _requestingThread;
    private volatile MatlabProxy _proxy;

    public GetProxyRequestCallback()
    {
        _requestingThread = Thread.currentThread();
    }

    @Override
    public void proxyCreated(MatlabProxy proxy)
    {
        _proxy = proxy;

        _requestingThread.interrupt();
    }

    public MatlabProxy getProxy()
    {
        return _proxy;
    }
}

是否有任何理由中断调用线程或这只是一个错误？

Answer 1

该RemoteMatlabProxyFactory.getProxy()方法创建一个实例GetProxyRequestCallback，然后休眠，等待该proxyCreated(...)方法被调用。因此，如果 proxyCreated() 没有中断最初创建请求的线程，则该线程将等待，直到达到超时。

在我看来，这是 matlabcontrol 库中的一个缺陷： Thread.interrupt() 不应该被滥用于此目的，因为被中断的线程可能有不同的原因，并且除了发出线程应该停止的信号之外不应该用于任何其他用途。

这应该在 matlabcontrol 库中通过等待互斥体来修复。

例如：

class RemoteMatlabProxyFactory implements ProxyFactory {
    // [...]

    @Override
    public MatlabProxy getProxy() throws MatlabConnectionException {
        GetProxyRequestCallback callback = new GetProxyRequestCallback();
        Request request = this.requestProxy(callback);
        return callback.getProxy(_options.getProxyTimeout());
    }

    // [...]
}

private static class GetProxyRequestCallback implements RequestCallback {
    private final Object _lock = new Object();
    private MatlabProxy _proxy;

    @Override
    public void proxyCreated(MatlabProxy proxy) {
        _proxy = proxy;

        _requestingThread.interrupt();
    }

    public MatlabProxy getProxy(long timeout) throws MatlabConnectionException {
        synchronized (_lock) {
            if (_proxy != null) {
                return _proxy;
            }
            try {
                _lock.wait(timeout);
            } catch (InterruptedException e) {
                Thread.currentThread().interrupt();
                throw new MatlabConnectionException("Thread was interrupted while waiting for MATLAB proxy", e);
            }
            if (_proxy == null) {
                throw new MatlabConnectionException("MATLAB proxy could not be created in " + timeout + " milliseconds");
            }
            return _proxy;
        }
    }
}