Timerクラスとgraphicsクラスでアニメーションの作成

Question

前提

Timerクラスを使って定期実行する描画のプログラミングを作りました。
デバックでだいぶ減らしたのですが、ここからどうすればいいのかわからなかったので教えてほしいです。
また、作ろうとしている動作が正常にできるかも判断してほしいです。描画のクラスは描画だけの時に動いたので問題ないと思います。

実現したいこと

正n角形を１秒ごとに正三角形から正n角形まで描画する。
角数が増えるごとに前の描画を消しています。

発生している問題・エラーメッセージ

PolygonTimer.java:20: エラー: 式の開始が不正です
        public void setI(){this.i++;}
        ^
PolygonTimer.java:22: エラー: <identifier>がありません
        System.out.print("How many squares do you want to draw?:");
                        ^
PolygonTimer.java:22: エラー: 型の開始が不正です
        System.out.print("How many squares do you want to draw?:");
                         ^
PolygonTimer.java:31: エラー: 型の開始が不正です
        if(i = n;){
        ^
PolygonTimer.java:31: エラー: <identifier>がありません
        if(i = n;){
            ^
PolygonTimer.java:31: エラー: 型の開始が不正です
        if(i = n;){

該当のソースコード

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.Graphics;
import java.util.Timer;
import java.util.TimerTask;
import java.util.Scanner;

class PolygonTimer{ 

        private int i = 3;
 	Scanner stdIn = new Scanner (System.in);
    public static void  main(String[] args){ 
 
   
        GameWindow gw = new GameWindow("circle and polygon",1000,1000); 
        gw.add(new DrawCanvas(i)); 
        gw.setVisible(true); 

	public void setI(){this.i++;}
	
	System.out.print("How many squares do you want to draw?:");
  	int n = stdIn.nextInt();
	Timer timer = new Timer();
	TimerTask task = new TimerTask(){
	 
	public void run(){
	gw.DrawCanvas(3);
	gw.setI();
	}
	if(i = n;){
	timer.cancel();}
	
	};

	 public void start(){timer.schedule(task,1000,1000);}
	

    }   
 
 
class DrawCanvas extends JPanel{  
 
 public void paintComponent(Graphics g) { 
  super.paintComponent(g); 
 

g.clearRect(0,0,1000,1000); 

  int x0 = 500;  
  int y0 = 500;  
  int r = 400;  
  int x[] = new int[i]; 
  int y[] = new int[i]; 
 
  double delta =  ( 2 *Math.PI )/ (double)i; 
 
  for(int l = 0; l<i; l++) { 
   x[l] = (int)(-Math.cos(Math.PI/2.0 + delta * l)*r) + x0; 
   y[l] = (int)(-Math.sin(Math.PI/2.0 + delta * l)*r) + y0; 
  } 
  
     g.setColor(Color.red); 
  g.drawPolygon(x, y, i); 
   i++; 

 }
 
    public void paint(Graphics g) { 
     super.paint(g); 
  g.setColor(Color.blue); 
     g.drawOval(100, 100, 800, 800); 
  } 
 
 } 
 
class GameWindow extends JFrame{ 
 public GameWindow(String title, int width, int height) { 
  super(title); 
  setDefaultCloseOperation(EXIT_ON_CLOSE); 
  setSize(width,height); 
  setLocationRelativeTo(null); 
  setResizable(false); 
   
 } 
}

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

デバッグしてください、ということでしょうか。
(コンパイルエラーを取るのはデバッグ以前ですが。)

少なくとも、メソッドの中にメソッドの定義があるのをおかしいと思えないのでは、基本が分かっていないようにしか見えません。

[退会済みユーザー]

学校の課題で授業ペースが早く、自分でもよくわかっていない状態で難しいものに手を出してしまった状態です、、、申し訳ないです、、
提出が近いのでその場しのぎで書いてもらって後から勉強したいです
おこがましいですが想像しているように直してもらうことは可能でしょうか

[m.ts10806]

作業依頼や課題代行に応える場ではないので、きっちり怒られるか、先生や同級生に聞くしかないと思います。

[m.ts10806]

ひとまずセミコロンは基本的に「文章としての終端」なので、「構文」として成り立たせるのなら、置いていいかどうかというのは考えてみてください。（つまりはエラーの通りということです）
=は代入で==は比較
ただ、さすがにこれを教えない先生も教科書もないはずなので、やりたいことの前に基礎を復習した方が結果的には早く進みますよ。ここで後回しにしてしまうと、ずっと後回しになって永遠に手に付けられることはないと思ってください。人間って楽な方に流れるような頭になってます。

[TN8001]

↓の質問をしたBiginnerさんですか？（後半部分が全く同じです）
[javaで正多角形の描画のプログラムをやっています。正n角形のｎの部分を入力するところでつまずいています。](https://teratail.com/questions/5dy91ppgg7ybxn)

もしそうであれば利用規約違反です。
[利用規約](https://teratail.com/legal)

同じ学校の別人あるいは先の質問を参考にした、ということであればすいません。

[dodox86]

一体どこの学校なのか。課題をこなした後はもうきっとプログラミングはしないのでしょうね。

[退会済みユーザー]

意見ありがとうございます。
現状進んだところで提出し、テストが終わってからゆっくりプログラミングを勉強しなおそうと思います。これからも何度か修正していくので変身していただけると助かります。
TN8001さんの返答としては、前つかっていたアカウントのパスワード等を忘れてしまい、新しくアカウントを作っていました。そうとは知らず申し訳ございません。

[m.ts10806]

ログイン画面に「パスワードを忘れた方はこちら」が見やすい位置にあるので、そこで新しいアカウントを作ろうという思考に飛んでしまうのはちょっと改めたほうが良いかと。

[dodox86]

それにしても前のご質問の投稿は今から2ヶ月以上前ですね。私はこれにて離脱します。

[jimbe]

課題に既視感を感じたのはそういうことでしたか。

[退会済みユーザー]

友達に教えてもらいながら解決することができました。描画するクラスをタイマータスク内で実行することで解決しました。インスタンス化では処理に飛ばないと勘違いしていたのが遠回りした原因でした。
たくさんの意見ありがとうございました。これからの勉強に役立てたいと思います。

[jimbe]

>友達に教えてもらいながら

何よりこれが一番ですね。

[TN8001]

> 前つかっていたアカウントのパスワード等を忘れてしまい、新しくアカウントを作っていました。

それはいけませんね。
このページの下部のヘルプから、パスワード再設定フォームに飛べます。
多分動いていると思いますが、返信がなかったりうまくいかない場合は直接「お問い合わせ」してください。

[ヘルプ | パスワードを忘れてしまった](https://teratail.com/help#forget-password)
[パスワード再設定リクエストを送る](https://teratail.com/login/forget)
[運営へのお問い合わせ](https://teratail.com/contact/input)

復旧しましたら質問を「解決済」にしたうえで、古いアカウントのほうは速やかに退会手続きをしてください。
[ヘルプ | 退会したい](https://teratail.com/help#about-chancel)
[退会申請](https://teratail.com/users/setting/leave)

[退会済みユーザー]

了解しました。回答も含めて最後までありがとうございました。

Answer 1

小さくしています。

PolygonTimer.java

import java.awt.*;
import java.awt.event.ComponentAdapter;
import java.awt.event.ComponentEvent;
import java.util.Scanner;
import java.util.Timer;
import java.util.TimerTask;

import javax.swing.*;

public class PolygonTimer extends JFrame {
  public static void main(String[] args) {
    int n = input();
    SwingUtilities.invokeLater(() -> new PolygonTimer(n).setVisible(true));
  }

  private static int input() {
    int n = 0;
    try(Scanner scanner = new Scanner(System.in)) {
      do {
        System.out.print("How many squares do you want to draw?:");
        n = scanner.nextInt();
      } while(n < 3 || 999 < n);
    }
    return n;
  }

  PolygonTimer(int n) {
    super("Circle and Polygon");
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(300, 300);
    //setLocationRelativeTo(null);

    add(new DrawCanvas(n));
  }
}

class Circle {
  private int x, y, w, h;

  Circle(int x0, int y0, int r) {
    x = x0 - r;
    y = y0 - r;
    w = h = r * 2;
  }

  void paint(Graphics2D g2) {
    g2.setColor(Color.blue);
    g2.drawOval(x, y, w, h);
  }
}

class Polygon {
  private int n, x[], y[];

  Polygon(int x0, int y0, int r, int n) {
    if(n < 3) throw new IllegalArgumentException("n は 3 以上でなければならない. n="+n);
    this.n = n;

    double delta = (2 * Math.PI) / n;
    x = new int[n];
    y = new int[n];
    for(int i=0; i<n; i++) {
      x[i] = (int)(-Math.cos(Math.PI / 2 + delta * i) * r) + x0;
      y[i] = (int)(-Math.sin(Math.PI / 2 + delta * i) * r) + y0;
    }
  }

  void paint(Graphics2D g2) {
    g2.setColor(Color.red);
    g2.drawPolygon(x, y, n);
  }
}

class DrawCanvas extends JPanel {
  private int numberOfVertices = 3, max;
  private Circle circle;
  private Polygon polygon;

  DrawCanvas(int max) {
    if(max < 3) throw new IllegalArgumentException("max error");
    this.max = max;

    addComponentListener(new ComponentAdapter() {
      @Override
      public void componentResized(ComponentEvent e) {
        createDrawingObjects();
      }
    });

    Timer timer = new Timer();
    timer.schedule(new TimerTask(){
      public void run(){
        if(countUp()) {
          timer.cancel();
          return;
        }
        createDrawingObjects();
        repaint();
      }
    }, 1000, 1000);
  }

  private void createDrawingObjects() {
    int x0 = getWidth() / 2;
    int y0 = getHeight() / 2;
    int r = (int)(Math.min(x0, y0) * 0.8);
    int n = numberOfVertices;
    circle = new Circle(x0, y0, r);
    polygon = new Polygon(x0, y0, r, n);
  }

  private boolean countUp() {
    if(numberOfVertices >= max) return true; //終わり
    numberOfVertices++;
    return false;
  }

  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);

    Graphics2D g2 = (Graphics2D)g;
    g2.setStroke(new BasicStroke(10));
    if(circle != null) circle.paint(g2);
    if(polygon != null) polygon.paint(g2);
  }
}

How many squares do you want to draw?:8

Answer 2

Timerクラスを使って定期実行する描画のプログラミングを作りました。

Swingではjava.util.Timerではなく、javax.swing.Timerを使用してください。
Timer (Java Platform SE 8 )

デバックでだいぶ減らしたのですが、ここからどうすればいいのかわからなかったので教えてほしいです。

メソッドの中にメソッドが書かれていたりif文すら正しく書けないとなると、もう一度基本から復習されたほうがいいと思います。

また、作ろうとしている動作が正常にできるかも判断してほしいです。

どういうことでしょうか？
出来るか出来ないかでいえば、思いつくことは大体できるでしょう。

描画のクラスは描画だけの時に動いたので問題ないと思います。

細かくクラスを分けるのがJavaっぽいのでしょうが、極力シンプルに１クラスだけにしました。

import javax.swing.*;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Scanner;

public class DrawCanvas extends JPanel implements ActionListener {
    public static void main(String[] args) {
        // 正n角形 最大値取得
        Scanner scanner = new Scanner(System.in);
        int max;
        while (true) {
            System.out.print("正何角形まで描きますか？3-999:");
            try {
                max = scanner.nextInt();
                if (3 <= max && max <= 999) break;
            } catch (Exception ignored) { scanner.next(); } // 不正入力スキップ
        }

        // JFrame（ウィンドウ）の準備
        JFrame frame = new JFrame("正多角形");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(1000, 1000);
        frame.setLocationRelativeTo(null);
        frame.add(new DrawCanvas(max)); // 最大値をコンストラクタ引数で渡す
        frame.setVisible(true);
    }

    private final int max; // 正n角形 最大値
    private int n = 3; // 正n角形 現在値

    public DrawCanvas(int m) { // コンストラクタ
        max = m; // 引数をフィールドに保存
        setBackground(Color.WHITE); // 背景 白
        new Timer(1000, this).start(); // タイマースタート！
    }

    @Override public void actionPerformed(ActionEvent e) { // １秒おきに呼ばれる
        n++;
        if (max < n) n = 3;
        repaint(); // 再描画
    }

    @Override public void paintComponent(Graphics g) { // 描画が必要になったとき呼ばれる
        super.paintComponent(g);

        // ウィンドウリサイズ対応
        int x0 = getWidth() / 2;
        int y0 = getHeight() / 2;
        int r = Math.min(x0, y0) / 5 * 4;

        int[] x = new int[n];
        int[] y = new int[n];
        double delta = 2 * Math.PI / n;

        for (int i = 0; i < n; i++) {
            x[i] = (int) (Math.cos(-Math.PI / 2 + delta * i) * r) + x0;
            y[i] = (int) (Math.sin(-Math.PI / 2 + delta * i) * r) + y0;
        }
        g.setColor(Color.RED);
        g.drawPolygon(x, y, n);
    }
}

---

タイマーの使い方はご理解いただけたかと思います（簡単ですよね？）

クラスを３つ使うのが必須条件なのかもしれませんが、（課題ということなので）答えをそのまま書く気はありません。
元々の動いていたコードにタイマー部分を追加するか、回答コードを３つに分割してください^^;

public class PolygonTimer {
    public static void main(String[] args) {
// mainを多少修正
    }
}

class DrawCanvas extends JPanel implements ActionListener {
// main以外の全部
}

class GameWindow extends JFrame {
// そのまま
}

Comments

[jimbe]

確か repaint は EDT 外からでも動作するとなっていたと思いますので、　java.util.Timer でも良いように思います。(心配事は増えますけども。)

[TN8001]

今回の内容（repaintを呼ぶだけ）ならどのタイマーでも問題ないということですね。

> (心配事は増えますけども。)

精度（遅延）を気にする必要がなければ、javax.swing.Timerが無難ですね^^

[jimbe]

厳しい条件なら遅延も問題ですが、普通にマルチスレッドになりますので、コンポーネントのフィールド(DrawCanvas.n)の更新のアトミック性とか、そのフィールドを何度も直接使用する場合に途中で変更されることへの考慮(paintComponent 中に n が変更されたら?)とかもですね。

Answer 3

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.Graphics;
import java.util.TimerTask;
import java.util.Scanner;
import java.util.Timer;
import java.util.TimerTask;
import java.text.ParseException;

class Polygonauto {

public static void  main(String[] args)throws ParseException{ 
  GameWindow gw = new GameWindow("円と正多角形",1000,1000); 

  Scanner stdIn = new Scanner(System.in);
  System.out.print("正n角形までの描画　nを入力:");
  int n = stdIn.nextInt() + 1;

  Timer timer = new Timer(false);
     TimerTask task = new TimerTask() {

              int cnt=3;

		@Override
		public void run() {
	
        
      gw.add(new DrawCanvas(cnt)); 
      gw.setVisible(true); 

      cnt++;
      if( cnt >= n ) timer.cancel();
		}
	};
	timer.schedule(task, 0, 500);





}  

}

class DrawCanvas extends JPanel{
private int n;

DrawCanvas(int n){
this.n = n;
}

public void paintComponent(Graphics g) {
super.paintComponent(g);

int x0 = 500;
int y0 = 500;
int r = 400;
int x[] = new int[n];
int y[] = new int[n];

double delta = ( 2 *Math.PI )/ (double)n;

for(int l = 0; l<n; l++) {
x[l] = (int)(-Math.cos(Math.PI/2.0 + delta * l)*r) + x0;
y[l] = (int)(-Math.sin(Math.PI/2.0 + delta * l)*r) + y0;
}

 g.setColor(Color.red); 

g.drawPolygon(x, y, n);

}

public void paint(Graphics g) { 
 super.paint(g); 

g.setColor(Color.blue);
g.drawOval(100, 100, 800, 800);
}

}

class GameWindow extends JFrame{
public GameWindow(String title, int width, int height) {
super(title);
setDefaultCloseOperation(EXIT_ON_CLOSE);
setSize(width,height);
setLocationRelativeTo(null);
setResizable(false);

}
}

Comments

[jimbe]

質問のコードのように、コードのマークダウンを利用して頂けると読み易くなって助かります。

paint を override して行っている青丸の描画は、 paintComponent の最後で行っても同じです。
swing は AWT と違い (paint では無く ) paintComponent を override する方針ですので、 paint から移動させて paint は削除したほうが良いです。

また、 TN8001 さんが回答での指摘や、私もそのコメントに少し書かせて頂きましたが、 Swing は java.util.Timer とは別に動作します。 Swing のコンポーネントが最初に表示状態になって以降、 Swing の EDT(イベントディスパッチスレッド) 以外から操作することは、一部ドキュメントで言及されていること以外は動作が保証されない行為です。
こちらのコードでは TimerTask での gw.add(new DrawCanvas(cnt)) がそれに当たります。
一応動作しているようですので課題としては OK が出るかもしれませんが、 Swing としては『あと少し』と思います。