JavaRush/Помощь по Java задачам/Помогите подобрать ключ к валидатору

41 уровень

10.08.2019
740просмотров
10комментариев

Помогите подобрать ключ к валидатору

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

Решен

С моей точки зрения - задача выполнена. Есть две обертки - одна для FileWriter другая для FileReader Данные перед чтением и записью привожу в формат, который указан в задании, пишу посимвольно. Непонятка только с аргументами комманд лайна. Или productName должен иметь возможность содержать пробел и тогда его нужно вбивать в аргументах вызова в " " или же не понятно вообще как аргументы командной строки парсить. Слить весь args в строку и брать подстроки по 8, 30 и так далее символов не выйдет, потому как если имя будет, например содержать 15 символов, после него сразу в такой строке будет price и следующие аргументы. Если брать по максимально возможным длинам из объединенного args - быстро вылетишь за границы строки. В общем - решил что подразумевается что productName может быть многострочным. А что подразумевали авторы курса - не известно. Вот аргс

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

CRUD для таблицы внутри файла.
Напиши программу, которая считывает с консоли путь к файлу для операций CRUD и при запуске со следующим набором параметров:
-c productName price quantity
добавляет товар с заданными параметрами с новой строки в конец файла, генерируя id (8 символов) самостоятельно путем инкремента максимального id, найденного в файле.

Значения параметров:
-c - флаг, который означает добавления товара.
productName - название товара, 30 символов.
price - цена, 8 символов.
quantity - количество, 4 символа.

В файле данные хранятся в следующей последовательности (без разделяющих пробелов):
id productName price quantity

Данные дополнены пробелами до их длины.

Для чтения и записи файла нужно использовать FileReader и FileWriter соответственно.

Пример содержимого файла:
19846 Шорты пляжные синие 159.00 12 198478 Шорты пляжные черные с рисунко173.00 17 19847983Куртка для сноубордистов, разм10173.991234

Требования:

Программа должна считать имя файла для операций CRUD с консоли.
В классе Solution не должны быть использованы статические переменные.
При запуске программы без параметров список товаров должен остаться неизменным.
При запуске программы с параметрами "-c productName price quantity" в конец файла должна добавится новая строка с товаром.
Товар должен иметь следующий id, после максимального, найденного в файле.
Форматирование новой строки товара должно четко совпадать с указанным в задании.
Созданные для файлов потоки должны быть закрыты.

19846 Шорты пляжные синие 159.00 12 198478 Шорты пляжные черные с рисунко173.00 17 19847983Куртка для сноубордистов, разм10173.991234 19847984Шорты пляжные зелен 151.0 32

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

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

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

Wladyslaw Java Developer

Master

11 августа 2019, 13:53

Ну ******

if (args.length != 0 && args[0].equals("-c"))

принял А изначальная версия

if(args.length == 0 || !args[0].equals("-c")){
    throw new IllegalArgumentException("Illegal command line args.");
}

Ему не нравилась. Ладно, завалидировало, наконец-то, но мне кажется вот это принуждение все писать не в ООП стиле не играет на руку тем кто по этому курсу учится.

Wladyslaw Java Developer

Master

11 августа 2019, 12:18

Окей - может ему выброшенное исключения не нравится? Делаем так

switch (args[0]) {
    case "-c": {


        if (args.length > 1) {
            BufferedReader fileReader = new BufferedReader(new FileReader(name));
            String line;
            while ((line = fileReader.readLine()) != null) {
                strings.add(line);
            }
            fileReader.close();

            int maxId = strings
                    .stream()
                    .map(s -> s.substring(0, 8).trim())
                    .mapToInt(Integer::parseInt)
                    .max()
                    .getAsInt();
            strings.add(String.format("%-8s%-30s%-8s%-4s", ++maxId, args[1], args[2], args[3]));
            BufferedWriter fileWriter = new BufferedWriter(new FileWriter(name));
            for (String string :
                    strings) {
                fileWriter.write(string);
                fileWriter.newLine();
            }
            fileWriter.close();
        }
    }
}

Больше в main ничего нет, кроме считывания имени файла и создания листа. ГДЕ ТАМ СПИСОК ТО МЕНЯЕТСЯ, ЕСЛИ БЕЗ ПАРАМЕТРОВ ЗАПУСТИТЬ, ЕСЛИ ОН ДАЖЕ ФАЙЛ НЕ БУДЕТ ЧИТАТЬ? Даже если один параметр "-c" будет - а остальных - нет, ничего делать программа не будет Первый раз он валидирует так, что я начинаю гневаться. Было все красиво, классы-обертки, все дела - не нравится. На тебе ту же логику в статическом контексте - нравится. Но не до конца.

