Не проходит валидацию

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

Решен

Все работает на тесте, но валидацию не проходит. Даже ошибка на закрытии потоков, хотя пробовал открывать и закрывать прямо в if

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 19843 Куртка для сноубордистов, разм10173.99123 198479 Майка 11.12 3 198480 Майкаbhhdvchdevchvhcvde 11.124343 198481 Майкаbhhdvchdevchvhcvde 11.124343

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

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

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

JustinianJudge в Mega City One

Master

19 августа 2019, 19:46

решение

У тебя все работает на тесте? Покажи скриншот для файла с примера:

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

Прежде чем задавать вопрос, кроме своих тестов ты должен быть уверен что на тесте с условий все отрабатывает. Игнорировать примеры в условиях не вариант.

Дмитрий

Уровень 22

19 августа 2019, 20:19

JustinianJudge в Mega City One

Master

19 августа 2019, 20:24

Я писал о файле-примере с условий. Ты показываешь свои тесты

Дмитрий

Уровень 22

19 августа 2019, 20:45

Спасибо! Понял в чем проблема была. id номер для последнего товара в примере сливается с названием товара, поэтому максимальный id не удается найти. Передал вот так. Может подскажете, что поинтереснее, чтобы можно было убрать лишние пробелы в строке?

private static int findLastID(BufferedReader fileReader) throws IOException {
        String s;
        int maxID = 0;
        while ((s = fileReader.readLine()) != null) {
            String sID = s.substring(0, 8);
            int id = 0;
            if (sID.contains(" ")) {
                id = Integer.parseInt(sID.substring(0, s.indexOf(' ')));
            } else {
                id = Integer.parseInt(sID);
            }

            if (id > maxID) {
                maxID = id;
            }
        }

        maxID++;
        return maxID;
    }

JustinianJudge в Mega City One

Master

19 августа 2019, 21:08

Обычно в этой задаче решают как:

id = Integer.parseInt(sID.substring(0, 8).trim());

как-то так. Также обращаю внимание, что вот это private static int findLastID( вместе с этим

maxID++;
return maxID;

очень плохо. Метод называется "искатьПоследнийАйдишникКоторый в файле, а возвращаешь ты уже максимальный + 1. Метод не делает то, что должен по названию. Поэтому либо findMaxId, либо getIdForNextProduct. maxID - так обычно не сокращают, maxId. sID - это тоже не совсем понятно, супер Айди или сильный Айди.. Для данного метода в принципе +/- норм, поэтому я изначально и ничего не говорил, но чем больше область видимости переменной, тем более четкое название. Какое, это уже другой вопрос. Удачи.

Дмитрий

Уровень 22

20 августа 2019, 04:14

Учту, спасибо!