JavaのTextareaの文章を自動更新する。

Question

#実現したいこと
JavaでTextAreaに表示したテキストを文字列が追加されるたびに自動で更新したい。

#現在の課題
以下にTextAreaにテキストファイルの文章を表示する部分のコードを示します。
JFrameのTextAreaに無限ループを使用して文字列が追加されるたびに文字列を追記で表示したいのですが、現段階の課題として、無限ループを使用するとTextAreaに文字が表示されません。

有識者の方々ご協力お願いします

	package logViewer;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;

import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;
import javax.swing.border.TitledBorder;


public class LogPanel2 extends JPanel implements ActionListener,Runnable{

	private JTextArea ta = new JTextArea();  //editer
	private JTextField tf = null; //title
	private JButton btn = null;   //ボタン

	private static int selected;

	private static File selectedFile;


	public LogPanel2() { //logAdd classでcheckBeforeReadfileを使用

	}

	public LogPanel2(String title) {
		super();
		this.setLayout(new BorderLayout());

		tf=new JTextField();
		btn=new JButton("Open");

		//btn.addActionListener(this);

		JPanel pnl=new JPanel();
		pnl.setLayout(new BorderLayout());
		pnl.add(tf,BorderLayout.CENTER);
		pnl.add(btn,BorderLayout.EAST);

		//ta = new JTextArea();
		JScrollPane sp = new JScrollPane(ta);
		sp.setAutoscrolls(true);

		this.add(pnl, BorderLayout.NORTH);
		this.add(sp, BorderLayout.CENTER);

		this.setBorder(new TitledBorder(title));
		this.setPreferredSize(new Dimension(400,200));

		btn.addActionListener(this);

	}


	/*	ファイルを開く
	 */

	public void actionPerformed(ActionEvent e) {
		JFileChooser filechooser = new JFileChooser();

		selected = filechooser.showOpenDialog(this);

		if (JFileChooser.APPROVE_OPTION == selected) {


		    selectedFile = filechooser.getSelectedFile();

			tf.setText("");

			String  strFileName = selectedFile.getName();
			tf.setText(strFileName);

			Thread thread = new Thread(new LogPanel2());
			thread.start();
		}


	}


	public static boolean checkBeforeReadfile(File file) {
		if(file.exists()) {
			if(file.isFile() && file.canRead()) {
				return true;
			}
		}

		return false;

	}

	public void run() {


		try {
			if(checkBeforeReadfile(selectedFile)) {

				String str;

				BufferedReader br = new BufferedReader(new FileReader(selectedFile));

				//StringBuilder sb = new StringBuilder();

				ta.setText("AA");

				while((str = br.readLine()) != null) {

						ta.append(str);
						ta.append("\n");


						System.out.println(str);

				}
				System.out.println("null");

				//ta.setText(new String(sb));

				br.close();

			}else {
				System.out.println("null");
			}


		} catch(IOException err) {

			JOptionPane.showMessageDialog(this,"IOException: ファイルを開くのに失敗しました。");
		}


	}


}

Comments

[TN8001]

> 文字列が追加されるたびに自動で更新したい。

「文字列が追加される」っていうのは「対象のファイルが更新された」って意味ですか？
だとするとファイル更新を監視するのが筋でしょうが、あまりにも今のコードとかけ離れます。
タイマーで何秒かおきに読み直すとかでしょうか。

[man_]

TN8001さん回答ありがとうございます。
対象のファイルが更新されたという意味であっています。
現段階では、TextAreaに定期的に文字列を表示する部分を書いていました。
そこの部分が実装できていない状態です。

マルチスレッドを実装して、run()のメソッドでTextAreaに文字列を表示したいのですが、何も表示されない状態です。

Answer 1

別スレッドからUIを触るとエラーになりますので（回避法もありますが）、タイマーでやった例です。

細かいテストはしていないです（読み書きがバッティングすると、表示が消えてしまうかもしれません）

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.io.IOException;
import java.nio.file.Files;
import java.util.stream.Collectors;
import java.util.stream.Stream;
import javax.swing.JButton;
import javax.swing.JFileChooser;
import javax.swing.JFrame;
import javax.swing.JTextArea;
import javax.swing.JTextField;
import javax.swing.Timer;
import javax.swing.filechooser.FileNameExtensionFilter;

class Main extends JFrame implements ActionListener {
    private JTextField textField;
    private JTextArea textArea;
    private File selectedFile;
    private Timer timer;

    public static void main(String[] args) {
        Main frame = new Main();
        frame.setVisible(true);
    }

    Main() {
        setSize(600, 400);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        textField = new JTextField();
        add(textField, BorderLayout.NORTH);

        textArea = new JTextArea();
        add(textArea, BorderLayout.CENTER);

        JButton button = new JButton("ファイル選択...");
        button.addActionListener(this);
        add(button, BorderLayout.SOUTH);

        timer = new Timer(1000, e -> showFile());
    }

    @Override public void actionPerformed(ActionEvent e) {
        textField.setText("");
        textArea.setText("");
        timer.stop();

        JFileChooser fileChooser = new JFileChooser();
        fileChooser.addChoosableFileFilter(new FileNameExtensionFilter("テキストファイル", "txt"));
        fileChooser.setAcceptAllFileFilterUsed(false);

        if (fileChooser.showOpenDialog(this) != JFileChooser.APPROVE_OPTION) return;

        selectedFile = fileChooser.getSelectedFile();
        if (!checkBeforeReadfile(selectedFile)) return;

        textField.setText(selectedFile.getName());
        showFile();
        timer.start();
    }

    private void showFile() {
        try {
            try (Stream<String> lines = Files.lines(selectedFile.toPath())) {
                textArea.setText(lines.collect(Collectors.joining("\n")));
            }
        } catch (IOException e) {
            textField.setText("");
            textArea.setText("");
            timer.stop();
//            e.printStackTrace();
        }
    }

    private boolean checkBeforeReadfile(File file) {
        return file.exists() && file.isFile() && file.canRead();
    }
}

Comments

[man_]

TN8001さん回答ありがとうございます。
ご教示いただいたコードを参考にコーディングしていきます。
ありがとうございます。

Answer 2

Swing はシングルスレッドです. ご提示のコードでは, actionPerformed が終わるまで, 再表示を含む Swing の全処理は停止しています.
読み込み及び文字列の設定を別スレッドで行うようにしてください.

Comments

[man_]

jimbeさん回答ありがとうございます。
別スレッドで実装してみましたが、TextAreaに表示されませんでした。
修正点等ご教示お願いします。

[jimbe]

ta に TextArea が設定されている個所が見当たりませんが, ちゃんと設定されているのでしょうか.

[jimbe]

いや, もしやこの actionPerformed が含まれているのが LogPanel2 ということでしょうか.
(こういうことがあるので出来るだけ全体をご提示頂きたいのですが...)
だとすれば
Thread thread = new Thread(new LogPanel2());
で動くスレッドはこのオブジェクトではありませんので ta 等は設定されていないはずです.
Thread thread = new Thread(this);
としてください.

[man_]

jimbeさん回答ありがとうございます。
コード全体を表示しました。

回答の通りThread thread = new Thread (this);とし、taの設定をグローバル変数部分で行ったところ表示されるようになりました。ありがとうございます。