JavaRush/Помощь по Java задачам/Нервный срыв

23 уровень

31.10.2020
354просмотров
5комментариев

Нервный срыв

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

Прайсы

Java Core, 8 уровень, 11 лекция

Решен

Подскажите в чем проблема? Ладно предпосление пункты, но поледний? Почему потоки не закрыты... Спасибо

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, после максимального, найденного в файле.
Форматирование новой строки товара должно четко совпадать с указанным в задании.
Созданные для файлов потоки должны быть закрыты.

package com.javarush.task.task18.task1827; import java.io.*; import java.util.Random; import java.util.Scanner; public class Test { }

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

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

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

hidden #2322530

Уровень 41

31 октября 2020, 13:51

полезный

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

info = String.format("%-8s%-30s%-8s%-4s\n", (getId(path)), args[1], args[2], args[3]);

подразумевается конечно, что в файле есть строки, НО id как правило начинается с 1, а не с 11111111

long newId = 11111111;
        Scanner s = new Scanner(new File(path));
        if (!s.hasNextLine()) {
            return newId;

если в файле будет id такой "12345 ", то здесь будет исключение!

long id = Long.parseLong(s.nextLine().substring(0, 8));

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

представь что там 5 строк, с такими id:

то у тебя вернёт 11111111, разве это следующий за максимальным? должен быть ведь 12346 откуда взялось число 17 ?

return newId + 17;

этот поток правильно закрыть сразу, как только ты путь к файлу считал.

r.close();

Valerii

Уровень 23

31 октября 2020, 14:12

Спасибо. Просто написано id из 8 символов, больше чем предыдущий, на выходе так и вышло. Попробую 1++; А то, что productName может быть больше 30 символов, это надо урезать до 30? Так же и с price и quantity? Или не записывать товар вообще?

hidden #2322530

Уровень 41

31 октября 2020, 14:15

А то, что productName может быть больше 30 символов, это надо урезать до 30? Так же и с price и quantity? Или не записывать товар вообще? урезать Просто написано id из 8 символов, больше чем предыдущий, на выходе так и вышло. это максимальная величина. может быть от до 1 до 8 символов. надо считать id, найти максимальный. и его увеличить на один. в помощь тебе метод trim() , он убирает пробелы с обоих концов

Valerii

Уровень 23

31 октября 2020, 14:58

Подскажи пожалуйста, в чем теперь проблема??? Послдение 4 пунка также эррорит

public static void main(String[] args) throws IOException {
        if (args.length != 4 || !args[0].equals("-c")) return;

        BufferedReader r = new BufferedReader(new InputStreamReader(System.in));
        String path = r.readLine();
        r.close();

        BufferedWriter bw = new BufferedWriter(new FileWriter(path, true));
        Scanner s = new Scanner(new File(path));

        long id = 1;
        while (s.hasNextLine()) {
            String string = s.nextLine().substring(0, 8).trim();
            long lastId = Long.parseLong(string);
            id = lastId + 1;
        }
        s.close();

        String productName = args[1];
        String price = args[2];
        String quantity = args[3];

        if (productName.length() > 30) productName = productName.substring(0, 30);
        if (price.length() > 8) price = price.substring(0, 8);
        if (quantity.length() > 4) quantity = quantity.substring(0, 4);

        String info = String.format("%-8s%-30s%-8s%-4s\n", id, productName, price, quantity);

        bw.write(info);
        bw.close();
    }

hidden #2322530

Уровень 41

31 октября 2020, 15:08

решение

long lastId = Long.parseLong(string);
id = lastId + 1;

у тебя всегда будет увеличивать id полученный последней строке. но он может быть и по середине файла. тут следует использовать поиск максимума, а уже потом его увеличивать. id у тебя типа long, а ты его пытаешься как String вставить

String.format("%-8s%-30s%-8s%-4s\n", id,

надо тогда так сделать: String.valueOf(id) и не надо в задачах с CRUD использовать File/new File, валидатор не пропустит