JavaRush/Помощь по Java задачам/Вопрос по Прайс 2

44 уровень

15.07.2021
332просмотров
15комментариев

Вопрос по Прайс 2

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

Решен

Здравствуйте. Натолкните, пожалуйста на мысль, что я делаю не так? По трем последним пунктам не проходит.

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

package com.javarush.task.task18.task1828; import java.io.*; import java.util.ArrayList; import java.util.List; import java.lang.String; /* Прайсы 2 */ public class Solution { public static void main(String[] args) throws IOException { if (args.length == 0) { System.exit(0); } BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String fileName = reader.readLine(); reader.close(); BufferedReader reader1 = new BufferedReader(new FileReader(fileName)); ArrayList<String> arrayList = new ArrayList<>(); while (reader1.ready()) { arrayList.add(reader1.readLine()); } String str = ""; BufferedWriter writer = new BufferedWriter(new FileWriter(fileName)); switch (args[0]) { case "-u": for (String list : arrayList) { if (args[1].equals(list.substring(0, 8).trim())){ arrayList.set(arrayList.indexOf(list), String.format("%-8d%-30s%-8s%-4s\n", args[1], args[2], Float.parseFloat(args[3]), Integer.parseInt(args[4]))); } } reader.close(); for (String list : arrayList) { writer.write(list); } writer.close(); break; case "-d": for (String list : arrayList) { if (args[1].equals(list.substring(0, 8).trim())) { arrayList.remove(list); } } reader.close(); for (String list : arrayList) { writer.write(list); } writer.close(); break; } } }

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

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

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

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 09:38

for (String list : arrayList) {
    writer.write(list);
}

Так ты запишешь все строки списка в одну, без переносов. И зачем цену и количество парсить в числа, чтобы тут же снова записать их как строки? Тем более что в шаблоне форматирования ты используешь для подстановки %s.

Ivan

Уровень 44

15 июля 2021, 09:45

Спасибо тебе большое, насчет переноса строк, 4 пункт уже проходит, 3 и 5 остались. После твоих подсказок код выглядит так:

public static void main(String[] args) throws IOException {
        if (args.length == 0) {
            System.exit(0);
        }
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String fileName = reader.readLine();
        reader.close();
        BufferedReader reader1 = new BufferedReader(new FileReader(fileName));
        ArrayList<String> arrayList = new ArrayList<>();
        while (reader1.ready()) {
            arrayList.add(reader1.readLine());
        }
        BufferedWriter writer = new BufferedWriter(new FileWriter(fileName));
        switch (args[0]) {
            case "-u":
                for (String list : arrayList) {
                    if (args[1].equals(list.substring(0, 8).trim())) {
                        arrayList.set(arrayList.indexOf(list), String.format("%-8d%-30s%-8s%-4s\n", args[1], args[2],args[3], args[4]));
                    }
                }
                reader1.close();
                for (String list : arrayList) {
                    writer.write(list +"\n");
                }
                writer.close();
                break;

            case "-d":
                for (String list : arrayList) {
                    if (args[1].equals(list.substring(0, 8).trim())) {
                        arrayList.remove(list);
                    }
                }
                reader1.close();
                for (String list : arrayList) {
                    writer.write(list +"\n");
                }
                writer.close();
                break;
            default:
                writer.close();
                break;
        }

    }
}

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 09:48

"%-8d%-30s%-8s%-4s\n" - тут \n не нужен И еще - что если имя товара будет больше 30 символов? И почему бы reader не закрыть сразу после использования?

Ivan

Уровень 44

15 июля 2021, 09:51

"%-8d%-30s%-8s%-4s\n" если \n убрать ничего не меняется. а вот если строку в таком формате записываю: %s-8d%-30s%-8s%-4s (не важно с \n или без) засчитывает 5-й пункт, если честно не понимаю как это связанно.

Ivan

Уровень 44

15 июля 2021, 09:53

reader перенес, пока получается вот так ( не принимает только 3 - й пункт):

