java awtの描画ツール処理について

Question

前提

javaでマウスを押したら丸い図形が描画されるツールを作成しています。（java awt）

実現したいこと

常に同じ数の図形が表示されるようにしたい（例えば10個だとしたら、１１個目を描画するときに一番古い図形を消す）です。できれば常に表示する数をコマンドライン引数で指定できるようにしたいです。

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

調べたのですがなかなか出てこなく、どなたか助けてほしいです。

該当のソースコード

package enshu_1102;

import java.awt.Color;
import java.awt.Frame;
import java.awt.Graphics;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.util.ArrayList;

public class Paint2 extends Frame implements MouseListener,MouseMotionListener{
int x,y;
int sum= 0;
ArrayList<Figure> objList; //描画する全オブジェクトを管理する
Figure obj; //実際に描画するオブジェクト
public static void main(String[] args) {
Paint2 f = new Paint2();
f.setSize(640,480);//ウインドウサイズ
f.setTitle("Paint Sample");//タイトル名
//f.getContentPane().setBackground( Color.RED );
f.addWindowListener(new WindowAdapter() {
@Override public void windowClosing(WindowEvent e) {
System.exit(0); //
}});
f.setVisible(true);
}
Paint2(){ //objListの準備（マウスなどの）
objList = new ArrayList<Figure>();
//マウス処理の登録
addMouseListener(this);
addMouseMotionListener(this);
}
@Override public void paint(Graphics g){ //必要な時に勝手に呼び出してくれるメソッド
Figure f;
for(int i =0; i<objList.size(); i++) {
f = objList.get(i);
f.paint(g); //その要素のpaintメソッド呼び出し
}
if(obj != null) obj.paint(g); //現在作成中の要素も描画
}
@Override public void mousePressed(MouseEvent e) {//マウスボタンが押された時の処理
x = e.getX();//クリックされた座標
y = e.getY();
obj = new Circle(); //objクラスの変数にCircleを代入

	obj.moveto(x,y);
	
	sum = sum + 1;
	System.out.println("Circle is :" + sum); //★1今までに描画した円の数
	
	//System.out.println(objList);
	//objList.add(0,obj);
	repaint(); //ウインドウを再描画（paintを呼び出してくれる）
}
@Override public void mouseReleased(MouseEvent e) {//マウスボタンが離された時の処理
	x = e.getX();
	y = e.getY();
	//obj = new Circle();
	obj.moveto(x,y);
	objList.add(0,obj);//図形を追加
	obj = null;
	repaint();
}
public void mouseClicked(MouseEvent e) {}
public void mouseEntered(MouseEvent e){}//コンポーネントにマウスが入ると呼び出される
public void mouseExited(MouseEvent e){} //コンポーネントからマウスが出ると呼び出される
public void mouseDragged(MouseEvent e){ //ボタンを押したまま動かした時の処理
	x = e.getX();
	y = e.getY();
	/*obj = new Circle();
	obj.moveto(x,y);
	objList.add(0, obj);
	repaint(); */
	if(obj != null) obj.moveto(x,y);
	repaint(); 
}
public void mouseMoved(MouseEvent e) {}
}

//マウスを話した瞬間にobjListに格納
//〇はobjに格納される

Comments

[jimbe]

コードの提示には「コードのマークダウン」を使用してください。
マークダウンを使用しないと、コード内の記号等がフォーマットに使用され、コピペしても実行できなかったりします。
マークダウンにつきましてはヘルプを参照してください。

[jimbe]

>調べたのですがなかなか出てこなく

何が起きていて何を調べたのでしょうか。

Answer 1

なぜ今更 AWT …昔の課題の使い回しでしょうか。
Figure や Circle がどんな定義なのか分からなかったのでテキトウに作りました。
ついでにドラッグでちらつきが酷かったのでダブルバッファリングしています。
Frame 直接では余計な描画がありそうだったので描画領域用に Panel を拡張して使っています。

「マウスを押したら丸い図形が描画される」ように、 mousePressed でリストに追加しています。それにより最新の Figure はリストの最後、最古の Figure はリストの最初となりますので、「例えば10個だとしたら、１１個目を描画するときに一番古い図形を消す」は、(描画時では無く)追加する時に先頭を削除で行い、「現在作成中」(ドラッグ中)なのは最新の Figure なので別変数で管理する必要は無くリストの最後のオブジェクトを変更しています。
コマンドラインからの個数指定は、main メソッドで解析して渡しています。

