.txtからの文字列の取得

Question

###質問内容
Arraylistを用いて，他ディレクトリにある.txtの内容を一行ずつ格納しようとしているのですが，現状だと何も格納できていません．何が原因でこのようになっているのかを教えていただきたいです．

###.txtから文を取得する関数

private static ArrayList<String> Qstring = new ArrayList<>();

public void loadQstrings(String QuestionFilePath) throws FileNotFoundException {
        System.out.println(QuestionFilePath);
        Responser.Qstring.add("");
        Scanner s = new Scanner(new File(QuestionFilePath));
        Responser.Qstring.clear();
        while (s.hasNext()){
            Responser.Qstring.add(s.nextLine());
        }
        s.close();
    }

###関数の呼び出し

responser.loadQstrings("data/questions/hard.txt");

###ディレクトリ

###格納できていないことの確認
次の関数を実行するときに中身を見ました．

public void sendGameData() throws IOException {
        try {
            ServerSocket s = new ServerSocket(8080); // make socket
            Socket socket = s.accept();  //wait setting demand of connection

            try {
                ObjectOutputStream objectOutput = new ObjectOutputStream(socket.getOutputStream());
                //send question
                objectOutput.writeObject(Responser.Qstring);
                
                System.out.println(Responser.Qstring.size());
                for(int i = 0; i < Responser.Qstring.size(); i++){
                    System.out.println(Responser.Qstring.get(i));
                }

                //send timelist
                objectOutput.writeObject(Responser.timeList);      
            } catch (IOException e) {
                e.printStackTrace();
            } 
            s.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
        
    }

結果

data/questions/hard.txt
0

###hard.txt

まおうがくいんのふてきごうしゃ
とあるまじゅつのいんでっくす
いたいのはいやなのでぼうぎょりょくにきょくふりしたいとおもいます
せいしゅうぶたやろうはばにーがーるせんぱいのゆめをみない
そーどあーとおんらいんおるたなてぃぶがんげいるおんらいん
にじがさきがくえんすくーるあいどるどうこうかい
まほうかこうこうのれっとうせい
やはりおれのせいしゅんらぶこめはまちがっている
ゆうきゆうなはゆうしゃである
わんだーえっぐぷらいおりてぃ

###追記
パスでの指定が駄目なのかと考え．ディレクトリ構造を次のようにし，以下のようにコードを変更したのですが改善しませんでした．

呼び出し

responser.loadQstrings("hard.txt");

読み込み

    public void loadQstrings(String QuestionFilePath) throws FileNotFoundException {
        System.out.println(QuestionFilePath);
        Responser.Qstring.add("");
        Scanner s = new Scanner(new File(QuestionFilePath));
        Responser.Qstring.clear();
        while (s.hasNext()){
            System.out.println("add");
            Responser.Qstring.add(s.nextLine());
        }
        s.close();
    }

結果

hard.txt
0

###Responser.java

import java.net.*;
import java.util.*;
import java.util.Map.Entry;
import java.io.*;

//loadlyrycslist->loadQstrings
//lyricslist -> Qstring
//lylicsFilePat -> QuestionFilePath

//EASY = 1;
//HARD = 2;

public class Responser{
    private static ArrayList<String> Qstring = new ArrayList<>();  //question
    private static ArrayList<Long> timeList = new ArrayList<>();      //waiting list

    public int loadDifficulty() throws IOException{
        ServerSocket s = new ServerSocket(8080); // make socket
        Socket socket = s.accept();  // wait setting demand of connection
        BufferedReader in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
        s.close();
        return  Integer.parseInt(in.readLine());    //accept difficulty
    }

    //load Qstringsdata from arraylist to file
    public void loadQstrings(String QuestionFilePath) throws FileNotFoundException {
        System.out.println(QuestionFilePath);
        Responser.Qstring.add("");
        Scanner s = new Scanner(new File(QuestionFilePath));
        Responser.Qstring.clear();
        while (s.hasNext()){
            System.out.println("add");
            Responser.Qstring.add(s.nextLine());
        }
        s.close();
    }

    //load timelist from arraylist to file
    public void loadTimeList(String timeFilePath) throws FileNotFoundException {
        Responser.timeList.add(0L);
        Scanner s = new Scanner(new File(timeFilePath));
        Responser.timeList.clear();
        while (s.hasNext()){
            Responser.timeList.add(s.nextLong());
        }
        s.close();
    }

    //send gamedata in response to client requests
    public void sendGameData() throws IOException {
        try {
            ServerSocket s = new ServerSocket(8080); // make socket
            Socket socket = s.accept();  //wait setting demand of connection

            try {
                ObjectOutputStream objectOutput = new ObjectOutputStream(socket.getOutputStream());
                //send question
                objectOutput.writeObject(Responser.Qstring);
                
                System.out.println(Responser.Qstring.size());
                for(int i = 0; i < Responser.Qstring.size(); i++){
                    System.out.println(Responser.Qstring.get(i));
                }

                //send timelist
                objectOutput.writeObject(Responser.timeList);      
            } catch (IOException e) {
                e.printStackTrace();
            } 
            s.close();
        } catch (Exception e) {
            e.printStackTrace();
        }
        
    }

    
    //accept score and register it to ranking
    public void updateRanking() throws IOException {
        ServerSocket s = new ServerSocket(8080); // make socket
        Socket socket = s.accept();  // wait setting demand of connection
        BufferedReader in = new BufferedReader(new InputStreamReader(socket.getInputStream()));
        String p_name = in.readLine();    //accept name
        String p_score = in.readLine();   //accept score
        int p_mode = Integer.parseInt(in.readLine());    //accept mode

        //System.out.println(p_mode);

        String rankingFilePath;
        if(p_mode == 1)    rankingFilePath = "data/ranking/easyranking.txt";
        else               rankingFilePath = "data/ranking/hardranking.txt";
        
        //key=ID, value=score
        Map<Integer,Integer> scores = new HashMap<Integer,Integer>();
        ArrayList<String> playerNameList = new ArrayList<String>();
        
        Scanner s2 = new Scanner(new File(rankingFilePath));
        int count = 1;
        
        int id = -1, score = -1;
        String name = "";
        while (s2.hasNext()){
            if(count % 3 == 1){
                id = Integer.parseInt(s2.next());
            }else if(count % 3 == 2){
                score = Integer.parseInt(s2.next());
            }else{
                name = s2.next();
                scores.put(id,score);
                playerNameList.add(name);
            }
            count++;
        }

        try{
            int nextId = playerNameList.size()+1;
            File file = new File(rankingFilePath);
            FileWriter filewriter = new FileWriter(file, true);
            filewriter.write(nextId + " " + p_score + " " + p_name + "\n");
            filewriter.close();
            scores.put(nextId, Integer.parseInt(p_score));
            playerNameList.add(p_name);
        }catch(IOException e){
            System.out.println(e);
        }

        // make entry list of scorelist
        List<Entry<Integer, Integer>> list_entries = new ArrayList<Entry<Integer, Integer>>(scores.entrySet());
        
        System.out.println("Sort start");
        //sort scorelist
        Collections.sort(list_entries, new Comparator<Entry<Integer, Integer>>() {
            public int compare(Entry<Integer, Integer> obj1, Entry<Integer, Integer> obj2) {
                return obj2.getValue().compareTo(obj1.getValue());
            }
        });


        PrintWriter out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(socket.getOutputStream())), true);  //setting of sending buffer

        //send ranking data to client
        String[] ordinalNumber = {"    1st", "    2nd", "    3rd", "    4th", "    5th"};
        out.println("--- Rank | Player | Score ---");
        for(int i=0; i<5; i++){
            out.println(ordinalNumber[i] + "    "  + playerNameList.get(list_entries.get(i).getKey()-1) + "\t " + list_entries.get(i).getValue());
            //System.out.println(ordinalNumber[i] + ", "  + playerNameList.get(list_entries.get(i).getKey()-1) + ", " + list_entries.get(i).getValue());
        }
        try{
            Thread.sleep(1000);
        }catch (Exception e){
            e.printStackTrace();
        }
        s.close();
        socket.close();
    }

}

