java timerについて

Question

public static void main(String[] args) {

    TimerTask timerTask = new TimerTask(){ 
    	
    	int cnt = 0;
    	
        public void run() { 
        	if(cnt < 5){
        		System.out.println("おはよう"); 
        		cnt++;
        	}
        } 
    }; 
    Timer timer = new Timer(); 
    timer.schedule(timerTask, 3000L, 3000L);

}

上記のようなタイマーを使って、３秒間隔で５回「おはよう」が出力されるプログラムを作り、一応その通りに動いたのですが、このプログラムは特になにか問題はないでしょうか？
きちんと動作していても問題点があると思い質問させていただきました。

Answer 1

あるプログラムの挙動に問題があるかは、要件次第というところもありますので
一概にこれは問題だとは言えないと思いますが、気になるところを書かせていただきます。

timer.schedule(timerTask, 3000L, 3000L); ですと、
３秒後から３秒間隔でtimerTaskが呼ばれ続けてしまいます。
つまり、５回「おはよう」と出力されたあともtimerTaskが呼ばれ続けます。
無駄にtimerTaskが呼び出されないように、cancelメソッドを呼んであげた方がいいと思います。

また今回のように軽い処理ではあまり関係ありませんが、timerTask内の処理が重い処理の場合には
scheduleメソッドに似たメソッドにscheduleAtFixedRateというものがあるので調べてみてください。

Comments

[kisaragizinzin7]

回答ありがとうございます＾＾

cancelメソッド加えたいと思います！
ありがとうございました。