javaで正多角形の描画のプログラムをやっています。正n角形のｎの部分を入力するところでつまずいています。

Question

前提

javaで正多角形の描画のプログラムをやっています。正n角形のｎの部分を入力するところでつまずいています。

実現したいこと

static Scanner stdIn = new Scanner(System.in);
System.out.println("正何角形？:");
static int n = stdIn.nextInt();
の部分で正多角形のnを入力してから、描画に移るプログラミングがしたいです。
n=9;などと置いてから実行した場合はできました。

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

Exception in thread "main" java.lang.NoSuchMethodError: 'void DrawCanvas.<init>(int)'
        at Test2.main(Test1.java:29)

該当のソースコード

該当の言語

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

public class Test1 {
	
    public static void  main(String[] args){
	Scanner stdIn = new Scanner(System.in);
	int n;

	while (true) {
		System.out.println("正何角形？(3-999):");
		try {
			n = stdIn.nextInt();
			if (3 <= n && n <= 999) break;
		} catch (Exception ignored) { stdIn.next(); }
	}
	

		double a = n;
		double b = 180/a;			
		double m = Math.toRadians(b);
		double l = a*(Math.sin(m));
		System.out.println("正"+n+"角形の円周率の近似:"+l);
		
		
		GameWindow gw = new GameWindow("円と正多角形",1000,1000);
		gw.add(new DrawCanvas(n));
		gw.setVisible(true);

 }
}

class DrawCanvas extends JPanel{
	private int n;
	public DrawCanvas(int n) {
        this.n = n;
		SpinnerNumberModel model = new SpinnerNumberModel(n, 3,999, 1);
        JSpinner spinner = new JSpinner(model);
        spinner.addChangeListener(e -> {
            this.n = (int) spinner.getValue();
            repaint();
        });
        add(spinner);
    }



	@Override 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 i= 0; i<n; i++) {
			x[i] = (int)(Math.cos(-Math.PI/2.0 + delta * i)*r) + x0;
			y[i] = (int)(Math.sin(-Math.PI/2.0 + delta * i)*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

[m.ts10806]

まず構文エラー直してビルドできるようになってからでないとスタートラインにすら立てないと思います

[TN8001]

> TN8001さんの修正内容で変更したところ次のようなエラーが出ました。TN8001さんのコードでも同じエラーが出ました。

気が付いていませんでした。すいません。
回答への返答・疑問等は、回答のコメント欄にお願いします。

質問を編集したことは**回答者には通知されません**
[編集した事を回答者にも通知したい | ヘルプ](https://teratail.com/help#notify-editing)

「質問への追記・修正依頼」をした人・低評価をした人には通知されます。

> Exception in thread "main" java.lang.NoSuchMethodError: 'void DrawCanvas.<init>(int)'
> at Test2.main(Test1.java:29)

DrawCanvasのコンストラクタ「public DrawCanvas(int n){}」が見つからないと言っています。
どういう環境でどうコンパイルしどう実行しましたか？

手元のIDE（IntelliJ IDEA）では問題なく実行できました。

Answer 1

そこにstaticは書けません。
閉じかっこ（}）もひとつ多いです。
そもそもmainはひとつしか実行されません。

Test2のmainの最初で入力を受け付けて、nをDrawCanvasのコンストラクタで渡せばいいんじゃないですかね。

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

public class Test2 {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        int n;
        while (true) {
            System.out.println("正何角形？3-20:");
            try {
                n = scanner.nextInt();
                if (3 <= n && n <= 20) break;
            } catch (Exception ignored) { scanner.next(); }
        }

        GameWindow gw = new GameWindow("円と正多角形", 1000, 1000);
        gw.add(new DrawCanvas(n));
        gw.setVisible(true);
    }
}

class DrawCanvas extends JPanel {
    private int n;

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

        SpinnerNumberModel model = new SpinnerNumberModel(n, 3, 20, 1);
        JSpinner spinner = new JSpinner(model);
        spinner.addChangeListener(e -> {
            this.n = (int) spinner.getValue();
            repaint();
        });
        add(spinner);
    }

    @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;

        g.setColor(Color.BLUE);
        g.drawOval(x0 - r, y0 - r, r * 2, r * 2);


        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.0 + delta * i) * r) + x0;
            y[i] = (int) (Math.sin(-Math.PI / 2.0 + delta * i) * r) + y0;
        }
        g.setColor(Color.RED);
        g.drawPolygon(x, y, n);
    }
}

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

---

通常GUIプログラムではコンソール入出力は使用しません。
UI部品（コンポーネント）を使って、簡単に操作できるようにしましょう。

---

追記
１クラスにまとめました。これなら間違いなく実行できるでしょう^^

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

public class Test1 extends JPanel {
    public static void main(String[] args) {
        Scanner stdIn = new Scanner(System.in);
        int n;
        while (true) {
            System.out.println("正何角形？3-999:");
            try {
                n = stdIn.nextInt();
                if (3 <= n && n <= 999) break;
            } catch (Exception ignored) { stdIn.next(); }
        }

        JFrame gw = new JFrame("円と正多角形");
        gw.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        gw.setSize(1000, 1000);
        gw.setLocationRelativeTo(null);

        gw.add(new Test1(n));
        gw.setVisible(true);
    }

    private int n;

    public Test1(int n) {
        this.n = n;
        printApproximationPI();

        SpinnerNumberModel model = new SpinnerNumberModel(n, 3, 999, 1);
        JSpinner spinner = new JSpinner(model);
        spinner.addChangeListener(e -> {
            this.n = (int) spinner.getValue();
            repaint();
            printApproximationPI();
        });
        add(spinner);
    }

    @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;

        g.setColor(Color.BLUE);
        g.drawOval(x0 - r, y0 - r, r * 2, r * 2);


        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);
    }

    private void printApproximationPI() {
        double a = n;
        double b = 180 / a;
        double m = Math.toRadians(b);
        double l = a * (Math.sin(m));
        System.out.println("正" + n + "角形の円周率の近似:" + l);
    }
}

Comments

[jimbe]

最初のうちは「main メソッド以外の static の記述は禁止」ぐらいのことにしても良いのかもしれません。

[TN8001]

私も同じこと言っていましたねｗ
[なぜか一回のクリックでイベントが複数回起こる。](https://teratail.com/questions/236651)

いま過去回答を見直して傾向等のデータを集めてみようと作業しだしたところでした（1000回答行ったら集計するつもり）