応答可能なグラフィカルインターフェイスを使用しているときにCallableを使用したマルチスレッド

Question

cmdコマンドを実行しているときに応答性の高いJavaFXグラフィカルインターフェイスを取得しようとしています。
実行しているコマンドは次のとおりです。

youtube-dl.exe --audio-format mp3 --extract-audio https://www.youtube.com/watch?v=l2vy6pJSo9c 

ご覧のとおり、YouTubeのリンクをmp3ファイルに変換するyoutube-downloaderです。 これは、メインのFXスレッドではなく、2番目のスレッドで実行します。

私は、StartDownloadingThreadクラスのインターフェイスCallableを実装することでこれを解決しました。

@Override 
public Process call() throws Exception { 
    Process p = null; 
    p = ExecuteCommand(localCPara1, localCPara2, localDirectory).start(); 
    try { 
     Thread.sleep(30); 
    }catch (InterruptedException e){} 
    return p; 
} 

ProcessBuilder

だけオブジェクトを返すExecuteCommand方法。

私はThread.sleepを使用して、プログラムをメインスレッドに戻し、アプリケーションを応答性にします。残念ながら、プログラムはまだフリーズします。

これは、メソッド呼び出しの呼び出し方法です。

ExecutorService pool = Executors.newFixedThreadPool(2); 
StartDownloadingThread callable = new StartDownloadingThread(parameter1, parameter2, directory); 
Future future = pool.submit(callable); 
Process p = (Process) future.get(); 
p.waitFor(); 

Callableを使用してGUIを応答させるにはどうすればよいですか？

Answer 1

executorを使用してgetメソッドを使用するだけで、タスクを実行するときに返されるFutureは、実際には元のスレッドを解放して他のタスクを続行しません。後で元のスレッドでwaitForメソッドを使用しても、Callableで行う処理よりもさらに時間がかかる可能性があります。

Task classは、イベントハンドラを使用してアプリケーションスレッドの成功/失敗を処理できるため、この目的のためにはより適切です。

また、タスクの実行が完了したら、ExecutorServiceがシャットダウンされていることを確認してください。

Task<Void> task = new Task<Void>() { 
    @Override 
    protected Void call() throws Exception { 
     Process p = null; 
     p = ExecuteCommand(localCPara1, localCPara2, localDirectory).start(); 

     // why are you even doing this? 
     try { 
      Thread.sleep(30); 
     }catch (InterruptedException e){} 

     // do the rest of the long running things 
     p.waitFor(); 
     return null; 
    } 
}; 
task.setOnSucceeded(event -> { 
    // modify ui to show success 
}); 

task.setOnFailed(event -> { 
    // modify ui to show failure 
}); 
ExecutorService pool = Executors.newFixedThreadPool(2); 

pool.submit(task); 

// add more tasks... 

// shutdown the pool not keep the jvm alive because of the pool 
pool.shutdown();