Javaでmainメソッドからreadメソッドを呼び出してArrayにcsvの内容を入れて返したい

Question

前提・実現したいこと

JavaでBufferedReaderを使ってcsvファイルを呼び出し、
それをArrayList(二次元配列)に格納したものを返したreadメソッドを作成したい。

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

エラーメッセージ
fileReader(String, List<String[]>) は引数 (String) に適用できません
型の不一致: List<String[]> から String[] には変換できません

該当のソースコード

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;


public class Main {
	public static void main(String[] args) throws Exception {
		List<String[]> list = new ArrayList<>();
		read("studentData.txt");
}

public static String[] read(String filename, List<String[]> list) {
	try {
		File csv = new File(filename);
		BufferedReader br = new BufferedReader(new FileReader(csv));
		String line;

		while ((line = br.readLine()) != null) {
			list.add(line.split(", ", 0));
		}
	} catch (Exception e) {
		System.out.println(e);
	} finally {
		return list;
	}
}

//実際はもう少しありますが見やすくするため少なくした
//番号、名前、性別、国、数、社、理、英
1, Aiba, 男, 50, 40, 30, 20, 10
2, Inoue, 女, 90, 80, 70, 60, 50
3, Ueda,  男, 10, 20, 30, 40, 50

試したこと

前に質問したのを進めたのですが、
それからずっとここで苦戦してます・・・。
どうしたらいいのでしょうか？

また、

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

Java8
Eclipse IDE 4.10.0

Answer 1

fileReader(String, List<String[]>) は引数 (String) に適用できません

mainメソッド内で

read("studentData.txt");

とString型1つを渡して呼び出してますが、
実際のreadメソッドは

String[] read(String filename, List<String[]> list)

と書いてあるように、String型とList<String[]>型の2つを引数として取ります。

要するに引数の数があってないということです。

型の不一致: List<String[]> から String[] には変換できません

readメソッドの定義を見ると

String[] read(String filename, List<String[]> list)

String[]型を返すように定義されています。
しかし、実際にreturnを見ると

return list;

変数listを返していて、この変数は引数でList<String[]>と定義されています。
つまり、String[]が返されるはずなのにList<String[]>が来てておかしいぞということです。

Comments

[Blue-sea]

回答ありがとうございます。
public static xxxx のxxxxのところはList<String>とやればいいのですね。
できないかと思い込んでいました。

Answer 2

＞ fileReader(String, List<String[]>) は引数 (String) に適用できません

read()メソッドの呼び出しを

read("studentData.txt", list);

とするか、read()メソッドの定義を

public static String[] read(String filename) {
   //returnは今は省略
}

としなければいけません。他でも回答が出ていますが引数の数が一致しません。

＞ Arrayにcsvの内容を入れて返したい
＞ 型の不一致: List<String[]> から String[] には変換できません

という事からArrayList<String[]>で返したいものと推測しますので以下の様な書き方になります。

public static ArrayList<String[]> read(String filename, ArrayList<String[]> list) {
   return list;
}

String[]で返すならreturnにはString[]型の変数、List<String[]>ならreturnにList<String[]>が必要です。

public static xxxx read() {
  xxxx modorichi;

  return modorichi;
}

また、List<String[]> list = new ArrayList<>();のArrayList<この中>()が推測できないので
エラーとなります。String[]を入れましょう。

以下は、finally内でなくfinally外に書き必ず実行するように書いた方が良いでしょう。

finally {
  return list;
}

また、BufferedReaderは使い終えたら閉じる必要があります。

BufferedReader br = new BufferedReader(new FileReader(csv));
br.close(); //閉じる

さらに、テキストの３行目のUeda, 男の間のスペースは２つ入っています。

※色々あって訳が分からないかもしれませんので作ってみました。
ある程度は元のソースをもとにしています。あしからず。

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.util.ArrayList;

public class Main {
	private static ArrayList<String[]> list = new ArrayList<String[]>();

	public static void main(String[] args) throws Exception{
               list = read("C:/TestFolder/studentData.csv");
               /***ここまででlistに目的の物が入っていると思いますのでご自由に使って下さい***/
               listout();
	}

	public static ArrayList<String[]> read(String filename) {
		File csv = new File(filename);
                String line;
        
		try { 
			BufferedReader br = new BufferedReader(new FileReader(csv));
	                while ((line = br.readLine()) != null) {
                           list.add(line.split(", ", 0));
	                }
	                br.close();
	        } catch (Exception e) {
	                System.out.println(e);
	        }
	        return list;
	}
	
        //ArrayList<String[]>の読み込み。以下のメソッドは確認用なので気にしないで。
	public static void listout() {
		String[] onegyo;
                String ws = ""; 
        
                for (int a = 0; a < list.size(); a++) {
        	    onegyo = list.get(a);
        	    for (int i = 0; i< onegyo.length; i++) {
        	        if (i == 0) {
        			ws = onegyo[i];
        		}
        		else {
        			ws = ws + ", " + onegyo[i] ;
        		}
        	}
        	System.out.println(ws);
        	ws = "";
        }
        //System.out.println(System.getProperty("file.encoding"));
	}
}

※確認用にコンソールで表示する時日本語の文字化けを起こすか分からなのでSystem.getProperty("file.encoding");で確認しcsvファイルの保存形式に注意してください。たぶん、EclipseのデフォルトならUTF-8だと思います。

Comments

[Blue-sea]

丁寧な回答をありがとうございます！
しかし、無事にエラーが出なくなったのですが、
listoutメソッドでprint実行結果、
男、女のところが文字化けします(男→・j : 女→・・になる)
保存形式を見るとUTF-8なのですが・・・。
どうしたらいいのでしょうか？教えていただけると嬉しいです

[shinami]

＞ System.out.println(System.getProperty("file.encoding"));
で確認するとUTF-8とコンソールに表示されるという事でしょうか？
テキストファイルそのものの保存形式がUTF-8で保存しているかです。
メモ帳だと名前を付けて保存する時「文字コード」ANSIとかでていませんか？
そこをUTF-8にして保存します。

[Blue-sea]

メモ帳の文字コードを確認したところANSIでしたので
UTF-8に直したら無事に表示されました！ありがとうございます！