DrawingFrame.java

import java.awt.*;
import java.awt.event.*;
import java.awt.image.VolatileImage;
import java.util.ArrayList;

public class DrawingFrame extends Frame {
  public static void main(String[] args) throws NumberFormatException {
    int max = 10;
    if(args.length >= 1) max = Integer.parseInt(args[0]);
    new DrawingFrame(max).setVisible(true);
  }

  DrawingFrame(int max) {
    super("Paint Sample");
    addWindowListener(new WindowAdapter() {
      @Override
      public void windowClosing(WindowEvent e) {
        System.exit(0);
      }
    });
    setSize(640,480);
    setResizable(false);

    add(new DrawingPanel(max));
  }
}

class DrawingPanel extends Panel {
  private java.util.List<Figure> figureList = new ArrayList<>();
  private VolatileImage backscreen;

  DrawingPanel(int max) {
    super(null);
    //setBackground(Color.YELLOW);

    MouseAdapter adapter = new MouseAdapter() {
      @Override
      public void mousePressed(MouseEvent e) {
        while(figureList.size() >= max) figureList.remove(0);
        figureList.add(new Circle(e.getX(), e.getY()));
        System.out.println("Circle is :" + figureList.size()); //★1今までに描画した円の数
        repaint();
      }
      @Override
      public void mouseDragged(MouseEvent e) {
        figureList.get(figureList.size()-1).moveto(e.getX(), e.getY());
        repaint();
      }
    };
    addMouseListener(adapter);
    addMouseMotionListener(adapter);
  }

  @Override
  public void update(Graphics g) {
    paint(g);
  }

  @Override
  public void paint(Graphics g) {
    super.paint(g);
    Dimension size = getSize();
    if(backscreen == null || backscreen.validate(getGraphicsConfiguration()) == VolatileImage.IMAGE_INCOMPATIBLE) {
      backscreen = createVolatileImage(size.width, size.height);
    }
    Graphics bg = backscreen.getGraphics();
    bg.setColor(getBackground());
    bg.fillRect(0, 0, size.width, size.height);
    for(Figure f : figureList) f.paint(bg);
    bg.dispose();

    g.drawImage(backscreen, 0, 0, this);
  }
}

class Circle extends Figure {
  Circle(int x, int y) {
    super(x, y);
  }
  @Override
  public void paint(Graphics g) {
    g.setColor(Color.GREEN);
    g.fillOval(x-15, y-15, 30, 30);
  }
}

abstract class Figure {
  protected int x, y;
  Figure(int x, int y) {
    this.x = x;
    this.y = y;
  }
  public void moveto(int x, int y) {
    this.x = x;
    this.y = y;
  }
  abstract public void paint(Graphics g);
}

Answer 2

常に同じ数の図形が表示されるようにしたい（例えば10個だとしたら、１１個目を描画するときに一番古い図形を消す）です。

objList.add(0,obj);//図形を追加

ここでリストの先頭に追加しているわけですが、１１個になっていたらリストの最後を削除するだけでは？（描画順的に最後に追加して最初を削除のほうが好みです）
ArrayList#remove (Java Platform SE 8 )

コマンドライン引数で指定できるようにしたいです。

java コマンドライン引数 使い方 - Google 検索
java 文字列 数値 変換 - Google 検索

調べたのですがなかなか出てこなく、どなたか助けてほしいです。

まあ気持ちはわかりますが、ドンピシャな答えがあるほうが珍しいのであって基本的にはすべて応用問題です。
プログラミングはその場で試せて結果も即出ます。エラーが出ても全く気にする必要はありません。
仮説を立てて確認の繰り返しです。思いついたことをどんどん試してみましょう。

import java.awt.*;
import java.awt.event.*;
import java.util.*;


public class Paint2 extends Frame implements MouseListener, MouseMotionListener {
    public static void main(String[] args) {
        int limit;
        try {
            limit = Integer.parseInt(args[0]);
        } catch (Exception e) { // 雑いｗ
            limit = 10; // デフォルト10個
        }
        new Paint2(limit).setVisible(true);
    }

    private final ArrayList<Figure> objList = new ArrayList<>();
    private final int limit;
    private Figure obj;

    Paint2(int limit) {
        this.limit = limit;

        setSize(640, 480);
        setTitle("Paint Sample");

        addWindowListener(new WindowAdapter() {
            @Override public void windowClosing(WindowEvent e) { System.exit(0); }
        });
        addMouseListener(this);
        addMouseMotionListener(this);
    }

