特定の形式のファイルを読み込みそれをそのまま表示するプログラム

Question

import java.util.*;
import java.io.*;

public class Graph{
    int num = 0;
    String filename;
    Node[] nodes;
    Graph(String filename){
	this.filename = filename;
	loadGraph(filename);
    }
    class Edge{
	private int to;
	private double omomi;
	Edge(int to,double omomi){ this.to = to; this.omomi=omomi;}
	int To(){ return to; }
	double Omomi(){return omomi;}
    }

    class Node{
	int id;
	ArrayList<Edge> list = new ArrayList<Edge>();
	ArrayList<Edge> list2 = new ArrayList<Edge>();

	Node(int id){
	    this.id = id;
	}
	void addToList(int nid,double omomi){
	    list.add(new Edge(nid,omomi));
	}
	ArrayList<Edge> getList(){
	    return list;
	}
	ArrayList<Edge> getList2(){
	    return list2;
	}
    }

    void loadGraph(String filename){
	this.nodes = new Node[1000];
	try{
	    FileReader fr = new FileReader(filename);
	    BufferedReader br = new BufferedReader(fr);
	    String line;
	    while((line = br.readLine()) != null){
		String[] node = line.split(":");
		int nOde = Integer.parseInt(node[0]);
		nodes[num] = new Node(nOde);
		String[] edge = node[1].split(",");

		for(int h=0;h<edge.length;h++){
		    String[] aaa=edge[h].split("@");
		    int y = Integer.parseInt(aaa[0]);
		    double z = Double.parseDouble(aaa[1]);
		    nodes[num].addToList(y,z);
		}
		
		num++;
	    }
	}catch(IOException e){
	    return ;
	}
    }

    void printGraph(){
	for(int i = 0;i < num;i++){
	    System.out.print(i+":");
	    for(int j = 0;j < nodes[i].getList().size();j++){
		System.out.print(nodes[i].getList().get(j).To()+"@"+nodes[i].getList2().get(j).Omomi());
		if(j < nodes[i].getList().size() - 1){
		    System.out.print(",");
		}
	    }
	    System.out.println();
	}
    }
}

class GraphTester{
    public static void main(String[] args){
	String filename = args[0];
	Graph graph = new Graph(filename);
	graph.printGraph();
    }
}

このプログラムをつかって
java GraphTester a.txtで（以下a.txtの中身です）

0:1@2.0,3@6.0,4@1.0
1:6@3.0

を

0:1@2.0,3@6.0,4@1.0
1:6@3.0

と元通り表示させたいのですが

C:\Users\user\Desktop>java GraphTester a.txt
0:Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 0, Size
: 0
        at java.util.ArrayList.rangeCheck(Unknown Source)
        at java.util.ArrayList.get(Unknown Source)
        at Graph.printGraph(Graph.java:69)
        at GraphTester.main(Graph.java:83)

が出てきてできません。
どこを直せばできるようになりますか？

Comments

[swordone]

コードが読みづらいです。ブロックごとに字下げしてください。

Answer 1

Nodeクラスにlist2が宣言されていますが、これに何も入れていません。
対して、printGraphからgetList2でListを取得し、そこから要素を取り出そうとしていますが、
何も入っていないので取り出せません。
omomiの情報もnidとともにEdgeクラスの情報として入っているため、list2自体が全くの無意味です。

Node[0]の中身
|    list    |   list2  |
-------------------------
|Edge(1, 2.0)|          |
|Edge(3, 6.0)|          |
|Edge(4, 1.0)|          |

この状態でprintGraph中、getList2からget(j)しようとすると、
何も入っていないlist2から取り出そうとするので例外発生

Comments

[kd3]

list2にはomomiを入れるつもりだったのですが、この状態だと、読み込むだけ読み込んでlist2に入れられてない状態、ということですか？

[swordone]

ご自分のコードをよく読んでください。omomiに相当する数はEgdeクラスのコンストラクタに渡され、Edgeインスタンスのメンバとして格納されます。つまり、nidもomomiもEgdeインスタンスの要素としてともにlistに入れられており、list2は一切使用されていません。
listからインスタンスを取り出せばどちらも取得できることになります。

[kd3]

ということは、list2そのものを考えないことにして、list2自体を消して、nodes[i].getList2().get(j).Omomi()をnodes[i].getList().get(j).Omomi()に変更することで正常に表示できる、ということですか？

[swordone]

見間違いをしました。そのとおりです。

[kd3]

解決しました。ありがとうございました。

[swordone]

私としては、文字列化をEdgeやNodeのtoStringメソッドに任せてしまいたいです。