C# と VB.NET の質問掲示板

お早う御座います。

Timerを使って観覧車を回転させようと考えています。

全く回転しません。
更に他のパネル等も加えて、何んとか、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 ) ;
	}

	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 ;

	Timer tm ;

	public Kanran_Panel1( )
	{
		this.setLocation( 300 , 0 ) ;
		this.setBackground( Color.black ) ;
	}

	public void paintComponent( Graphics g )
	{
		super.paintComponent( g ) ;
		g2 = (Graphics2D) g ;

		g2.rotate( rr * Math.PI / 180 , xc , yc ) ;

		//Outer Cicle Line
		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 run()
	{
		tm = new Timer() ;
		{
			tm.schedule
			(
				new TimerTask()
				{
					public void run()
					{
						rr = ++rr % 360 ;
						repaint() ;
					}
				} , 0 , 50 ) ;
		}
	}
}

//-----------------------------------------------------------------------

shu さん 回答ありがとうございます。


「Timer　schedul」を使わずに「SwingのTimer」で書いた時には
正常に動いています。

今回、「Timer　schedul」を使かうために書き直したら、動作しません。
つまり、「Timer　schedul」の書き方、書く位置が原因と考えていますが、
修正の仕方が分かりません。


宜しくお願いします。


//-----------------------------------------------------------

public class Kanran_Frame extends JFrame 
{		
	public Kanran_Frame( )
	{				
		Container cnt = this.getContentPane() ;
		
		this.setSize( 800 , 600 ) ;
		cnt.setBackground( Color.black ) ;
		this.setLayout( null ) ;
		
		Kanran_Panel1 k_can1 = new Kanran_Panel1() ;
		k_can1.setLayout( null ) ;
		k_can1.setSize( 350 , 350 ) ;
		k_can1.setBackground( Color.black ) ;
		cnt.add( k_can1 ) ;
		
		new Timer( 30 , k_can1 ).start() ;
	}

	public static void main( String[] args )
	{
		Toolkit.getDefaultToolkit().setDynamicLayout( true ) ;
		
		new Kanran_Frame() ;
		( new Kanran_Frame() ).setVisible( true ) ;
	}
}

//--------------------------------------------

public class Kanran_Panel1 extends JPanel implements ActionListener
{
	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 ) ;
	}

	public void paintComponent( Graphics g )
	{
		super.paintComponent( g ) ;
		g2 = (Graphics2D) g ;
		
		g2.rotate( rr * Math.PI / 180 , xc , yc ) ;
	
		//Outer Cicle Line
		g2.setColor( Color.GREEN ) ;
		g2.drawOval( 100 , 100 , 200 , 200 ) ;

		g2.setColor( Color.magenta ) ;
		g2.drawOval( 101 , 101 , 198 , 198 ) ;

		g2.setColor( Color.orange ) ;
		g2.drawOval( 102 , 102 , 196 , 196 ) ;

		//Scue　Arm Line
		//Backets
		for( kakudo = 0.0 ; kakudo < 360 ; kakudo += 20.0 )
		{
			//Scue　Arm Line
			x = xc + (int)( hankei * Math.cos( Math.toRadians( kakudo ) ) ) ;
			y = yc + (int)( hankei * Math.sin( Math.toRadians( kakudo ) ) ) ;
			g2.drawLine( xc , yc , x , y ) ;

			//Backets
			g2.fillRect( x - 5 , y - 2 , 11 , 11 ) ;
		}
	}
//	@Override
	public void actionPerformed( ActionEvent e )
	{
		// TODO 自動生成されたメソッド・スタブ
		rr = ++rr % 360 ;
		repaint() ;
	}
}

■No81460 (かんたろう さん) に返信

Kanran_Panel1のrunメソッドがどこからも呼ばれてないので
runメソッドを呼ぶようにすれば観覧車は回るはずですよ、ぐーるぐるですよ

java.util.Timerを使う場合TimerTaskはGUIの描画処理を行うスレッドとは
別のスレッドで実行されますのでちゃんと実装するならこんな感じで

new TimerTask() {
    public void run() {
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                rr = ++rr % 360;
                repaint();
            }
        });
    }
}

もりお さん　回答有難う御座います。


