Javaでウィンドウに四角を複数、座標を変えて描画する簡潔なコードの書き方が知りたいです

Question

前提・実現したいこと
ウィンドウに四角を複数描画したいが、座標を変えて同じものを描画するだけなので無駄にコードを増やしたくなく、省略する方法を探しています。

該当のソースコード

class Window extends JFrame{
	static JFrame mainFrame = new JFrame("The Mountain");;
	//contentpaneを取得しそこにパネルを配置
	static Container contentPane = mainFrame.getContentPane();
    	public static void Win() {
    		Square sq = new Square();
    		Container cp = mainFrame.getContentPane();
    		cp.add( sq );
 mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            mainFrame.setSize(330, 650);
            mainFrame.setLocationRelativeTo(null);
            //フレームを表示
            mainFrame.setVisible(true);
    	}
    }
class Square extends JPanel{
	public void paint(Graphics g) {
		super.paintComponent(g);
		g.fillRect(260,400,30,30);//塗りつぶし
	}
	}

これによって四角が一つは描画できている状態です。

自分で調べたことや試したこと
Squareクラスの引数をg1,g2...のように増やしていってsuper.paintComponent(g○);とg○.fillRect();を増やしてみたのですが二つ以上は描画されませんでした。また、この方法でできたとしてもプログラムが長くなってしまうので望んでいる形ではありません。Squareクラスのインスタンスを複数生成してリストにいれて簡潔に管理するという方法はできるのかなど考えましたが描画する座標を変えたいのでそれはできないか..という感じで手詰まりです。

使っているツールのバージョンなど補足情報
JavaSE-17,EclipseIDE
プログラミング言語を学ぶのはJavaが初めてで学び始めて1週間程度ですが、独習Javaという本を読んだだけなのでまずは作りたいものを作りながら学んでいこうと思い一昨日辺りから書き始めました。一つ一つのコードを理解しながら書いていきたいと思っていますがstatic Container contentPane = mainFrame.getContentPane();とContainer cp = mainFrame.getContentPane();二つは何を表しているのかがなんとなくでしかわかっていませんのでおかしいと思われる箇所は他にもご指摘いただきたいです。

Comments

[jimbe]

コードはマークダウン記法をご利用ください。
マークダウン記法に付きましてはヘルプをご参照ください。

[jimbe]

ご提示のコードはそのままでは実行できません。
どのように実行されましたでしょうか。

[jimbe]

> ウィンドウに四角を複数描画したいが、座標を変えて同じものを描画するだけなので無駄にコードを増やしたくなく、省略する方法を探しています。

まず複数 (3 つ程度 ?) 描画するコードを作っては如何でしょう。

[H-JP]

同じファイルの中でDisplayクラスというものを作り、ボタンやラベルの追加を行い、最後にWindowクラスのWinメソッドを呼び出すというメソッドを作りました。そしてそのメソッドを別のファイル(ゲーム進行を行うためのもの)にあるメソッドで呼び出すという形で実行しています。調べたところゲームを作る際は画面表示を行うクラスとゲームを進行するクラスは分けた方が良いと書いてあったので分けてみたのですがファイルも別々のものにするのはなぜだかいまいちわかっておりません。

[jimbe]

> おかしいと思われる箇所は他にもご指摘いただきたいです。

ということから言いますと、恐らく全体構造は分かり難い方向になっているように思います。(個人の感想です。念の為。)
画面表示とゲーム進行のクラスは分けるのかもしれませんが、 Window と Display は何故分けるのでしょうか。
ボタンやラベルの追加を Window のコンストラクタで行ってしまえば良いように思います。
(蛇足ですが、 "Window" というクラスは Swing/AWT にありますので紛らわしいです。 "Mountain" の JFrame なのでしたら、"MountainFrame" 等としては如何でしょうか。)

[H-JP]

アドバイスありがとうございます、今一度クラスを整理しようと思います。

Answer 1

Square クラスをパネル内で "四角を書くクラス" としてみました。

import java.awt.*;
import java.util.ArrayList;
import java.util.List;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class MountainFrame extends JFrame {
  public static void main(String[] args) {
    new MountainFrame().setVisible(true); //フレームを生成・表示
  }

  public MountainFrame() {
    super("The Mountain");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setSize(330, 650);
    setLocationRelativeTo(null);

    DrawingPanel drawingPanel = new DrawingPanel();
    drawingPanel.addSquare(260,400,30,30,Color.RED);
    drawingPanel.addSquare(100,100,30,30,Color.BLUE);
    drawingPanel.addSquare(150,300,20,40,Color.GREEN);

    //contentpaneを取得し…なくても直接 add で配置出来る
    add(drawingPanel);
  }
}

