Работает со всеми проверками. Но валидацию не проходит

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

Решен

public static List<LineItem> lines = new ArrayList<LineItem>();
    public static ArrayList<String> n1 = new ArrayList<String>();
    public static ArrayList<String> n2 = new ArrayList<String>();
    public static void main(String[] args)throws Exception {
        BufferedReader rd = new BufferedReader(new InputStreamReader(System.in));
        String fileName1 = rd.readLine();
         String fileName2 = rd.readLine();
         rd.close();

             try{
             FileReader fr1 = new FileReader(fileName1);
             BufferedReader fileReader = new BufferedReader(fr1);
             FileReader fr2 = new FileReader(fileName2);
             BufferedReader fileReader2 = new BufferedReader(fr2);
             while(fileReader.ready())
             {
                 String line = fileReader.readLine();

                 n1.add(line);

             }
            fileReader.close();

             while(fileReader2.ready())
             {
                  String line = fileReader2.readLine();
                  n2.add(line);

             }
             fileReader2.close();
             Solution.Check();


             }
             catch(Exception e)
             {

             }

    }
    public static void Check() // метод который заполняет Массив и уравниеват массивы по длине
    {
         while(true)
        {
            if(n1.size()<=n2.size())
            {
            n1.add("");
            }
            else{
                break;
            }
        }
        while(true)
        {
            if(n2.size()<=n1.size())
            {
            n2.add("");
            }
            else{
                break;
            }
        }

        for(int i=0; i<n1.size();i++)
        {
            if(n1.get(i).equals("")&&!n2.get(i).equals(""))
            {
                lines.add(new LineItem(Type.ADDED,n2.get(i)));
            }
           else if(!n1.get(i).equals("")&&n2.get(i).equals(""))
            {
                lines.add(new LineItem(Type.REMOVED,n1.get(i)));
            }

            else if(!n2.get(i).equals("")&&!n1.get(i).equals(i))
            {
                lines.add(new LineItem(Type.SAME,n1.get(i)));
            }
        }

    }


    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;
        }
    }
}

Считать с консоли 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.util.*; import java.io.*; /* Отслеживаем изменения */ public class Solution { public static List<LineItem> lines = new ArrayList<LineItem>(); public static ArrayList<String> n1 = new ArrayList<String>(); public static ArrayList<String> n2 = new ArrayList<String>(); public static void main(String[] args)throws Exception { BufferedReader rd = new BufferedReader(new InputStreamReader(System.in)); String fileName1 = rd.readLine(); String fileName2 = rd.readLine(); rd.close(); try{ FileReader fr1 = new FileReader(fileName1); BufferedReader fileReader = new BufferedReader(fr1); FileReader fr2 = new FileReader(fileName2); BufferedReader fileReader2 = new BufferedReader(fr2); while(fileReader.ready()) { String line = fileReader.readLine(); n1.add(line); } fileReader.close(); while(fileReader2.ready()) { String line = fileReader2.readLine(); n2.add(line); } fileReader2.close(); Solution.Check(); } catch(Exception e) { } } public static void Check() { while(true) { if(n1.size()<=n2.size()) { n1.add(""); } else{ break; } } while(true) { if(n2.size()<=n1.size()) { n2.add(""); } else{ break; } } for(int i=0; i<n1.size();i++) { if(n1.get(i).equals("")&&!n2.get(i).equals("")) { lines.add(new LineItem(Type.ADDED,n2.get(i))); } else if(!n1.get(i).equals("")&&n2.get(i).equals("")) { lines.add(new LineItem(Type.REMOVED,n1.get(i))); } else if(!n2.get(i).equals("")&&!n1.get(i).equals(i)) { lines.add(new LineItem(Type.SAME,n1.get(i))); } } } 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)

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

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

Pavlo Plynko Java-разработчик в CodeGym

Expert

23 октября 2017, 10:07

решение

Первый файл:

111
222

Второй файл:

111
333
222

Результат твоего кода:

LineItem{type=SAME, line='111'}
LineItem{type=SAME, line='222'}
LineItem{type=ADDED, line='222'}

Сергеев Виктор

Уровень 40

Master

20 октября 2017, 15:40

немного по коду: Зачем :

while(true)
        {
            if(n1.size()<=n2.size())
            {
            n1.add("");
            }
            else{
                break;
            }
        }

если так прочитать и написать проще:

while(n1.size()<=n2.size()) {
            n1.add("");
           }

дальше по логике разбора строк:

else if(!n2.get(i).equals("")&&!n1.get(i).equals(i))

зачем вы пытаетесь сравнить содержимое строки с её номером?

Артем Грушевець

Уровень 25

22 октября 2017, 20:48

else if(!n2.get(i).equals("")&&!n1.get(i).equals(""))

Исправил, валидацию не проходит

Сергеев Виктор

Уровень 40

Master

22 октября 2017, 21:00

результат тестовых данных из задания приложите