java.lang.StringIndexOutOfBoundsExceptioの意味が分からない。

Question

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

public class Jfile {
  //javaファイルを保存する
  private File jfile;

  //getStringメソッドに使用
  //条件に合う文を格納するリスト
  public ArrayList<String> list = new ArrayList<>();

  Jfile(File f) {
    this.jfile = f;
  }

  //ファイル名を返す関数
  public String getName() {
    return this.jfile.getName();
  }

  //パスを返す関数
  public String getPath() {
    return this.jfile.getAbsolutePath();
  }

  //ファイルの行数を返す関数
  public int getLineCount() {
    int count = 0; //行数を数えるための変数
    try {
      //ファイルの中身を読み込む
      FileReader fr = new FileReader(this.jfile);
      BufferedReader br = new BufferedReader(fr);
      
      while(br.readLine() != null) {
        count++;
      }

      br.close();
      fr.close();
    } catch(IOException e) {
        System.out.println(e);
    }
    return count;
  }

  //ファイルのクラス、パッケージ、importを返す関数
  public String getString(String search_str) {
    try {
      //ファイルの中身を読み込む
      FileReader fr = new FileReader(this.jfile);
      BufferedReader br = new BufferedReader(fr);

      //条件に合う行をリストに格納
      String line;
      while((line = br.readLine()) != null) {
        Pattern p = Pattern.compile(search_str);
        Matcher m = p.matcher(line);

        if(m.find()) {
          list.add(line);
        }
      }

      br.close();
      fr.close();
    } catch(IOException ex) {
      ex.printStackTrace();
    }

    //リストの中身を一つの文にする。
    String rt_line = "";

    while(list.size() != 0) {
      rt_line += list.get(0) + " 、";
      list.remove(0);
    }

    return rt_line;
  }
}

import java.io.*;
import java.util.*;
public class Main {
  private static final String FILENAME = "file2.csv";

  public static void main(String[] args) {

    //Javaファイルを保存するクラス
    ArrayList<Jfile> javalist = new ArrayList<Jfile>();
    //中を探索するためのディレクトリを保存するリスト
    ArrayList<File> ser_dir = new ArrayList<File>();

    //最初のディレクトリをinputに代入
    Scanner s = new Scanner(System.in);
    System.out.print("ディレクトリ名 → ");
    String input = s.nextLine();

    s.close();

    File dir = new File(input);
    ser_dir.add(dir);

    while(ser_dir.size() != 0) {
      File[] dirFilelist = ser_dir.get(0).listFiles();
      for(int i=0; i<dirFilelist.length; i++) {
        String extension = "";
        String fileName = dirFilelist[i].getName();
        //ファイルの拡張子を取得
        extension = fileName.substring(fileName.lastIndexOf("."));

        //ファイルがディレクトリなら中身を探索
        if(dirFilelist[i].isDirectory()) {
          ser_dir.add(new File(dirFilelist[i].getPath()));

        } else if (extension == ".java") { //ファイル拡張子が.javaならjavalistに追加
        javalist.add(new Jfile(dirFilelist[i]));
        }
      }  
      ser_dir.remove(0);
    }
  

    //csvファイルに書き込む
    try(PrintWriter w = new PrintWriter(new BufferedWriter(new FileWriter(FILENAME)))){
      for(int i=0 ; i < javalist.size() ; i++){
          w.println(javalist.get(i).getPath() + "," +
                       javalist.get(i).getName() + "," + 
                       javalist.get(i).getLineCount() + "," + 
                       javalist.get(i).getString("class") + "," +
                       javalist.get(i).getString("package") + "," +
                       javalist.get(i).getString("import")
                       );
      }
    }catch(IOException e){
          System.out.println(e);
    }
  }
}

実行結果と表示されるエラーメッセージ

ディレクトリ名 → ディレクトリ1
Exception in thread "main" java.lang.StringIndexOutOfBoundsException: begin -1, end 7, length 7
at java.base/java.lang.String.checkBoundsBeginEnd(String.java:3734)
at java.base/java.lang.String.substring(String.java:1903)
at java.base/java.lang.String.substring(String.java:1876)
at Main.main(Main.java:29)

試したこと
こちらのエラーについて調べました。配列のインデックスに不適切な値が入れられていると表示されるらしいです。今回はなぜ表示されているのか自分では解決できなかったので質問させていただきます。

Answer 1

//ファイルの拡張子を取得
extension = fileName.substring(fileName.lastIndexOf("."));

ここで fileName.substring() に -1 を渡していて、例外が起きています。-1 を渡さないようにしなければなりません。fileName に . が含まれていない場合は何をしたいのかを考えましょう。

Comments

[K.M.PEANUTS]

理解できました！
ありがとうございます！！