JavaRush/Помощь по Java задачам/Не проходят пункты 2 и 3. Что не так?

26 уровень

31.05.2022
287просмотров
12комментариев

Не проходят пункты 2 и 3. Что не так?

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

Обсуждается

При тестировании всё происходит так, как должно быть - если параметр "-u", то в файле обновляется информация по товару с определенным id, если параметр "-d", то удаляется. Да, по сути, я просто перезаписываю файл новыми данными, не обновляю или удаляю данные прямо в файле, но в итоге же получается файл нужного формата. Помогите разобраться, что не так?

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

Значения параметров:
-u - флаг, который означает обновление данных товара с заданным id
-d - флаг, который означает физическое удаление товара с заданным id (из файла удаляются все данные, которые относятся к переданному id)
id - id товара, 8 символов
productName - название товара, 30 символов
price - цена, 8 символов
quantity - количество, 4 символа

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

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

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

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

Требования:

Программа должна считать имя файла для операций CRUD с консоли.
При запуске программы без параметров список товаров должен остаться неизменным.
При запуске программы с параметрами "-u id productName price quantity" должна обновится информация о товаре в файле.
При запуске программы с параметрами "-d id" строка товара с заданным id должна быть удалена из файла.
Созданные для файлов потоки должны быть закрыты.

5632 Жакетик 4500 12 750 Носочки красные 150 5 22 Колготы для мальчиков 7500000 8992 464 Кардиган(бежевый) 1699 10 5634 Картофель(жаренный) 500 1

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

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

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

Артем Тимонин

Уровень 26

1 июня 2022, 14:08

Вот так тоже не проходит...

