ペイントツール 色の選択とボタンを押して線を消したい

Question

前提・実現したいこと

簡単なペイントツールを解説サイトを見ながら作成していて
見つけた仕組みを組み合わせて
ボタンを押して線の色を変更したいのと
クリアのボタンを押してウィンドウの線を消したいのですが
何を見て解決したらよいのかわからず困っています。

ここを見ながら作っています
http://www.ibe.kagoshima-u.ac.jp/edu/ProgramingJava/StreamIO/s5.html

理解が及んでいないことは重々承知しているのですが
知恵をお貸しください

よろしくお願い致します。

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

エラーメッセージ

該当のソースコード

import java.awt.Button;
import java.awt.Color;
import java.awt.Event;
import java.awt.Graphics;
import java.awt.Point;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseMotionListener;
import java.util.ArrayList;
import java.util.Enumeration;
import java.util.List;
import java.util.Vector;

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

public class simplePainter {
    static Vector data;

    public static void main(String[] args) {
        JFrame frame = new JFrame("SimplePainter");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setSize(1000, 1000);
        frame.setLocationRelativeTo(null);

        Canvas canvas = new Canvas();
        frame.add(canvas);

        frame.setVisible(true);

        data = new Vector();


    }

    public void draw(Graphics g) {


    }

}

class Canvas extends JPanel implements MouseListener, MouseMotionListener {

    List<List<Point>> pintsList = new ArrayList<>();

    List<Point> currentPoint = new ArrayList<>();

    boolean isDraw = false;

    private Object data;
    public Canvas() {
        addMouseListener(this);
        addMouseMotionListener(this);
        setSize(1000,1000);
        add(new Button("Quit"));
        add(new Button("Clear"));

        currentPoint = new Vector();

    }

    void quit(){
        System.exit(0);
    }

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



        Color c = new Color(100,255,100);
        g.setColor(c);
        }




    void paintFreeCurvedLines(Graphics g) {
        if (isDraw) {
            pintsList.add(currentPoint);
            for (List<Point> points : pintsList) {
                for (int i = 1; i < points.size(); i++) {
                    Point start = points.get(i-1);
                    Point end = points.get(i);
                    g.drawLine(start.x, start.y, end.x, end.y);
                    Color c = new Color(100,255,100);
                    g.setColor(c);

                }
                }

                }
                    currentPoint = new ArrayList<>();


                }

    @Override
        public void mouseClicked(MouseEvent e) {

    }

    @Override
        public void mousePressed(MouseEvent e) {
        isDraw = false;
       currentPoint.add(new Point(e.getX(),e.getY()));

    }

    @Override
        public void mouseReleased(MouseEvent e) {
        isDraw =true;
       currentPoint.add(new Point(e.getX(), e.getY()));
        super.repaint();


    }

    @Override
        public void mouseEntered(MouseEvent e) {


    }

    @Override
        public void mouseExited(MouseEvent e) {

    }

    @Override
        public void mouseDragged(MouseEvent e) {
        currentPoint.add(new Point(e.getX(),e.getY()));


    }

    @Override
        public void mouseMoved(MouseEvent e) {

    }

    public void clear() {
       data.removeAllElement();
        repaint();


    }

    public void paint1(Graphics g){
        for(Enumeration e=data.elements();e.hasMoreElements();)
          ((simplePainter)e.nextElement()).draw(g);
      }

    public boolean action(Event ev,Object arg){
        if(ev.target instanceof Button){
            if(arg.equals("Quit"))
                quit();
            if(arg.equals("Clear"))
                clear();
        }
        return true;
 }

    @SuppressWarnings("deprecation")
    public boolean handleEvent(Event ev){
        if(ev.id == Event.WINDOW_DESTROY)
            quit();
        return super.handleEvent(ev);
    }

}

試したこと

カラーパレットを一度組み込もうとしたが適応方法がわからず
上手く色が実装できないため現在はコードを消しています

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

Eclipse を使っています。

Comments

[m.ts10806]