教えて頂いたコードを下記のように修正してやりましたが、

//-------------------
@Override
public void run() 
{
rr = ++rr % 360;
repaint();
}
//-------------------
の個所で
「new Runnable(){}のメソッドrun()はスーパークラスのメッソドをオーバライドする必要があります」
というエラーになります。

どこをどのように修正してやればいいのでしょうか。
宜しくご教示お願い致します。


//--------------------------------------

class Kanran_Panel1 extends JPanel implements Runnable 
{
	int     x , y ;
	int     xc = 200 ;
	int     yc = 200 ;
	double  hankei = 100.0 ;
	double  kakudo ;

	int rr = 0 ;
	Graphics2D g2 ;

	Timer tm ;

	public Kanran_Panel1( )
	{
		this.setLocation( 300 , 0 ) ;
		this.setBackground( Color.black ) ;
	}

	public void paintComponent( Graphics g )
	{
		super.paintComponent( g ) ;
		g2 = (Graphics2D) g ;

		g2.rotate( rr * Math.PI / 180 , xc , yc ) ;

		//Outer Cicle Line
		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 run()
	{
		tm = new Timer() ;
		{
			tm.schedule
			(
				new TimerTask() 
				{
					public void run() 
					{
						SwingUtilities.invokeLater
						(
							new Runnable() 
							{
								@Override
								public void run() 
								{
									rr = ++rr % 360;
									repaint();
								}
							}
						);
					}
				} , 0 , 50 ) ;
		}		
	}	
}
//------------------------------------------------------------------

もりお さん 回答有難う御座います。

//---------------------------------------------------------

new TimerTask() {
    public void run() {
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                rr = ++rr % 360;
                repaint();
            }
        });
    }
}

//---------------------------------------------------------

との回答をいただきましたが、

//---------------------------------------------------------

new TimerTask() {
    public void run() {
                rr = ++rr % 360;
                repaint();
                }
}

//---------------------------------------------------------

と書くと何故間違いのでしょうか。

色々と調べてみましたが、よくわかりませんでした。

宜しくお願いします。

■No81510 (かんたろう さん) に返信

時間も経っていてあれだけど、やっぱり気になるので。

Javaはわからないのだけど、観覧車を描いているところ、
特に「g2.rotate( rr * Math.PI / 180 , xc , yc ) ;」が気になって調べました。
C# では、次のようなコードになると思います。
        private Image MakeImage(int turn)
        {
            Bitmap bmp = new Bitmap(200, 200);
            using (Graphics g = Graphics.FromImage(bmp))
            {
                g.DrawRectangle(Pens.Brown, new Rectangle(0, 0, 199, 199));
                g.TranslateTransform(center.X, center.Y);    // ここから3行
                g.RotateTransform(turn);                     //
                g.TranslateTransform(-center.X, -center.Y);  // ここまでが、g2.rotate( rr * Math.PI / 180, xc, yc );
                for (int angle = 0; angle < 360; angle += 20)
                {
                    float x = (float)(center.X + radius * Math.Cos((angle/* + turn*/) * Math.PI/180));
                    float y = (float)(center.Y + radius * Math.Sin((angle/* + turn*/) * Math.PI/180));
                    g.FillRectangle(Brushes.LightGreen, x - 4, y - 2, 8, 10);
                    g.DrawLine(Pens.Black, center, new PointF(x, y));
                }
                g.DrawEllipse(Pens.Blue, center.X - radius, center.Y - radius, radius * 2, radius * 2);
            }

            return bmp;
        }

つまり、(xc, yc) を中心に、rr * Math.PI / 180 ラジアン回転させる、と。
で、これを実行すると、次のような結果が得られます。
https://1drv.ms/i/s%21AiQzlIklFkpNjRzmIBtWSpTl-RXt

どういうことかというと、観覧車が回っているのではなく、観覧車を見ている人が回っています。
g.rotate で、これから描くときに指示する座標に対して、演算を行います。
これによって、座標変換による描画の変更、特に拡大縮小や回転を簡単にできるのですが、
今回は「結果を回す」のではなく、「回っているところを描く」のだから、不適切かな、と。
観覧車を回すには、.rotate を削除して、支柱の座標を求める角度に、angle を足しこみます。