public static void main(String[] args) throws IOException {
        if (args.length == 0) {
            System.exit(0);
        }
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String fileName = reader.readLine();
        reader.close();
        BufferedReader reader1 = new BufferedReader(new FileReader(fileName));
        ArrayList<String> arrayList = new ArrayList<>();
        while (reader1.ready()) {
            arrayList.add(reader1.readLine());
        }
        reader1.close();
        BufferedWriter writer = new BufferedWriter(new FileWriter(fileName));
        switch (args[0]) {
            case "-u":
                for (String list : arrayList) {
                    if (args[1].equals(list.substring(0, 8).trim())) {
                        arrayList.set(arrayList.indexOf(list), String.format("%s-8d%-30s%-8s%-4s", args[1], args[2], args[3], args[4]));
                    }
                }
                for (String list : arrayList) {
                    writer.write(list + "\n");
                }
                writer.close();
                break;
            case "-d":
                for (String list : arrayList) {
                    if (args[1].equals(list.substring(0, 8).trim())) {
                        arrayList.remove(list);
                    }
                }
                for (String list : arrayList) {
                    writer.write(list + "\n");
                }
                writer.close();
                break;
            default:
                writer.close();
                break;
        }

    }
}

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 10:11

решение

что если имя товара будет больше 30 символов? writer.close() можно просто вынести из switch. Зачем повторять одно и то же 3 раза?

Ivan

Уровень 44

15 июля 2021, 10:20

Добавил вот такую конструкцию:

if (args[1].equals(list.substring(0, 8).trim())) {
                      String str = args[2];
                      while (str.length() < 30) {
                          StringBuilder stringBuilder = new StringBuilder(str);
                          stringBuilder.append(" ");
                          str = stringBuilder.toString();
                      }
                      if (str.length() > 30) {

                         str = str.substring(0, 30);
                      }
                      arrayList.set(arrayList.indexOf(list), String.format("%-8d%-30s%-8s%-4s", args[1], str, args[3], args[4]));
                  }
              }

Ivan

Уровень 44

15 июля 2021, 10:23

вынес из switch. 3-й пункт пока что так и не принимает. Уже не знаю куда смотреть, думаю дело в этой строке:

arrayList.set(arrayList.indexOf(list), String.format("%s-8d%-30s%-8s%-4s", args[1], str, args[3], args[4]));

%s-8d%-30s%-8s%-4s это конструкция не верная, но 5-й пункт только так принимает.

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 10:24

решение

А строку из String.format() ты просто скопировал откуда-то? потому что такое впечатление, что ты не очень понимаешь, что в ней происходит. И у тебя там сейчас ошибка.

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 10:28

решение

5-й пункт не принимает именно из-за того, что у тебя в этой строке параметры напутаны, и код падает с ошибкой.

Ivan

Уровень 44

15 июля 2021, 10:31

вроде я в ней разобрался, ничего сложного честно, да я ее скопировал, с обсуждений, но вроде посмотрел, вариант верный, изначально он вот такой: %-8d%-30s%-8s%-4s

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 10:35

решение

Если бы ты в ней действительно разобрался, то не стал бы наворачивать конструкцию с while и StringBuilder. Кроме того, тут важна не только сама строка, но и аргументы, которые к ней прилагаются. Тип должен соответствовать. А еще хорошо бы запускать свой код.

Ivan

Уровень 44

15 июля 2021, 10:38

код мой, кроме этой строки. почитал про стринг формат, переписал эту строку и все решил, спасибо тебе большое!!!))))))

Ivan

Уровень 44

15 июля 2021, 10:47

Ты имеешь ввиду, что while и stringbuilder можно заменить лишь этим: String.format("%-30s", args[2]); Получается while и builder можно было вообще не писать?

Ksenia Volkova Java Developer в DXC

Master

15 июля 2021, 10:57

полезный

Именно. А еще вместо substring(0, 30) можно было сделать "%-30.30s"