>Vector　を使うとクリアが実装できるとあったが
どこでしょうか。出典明示してください。

[Kazumannbo-]

確認したのですが一度描画した線を再現するためにVectorを使っていました
言葉足らずで申し訳ありません
ここを見ながら作っています
http://www.ibe.kagoshima-u.ac.jp/edu/ProgramingJava/StreamIO/s5.html

[m.ts10806]

質問は編集できるので情報は基本的に質問本文に記載してください

Answer 1

Vector は, 言ってしまえば List の古い版です.
今わざわざ　Vector を( List を使っているコードと混ぜて) 使う意味は無いのではと思います.

「何を見て解決」というなら, 参考にされている記事・コードしかありません.
要は, 線をどうデータとして表現しているか, その表現の中で"消す"というのはどういうことかを理解しているかどうかです.
複数のプラモデルを組み合わせて新しいものを作ろうとするなら, 元のプラモデルの構造を理解している必要があるはずです.
(「理解が及んでいないことは重々承知している」なればこそ)参考にされている記事・コードの理解を進めることをお勧めします.

#追加
リンク先の SimpleDraw02 を Swing 化するとこのような感じでしょうか.
なるべく元の状態を維持しつつ…と思ったのですが, 描画領域は専用クラス(CanvasPanel)にしないと面倒そうでしたので諦め, 各リスナも無名クラス化してしまいました.

//package teratail.q249928;

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

import javax.swing.*;

/**
 * 線分情報クラス <p>
 * 描画された1本の線分の情報を記憶しておくためのクラスです。
 * 線分情報には、始点座標と終点座標とがあります。
 */
class SegmentInfo {

  /** 始点座標 */
  private int x1, y1;

  /** 終点座標 */
  private int x2, y2;

  /** 色 */
  private Color color;

  /** 始点と終点を与えて生成します。 */
  SegmentInfo(int x1, int y1, int x2, int y2, Color color) {
    this.x1 = x1;
    this.y1 = y1;
    this.x2 = x2;
    this.y2 = y2;
    this.color = color;
  }

  /** 自分自身を描画します。 */
  void draw(Graphics g) {
    g.setColor(color);
    g.drawLine(x1, y1, x2, y2);
  }
}

class CanvasPanel extends JPanel {
  /** 線分の描画のための座標. */
  private int lx1, ly1, lx2, ly2;

  /** 線分情報を記憶しておくためのベクトル. */
  private List<SegmentInfo> data = new ArrayList<SegmentInfo>();

  private Color color = new Color(100,255,100);

  CanvasPanel() {
    super(null);
    setBackground(Color.WHITE);

    addMouseListener(new MouseAdapter() {
      /**
       * マウスボタンが押されると、その後のマウスの動きを追跡して仮の線を引く必要があります。
       */
      @Override
      public void mousePressed(MouseEvent e) {
        lx1 = lx2 = e.getX();
        ly1 = ly2 = e.getY();
        drawXORLine();
      }

      /**
       * マウスボタンが離されると、以前の仮の線を消して、確定した直線を描きます。
       */
      @Override
      public void mouseReleased(MouseEvent e) {
        drawXORLine();
        lx2 = e.getX();
        ly2 = e.getY();
        data.add(new SegmentInfo(lx1, ly1, lx2, ly2, color));
        repaint();
      }
    });

    addMouseMotionListener(new MouseMotionAdapter() {
      /**
       * マウスがドラッグされると、それまでの仮の線を消して、新しい仮の線を描きます。
       */
      @Override
      public void mouseDragged(MouseEvent e) {
        drawXORLine();
        lx2 = e.getX();
        ly2 = e.getY();
        drawXORLine();
      }
    });
  }

  /** 仮の線分の描画. */
  private void drawXORLine() {
    // XORモードで仮の線分を描画します。
    // XORモードで描かれた線分は、再度同じ線分を描画すると消えます。
    Graphics g = getGraphics();
    g.setXORMode(getBackground());
    g.drawLine(lx1, ly1, lx2, ly2);
    g.dispose();
  }