public class Solution {
    public static void main(String[] args) {
        if (args.length == 0) return;
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));

        String filename = "";

        try {
            filename = bufferedReader.readLine();
        } catch (IOException e) {
            e.printStackTrace();
        }

        ArrayList<String> list = new ArrayList<>();
        int keyIndex = -1;
        try (BufferedReader bufferedFileReader = new BufferedReader(new FileReader(filename))){
            String data;
            while (bufferedFileReader.ready() && !(data = bufferedFileReader.readLine()).equals("")) {
                list.add(data);
                int index = Integer.parseInt(data.split(" ")[0].trim());
                if (index == Integer.parseInt(args[1])) keyIndex = list.size() - 1;
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

        if (keyIndex >= 0) {
            if (args[0].equals("-u")) list.set(keyIndex, String.format("%-8s%-30s%-8s%-4s", args[1], args[2], args[3], args[4]));
            else if (args[0].equals("-d")) list.remove(keyIndex);
        }

        try (BufferedWriter writer = new BufferedWriter(new FileWriter(filename))) {
            for (String data : list) {
                writer.write(data);
                writer.newLine();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }

    }
}

Anonymous #2963897

Уровень 31

1 июня 2022, 11:22

когда ты делаешь add ты просто сдвигаешь, а тебе нужно заменить list.set(). как уже написали. если не пройдет надо будет еще поковырять.

Артем Тимонин

Уровень 26

1 июня 2022, 14:08

Я ведь просто заполняю список данными с нуля и если у нас параметр -u то подменяю новую строку на данные из параметров, а если парметр -d, то просто ничего не добавляю в список. За счет этого не нужны лишние манипуляции и заполнить список можно за один проход по файлу. Переписал вот так, но пункты 2 и 3 также не проходят, хотя всё работает...

public class Solution {
    public static void main(String[] args) {
        if (args.length == 0) return;
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));

        String filename = "";

        try {
            filename = bufferedReader.readLine();
        } catch (IOException e) {
            e.printStackTrace();
        }

        ArrayList<String> list = new ArrayList<>();
        int keyIndex = -1;
        try (BufferedReader bufferedFileReader = new BufferedReader(new FileReader(filename))){
            String data;
            while (bufferedFileReader.ready() && !(data = bufferedFileReader.readLine()).equals("")) {
                list.add(data);
                int index = Integer.parseInt(data.split(" ")[0].trim());
                if (index == Integer.parseInt(args[1])) keyIndex = list.size() - 1;
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

        if (keyIndex >= 0) {
            if (args[0].equals("-u")) list.set(keyIndex, String.format("%-8s%-30s%-8s%-4s", args[1], args[2], args[3], args[4]));
            else if (args[0].equals("-d")) list.remove(keyIndex);
        }

        try (BufferedWriter writer = new BufferedWriter(new FileWriter(filename))) {
            for (String data : list) {
                writer.write(data);
                writer.newLine();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }

    }
}

Anonymous #2963897

Уровень 31

1 июня 2022, 14:23

А зачем тут

if (index == Integer.parseInt(args[1])) keyIndex = list.size() - 1;

keyIndex присваиваешь размер листа минус один? ты же потом его используешь для удаления и замены, а он у тебя будет не тот который задан в args. наверное там должно быть keyIndex = index?

Артем Тимонин

Уровень 26

1 июня 2022, 19:06

В данном блоке мы проверяем - если индекс текущей строки = индексу, с которым мы хотим работать потом (обновить или удалить), то сохраняем этот самый индекс, чтобы потом по нему уже найти нужный элемент в списке. Проще всего, как мне показалось, сделать это можно просто получить индекс последнего элемента СРАЗУ после того, как мы добавили в список элемент, удовлетворяющий условию выше. Больше значение переменной keyIndex меняться не будет, так как мы нашли нужный индекс и он у нас всего один. В итоге я не выдержал и подсмотрел верное решение. Так, конечно, всё несколько мудрено решено - с помощью дополнительного класса и его объектов (по мне так немного избыточно), но конкретно мой явный косяк был в том, что я не учел тот факт, что наименование товара, который нужно обновить может состоять из нескольких слов. То есть фактически, параметров args может быть гораздо больше, чем 4.

WillMage

Уровень 36

1 июня 2022, 08:55

list.add(data); Обновиться то есть замениться. думаю set() нужен. И при - d удалиться.

Артем Тимонин

Уровень 26

1 июня 2022, 14:07

public class Solution {
    public static void main(String[] args) {
        if (args.length == 0) return;
        BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in));

        String filename = "";

        try {
            filename = bufferedReader.readLine();
        } catch (IOException e) {
            e.printStackTrace();
        }

        ArrayList<String> list = new ArrayList<>();
        int keyIndex = -1;
        try (BufferedReader bufferedFileReader = new BufferedReader(new FileReader(filename))){
            String data;
            while (bufferedFileReader.ready() && !(data = bufferedFileReader.readLine()).equals("")) {
                list.add(data);
                int index = Integer.parseInt(data.split(" ")[0].trim());
                if (index == Integer.parseInt(args[1])) keyIndex = list.size() - 1;
            }

        } catch (IOException e) {
            e.printStackTrace();
        }

        if (keyIndex >= 0) {
            if (args[0].equals("-u")) list.set(keyIndex, String.format("%-8s%-30s%-8s%-4s", args[1], args[2], args[3], args[4]));
            else if (args[0].equals("-d")) list.remove(keyIndex);
        }

        try (BufferedWriter writer = new BufferedWriter(new FileWriter(filename))) {
            for (String data : list) {
                writer.write(data);
                writer.newLine();
            }
        } catch (IOException e) {
            e.printStackTrace();
        }

    }
}

WillMage

Уровень 36

1 июня 2022, 14:30

data.split(" ")[0].trim() а не может быть тут беда?? может же быть такое что между этими частями строки не куда ставить пробел???

Артем Тимонин

Уровень 26

1 июня 2022, 19:07

Все тесты у меня проходили нормально, но есть одно НО) В итоге я не выдержал и подсмотрел верное решение. Так, конечно, всё несколько мудрено решено - с помощью дополнительного класса и его объектов (по мне так немного избыточно), но конкретно мой явный косяк был в том, что я не учел тот факт, что наименование товара, который нужно обновить может состоять из нескольких слов. То есть фактически, параметров args может быть гораздо больше, чем 4.

WillMage

Уровень 36

2 июня 2022, 11:52

И все же метод split() точно переписал?))))

Артем Тимонин

Уровень 26

2 июня 2022, 12:00

В файле данные лежат в формате "индекс товар количество цена". То есть все части строки разделены пробелами. На первом месте всегда стоит индекс, так что о том, что там дальше в строке - можно не беспокоиться. Единственное, тут, вероятно, лишним является метод trim(), удаляющий пробелы в строке слева и справа, так как Integer.parseInt() вполне парсит число и с ними.

WillMage

Уровень 36

2 июня 2022, 12:08

В условии задачи есть пример содержания файла и описание какое количество символов должно быть отведено под все элементы "индекс товар количество цена" 8 символов под id и после нет места пробелу. И принимая из файла такую забитую строку, методом сплит с разделителем " " вы не получите правильного разделения строки.