read files with readAllLines and usage of reverse method

Question

前提・実現したいこと

<前提>
コンソールから　Text1.txt Text2.txt Text3.txtとして
プログラムを起動すると、まずtext3.txtが出力され、次にtext2.txtが出力されるようにしたい。

メソッド作成
以下を使用できたらなおよい。
List<String> allLines = Files.readAllLines(Paths.get("fileName"));
allLines.forEach(System.out::Println);
<実現させたこと>
ファイルを逆順に出力

<わからないこと>
メソッドを作成して、
ラムダだが、メソッド参照を使用して逆順ファイルを出力

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

for(String l : lists) {//ここでおこられる
System.out.println(get(l));
にすると以下のようにerrorが発生します。
エラーメッセージ
型の不一致: 要素タイプ List<String> から String には変換できません

該当のソースコード

import java.io.IOException;
import java.nio.file.Files;
import java.nio.file.Paths;
import java.util.Collections;
import java.util.LinkedList;
import java.util.List;



public class Demo8 {


	public static void main(String[] args) {
		try {
			List<String> list = Files.readAllLines(Paths.get(args[0]));
			List<String> list1 = Files.readAllLines(Paths.get(args[1]));
			List<String> list2 =Files.readAllLines(Paths.get(args[2]));

			LinkedList<List <String>> lists = new LinkedList<List <String>>();
			lists.add(list);
			lists.add(list1);
			lists.add(list2);
			Collections.reverse(lists);//要素を逆順にする

			for (String l : lists) {
				System.out.println(get(l);
				//for (String l : lists.get(0)) {
				System.out.println(l);
			}
		

		} catch (IOException e) {

			e.printStackTrace();
		}

	}
}

試したこと

まず、逆順にソートさせてText3のファイルの中身が全行読み込まれて出力出来るかどうか。

public static void main(String[] args) {
		try {
			List<String> list = Files.readAllLines(Paths.get(args[0]));
			List<String> list1 = Files.readAllLines(Paths.get(args[1]));
			List<String> list2 =Files.readAllLines(Paths.get(args[2]));

			LinkedList<List <String>> lists = new LinkedList<List <String>>();
			lists.add(list);
			lists.add(list1);
			lists.add(list2);
			Collections.reverse(lists);//要素を逆順にする
            for (String l : lists.get(0)) {
				System.out.println(l);
			}

以下のサイトを参照して学習してみた
ラムダだが、メソッド参照
Class Files
Rreverse method
###追記
降順で出力できたけど、結果が"[]"になった状態で出力されてしまった。
どうしたら解決しますでしょうか？

			LinkedList<List <String>> lists = new LinkedList<List <String>>();
			lists.add(list);
			lists.add(list1);
			lists.add(list2);
			Collections.reverse(lists);//要素を逆順にする
			for (int i = lists.size() - 1; i >= 0; i--) {
				System.out.println(lists.get(i));
			}
		} catch (IOException e) {

			e.printStackTrace();
		}

	}
}

###The result is as bellow

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

Java version 8.0
使用しているeditor:Eclipse

Comments

[dodox86]

> コンソールから　Text1.txt Text2.txt Text3.txtとして

ファイル名でのソートでしょうか。コンソールから「Text3.txt Text1.txt Text2.txt」と指定されたらどう動きますか。
ファイル名での逆順ソートなら「Text3.txt Text2.txt Text1.txt」の順ですが、
コンソールでのコマンドライン引数の並びの逆順ソートなら、「Text2.txt Tex1.txt Text3.txt」の順になります。

[MilagrosS]

コマンドライン引数の並びでソートかけているから問題なんですね
つまりファイル名でソートかければいいのですね。

[dodox86]

いえ、そうではなくMilagrosSさんがどちらの方法をしたいのか、ということです。

[dodox86]

ファイル名でソートする方が簡単ですが、質問のエラーはそれとは別の問題です。

[MilagrosS]

質問の意図を理解出来てなくて申し訳ございません。

ファイル名でソートしたいです。

Answer 1

「コマンドライン引数の並び順に関係なく、ファイル名を降順でソートし、その順番でファイルの内容を出力したい」と言うのがご要望だとします。

降順で出力できたけど、結果が"[]"になった状態で出力されてしまった。

本当に、降順で出力できていますか？ Collections.reverse(lists)でせっかく降順にしたのに、for (int i = lists.size() - 1; i >= 0; i--) {のループで
昇順（後ろから）で要素を取り出しています。更に、結果が[]で出力されるのは、LinkedListのtoString()メソッドが暗黙的に使われているためです。

Collections.reverse(lists);//要素を降順にする
for (int i = lists.size() - 1; i >= 0; i--) {
    System.out.println(lists.get(i));
}

質問者さんのコードの延長で修正するならば、以下のようになります。

import java.util.*;
import java.nio.file.*;
import java.io.*;

public class t1 {
    public static void main(String[] args) {
        try {
            List<String> list = Files.readAllLines(Paths.get(args[0]));
            List<String> list1 = Files.readAllLines(Paths.get(args[1]));
            List<String> list2 =Files.readAllLines(Paths.get(args[2]));

            LinkedList<List <String>> lists = new LinkedList<List <String>>();
            lists.add(list);
            lists.add(list1);
            lists.add(list2);
            Collections.reverse(lists);//要素を降順にする

//!NG せっかく降順にしたリストから、昇順に取り出している
//          for (int i = lists.size() - 1; i >= 0; i--) {
            for (int i = 0; i < lists.size(); i++) {
                // System.out.println(lists.get(i));
                List<String> nlist = lists.get(i);
                for (int j = 0; j < nlist.size(); j++) {
                    System.out.println(nlist.get(j));
                }
            }
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Windows 10 のWSL/Ubuntu上で動かします。text1.txt～text3.txtのファイルの内容がそれぞれ以下のようなものだとして、

$ cat text1.txt
text1.txt - LINE1
text1.txt - LINE2
text1.txt - LINE3
text1.txt - LINE4

$ cat text2.txt
text2.txt - LINE1
text2.txt - LINE2
text2.txt - LINE3
text2.txt - LINE4

$ cat text3.txt
text3.txt - LINE1
text3.txt - LINE2
text3.txt - LINE3
text3.txt - LINE4

それで実行すると、以下のように正しく出力されます。

$ java t1 text1.txt text2.txt text3.txt
text3.txt - LINE1
text3.txt - LINE2
text3.txt - LINE3
text3.txt - LINE4
text2.txt - LINE1
text2.txt - LINE2
text2.txt - LINE3
text2.txt - LINE4
text1.txt - LINE1
text1.txt - LINE2
text1.txt - LINE3
text1.txt - LINE4

上記のコードで動作はしますが、本来LinkedListである必要もないし、3つのファイルを最初から全部読み出しておく必要も無いと思います。それで書き直してみたのが以下のコードです。同じ結果になりますので試してみてください。コマンドライン引数で指定するファイルも3つ固定ではなく、指定した分だけ処理できます。

import java.util.*;
import java.nio.file.*;
import java.io.*;

public class t2 {
    public static void main(String[] args) {
        // コマンドライン引数をリストにして、降順ソート
        List<String> argList = Arrays.asList(args);
        Collections.reverse(argList);

        // 降順にしたファイル名をひとつづつ処理
        for (String fileName : argList) {
            try {
                // ファイルから全行を読み出し
                List<String> lineList = Files.readAllLines(Paths.get(fileName));

                // 1行づつ出力
                for (int i = 0; i < lineList.size(); i++) {
                    System.out.println(lineList.get(i));
                }
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
    }
}

Comments

[MilagrosS]

すみません。確認したところ、ご回答者様のおっしゃる通り、昇順（後ろから）で要素を取り出していました。
また、「結果が[]で出力されるのは、LinkedListのtoString()メソッドが暗黙的に使われているため」ということは勉強不足で知らなかったのでとても勉強になりました。

[dodox86]

> 「結果が[]で出力されるのは、LinkedListのtoString()メソッドが暗黙的に使われているため」ということは勉強不足で知らなかったので

はい。JavaのすべてのクラスのルートとなるObjectクラスはtoString()メソッドを持っていて、継承したクラスは必要に応じてオーバーロードしています。
https://docs.oracle.com/javase/jp/8/docs/api/java/lang/Object.html#toString--

※心配になったのでLinkedListについて一応確認しました。以下のサンプルコードをご覧ください。
----
import java.util.*;

public class t1b {
public static void main(String[] args) {
List<String> list = Arrays.asList("item1", "item2", "item3");

LinkedList<List<String>> lists = new LinkedList<List<String>>();
lists.add(list);

System.out.println(lists.get(0));
System.out.println(lists.get(0).toString());
}
}
----
これは、こう出力されます。toString()の出力内容と同じです。
$ java t1b
[item1, item2, item3]
[item1, item2, item3]

[MilagrosS]

ご親切にご回答とサンプルのご教示ありがとうございます。
これからの学習の際に参考にさせて頂きます。