    @Override public void paint(Graphics g) {
        for (Figure f : objList) f.paint(g);
        if (obj != null) obj.paint(g);
    }

    @Override public void mousePressed(MouseEvent e) {
        obj = new Circle();
        obj.moveTo(e.getX(), e.getY());
        repaint();
    }

    @Override public void mouseReleased(MouseEvent e) {
        obj.moveTo(e.getX(), e.getY());

        objList.add(obj); // 図形を最後に追加
        if (limit < objList.size()) { // リミットを超えていたら...（リミットが可変ならwhileのほうがいい）
            objList.remove(0); // 先頭を削除
        }

        obj = null;
        repaint();
    }

    @Override public void mouseDragged(MouseEvent e) {
        if (obj != null) {
            obj.moveTo(e.getX(), e.getY());
            repaint();
        }
    }

    @Override public void mouseClicked(MouseEvent e) { }

    @Override public void mouseEntered(MouseEvent e) { }

    @Override public void mouseExited(MouseEvent e) { }

    @Override public void mouseMoved(MouseEvent e) { }
}

class Circle extends Figure {
    private final int r = random.nextInt(30, 60);

    @Override public void paint(Graphics g) {
        g.setColor(color);
        g.fillOval(x - r, y - r, r * 2, r * 2);
    }
}

abstract class Figure {
    protected static final Random random = new Random();
    protected int x, y;
    protected Color color = new Color(random.nextInt(0xFFFFFF));

    public abstract void paint(Graphics g);

    public void moveTo(int x, int y) {
        this.x = x;
        this.y = y;
    }
}

---

（おそらく課題なのでしょうから）AWTはマストなのかもしれませんが、ちらつきが気になるのでSwing版。

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;
import java.util.LinkedList;
import java.util.Random;


public class Paint2 extends JPanel {
    public static void main(String[] args) {
        int limit;
        try {
            limit = Integer.parseInt(args[0]);
        } catch (Exception e) {
            limit = 10;
        }
        JFrame frame = new JFrame("Paint Sample");
        frame.setSize(640, 480);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setLocationRelativeTo(null);
        frame.add(new Paint2(limit));
        frame.setVisible(true);
    }

    private final LimitedQueue<Figure> figures;
    private Figure current;

    Paint2(int limit) {
        figures = new LimitedQueue<>(limit);
        setBackground(Color.WHITE);

        MouseAdapter listener = new MouseAdapter() {
            @Override public void mousePressed(MouseEvent e) {
                current = new Circle();
                current.moveTo(e.getX(), e.getY());
                repaint();
            }

            @Override public void mouseReleased(MouseEvent e) {
                current.moveTo(e.getX(), e.getY());
                figures.add(current);
                current = null;
                repaint();
            }

            @Override public void mouseDragged(MouseEvent e) {
                if (current != null) {
                    current.moveTo(e.getX(), e.getY());
                    repaint();
                }
            }
        };
        addMouseListener(listener);
        addMouseMotionListener(listener);
    }

    @Override protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        for (Figure f : figures) f.paint(g);
        if (current != null) current.paint(g);
    }
}

class Circle extends Figure {
    private final int r = random.nextInt(30, 60);

    @Override public void paint(Graphics g) {
        g.setColor(color);
        g.fillOval(x - r, y - r, r * 2, r * 2);
    }
}

abstract class Figure {
    protected static final Random random = new Random();
    protected int x, y;
    protected Color color = new Color(random.nextInt(0xFFFFFF));

    public abstract void paint(Graphics g);

    public void moveTo(int x, int y) {
        this.x = x;
        this.y = y;
    }
}

// サイズ制限のあるリスト（やってることは同じ）
// [collections - Size-limited queue that holds last N elements in Java - Stack Overflow](https://stackoverflow.com/questions/5498865/size-limited-queue-that-holds-last-n-elements-in-java)
class LimitedQueue<E> extends LinkedList<E> {
    private final int limit;

    public LimitedQueue(int limit) {
        this.limit = limit;
    }

    @Override public boolean add(E o) {
        boolean added = super.add(o);
        while (added && limit < size()) remove();
        return added;
    }
}

Comments

[TN8001]

jimbeさんの回答と大差ないですが、Swing版も欲しかったので^^;

[jimbe]

> LimitedQueue

取り出しで無くなるわけでは無いので、配列でリングバッファでも良いかもしれません。