Почему я получаю эксепшн ArrayIndexOutOfBoundsException?

Question

[v_vladik]

У меня есть такой вот образец кода. В моих планах считывать рядок из файла, в котором хранятся слова и их переводы. Англ слова сохраняются в лист "words", а их переводы - "translation". Я реализовал два геттера, но со вторым возникает проблема ArrayIndexOutOfBoundsException. В чем проблема?

public class Main {
    public static void main(String[] args) throws IOException {
        File file = new File("english.txt");
        Scanner sc = new Scanner(file);
        int size = 0;
        List<String> list = new ArrayList<>();
        List<String> words = new ArrayList<>();
        List<String> translation = new ArrayList<>();
        while (sc.hasNextLine()) {
            list.add(sc.nextLine());
            size++;
        }
        for (int i = 0; i < size; i++) {
            words.add(getFirstLine(list.get(i)));
            translation.add(getSecondLine(list.get(i)));
        }
    }
    public static String getFirstLine(String line) {
        String[] lines = line.split("-");
        return lines[0];
    }
    public static String getSecondLine(String line) {
        String[] lines = line.split("-");
        return lines[1];
    }
}

Answer 1

[Алексей Уколов]

Очевидно, что в файле у вас есть строки, в которых нет "-", и поэтому второго элемента в массиве после split нет.

Comments

[v_vladik]

Этот вариант исключается. Я точно уверен, что в файле разделитель присутствует.

[Алексей Уколов]

А, ну в таком случае — это волшебные силы электричества. Или домовой балуется. Ну или в файле есть пустые строки.

[Dmitry Roo]

v_vladik, добавьте какое-то логирование строки, которую вы обрабатываете. Тогда все станет понятно

[Денис Загаевский]

v_vladik, сколько бы ты ни был уверен, так делать нельзя. Обязательно надо проверять данные перед использованием.

[v_vladik]

Денис Загаевский, Эти данные были введени лично мной пока что там только три строки:
Home - Дом
Pen - Ручка
Milk - Молоко
Для начала я не вводил много строк, чтобы создать робочую версию програму.

[Алексей Уколов]

Вам уже пять раз сказали - сделайте вывод строки в консоль и сами убедитесь, что я прав.
Иного тут просто быть не может - index out of bounds означает, что элемента 1 в массиве нет. Нет его там потому, что в какой-то строке нет дефиса. Это может быть пустая строка в конце файла, например.

Answer 2

[Орхан Гасанлы]

v_vladik,

Этот вариант исключается. Я точно уверен, что в файле разделитель присутствует.

Учтите еще одно, что дефис и тире разные символы.
Например, https://www.compart.com/en/unicode/category/Pd
Что если у вас например, тире, вместо дефиса или еще что...

Не лучше ли вместо того, чтобы плодить кучу методов (получить первую строку, получить вторую, третью, четвертую и т.д. ), создать метод, который принимает строку и номер строки, которую нужно получить? Соответственно, будет один метод.

public static String getFirstLine(String line) {
        String[] lines = line.split("-");
        return lines[0];
    }
    public static String getSecondLine(String line) {
        String[] lines = line.split("-");
        return lines[1];
    }

english.txt

Было бы неплохо увидеть пример содержимого файла.

И еще вполне возможно, что у вас тут проблема:

for (int i = 0; i < size; i++) {
            words.add(getFirstLine(list.get(i)));
            translation.add(getSecondLine(list.get(i)));
        }

А чего гадать-то? Сделайте дебаг и гляньте что именно ему не нравится