Результат правильный, если брать данные из условий. Говорит, что я неправ и потоки не закрыл

Вопрос по задаче

Решен

Сдался на вашу милость. Результат выдает правильно, при всех костылях.. раньше говорил, что не везде расставляет Ремувд, теперь Аддед.. но еще добавил, что я не закрыл поток второго файлридера.. подскажите, не хочу уходить на путь 2х аррайлистов

Считать с консоли 2 пути к файлам - file1, file2.
Файлы содержат строки. Так как file2 является обновленной версией file1, то часть строк совпадает.
Нужно создать объединенную версию строк из обоих файлов и записать эти строки в список lines.
Правила объединения:

Если строка в обоих файлах совпадает, то в результат она попадает с операцией (приставкой) SAME.
Например, SAME строка1.
Если строка есть в file1, но ее нет в file2, то считаем, что строку удалили и в результат она попадает с операцией (приставкой) REMOVED.
Например, REMOVED строка2.
Если строки нет в file1, но она есть в file2, то считаем, что строку добавили и в результат она попадает с операцией (приставкой) ADDED.
Например, ADDED строка0.

Операции ADDED и REMOVED не могут идти подряд, они всегда разделены SAME.
Пустые строки даны в примере для наглядности и означают, что этой строки нет в определенном файле.
В оригинальном и редактируемом файлах пустых строк нет!

Пример 1:
содержимое оригинального файл (file1):
строка1 строка2 строка3 строка4 строка5 строка1 строка2 строка3 строка5 строка0
содержимое "редактированного" файла (file2):
строка1 строка3 строка5 строка0 строка1 строка3 строка4 строка5
результат объединения:
оригинальный редактированный общий file1: file2: результат:(lines) строка1 строка1 SAME строка1 строка2 REMOVED строка2 строка3 строка3 SAME строка3 строка4 REMOVED строка4 строка5 строка5 SAME строка5 строка0 ADDED строка0 строка1 строка1 SAME строка1 строка2 REMOVED строка2 строка3 строка3 SAME строка3 строка4 ADDED строка4 строка5 строка5 SAME строка5 строка0 REMOVED строка0

Пример 2:
содержимое оригинального файла (file1):
строка1
содержимое "редактированного" файла (file2):
строка1 строка0
результат объединения:
оригинальный редактированный общий file1: file2: результат:(lines) строка1 строка1 SAME строка1 строка0 ADDED строка0

Требования:

Класс Solution должен содержать класс LineItem.
Класс Solution должен содержать enum Type.
Класс Solution должен содержать публичное статическое поле lines типа List<LineItem>, которое сразу проинициализировано.
В методе main(String[] args) программа должна считывать пути к файлам с консоли (используй BufferedReader).
В методе main(String[] args) BufferedReader для считывания данных с консоли должен быть закрыт.
Программа должна считывать содержимое первого и второго файла (используй FileReader).
Потоки чтения из файлов (FileReader) должны быть закрыты.
Список lines должен содержать объединенную версию строк из файлов, где для каждой строки указана одна из операций ADDED, REMOVED, SAME.

line1 line2 line3 line4 line5 line1 line2 line3 line5 line0

Комментарии (3)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

y-grek

Уровень 20

30 сентября 2020, 18:13

вот что значит на свежую голову и не усложнять способ решения (хотя задание я понял далеко не сразу, очень размыто было)

while (!list1.isEmpty() && !list2.isEmpty()) {
    if (list1.get(0).line.equals(list2.get(0).line)) {
        lines.add(list1.get(0));
        list1.get(0).type = Type.SAME;
        list1.remove(0);
        list2.remove(0);
    } else if (list2.size() > 1 && list1.get(0).line.equals(list2.get(1).line)) {
        lines.add(list2.get(0));
        list2.get(0).type = Type.ADDED;
        list2.remove(0);
    } else {
        lines.add(list1.get(0));
        list1.get(0).type = Type.REMOVED;
        list1.remove(0);
    }
}
if (!list1.isEmpty() && list2.isEmpty()) {
    lines.add(list1.get(0));
    list1.get(0).type = Type.REMOVED;
} else if (list1.isEmpty() && !list2.isEmpty()) {
    lines.add(list2.get(0));
    list2.get(0).type = Type.ADDED;
}

JustinianJudge в Mega City One

Master

28 сентября 2020, 22:45

если пишет что не закрыты потоки, а ты уверен что они закрыты и это есть в коде, это значит что твой код бросает исключения, из-за которых не проходят тесты на закрытие потоков, поэтому тебе и будет писать что потоки не закрыты. И мест где твой код бросает исключения хватает, ты инкрементишь индексы и обращаешься к списку без проверок выхода за границы. И кстати, а тип DEFAULT был или ты добавил? Если этого типа не было раньше и ты сам его добавил - удали и верни предыдущий вид. Уже достаточно скоро будешь изучать принципы написания кода, один из которых гласит, код должен максимально добавлять новых строк и минимально менять старые строки, в реальной программе компоненты используются многими другими модулями программы, ты не можешь просто так добавлять для своего удобства там где хочется. Если дается какой-то готовый код и по условиям нет задания его явно поменять или это не следует с логики программы, ты его оставляешь как есть. Но может условия поменяли и такой тип добавили, я не берусь утверждать.

y-grek

Уровень 20

30 сентября 2020, 17:24

спасибо за ответ добавлял Дефолт сам, еще при первых попытках, когда считывал строки из файла в лист ЛайнАйтемов, поскольку тайп ставил нулл.. и потом хотел избежать НПЕ. с АутОфБаундами заметил свои ошибки сразу после поста этой темы.. плюс еще парочку Переделаю с нуля или с 2мя листами ..на свежую голову