「Util.Timer」を使って車輪が回転するコードを作成しましたが、回転しない原因がわかりません

Question

今晩は。

車輪が回転するコードを作成しました。
エラ-は出ていませんが、車輪は回転しません。

宜しくご教示お願い致します。

このコードは、「Util.Timer」を使って周期的に動かそうとしていますが、
「Swing.Timer」を使って書き換えると問題なく動きました。

問題は「TimerTask」、「Timer」の箇所に原因がある
と考えていますが、どこをどのように修正すればよいのか分かりません。

詳しい方宜しくお願い致します。
//----------------------------------------------------

public class Kanran_Frame extends JFrame
{
	public Kanran_Frame( )
	{
		Container cnt = this.getContentPane() ;
		cnt.setBackground( Color.black ) ;

		this.setSize( 800 , 600 ) ;
		this.setLayout( null ) ;

		Kanran_Panel1 k_can1 = new Kanran_Panel1() ;
		k_can1.setLayout( null ) ;
		k_can1.setSize( 350 , 350 ) ;

		cnt.add( k_can1 ) ;

		Timer timer = new Timer() ;
		timer.schedule( ( new MyTimer() ) , 500 , 1000 ) ;
	}

	public static void main( String[] args )
	{
		Toolkit.getDefaultToolkit().setDynamicLayout( true ) ;
		new Kanran_Frame() ;
		( new Kanran_Frame() ).setVisible( true ) ;
	}
}

//---------------------------------------------------------

public class Kanran_Panel1 extends JPanel
{
	int     x , y ;
	int     xc = 200 ;
	int     yc = 200 ;
	double  hankei = 100.0 ;
	double  kakudo ;

	int rr = 0 ;
	Graphics2D g2 ;

	public Kanran_Panel1( )
	{
		this.setLocation( 300 , 0 ) ;
		this.setBackground( Color.black ) ;
	}

	public void paintComponent( Graphics g )
	{
		super.paintComponent( g ) ;
		g2 = (Graphics2D) g ;

		System.out.println( "rr_00 =" + rr ) ;
		g2.rotate( rr * Math.PI / 180 , xc , yc ) ;

		//Outer CicleLine
		g2.setColor( Color.GREEN ) ;
		g2.drawOval( 100 , 100 , 200 , 200 ) ;

		//Frame
		for( kakudo = 0.0 ; kakudo < 360 ; kakudo += 20.0 )
		{
			x = xc + (int)( hankei * Math.cos( Math.toRadians( kakudo ) ) ) ;
			y = yc + (int)( hankei * Math.sin( Math.toRadians( kakudo ) ) ) ;

			g2.drawLine( xc , yc , x , y ) ;
		}
	}

	public void move()
	{
		rr = ++rr % 360 ;

		System.out.println( "rr_11 =" + rr ) ;

	}
}

//-----------------------------------------------------------------

public class MyTimer extends TimerTask
{
	Kanran_Panel1 kp1 ;

	public MyTimer()
	{
		kp1 = new Kanran_Panel1() ;
	}

	@Override
	public void run( )
	{
		 kp1.move() ;
		 kp1.repaint() ;
	}
}

//----------------------------------------------------------------

takashi8842 · Accepted Answer

No.1です。

「rr_00」が抜けるということは、paintComponentが呼び出されていないということです。

kp1.repaint() を実行したからといって、必ずしもpaintComponentが呼び出されるとは限りません。

APIのドキュメントによると、
http://docs.oracle.com/javase/jp/8/docs/api/java/awt/Component.html#repaint--
「このメソッドはこのコンポーネントのpaintメソッドをできるだけ早く呼び出します。」
とあります。
repaint自体は、再描画のリクエストを出すだけで、実際の再描画は状況によっては遅延される可能性があります。

何かの処理が重くて、タイマーでrunを呼び出すタイミングに追い付かないときは、paintComponentが呼び出されず、何回かのrepaintがまとめられて１回のpaintComponent呼び出しになってしまうのでしょう。

多分、最初にフレーム等を生成・表示する処理に手間取って、最初だけ飛ばされてしまうのかもしれません。

takashi8842 · Answer

質問から何日か経っていますが解決したでしょうか？

念のため、気がついたところを指摘しておきます。

まず、回転しない原因ですが、これは、Kanran_Panel1のインスタンスが２つ作られ、一つが回転しないで表示され、もう一つが表示されずに回転しているせいです。

new Kanran_Panel1() が２か所にあります。
public Kanran_Frame( )の中で作られたものが表示されて、public MyTimer()の中で作られたものが表示されずに回転しています。

修正案としては、Kanran_Frame( )で作ったKanran_Panel1のインスタンスをMyTimer()に渡すやり方があります。

public Kanran_Frame( )の中の
timer.schedule( ( new MyTimer() ) , 500 , 1000 ) ;
を
timer.schedule( ( new MyTimer(k_can1) ) , 500 , 1000 ) ;
に変更し、さらに、
public MyTimer()
 {
 kp1 = new Kanran_Panel1() ;
 }
 を
public MyTimer(Kanran_Panel1 p)
 {
 kp1 = p;
 }
 に変更します。

これでとりあえず回転するようになると思います。

もう一つ問題になるのが、mainメソッドの中の、
 new Kanran_Frame() ;
 ( new Kanran_Frame() ).setVisible( true ) ;
の部分です。
これでは、Kanran_Frameが２つ作られ１つが表示されていますので、
( new Kanran_Frame() ).setVisible( true ) ;
だけにしてください。

さらにもう一つ、このままでは、表示されたKanran_Frameの右上の閉じるボタンをクリックしても、フレームは閉じますが、タイマーは動いたまま残ります。
とりあえずこのアプリケーションを終了させてしまえばいいのであれば、Kanran_Frame( )の中の
this.setLayout(null);の次の行あたりに
this.setDefaultCloseOperation(EXIT_ON_CLOSE);
を入れておきます。

「Util.Timer」を使って車輪が回転するコードを作成しましたが、回転しない原因がわかりません

質問から何日か経っていますが解決したでしょうか？

No.1です。

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング