JavaRush/Помощь по Java задачам/Накидайте вариантов для теста

29 уровень

19.03.2020
582просмотров
17комментариев

Накидайте вариантов для теста

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

Решен

Почему не стал использовать для чтения из файла BufferedReader? Да потому что в требованиях написано использовать FileReader, больше просто так я валидатору не доверяю :) Тестировал на вариантах из условия, ну еще и на таком: Файл 1: Файл 2: строка1 строка1 строка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.

package com.javarush.task.task19.task1916; import java.io.*; import java.util.ArrayList; import java.util.List; /* Отслеживаем изменения C:\Users\kerel\Desktop\JAVA\fortesting\Test1.txt */ 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)); FileReader fileReader1 = new FileReader(reader.readLine()); FileReader fileReader2 = new FileReader(reader.readLine()); reader.close(); ArrayList<String> fromfile1 = new ArrayList<>(); ArrayList<String> fromfile2 = new ArrayList<>(); /*BufferedReader readerFile = new BufferedReader(fileReader); BufferedWriter writerFile = new BufferedWriter(fileWriter);*/ String line1 = ""; while (fileReader1.ready()) { char r; while ((fileReader1.ready()) && ((r = (char) fileReader1.read()) != '\r' || (r = (char) fileReader1.read()) != '\n')) { line1 += r; } if (!line1.equals("")) fromfile1.add(line1); line1 = ""; } fileReader1.close(); String line2 = ""; while (fileReader2.ready()) { char r; while ((fileReader2.ready()) && ((r = (char) fileReader2.read()) != '\r' || (r = (char) fileReader2.read()) != '\n')) { line2 += r; } if (!line2.equals("")) fromfile2.add(line2); line2 = ""; } fileReader2.close(); if (fromfile1.size() > fromfile2.size()) { while (fromfile2.size() != fromfile1.size()) fromfile2.add("Z"); } else if (fromfile1.size() < fromfile2.size()) { while (fromfile1.size() != fromfile2.size()) fromfile1.add("Z"); } for (int i = 0; i < fromfile1.size(); i++) { if (fromfile1.get(i).equals(fromfile2.get(i))) lines.add(new LineItem(Type.SAME, fromfile1.get(i))); else if (fromfile1.get(i).equals("Z")) { lines.add(new LineItem(Type.ADDED, fromfile2.get(i))); } else if (fromfile2.get(i).equals("Z")) { lines.add(new LineItem(Type.REMOVED, fromfile1.get(i))); } else if (fromfile1.get(i).equals(fromfile2.get(i+1))) { lines.add(new LineItem(Type.ADDED, fromfile2.get(i))); fromfile1.add(i,"Z"); } else if (fromfile2.get(i).equals(fromfile1.get(i+1))) { lines.add(new LineItem(Type.REMOVED, fromfile1.get(i))); fromfile2.add(i, "Z"); } } } 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; } } }

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

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

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

JustinianJudge в Mega City One

Master

19 марта 2020, 11:22

Да потому что в требованиях написано использовать FileReader,
больше просто так я валидатору не доверяю :)

Как говорится, сам себе злобный буратино, но наша жизнь состоит из цепочки решений и принятия ответственности за них, так что твоя зона ответственности. По поводу вопроса, старайся прошерстить весь раздел Помощь по этой задаче, там десятки вариантов тесткейсов в комментариях. У меня твой код не отработал даже на примере с условий: Первый файл:

Второй файл:

а что у тебя выдает при таких файлах?

Кирилл

Уровень 29

19 марта 2020, 11:45

SAME 1 REMOVED 2 SAME 3 REMOVED 4 SAME 5

Кирилл

Уровень 29

19 марта 2020, 11:45

0 пропал, ага, вижу

JustinianJudge в Mega City One

Master

19 марта 2020, 12:24

у меня экспшен вообще вылез, если ничего еще не делал и не менял кода, сбрось скриншот файлов и твоего кода Если что-то нашел, или поменял код, тогда просто полностью новую версию кода сбрось Но если там нашел за что зацепиться - кейс с 0 пропавшим, то реши его, и тогда проверяй как я говорил - на кейсах с комментов других вопросов в разделе Помощь по этой задаче. Как оттестируешь на всех кейсах и везде будет зеленое, но код проходить не будет. Тогда маякуй , сбрасывая попутно последнюю версию кода

Кирилл

Уровень 29

19 марта 2020, 14:41

Ошибка была в том, что если действия были связаны с одним лишь удалением или добавлением, то в итоге один из массивов вылетал за границы цикла, либо вылетал эксепшен. Я эту ситуцацию исправил и теперь работает вообще как часы. И так и наоборот, меняя файлы местами. Но валидатору все равно не понравилось

Кирилл

Уровень 29

19 марта 2020, 14:42

Исправленная часть кода