Comments

[swordone]

このコードを書いているクラスはResponserという名前のクラスですか？

[akaevaka]

はい，ファイルから取得する関数，中身を確認する関数の両方がResponserクラスに含まれています．

[swordone]

どこから実行しているんですか？mainメソッド？

[akaevaka]

Serverディレクトリ下のServerMain.javaで実行しています
```
import java.io.IOException;

//EASY = 1;
//HARD = 2;

public class ServerMain {
public static void main(String[] args) throws IOException {
Responser responser = new Responser();

int mode = responser.loadDifficulty();

if(mode == 1){
responser.loadQstrings("data/questions/easy.txt");
responser.loadTimeList("data/questions/easytime.txt");
System.out.println("easy");
}
else{
//responser.loadQstrings("data/questions/hard.txt");
responser.loadQstrings("hard.txt");
responser.loadTimeList("data/questions/hardtime.txt");
System.out.println("hard");
}

//send gamedata in response to client requests
responser.sendGameData();

System.out.println("sendGameData");

//accept and regist score, send ranking
responser.updateRanking();

System.out.println("sendRanking");

}
}
```

[swordone]

easyの方は読み込めるのですか？

[akaevaka]

いえ，hard同様読み込めていません

[Luice]

ResponserクラスにはloadQstringsメソッド以外にも
loadDifficulty,loadTimeList,sendGameDataなどのメソッドがある様です。
これらのメソッドも開示してください。

[akaevaka]

直接関係はなさそうと判断して，見やすさのために省いたのですが，追記の方jにResponser.javaの全文を載せておきます．

Answer 1

while (s.hasNext()){
  System.out.println("add");
  Responser.Qstring.add(s.nextLine());
}    

s.hasNext()が常にfalseです。
よってQstringに値が追加されることはありません。
結果Qstringのsizeは0で中身は空となります。

Scanner使ってファイル読み込むとかやったことないのですが、
最近はFilesでファイルを読み込むのが一般的だと思います。
(文字コードはutf8指定なので、違う場合は適宜変更してください)

import java.nio.charset.StandardCharsets;
import java.nio.file.Files;
import java.nio.file.Path;

  public void loadQstrings(String QuestionFilePath) throws IOException {
    System.out.println(QuestionFilePath);
    var path = Path.of(QuestionFilePath);
    var lines = Files.readAllLines(path, StandardCharsets.UTF_8);
    // 置き換えたいなら上書き
    Responser.Qstring = lines;
    // 追記ならaddAll
    //Responser.Qstring.addAll(lines);
  }

既存のコードではQstringをclearしてaddしてますが、置換ならこれで十分です。

Comments

[akaevaka]

なるほど，読み込み方は他にもあったのですね
少し調べて上に出てきたのが今回の方法だったので，それを使用していました．

教えていただいたようにコードを書き替えたのですが，
.\Responser.java:46: エラー: 不適合な型: List<String>をArrayList<String>に変換できません:
Responser.Qstring = lines;
　　　^
というようなエラーが出てしまったのですが，どのように修正すればよいでしょうか
他のっプログラムでもArrayListで記述しているので，ArrayListで処理したいと考えております．

[akaevaka]

解決しました