Убедись, что список lines в нужных местах содержит операции REMOVED c нужными строками.

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

Решен

Что с ним не так?

Считать с консоли 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.

package com.javarush.task.task19.task1916; import java.io.*; import java.util.ArrayList; import java.util.List; /* Отслеживаем изменения */ public class Solution { public static List<LineItem> lines = new ArrayList<LineItem>(); public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String file1 = reader.readLine(); String file2 = reader.readLine(); BufferedReader freader1 = new BufferedReader(new FileReader(file1)); BufferedReader freader2 = new BufferedReader(new FileReader(file2)); reader.close(); ArrayList<String> A = new ArrayList<>(); ArrayList<String> B = new ArrayList<>(); while (freader1.ready()){ A.add(freader1.readLine()); } freader1.close(); while (freader2.ready()){ B.add(freader2.readLine()); } freader2.close(); while(true){ if(A.size()==0 || B.size()==0) break; if(A.get(0).equals(B.get(0))){ lines.add((new LineItem(Type.SAME,A.get(0)))); A.remove(0); B.remove(0); if(A.size()==0 || B.size()==0) break; } if(!A.get(0).equals(B.get(0))){ if(A.get(1).equals(B.get(0))){ lines.add(new LineItem(Type.ADDED,A.get(0))); A.remove(0); if(A.size()==0) break; } } if(A.get(0).equals(B.get(1))){ lines.add(new LineItem(Type.REMOVED, B.get(0))); B.remove(0); if(B.size()==0) break; } } for(String a :A){ lines.add(new LineItem(Type.REMOVED,A.get(0))); } for(String b :B){ lines.add(new LineItem(Type.ADDED,B.get(0))); } } public static enum Type { ADDED, //добавлена новая строка REMOVED, //удалена строка SAME //без изменений } public static class LineItem { public Type type; public String line; public LineItem(Type type, String line) { this.type = type; this.line = line; } } }

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

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

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

Дмитрий Каменщиков Junior QA Engineer в Перформанс Лаб

2 ноября 2019, 20:23

решение

У тебя в блоке while(true) перепутаны ADDED и REMOVED, поменяй местами

Валентин

Уровень 35

2 ноября 2019, 18:46

полезный

Глянь этот код с объяснениями - в нем работа с оставшимися строками в файлах организована похуже, но если ты похожим образом организуешь свое решение - ошибку найти будет гораздо проще.

try (
                BufferedReader scanConsole = new BufferedReader(new InputStreamReader(System.in));
                BufferedReader fileBufferReader1 = new BufferedReader(new FileReader(scanConsole.readLine()));
                BufferedReader fileBufferReader2 = new BufferedReader(new FileReader(scanConsole.readLine()))
        ){
            List<String> linesFile1 = fileBufferReader1.lines().collect(Collectors.toList());

            List<String> linesFile2 = fileBufferReader2.lines().collect(Collectors.toList());

            String firstLineFirstFile = getLine(linesFile1, indexFirstFile);
            String firstLineSecondFile = getLine(linesFile2, indexSecondFile);

            while(firstLineFirstFile != null || firstLineSecondFile != null){
                String secondLineFirstFile = getLine(linesFile1, indexFirstFile + 1);
                String secondLineSecondFile = getLine(linesFile2, indexSecondFile + 1);

                //проверка для конца файла - если в одном файле закончатся элементы, а в другом останутся
                if(firstLineFirstFile == null){
                    setAddedLine(firstLineSecondFile);
                } else if(firstLineSecondFile == null){
                    setRemovedLine(firstLineFirstFile);
                }

Валентин

Уровень 35

2 ноября 2019, 18:46

//Проверка для заполненной части файла //по условию задачи у нас встречаются неизмененные строки не реже чем через одну строку //а также в файлах нет пропуска строк //другими словами у нас може быть только три ситуации // Файл1 Файл2 //строка1 строка1 - неизменные строки //строка1 строка2 - строка1 была удалена в файле2 можно представить так //строка1 //строка2 строкаПофиг(но не строка1 по условию задачи) //строка2 строка2 //строка1 строка2 - строка2 была добавлена в файл2 можно предстивить так // строка2 //строкаПофиг строка1 //строка1 строка1 //(но не строка2) // Проверка неизменности строк // х х else if(isSame(firstLineFirstFile, firstLineSecondFile)){ setSameLine(firstLineFirstFile); //Проверка удалена ли строка // 0 х // х } else if(isRemove(secondLineFirstFile, firstLineSecondFile)){ setRemovedLine(firstLineFirstFile); //Проверка добавлена ли строка // х 0 // х } else if (isAdded(firstLineFirstFile, secondLineSecondFile)){ setAddedLine(firstLineSecondFile); } firstLineFirstFile = getLine(linesFile1, indexFirstFile); firstLineSecondFile = getLine(linesFile2, indexSecondFile); } lines.forEach(System.out::println); } catch (Exception e) { e.printStackTrace(); }

Валентин

Уровень 35

2 ноября 2019, 18:40

Логично блок REMOVED включить в тот же блок условий что и ADDED

if(!A.get(0).equals(B.get(0))){
         if(A.get(1).equals(B.get(0))){
             lines.add(new LineItem(Type.ADDED,A.get(0)));
             A.remove(0);
             if(A.size()==0) break;
         }
        if(A.get(0).equals(B.get(1))){
           lines.add(new LineItem(Type.REMOVED, B.get(0)));
           B.remove(0);
           if(B.size()==0) break;
     }
  }

А лучше упрости код. Например, заменить во всех скобках проверки размера массива на continue, для того чтобы за один цикл выполнялось лишь одно условие. и убрать условие if(!A.get(0).equals(B.get(0))){, которое теперь не нужно это облегчит дебаг (Ctrl+F8 Shift+F9).