for (int i = 0; i < fromfile1.size(); i++) {
           if (fromfile1.get(i).equals(fromfile2.get(i))) lines.add(new LineItem(Type.SAME, fromfile1.get(i)));
           else if (fromfile1.get(i).equals("Z")) {
               lines.add(new LineItem(Type.ADDED, fromfile2.get(i)));
               break;
           }
           else if (fromfile2.get(i).equals("Z")) {
               lines.add(new LineItem(Type.REMOVED, fromfile1.get(i)));
               break;
           }
           else if (fromfile1.get(i).equals(fromfile2.get(i+1))) {
               lines.add(new LineItem(Type.ADDED, fromfile2.get(i)));
               fromfile1.add(i,"Z");
               fromfile2.add("Z");
           }
           else if (fromfile2.get(i).equals(fromfile1.get(i+1))) {
               lines.add(new LineItem(Type.REMOVED, fromfile1.get(i)));
               fromfile2.add(i, "Z");
               fromfile1.add("Z");
           }
        }

Кирилл

Уровень 29

19 марта 2020, 14:45

Пока особо новых текст кейсов не нахожу в разделе

JustinianJudge в Mega City One

Master

19 марта 2020, 14:47

1. Всегда сбрасывай полностью код. На СТО не показывают исправленную или поломанную часто авто. На СТО пригоняют весь автомобиль, мастер сам должен решать на что смотреть. Как и врач лечит пациента, а не его отдельные части, которые принесли. Такая процедура. 2. Ты прошерстил ВСЕ вопросы с комментами и перепробовал ВСЕ тест-кейсы найденные там? Советую сделать себе файлик, чтобы при изменении твоего кода ты мог прогнать еще раз по ним. А то один момент исправишь, другой отвалится.

Кирилл

Уровень 29

19 марта 2020, 15:00

Нашел еще ошибку) Тестирование наше все)

Кирилл

Уровень 29

19 марта 2020, 15:10

package com.javarush.task.task19.task1916;

import java.io.*;
import java.util.ArrayList;
import java.util.List;

/*
Отслеживаем изменения
C:\Users\kerel\Desktop\JAVA\fortesting\testpapka\Test1.txt
*/

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));
        FileReader fileReader1 = new FileReader(reader.readLine());
        FileReader fileReader2 = new FileReader(reader.readLine());
        reader.close();
        ArrayList<String> fromfile1 = new ArrayList<>();
        ArrayList<String> fromfile2 = new ArrayList<>();

        /*BufferedReader readerFile = new BufferedReader(fileReader);
        BufferedWriter writerFile = new BufferedWriter(fileWriter);*/

        String line1 = "";
        while (fileReader1.ready()) {
            char r;
            while ((fileReader1.ready()) && ((r = (char) fileReader1.read()) != '\r' || (r = (char) fileReader1.read()) != '\n')) {
                line1 += r;
            }
            if (!line1.equals("")) fromfile1.add(line1);
            line1 = "";
        }
        fileReader1.close();

        String line2 = "";
        while (fileReader2.ready()) {
            char r;
            while ((fileReader2.ready()) && ((r = (char) fileReader2.read()) != '\r' || (r = (char) fileReader2.read()) != '\n')) {
                line2 += r;
            }
            if (!line2.equals("")) fromfile2.add(line2);
            line2 = "";
        }
        fileReader2.close();

Кирилл

Уровень 29

19 марта 2020, 15:10

if (fromfile1.size() > fromfile2.size()) {
            while (fromfile2.size() != fromfile1.size()) fromfile2.add("Z");
        } else if (fromfile1.size() < fromfile2.size()) {
            while (fromfile1.size() != fromfile2.size()) fromfile1.add("Z");
        }

        for (int i = 0; i < fromfile1.size(); i++) {
            if (fromfile1.get(i).equals("Z") && fromfile2.get(i).equals("Z")) break;
           else if (fromfile1.get(i).equals(fromfile2.get(i))) lines.add(new LineItem(Type.SAME, fromfile1.get(i)));
           else if (fromfile1.get(i).equals("Z")) {
               lines.add(new LineItem(Type.ADDED, fromfile2.get(i)));
               break;
           }
           else if (fromfile2.get(i).equals("Z")) {
               lines.add(new LineItem(Type.REMOVED, fromfile1.get(i)));
               break;
           }
           else if (fromfile1.get(i).equals(fromfile2.get(i+1))) {
               lines.add(new LineItem(Type.ADDED, fromfile2.get(i)));
               fromfile1.add(i,"Z");
               fromfile2.add("Z");
           }
           else if (fromfile2.get(i).equals(fromfile1.get(i+1))) {
               lines.add(new LineItem(Type.REMOVED, fromfile1.get(i)));
               fromfile2.add(i, "Z");
               fromfile1.add("Z");
           }
        }

        for (LineItem lineitem: lines) {
            System.out.println(lineitem);
        }

    }


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

        @Override
        public String toString() {
            return type.toString() + " " + line;
        }
    }
}

Кирилл

Уровень 29

19 марта 2020, 15:11

