JAVAのリストボックスについて

Question

### 前提・実現したいこと
JAVAを勉強中です。
社内のサーバーのフォルダにあるファイルをリストボックスで表示して、必要に応じてファイルを開く仕様です。

url1はテストで自分のPCを指定しておりますが、実際は社内のサーバーにアクセスします。

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

フォルダのファイルを表示しようとすると、リストボックスがつまって表示されます。

該当のソースコード

import java.io.File;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;

public class TEST1 extends JPanel implements ListSelectionListener {

static String url1;
String url2;
String[] file_name = new String[200];
String[] folder = new String[200];
String[] name = new String[200];

public TEST1() {

JFrame frame = new JFrame();
frame.setSize(1300, 750);
frame.setLayout(null);

url1 = "C:\Users\表示テスト";

//aフォルダの一覧を取得する
File file4 = new File(url1);
File files5[] = file4.listFiles();
for (int i=0; i<files5.length; i++) {
file_name[i] = files5[i].getName();
File item = files5[i];
if(item.isDirectory()) {
folder[i] = item.getName();
}
if (item.isFile()) {
name[i] = item.getName();
}
}

JList lst0 = new JList(file_name);
lst0.addListSelectionListener(this);
JScrollPane lst1 = new JScrollPane(lst0);
lst1.setBounds(240, 150, 470,400);
frame.getContentPane().add(lst1);

frame.setVisible(true);

}

public void valueChanged(ListSelectionEvent e) {

JList lst1 = (JList)e.getSource();
url2 = url1+lst1.getSelectedValue();

//aフォルダの一覧を抽出する
File file2 = new File(url2);
File files[] = file2.listFiles();
for (int i=0; i<files.length; i++) {
file_name[i] = files[i].getName();
}
}

public static void main(String[] args) {
new TEST1();
}

}

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

JAVA8、Eclipseで開発しております。

どこを修正すれば良いか教えて下さい。
宜しくお願い致します。

Answer 1

コメントで質問を投げたままですが, 原因は分かっているのでコードを示します.
Javaを勉強中, しかも会社でということで, なるべく修正量を少なくしています.

原因は JList に固定長の配列 file_name を設定していることです.

JList は 200 個分の String が入る file_name の全てを表示しようとしますが, 例えば(フォルダの中にファイルが 10 個しか無くて) listFiles() が 10 件しか返さなかったら, 残り 190 個には何も入っていません. それでも表示しようとして'混乱'してしまうのです. (JList の中の話までは範疇外と思いますのでこのような表現をしています.)

ということで, file_name をフォルダ/ファイルの数が分かった時点で作成するように変更しました.

import java.io.File;

import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.event.ListSelectionEvent;
import javax.swing.event.ListSelectionListener;

public class TEST1 extends JPanel implements ListSelectionListener {

	static String url1;
	String url2;
	String[] file_name; /* = new String[200];*/	//ここでは必要な大きさが分からないので宣言のみ
	String[] folder = new String[200];
	String[] name = new String[200];

	public TEST1() {

		JFrame frame = new JFrame();
		frame.setSize(1300, 750);
		frame.setLayout(null);

		url1 = "C:\Users\表示テスト\";
		//url1 = "T:\";

		//aフォルダの一覧を取得する
		File file4 = new File(url1);
		File files5[] = file4.listFiles();
		file_name = new String[files5.length];	//ここで必要な大きさが分かるので実態を作成
		for (int i=0; i<files5.length; i++) {
			file_name[i] = files5[i].getName();
			File item = files5[i];
			if(item.isDirectory()) {
				folder[i] = item.getName();
			}
			if (item.isFile()) {
				name[i] = item.getName();
			}
		}

		JList lst0 = new JList(file_name);
		lst0.addListSelectionListener(this);
		JScrollPane lst1 = new JScrollPane(lst0);
		lst1.setBounds(240, 150, 470,400);
		frame.getContentPane().add(lst1);

		frame.setVisible(true);

	}

	public void valueChanged(ListSelectionEvent e) {

		JList lst1 = (JList)e.getSource();
		url2 = url1+lst1.getSelectedValue();

		//aフォルダの一覧を抽出する
		File file2 = new File(url2);

		if(!file2.isDirectory()) return;		//フォルダでなければ何もしない, という風にしないとファイルをクリックしたときに止まります.

		File files[] = file2.listFiles();
		file_name = new String[files.length];	//クリックされたフォルダに必要な大きさがここで分かるので実態を再作成
		for (int i=0; i<files.length; i++) {
			file_name[i] = files[i].getName();
		}
		lst1.setListData(file_name);			// file_name を再作成したので JList に再設定
	}

	public static void main(String[] args) {
		new TEST1();
	}

}

Comments

[qwxpc243]

jimbeさん
ありがとうございました。
表示出来るようになりました。
非常に助かりました。

Answer 2

「リストボックスがつまって表示」の問題が分かりかねます.
qwxpc243さんのイメージされている表示よりも行間隔が狭いということとは思いますが, ではどれくらいの間隔になるはずと思われているのか...
手書きでも他の方のサンプルのスクリーンショットでもいいので, 「こういう表示(間隔)になる」はずなのに「こうなってしまった」というイメージ画像を追加していて頂くと良いかと思います.

Comments

[qwxpc243]

ご連絡ありがとうございます。
画像を添付致しました。
更に一個下のフォルダにファイルが多数あると行間隔が狭くなってしまいます。
宜しくお願い致します。

[jimbe]

確認ですが, この画面になったフォルダにはファイルやフォルダが幾つあるのでしょうか.