Wladyslaw Java Developer

Master

11 августа 2019, 12:10

У валидатора какая-то аллергия на самописные классы, видимо. Вынес все в main(), логику не менял, но валидатору так больше понравилось.

public static void main(String[] args) throws Exception {

        if (args.length == 0 || !args[0].equals("-c")) {
            throw new IllegalArgumentException("Illegal command line args.");
        } else {


            BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
            String name = reader.readLine();
            reader.close();
            //List<String> strings = Files.readAllLines(Paths.get(name));
            List<String> strings = new ArrayList<>();

            BufferedReader fileReader = new BufferedReader(new FileReader(name));
            String line;
            while ((line = fileReader.readLine()) != null) {
                strings.add(line);
            }
            fileReader.close();

            switch (args[0]) {
                case "-c": {
                    int idMax = strings
                            .stream()
                            .map(s -> s.substring(0, 8).trim())
                            .mapToInt(Integer::parseInt)
                            .max()
                            .getAsInt();
                    strings.add(String.format("%-8s%-30s%-8s%-4s", ++idMax, args[1], args[2], args[3]));
                }
            }
            //Files.write(Paths.get(name), strings);
            BufferedWriter fileWriter = new BufferedWriter(new FileWriter(name));
            for (String string :
                    strings) {
                fileWriter.write(string);
                fileWriter.newLine();
            }
            fileWriter.close();





        }
    }

Принимает все - кроме "При запуске программы без параметров список товаров должен остаться неизменным." Ну так вон же - не будет параметров - вылетит исключение, что теперь то не так?

IceBerg

Уровень 41

Master

10 августа 2019, 16:36

Не думаю, что параметры нужно передавать с отступами, в любом случае смысла нету, деление всегда идет по пробелу, а форматирование касается их хранения в файле. Даже наоборот, тот же productName может содержать и больше 30 символов, а в файле их нужно обрезать. По кавычкам всё просто, есть пробелы - ставишь, нету - можешь не ставить.

-с Шорты 1 2
-с "Шорты пляжные синие" 1 2
-с "Шорты пляжные синие, шорты пляжные черные с рисунком" 123456789 123456789

Wladyslaw Java Developer

Master

10 августа 2019, 16:40

Ну, в файл уйдет обрезанный параметр. Или дополненный, по ситуации.

if(productName.length() != 30){
    if(productName.length()<30) {
        int charsquantityToAppend = 30 - productName.length();
        for (int charsCounter = 0; charsCounter < charsquantityToAppend; charsCounter++) {
            productName += " ";
        }
    } else {
        productName = productName.substring(0,29);
    }

Потому не думаю что валидатор на это ругается Ну да, они там и не учитываются. Просто каждый хотя бы один пробел - значит новый параметр, у меня в итоге массив длинной 4 выходит, все как надо. Не знаю зачем я туда пробелов наставил, вроде как копировал откуда-то.

IceBerg

Уровень 41

Master

10 августа 2019, 16:43

Не, не должен каждый пробел создавать новый параметр. Ну в смысле если подряд идут, недопонял тебя наверное)

Wladyslaw Java Developer

Master

10 августа 2019, 16:47

Ну, я имею ввиду

-с "Шорты пляжные синие, шорты пляжные черные с рисунком" 123456789 123456789

4 параметра А так

-с Шорты пляжные синие, шорты пляжные черные с рисунком 123456789 123456789

11 параметров. Вот во втором случае хрена с два я там смогу имя запарсить. Не ясно что считать его границей

IceBerg

Уровень 41

Master

10 августа 2019, 16:50

То что ты задачки делаешь с учётом ООП и максимально универсальными это круто конечно, но тестировать такое желание отпадает)) Параметров по условию всегда 4, если в productName нужны пробелы, значит там будут кавычки, но какая разница, значение аргумента кавычки то не хранит.

Wladyslaw Java Developer

Master

10 августа 2019, 16:56

Примерно такой подход я и выбрал для работы с параметрами. Просто в условии можно было бы дописать. Хотя, это особенности работы терминала винды(не знаю как там на других осях), так что может - само собой подразумевается. Видимо потому у моих вопросов обычно и комментов мало) Найду рано или поздно в чем проблема, конечно, заодно с JUnit попрактикуюсь. Но, если будет желание тоже поискать баги - то я всегда открыт для критики)

Wladyslaw Java Developer

Master

10 августа 2019, 15:45

Индексы в substring'ах во Writer'е уже поправил, если что, но не в них дело