Думаю теперь все, но все равно не принимает

JustinianJudge в Mega City One

Master

19 марта 2020, 15:57

в данном случае, public static class LineItem { и public static class LineItem { это код который дается изначально и не должен меняться, поэтому его можно опустить то что над классом 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));
      FileReader fileReader1 = new FileReader(reader.readLine());
      FileReader fileReader2 = new FileReader(reader.readLine());
      reader.close();
      ArrayList<String> fromfile1 = new ArrayList<>();
      ArrayList<String> fromfile2 = new ArrayList<>();

и так далее по тексту, кроме вышеобозначенных классов. Ну то что "все" ты думал еще когда писал вопрос, вон сколько уже сам нашел. По самому коду немного:

ArrayList<String> fromfile1

коллекции мы объявляем через интерфейсы, чтобы использовать такой принцип ООП как полиморфизм. List<String> fromFile1 и тд Также fromfile - это нарушение Java Naming Convention. в джава camelCase, этоЗначитЧтоСловаРазделяютсяБольшойБуквойИУдобнейТакЧитать правдаудобнейчемвоттакойвоттекстнеправдали поэтому fromFile1 потом строка:

while ((fileReader1.ready()) && ((r = (char) fileReader1.read()) != '\r'
|| (r = (char) fileReader1.read()) != '\n')) {

так циклы не пишут. Сравни команду: -Коля, купи хлеб. -Маша сходи в супермаркет, а потом зайди в банк. И сравни с: -Коля когда ты не будешь не там где ты был через там где ты не будешь вот там тогда зайди туда не туда где был там а там где не был ты не там но там Как тебе? Не пишутся такие условия, вот не пишут и все. Эту задачу с большой натяжкой можно отнести к алгоритмическим поэтому некоторые посла

JustinianJudge в Mega City One

Master

19 марта 2020, 15:57

решение

послабления к клин коду есть, к алгоритмическому и низкоуровневому коду другие требоваия. Но даже с учетом послаблений эта строка превышает все разумные нормы. Я насчитал в этом условии while 22 различных оператора, использование трех различных примитивов, три метода, и это все соединено вместе. Сплетать нитки затягивая сложные узлы вроде и ничего так. Ровно до тех пор пока оно работает или не надо менять. Раз уж ты принципиально не хочешь использова буффередРидер и тебе нравится вместо получения зеленой галки валидатора, тратить время на поиск ошибок (чем сложнее логику программы ты делаешь, тем больше концентрации требует задача и как следствие больше ошибок начинает появляться), то тогда вынеси это хоть в отдельные какие-то методы, перепиши не знаю.

else if (fromfile1.get(i).equals(fromfile2.get(i))) lines.add(new LineItem(Type.SAME, fromfile1.get(i)));

давай здесь считать, 15 операторов, 5 методов, классов, объектов и переменных уж и считать не хочется Запомни, одна строка кода должна содержать РАЗУМНОЕ количество операций. Это субъективная характеристика, но эта строка слишком перегружена. Хотя здесь проще решается, ведь у тебя нарушено еще и другое жесткое правило - это то, что после циклов и условий - while, for, if /else if ВСЕГДА должны стоять фигурные скобки. Есть очень редкие исключения для общепринятых случаев, но пока что до отдельных указаний, пиши всегда. Поставив там фигурную скобку, строка разбивается, уже можно хоть визуально отделить где условие, где что происходит, уже легче. Ты не ответил на вопрос: 2. Ты прошерстил ВСЕ вопросы с комментами и перепробовал ВСЕ тест-кейсы найденные там? Советую сделать себе файлик, чтобы при изменении твоего кода ты мог прогнать еще раз по ним. А то один момент исправишь, другой отвалится. Если ты внес изменения в код, валидатор не принимает значит снова нужно прогнать по всем этим тест кейсам, а там их десятки.

Кирилл

Уровень 29

19 марта 2020, 16:16

В общем, подниму вам немного настроение, но это "Раз уж ты принципиально не хочешь использова буффередРидер и тебе нравится вместо получения зеленой галки валидатора, тратить время на поиск ошибок..." оказалось правдой хД. Заменил велосипед на буффер и все, засчитало Большое вам спасибо, мистер Дредд)

Кирилл

Уровень 29

19 марта 2020, 16:19

Ну а с вопросом вы и сами все понимаете)

JustinianJudge в Mega City One

Master

19 марта 2020, 16:22

полезный

На здоровье :) красавчик, все решил сам, и мне от работы не пришлось отвлекаться, и задачу решил и закрыл, отлично ) Ну и запомни выводы, в джава существует вагон и маленькая тележка правил которые направлены на то, чтобы код получался легкочитаемым, понятным, ошибиться в таком коде будет гораздо сложнее. Легче же просто крутить рулем и нажимать педали, чем управлять Боингом или подводной лодкой. Чем более сложная структура, тем больше ресурсов нужно чтобы все делать правильно. Успехов )