javaのcsvファイルの読み込みとGUIについて

Question

前提

csvファイルを拡張for文を利用して読み込み、中に書かれている内容に応じて描画をするプログラムを書きたいのですが、「描画」ボタンを押しても描画されませんでした。
ちなみにcsvファイルには

四角形 黒 50 50 230 150
塗りつぶし四角形 青 50 50 230 150
塗りつぶし四角形 黄色 110 50 25 150
塗りつぶし四角形 黄色 50 110 230 25

と記載されています。

該当のソースコード

import java.awt.Color;
import java.awt.Container;
import java.awt.Graphics;
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.List;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.WindowConstants;
import java.awt.BorderLayout;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class finalReport extends JFrame implements ActionListener{
    private JPanel panel1;
    private JTextField textField1;
    private JButton button1;
    private String fileName;
    public finalReport() {
        setLayout(new BorderLayout());
        panel1 = new JPanel();
        textField1 = new JTextField("sec.csv", 10);
        button1 = new JButton("描画");
        button1.addActionListener(this);
        panel1.setLayout(new FlowLayout());
        panel1.add(textField1);
        panel1.add(button1);
        Container container = getContentPane();
        container.setLayout(new BorderLayout());
        add(panel1, BorderLayout.NORTH); 
        setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        setSize(700, 300);
        setLocationRelativeTo(null);
        setVisible(true);
    }
    
    class MyPanel extends JPanel {
        public void paint(Graphics g) {
            try {
            		Path path = Paths.get(fileName);
            		List<String> list = Files.readAllLines(path, StandardCharsets.UTF_8);
            		for (String s : list) {
            			String[] raw = s.split(",");
            			String shape = raw[0];
            			String color = raw[1];
            			int x_axis = Integer.parseInt(raw[2]);
            			int y_axis = Integer.parseInt(raw[3]);
            			int width = Integer.parseInt(raw[4]);
            			int height = Integer.parseInt(raw[5]);
            			if (shape.equals("四角形")) {
            				if (color.equals("黒")) {
            					g.setColor(Color.BLACK);
            				}
            				g.drawRect(x_axis, y_axis, width, height);
            			} else if (shape.equals("塗りつぶし四角形")) {
            				if (color.equals("青")) {
            					g.setColor(Color.BLUE);
            				}else if (color.equals("黄色")) {
            					g.setColor(Color.YELLOW);
            				}g.fillRect(x_axis, y_axis, width, height);
            			} 
            		}
            	} catch (IOException e) {
            		e.printStackTrace();
            	}
        	}
    	}
    @Override
    public void actionPerformed(ActionEvent e) {
        fileName = textField1.getText();
        if (fileName.equals("sec.csv")) {
        	MyPanel myPanel = new MyPanel();
        	add(myPanel, BorderLayout.CENTER);
        	myPanel.repaint();
    }}
    public static void main(String[] args) {
        new finalReport();
    }
}

試したこと

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

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

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

Comments

[jimbe]

paint() はウインドウの重なり等で反映に実行されます。
ですので極力軽くする必要があり、ファイルアクセス等描画以外の(相対的に)重い処理を行ってはいけません。

Answer 1

Swing では paint() では無く paintComponent() を override し、描画以外は極力行わないようにしてください。

FinalReport.java

import java.awt.*;
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.*;
import java.util.*;
import java.util.List;

import javax.swing.*;

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

  public FinalReport() {
    super("FinalReport");
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    setSize(700, 300);
    setLocationRelativeTo(null);
    setLayout(new BorderLayout());

    JTextField textField = new JTextField("sec.csv", 10);
    JButton button = new JButton("描画");

    JPanel panel = new JPanel(new FlowLayout());
    panel.add(textField);
    panel.add(button);
    add(panel, BorderLayout.NORTH);

    MyPanel myPanel = new MyPanel();
    add(myPanel, BorderLayout.CENTER);

    button.addActionListener(v -> {
      String fileName = textField.getText();
      try {
        myPanel.readFile(fileName);
      } catch(IOException e) {
        e.printStackTrace();
      }
    });
  }

  private class MyPanel extends JPanel {
    private static final Map<String,Color> colorMap = new HashMap<>();
    static {
      colorMap.put("黒", Color.BLACK);
      colorMap.put("青", Color.BLUE);
      colorMap.put("黄色", Color.YELLOW);
    }

    private enum Shape {
      四角形 {
        @Override
        void draw(Graphics g, int x, int y, int w, int h) {
          g.drawRect(x, y, w, h);
        }
      },
      塗りつぶし四角形 {
        @Override
        void draw(Graphics g, int x, int y, int w, int h) {
          g.fillRect(x, y, w, h);
        }
      };
      abstract void draw(Graphics g, int x, int y, int w, int h);
    }

    private class DrawingObject {
      private Shape shape;
      private Color color;
      private int x, y, w, h;
      DrawingObject(Shape shape, Color color, int x, int y, int w, int h) {
        this.shape = shape;
        this.color = color;
        this.x = x;
        this.y = y;
        this.w = w;
        this.h = h;
      }
      void draw(Graphics g) {
        g.setColor(color);
        shape.draw(g, x, y, w, h);
      }
    }

    private List<DrawingObject> doList = new ArrayList<>();

    MyPanel() {
      super(null);
      setBackground(Color.WHITE);
    }

    void readFile(String fileName) throws IOException {
      doList.clear();

      Path path = Paths.get(fileName);
      List<String> list = Files.readAllLines(path, StandardCharsets.UTF_8);
      for(String s : list) {
        String[] raw = s.split(",");
        String shape = raw[0];
        String color = raw[1];
        int x = Integer.parseInt(raw[2]);
        int y = Integer.parseInt(raw[3]);
        int w = Integer.parseInt(raw[4]);
        int h = Integer.parseInt(raw[5]);
        doList.add(new DrawingObject(Shape.valueOf(shape), colorMap.get(color), x, y, w, h));
      }
      repaint();
    }

    @Override
    public void paintComponent(Graphics g) {
      super.paintComponent(g);
      for(DrawingObject obj : doList) obj.draw(g);
    }
  }
}

