新しいスレッドでスケジュールを実行する

Question

APIをクエリしてレポートを作成し、そのレポートを5分後に読み込みたいとします。私はそのためにScheduledExecutorServiceを使用したいと思います。エグゼクティブにスレッドをブロックさせたくないので、新しいスレッドを作成していますが、これが正しい方法かどうかはわかりません。ここに私のコードは次のとおりです。

Thread thread = new Thread() { 
     public void run() { 
      log.info("Starting..."); 
      new RequestReport().runScheduledTask(requestId); 
     } 
    }; 
thread.start(); 

---

private void runScheduledTask(String requestId) { 
    log.info("Starting five-minute countdown now..."); 
    ScheduledFuture<?> countdown = scheduler.schedule(() -> { 
     try { 
      new GetReportList().run(requestId); 
     } catch (Exception e) { 
      e.printStackTrace(); 
     } 
    }, 5, TimeUnit.MINUTES); 

    try { 
     countdown.get(); 
    } catch (InterruptedException | ExecutionException e) { 
     log.info("catched Exception"); 
     e.printStackTrace(); 
    } 
    scheduler.shutdown(); 
} 

は、5分後に別の機能を実行するためのより良い方法はありますか？私はこれをやっているのですか？私は何を変えるべきですか？

ところで、私は春を使用しています - それを改善できるものはありますか？

Answer 1

ScheduledExecutorServiceに言われたクラスのインスタンスに沿って渡すことができますあなたはそれを間違って使用します：

まず、Threadを作成する必要はありません。機能に何も追加せず、リソースを浪費します。

第2に、shutdown()を呼び出した後、schedulerはタスクを受け付けなくなります。複数のレポートを生成する必要がある場合は、これが問題になります。

第3に、タスクの完了後にコードが何もしないので、get()を呼び出す必要はありません。

だから、あなたが必要とする唯一のコードは次のとおりです。

scheduler.schedule(() -> { 
    try { 
     new GetReportList().run(requestId); 
    } catch (Exception e) { 
     e.printStackTrace(); 
    } 
}, 5, TimeUnit.MINUTES); 

それは、タスクのスケジュールを設定し、すぐにスレッドを解放します。タスクは5分後に別のスレッドで実行され、そのスレッドはschedulerによって管理されます。

予約タスク（状態の確認、キャンセルなど）が必要な場合は、をschedule()から取得してどこかに保存しますが、質問のコードに基づいて必要はありません。

Answer 2

あなたは、コードの詳細「正しい」になりたい場合は、あなたがRunnableインターフェイスを実装して独自のクラスにRequestReportを分離し、スレッドのコンストラクタ