  /** すべての線分を描画します。 */
  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);
    for(SegmentInfo e : data) e.draw(g);
  }

  /** 全データを消して、再描画します。 */
  void clear() {
    data.clear();
    repaint();
  }
}

public class SimpleDraw02 extends JFrame {

  public SimpleDraw02() {
    super("SimpleDraw ver 0.2(Swing)");
    setSize(1000, 1000);
    setLocationRelativeTo(null);

    setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
    addWindowListener(new WindowAdapter() {
      @Override
      public void windowClosing(WindowEvent e) {
        quit();
      }
    });

    CanvasPanel canvas = new CanvasPanel();
    add(canvas, BorderLayout.CENTER);

    JPanel eastPanel = new JPanel(new FlowLayout());
    add(eastPanel, BorderLayout.EAST);

    JButton quitButton = new JButton("Quit");
    quitButton.addActionListener(new ActionListener() {
      /** Quit ボタンが押されたときの処理を行います。 */
      @Override
      public void actionPerformed(ActionEvent e) {
        quit();
      }
    });
    eastPanel.add(quitButton);

    JButton clearButton = new JButton("Clear");
    clearButton.addActionListener(new ActionListener() {
      /** Clear ボタンが押されたときの処理を行います。 */
      @Override
      public void actionPerformed(ActionEvent e) {
        canvas.clear();
      }
    });
    eastPanel.add(clearButton);
  }

  /**
   * プログラムを終了します。
   * 終了にはいくつかの方法がありますが、すべてここでその処理を行うことで、動作の統一を図ります。
   */
  void quit() {
    System.exit(0);
  }

  /**
   * クラスを直接実行すると、このメソッドから開始されます。
   * ここでは、クラス自身のインスタンスを作ってそれを表示しています。
   * これにより、クラスの実行が開始されます。
   */
  public static void main(String[] args) {
    new SimpleDraw02().setVisible(true);
  }
}

#追加2
リンク先での「線の色を指定できるように」という課題でのヒントにおきまして「『Javaって何ですか？』の章の例題が参考」とありますが, アプレット(AWT)ですので, アプリケーション(Swing)化しました.
色の設定は Choice(AWT) から JComboBox(Swing) に変わりますが, 扱い方(addItem 等)はあえて変えていません.

//package teratail.q249928;

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

import javax.swing.*;

// Lineクラス
class Line {
  Point sp, ep; // 始点と終点
  Color color; // 色

  // コンストラクタ
  Line(Point p1, Point p2, Color c) {
    sp=p1; ep=p2; color=c;
  }
  // 描画
  void draw(Graphics g) {
    g.setColor(color);
    g.drawLine(sp.x, sp.y, ep.x, ep.y);
  }
}

// JControlPanelクラス
class JControlPanel extends JPanel implements ActionListener {
  private JSimpleDrawMain app;
  // コンストラクタ
  public JControlPanel(JSimpleDrawMain app) {
    super(new FlowLayout());
    this.app = app;
    JComboBox<String> ch = new JComboBox<>();
    ch.addItem("Black");
    ch.addItem("Red");
    ch.addItem("Blue");
    ch.addItem("Green");
    ch.addItem("Yellow");
    ch.addItem("Gray");
    ch.addItem("Pink");
    ch.addActionListener(this);
    add(ch);

    JButton clearButton = new JButton("Clear");
    clearButton.addActionListener(this);
    add(clearButton);
  }
  // アクション処理
  @Override
  public void actionPerformed(ActionEvent e) {
    if(e.getSource() instanceof JComboBox) {
      String arg = (String)((JComboBox<?>)e.getSource()).getSelectedItem();
      if(arg.equals("Black")) app.color=Color.BLACK;
      else if(arg.equals("Red")) app.color=Color.RED;
      else if(arg.equals("Blue")) app.color=Color.BLUE;
      else if(arg.equals("Green")) app.color=Color.GREEN;
      else if(arg.equals("Yellow")) app.color=Color.YELLOW;
      else if(arg.equals("Gray")) app.color=Color.GRAY;
      else if(arg.equals("Pink")) app.color=Color.PINK;
    }
    if(e.getSource() instanceof JButton) {
      String arg = ((JButton)e.getSource()).getText();
      if(arg.equals("Clear")) app.clear();
    }
  }
}