sec.csv

塗りつぶし四角形,青,50,50,230,150
塗りつぶし四角形,黄色,110,50,25,150
塗りつぶし四角形,黄色,50,110,230,25
四角形,黒,50,50,230,150

描画ボタン押下結果

Answer 2

「描画」ボタンを押しても描画されませんでした。

ウィンドウのサイズを変えたりすると表示されますね。
この辺私もいまだに理解できていないのですが、パネルの追加等レイアウトにかかわる変更の時は、revalidateを呼んだ上でrepaintを呼んでください。

//myPanel.repaint();
revalidate();
repaint();

JComponent#revalidate (Java Platform SE 8 )

---

しかしこのままですと、ボタンを押すたびにMyPanelが増え続けます。
MyPanelを最初から追加しておけば、myPanel.repaint()のままでよかったということになります。

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.nio.file.*;
import java.util.List;


public class FinalReport extends JFrame implements ActionListener {
    public static void main(String[] args) {
        new FinalReport().setVisible(true);
    }

    private final JTextField textField1;
    private final MyPanel myPanel;
    private String fileName;


    public FinalReport() {
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(700, 300);
        setLocationRelativeTo(null);

        JPanel panel1 = new JPanel();
        add(panel1, BorderLayout.NORTH);

        textField1 = new JTextField("sec.csv", 10);
        panel1.add(textField1);

        JButton button1 = new JButton("描画");
        button1.addActionListener(this);
        panel1.add(button1);

        myPanel = new MyPanel();
        add(myPanel, BorderLayout.CENTER);
    }

    @Override public void actionPerformed(ActionEvent e) {
        fileName = textField1.getText();
        if (fileName.equals("sec.csv")) {
            myPanel.repaint();
        }
    }


    class MyPanel extends JPanel {
        @Override public void paintComponent(Graphics g) {
            super.paintComponent(g);
            if (fileName == null) return;

            try {
                Path path = Paths.get(fileName);
                List<String> list = Files.readAllLines(path, StandardCharsets.UTF_8);
                for (String s : list) {
                    String[] raw = s.split(",");
                    String shape = raw[0];
                    String color = raw[1];
                    int x_axis = Integer.parseInt(raw[2]);
                    int y_axis = Integer.parseInt(raw[3]);
                    int width = Integer.parseInt(raw[4]);
                    int height = Integer.parseInt(raw[5]);

                    if (color.equals("黒")) {
                        g.setColor(Color.BLACK);
                    } else if (color.equals("青")) {
                        g.setColor(Color.BLUE);
                    } else if (color.equals("黄色")) {
                        g.setColor(Color.YELLOW);
                    }

                    if (shape.equals("四角形")) {
                        g.drawRect(x_axis, y_axis, width, height);
                    } else if (shape.equals("塗りつぶし四角形")) {
                        g.fillRect(x_axis, y_axis, width, height);
                    }
                }
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
}

Answer 3

        if (fileName.equals("sec.csv")) {
        	MyPanel myPanel = new MyPanel();
+          myPanel.setSize(200, 300);
        	add(myPanel, BorderLayout.CENTER);

Comments

[shiketa]

jimbeさん、TN8001さんご両名の忠告は、きちんとご理解されたほうがいいとおもいますよ。わたしは「描画されませんでした」への対応案を提示しただけですので。

[TN8001]

なかなか難しい問題ですよね...（以下誰かへの意見とかではなく個人の感想です）

> jimbeさん、TN8001さんご両名の忠告は、きちんとご理解されたほうがいいとおもいますよ。

ほとんどの質問者は解決後に質問を見ることはなく、次に質問しようとしたときに気が付くかどうかでしょうね（そして次は１年後かもしれないしｗ
メールアドレスを登録しておけば通知は行くはずですが、みなさんSNS登録なんですかね？（あるいは捨てアド？ｗ

> わたしは「描画されませんでした」への対応案を提示しただけですので。

説明不足に感じたのとリークが気になったので急いで回答しましたが、言葉足らずでjimbeさんの回答を呼んでしまいました（気にはなっていたんですが一言でも触れておくべきでした^^;

teratailでは「質問から15分以内に回答をし、ベストアンサーに選ばれる」と追加スコアがあることから、素早い回答を期待されているんだと思います。
また「価値のあるコンテンツを世の中に残し届ける事」とあるように、閲覧者に役に立つあるいは正しい情報を残すことも求められています。

この両立は大変難しく（当初の設計では回答のブラッシュアップを期待していた？）修正をお仕着せするよりは、多様な回答があったほうが望ましい気がします（明確に間違っている場合はコメントしますが）
個人個人の回答ポリシーで回答していれば、「結果的に充実するのかなー」と思います^^

何が言いたいのかよくわからんコメントになってしまった（コードを書くのは好きだけど文章は苦手^^;