class DrawingPanel extends JPanel {
  //四角塗りつぶし
  private static class Square {
    final int x, y, width, height;
    final Color color;
    Square(int x, int y, int width, int height, Color color) {
      this.x = x;
      this.y = y;
      this.width = width;
      this.height = height;
      this.color = color;
    }
    void draw(Graphics g) {
      g.setColor(color);
      g.fillRect(x, y, width, height);
    }
  };
  private List<Square> squareList = new ArrayList<>();

  void addSquare(int x, int y, int width, int height, Color color) {
    synchronized(squareList) {
      squareList.add(new Square(x, y, width, height, color));
    }
    repaint();
  }

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

    Square[] squares;
    synchronized(squareList) {
      squares = squareList.toArray(new Square[squareList.size()]);
    }
    for(Square square : squares) square.draw(g);
  }
}

Comments

[H-JP]

ありがとうございます、ContentPaneを取得しなくてもよいというのはどこにも載っていなかったので知ることができてよかったです。

Answer 2

Squareクラスの引数をg1,g2...のように増やしていってsuper.paintComponent(g○);とg○.fillRect();を増やしてみたのですが二つ以上は描画されませんでした。

増やすのはfillRectのほうです。

g.fillRect(260, 400, 30, 30);
g.fillRect(260, 440, 30, 30);

また、この方法でできたとしてもプログラムが長くなってしまうので望んでいる形ではありません。

ということは何か法則性があるってことですね？
だったらfor等を使えば短く書けますね。

static Container contentPane = mainFrame.getContentPane();とContainer cp = mainFrame.getContentPane();二つは何を表しているのかがなんとなくでしかわかっていません

この辺まあいろいろあるんですが、mainFrame.add(sq);で意味は同じになるんで「JFrameに追加するのね」位の認識でよろしいかと思います。

仕組みがわからないと寝られないタイプの方は、↓らへんを見てください。
How to Use Root Panes (The Java™ Tutorials > Creating a GUI With Swing > Using Swing Components)

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.Container;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Point;
import java.awt.Rectangle;


class Window extends JFrame {
    // 間違いというわけではないが、提示コードの範囲ではローカル変数で十分に見える。
    // ほかのクラスで使用したいということなら、その限りではない。
    // static JFrame mainFrame = new JFrame("The Mountain");;
    // static Container contentPane = mainFrame.getContentPane();

    // public static void Win() {
    public static void main(String[] args) {
        JFrame mainFrame = new JFrame();
        mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        mainFrame.setSize(330, 650);
        mainFrame.setLocationRelativeTo(null);

        Square sq = new Square();

        // Container cp = mainFrame.getContentPane();
        // cp.add(sq);
        // ↑↓同じ意味なので↓の方がスッキリ
        mainFrame.add(sq);

        mainFrame.setVisible(true);
    }
}

class Square extends JPanel {
    private final Point[] points = new Point[]{
            new Point(120, 120),
            new Point(240, 80),
            new Point(180, 220),
    };

    private final Rectangle[] rectangles = new Rectangle[]{
            new Rectangle(120, 340, 30, 30),
            new Rectangle(160, 400, 50, 20),
            new Rectangle(80, 480, 40, 30),
    };

    // SwingではpaintComponentをオーバーライドする
    // public void paint(Graphics g) {
    @Override public void paintComponent(Graphics g) {
        super.paintComponent(g);

        // 複数描きたいなら続けて描くだけ
        g.fillRect(260, 400, 30, 30);
        g.fillRect(260, 440, 30, 30);
        g.fillRect(260, 480, 30, 30);

        // 何か法則性があるならループを使えば行数を減らせる
        g.setColor(Color.RED);
        for (int i = 0; i < 10; i++) {
            g.fillRect(40, i * 40 + 40, 30, 30);
        }

        // 法則性がないなら事前に座標を配列等でもっておく？（行数は減らないが^^;
        g.setColor(Color.GREEN);
        for (Point p : points) {
            g.fillRect(p.x, p.y, 30, 30);
        }

        // サイズもバラバラな場合はRectangleクラスがある
        g.setColor(Color.BLUE);
        Graphics2D g2 = (Graphics2D) g;
        for (Rectangle r : rectangles) {
            g2.fill(r);
        }
    }
}

Comments

[H-JP]

わざわざ実行結果画面までつけて頂きありがとうございます、とても為になりました。