// JDrawCanvasクラス
class JDrawCanvas extends JPanel {
  private JSimpleDrawMain app;
  private Point p1, p2; // 描画中の始点と終点

  // コンストラクタ
  public JDrawCanvas(JSimpleDrawMain app) {
    super(null);
    this.app = app;
    setBackground(Color.WHITE);

    addMouseListener(new MouseAdapter() {
      // マウスボタンが押された
      @Override
      public void mousePressed(MouseEvent e) {
        Graphics g = getGraphics();
        g.setXORMode(getBackground());
        p2 = p1 = new Point(e.getX(), e.getY());
        g.drawLine(p1.x, p1.y, p2.x, p2.y);
        g.dispose();
      }

      // マウスボタンが離された
      @Override
      public void mouseReleased(MouseEvent e) {
        Graphics g = getGraphics();
        g.setXORMode(getBackground());
        g.drawLine(p1.x, p1.y, p2.x, p2.y);
        g.setPaintMode();
        p2 = new Point(e.getX(), e.getY());
        if(app.line_num < JSimpleDrawMain.LINEMAX) {
          app.lines[app.line_num] = new Line(p1, p2, app.color);
          app.lines[app.line_num++].draw(g);
        }
        g.dispose();
      }
    });

    addMouseMotionListener(new MouseMotionAdapter() {
      // マウスがドラッグされた
      @Override
      public void mouseDragged(MouseEvent e) {
        Graphics g = getGraphics();
        g.setXORMode(getBackground());
        g.drawLine(p1.x, p1.y, p2.x, p2.y);
        p2 = new Point(e.getX(), e.getY());
        g.drawLine(p1.x, p1.y, p2.x, p2.y);
        g.dispose();
      }
    });
  }
  // 再描画
  public void paintComponent(Graphics g) {
    super.paintComponent(g);
    for(int i = 0; i < app.line_num; i++)
      app.lines[i].draw(g);
  }
}

public class JSimpleDrawMain extends JPanel {
  static final int LINEMAX = 100; // 最大線数
  int line_num; // 現在の線数
  Line[] lines; // 線データ
  Color color; // 現在の色
  JDrawCanvas dc; // 描画キャンバス

  // 初期化
  public void init() {
    setBackground(Color.white);
    line_num = 0;
    lines = new Line[LINEMAX];
    color = Color.BLACK;
    setLayout(new BorderLayout());
    add(new JControlPanel(this), BorderLayout.SOUTH);
    add(dc = new JDrawCanvas(this), BorderLayout.CENTER);
  }

  // メイン
  public static void main(String[] args) {
    JFrame f = new JFrame("JSimpleDrawMain");
    f.setSize(400,300);
    JSimpleDrawMain a = new JSimpleDrawMain();
    f.add(a, BorderLayout.CENTER);
    a.init();
    f.setVisible(true);
  }

  // クリア
  public void clear() {
    line_num = 0;
    dc.repaint();
  }
}

Comments

[jimbe]

にしても, 鹿児島大学は未だに Vector とか AWT とか アプレットとかの記事を使っているのでしょうか.

[Kazumannbo-]

ありがとうございます。
まだ初心者向けの参考書をみて学習している段階ですが色々と調べてみます

[jimbe]

ついで…と言いますか, JFrame は Swing で Button は AWT です.
Swing と AWT は混在させると異常動作を引き起こしますし, 基本設定から異なったり, 同じ処理でも使用すべきメソッドが異なったりします. (paint では無く paintComponent とか)
恐らく Swing もあまりお分かりになっていないと思いますので, そちらも頑張ってみてください.

[Kazumannbo-]

ありがとうございます！
一通り調べて機能追加頑張ります