DefaultListModelの内容がJListに反映されない

Question

前提・実現したいこと

Javaで住所録のデスクトップアプリを作成しています。
ファイルからデータを読みこんでJListに反映させたいのですが、うまく動作しません。
どこに問題があるのかご教示をお願い致します。

DefaultListModelを使い、読み込んだデータを動的に扱えるようにしたいです。

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

DefaultListModelにデータを読み込むことはできたのですが、JListに表示されません。

該当のソースコード

質問用に簡略化しています。

import java.awt.Dimension;

import java.io.BufferedInputStream;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.ObjectInputStream;

import javax.swing.DefaultListModel;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;

public class MainFrame extends JFrame {
	JPanel panel;
	static DefaultListModel<PersonalData> dataListModel;
	JList<PersonalData> dataList;
	
	MainFrame(){
		this.setSize(600,400);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	
		panel = new JPanel();
		dataListModel = new DefaultListModel<PersonalData>();
		dataList = new JList<PersonalData>(dataListModel);
		JScrollPane sp = new JScrollPane();
	    sp.setPreferredSize(new Dimension(500, 300));
		
		panel.add(sp);
		this.add(panel);
	}
	
	

	public static void main(String[] args) {
		MainFrame mainFrame = new MainFrame();
		mainFrame.setVisible(true);
		fileRead("data.dat");
	}
	
	public static void fileRead(String path) {
		//ファイルの読み込み
		try {
			FileInputStream f = new FileInputStream(path);
			BufferedInputStream b = new BufferedInputStream(f);
			ObjectInputStream in = new ObjectInputStream(b);
			DefaultListModel<PersonalData> list = (DefaultListModel<PersonalData>)in.readObject();
			System.out.println("list:" + list);
			//読み込んだデータを配列にコピーし、dataListModelに追加する
			PersonalData[] array = new PersonalData[list.size()];
			list.copyInto(array);
			for(PersonalData data:array) {
				dataListModel.addElement(data);
			}
			System.out.println("dataListModel:" + dataListModel);
			
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		} catch (IOException e) {
			e.printStackTrace();
		}
		
	}
}

import java.io.Serializable;

public class PersonalData implements Serializable {
	String name,address,tel;
	
	PersonalData(String n,String a,String t){
		this.name = n;
		this.address = a;
		this.tel = t;
	}

	@Override
	public String toString() {
		return this.name + this.address + this.tel;
		
	}
}

Comments

[TN8001]

JScrollPane sp = new JScrollPane();
を
JScrollPane sp = new JScrollPane(dataList);
か
JScrollPane sp = new JScrollPane();
sp.getViewport().setView(dataList);
にすればなにかしらでませんか？

fileReadが不可解ですが本来はいろんな処理があるのかな？

[warawaraby]

ありがとうございます！
本来の方では
JScrollPane sp = new JScrollPane(dataList);
としていましたが、タイミングに問題があったようで、そちらを見直すことで無事に解決いたしました。

fileReadは基本的な処理の流れは同じです。
ファイルを暗号化しているためCipherInputStreamを挟む程度の差ですが、不可解なのはこちらでしょうか？
DefaultListModel<PersonalData> list = (DefaultListModel<PersonalData>)in.readObject();
//読み込んだデータを配列にコピーし、dataListModelに追加する
PersonalData[] array = new PersonalData[list.size()];
list.copyInto(array);
for(PersonalData data:array) {
dataListModel.addElement(data);
}

最初はDefaultListModelをnewせず、JListのインスタンス生成時の引数を空にして
dataListModel = (DefaultListModel<PersonalData>)in.readObject();
dataList.setModel(dataListModel)
とするとエラーが出たのでこのような形にしてしまいました。
質問内容部分の修正に伴い、setModelが使えるようになったのでこちらも修正いたしました。
その他の部分でおかしなところがあればご指摘いただければ幸いです。

[TN8001]

解決されて何よりです＾＾
お手数ですが自己回答で解決済みにしてください。

> fileReadが不可解ですが
簡略化版で本来のコードとは違うのでしょうが、

1. static にする意味を感じない。そのせいでdataListModelまでstaticにせざるを得なくなっている。
2. 初期値で入ることが決まっているなら
dataList = (DefaultListModel<PersonalData>)in.readObject();
で十分では？（ここは改善されたようですね）
3. 何個か読むor後から読むような場合は
for (int i = 0; i < list.getSize(); i++) {
dataListModel.addElement(list.elementAt(i));
}
でいいのでは？
4. Streamのclose

あたりが気になりました。

[warawaraby]

追加でご回答いただき、ありがとうございます！
自己回答の件、承知いたしました。
１に関しては、再度見直してみます。
３は読み込みが最初だけなので２の状態でいこうと思いますが、今後の参考にさせていただきます。
４は単純に忘れておりました・・・ご指摘ありがとうございます。

入門書の読了後、初めてコードを書いているため、ご指摘いただいた内容が非常に参考になりました。
ご丁寧にありがとうございました。重ねてお礼申し上げます。

Answer 1

TN8001様のおかげで解決いたしました。ありがとうございました。