JavaRush/Помощь по Java задачам/Помогите пожалуйста, по тесту всё ок.

19 уровень

15.10.2020
422просмотров
8комментариев

Помогите пожалуйста, по тесту всё ок.

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

Решен

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

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.ArrayList; import java.util.List; /* Прайсы */ public class Solution { public static void main(String[] args) throws Exception { // args = new String[]{"-c", "T-short", "32.01", "110"}; BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); if (args[0].equals("-c") && args.length == 4) { String address = reader.readLine(); BufferedWriter writer = new BufferedWriter(new FileWriter(address, true)); // System.out.printf("\n%-8d%-30s%-8.2f%-4d", // findNumberNewProduct(address) + 1, args[1], Double.parseDouble(args[2]), Integer.parseInt(args[3])); String line = String.format("\n%-8d%-30s%-8.2f%-4d", findNumberNewProduct(address) + 1, args[1], Double.parseDouble(args[2]), Integer.parseInt(args[3])); writer.write(line); writer.close(); } reader.close(); } public static int findNumberNewProduct(String address) throws IOException { //looking max ID in all lines (in file) FileReader fileReader = new FileReader(new File(address)); BufferedReader bufferedReader = new BufferedReader(fileReader); String line = ""; int number = 0; while ((line = bufferedReader.readLine()) != null) { int indexId; if ((indexId = Integer.parseInt(line.substring(0, line.indexOf(' ')))) > number) { number = indexId; } } bufferedReader.close(); fileReader.close(); return number; } }

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

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

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

Natasha Martova

Уровень 41

17 октября 2020, 18:53

Мне кажется, сначала нужно вообще проверить наличие аргументов, если 0 - выход

hidden #2322530

Уровень 41

15 октября 2020, 19:17

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

если аргументов нет, тут будет исключение ArrayIndexOutOfBoundsException. надо вначале проверить что длина = 4, а потом уже проверять нулевой аргумент.

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 0 out of bounds for length 0

а еще здесь:

if ((indexId = Integer.parseInt(line.substring(0, line.indexOf(' ')))) > number) {

в строке между Id и названием может и не быть пробелов. но во входном файле первые 8 символов (включая пробелы) это будет id

Exception in thread "main" java.lang.NumberFormatException: For input string: "19847983Куртка"

Vitalik

Уровень 19

18 октября 2020, 17:32

благодарю, очень помогло. Всё равно валидатор не даёт завершить задачу. Теперь почему-то ему не нравится то, что id не следующий после максимального, форматирование строки не нравится и не нравится, что добавляется строка. Может не нравится, что при пустом файле записывается строка с переносом? Подскажите пожалуйста.

public static void main(String[] args) throws Exception {
        args = new String[]{"-c", "T-shortsssss", "32.01", "110"};

        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        if (args.length == 4 && args[0].equals("-c")) {
            String address = reader.readLine();
            BufferedWriter writer = new BufferedWriter(new FileWriter(address, true));
            String line = String.format("\n%-8d%-30s%-8.2f%-4d",
                    findNumberNewProduct(address) + 1, args[1], Double.parseDouble(args[2]), Integer.parseInt(args[3]));

            writer.write(line);
            writer.close();
        }
        reader.close();
    }

    public static int findNumberNewProduct(String address) throws IOException {     //looking for a max ID in all lines (in file)
        FileReader fileReader = new FileReader(new File(address));
        BufferedReader bufferedReader = new BufferedReader(fileReader);
        String line = "";
        int number = 0;
        while ((line = bufferedReader.readLine()) != null) {
            String textId = line.substring(0, 8);
            int indexId;
            if (textId.contains(" ")) {
                if ((indexId = Integer.parseInt((line.substring(0, 8)).substring(0, line.indexOf(' ')))) > number) {
                    number = indexId;
                }
            } else {
                if ((indexId = Integer.parseInt(textId)) > number) {
                    number = indexId;
                }
            }
        }
        bufferedReader.close();
        fileReader.close();
        return number;
    }
<

hidden #2322530

Уровень 41

18 октября 2020, 18:00

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

args = new String[]{"-c", "T-shortsssss", "32.01", "110"};

перед отправкой на валидацию, это надо убирать. ещё надо учитывать, что аргументы могут быть длинее указанных цифр. например наименование может быть 100 символов, его надо тогда обрезать до 30 вот это всё:

FileReader fileReader = new FileReader(new File(address));
        BufferedReader bufferedReader = new BufferedReader(fileReader);
        String line = "";
        int number = 0;
        while ((line = bufferedReader.readLine()) != null) {
            String textId = line.substring(0, 8);
            int indexId;
            if (textId.contains(" ")) {
                if ((indexId = Integer.parseInt((line.substring(0, 8)).substring(0, line.indexOf(' ')))) > number) {
                    number = indexId;
                }
            } else {
                if ((indexId = Integer.parseInt(textId)) > number) {
                    number = indexId;
                }
            }
        }
        bufferedReader.close();
        fileReader.close();
        return number;

можно переписать вот так:

public static int findNumberNewProduct(String address) throws IOException {     //looking for a max ID in all lines (in file)
    int number = 1;
    try (BufferedReader bufferedReader = new BufferedReader(new FileReader(address))) {
        String line = "";
        while ((line = bufferedReader.readLine()) != null) {
            number = Math.max(number, Integer.parseInt(line.substring(0, 8).trim()));
        }
    }
    return number;
}

Vitalik

Уровень 19

18 октября 2020, 20:21

записанный аргумент я в комменты кидаю конечно же) это для себя сделано, чтобы проверять) по поводу записи в 100 символов и т.д. Есть же форматирование: "\n%-8d%-30s%-8.2f%-4d" оно почему-то у меня не работает, что очень странно. Если есть возможность, объясните пожалуйста, потому что в разборе видео по форматированию нужно писать именно 30s, а не 30.30s. Почему так? Исправил именно на 30.30s и начало правильно обрезать. Не знал о методе trim очень выручает в этом случае, огромное спасибо.

Vitalik

Уровень 19

19 октября 2020, 10:36

Большое спасибо за подсказки, но если удастся ещё раз помочь, буду весьма благодарен. Ругается валидатор на то, что при запуске с правильными параметрами, у меня якобы строка не добавляется в конец файла. Не пойму. Проверил по всякому, всё работает как часы.

public static void main(String[] args) throws Exception {
//        args = new String[]{"-c", "Футболка красная, мужская", "17.2", "1000"};

        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        if (args.length == 4 && args[0].equals("-c")) {
            String address = reader.readLine();
            BufferedWriter writer = new BufferedWriter(new FileWriter(address, true));
            File file = new File(address);
            if (file.length() > 0) {
                writer.write(String.format(System.lineSeparator() + "%-8d%-30.30s%-8.2f%-4d",
                        findNumberNewProduct(address) + 1, args[1], Float.parseFloat(args[2]), Integer.parseInt(args[3])));
            } else {
                writer.write(String.format("%-8d%-30.30s%-8.2f%-4d",
                        findNumberNewProduct(address) + 1, args[1], Float.parseFloat(args[2]), Integer.parseInt(args[3])));
            }
            writer.close();
        }
        reader.close();
    }

    public static int findNumberNewProduct(String address) throws IOException {     //looking for a max ID in all lines (in file)
        String line = "";
        int number = 0;
        try (BufferedReader bufferedReader = new BufferedReader(new FileReader(address))) {
            while ((line = bufferedReader.readLine()) != null) {
                number = Math.max(number, Integer.parseInt(line.substring(0, 8).trim()));
            }
            bufferedReader.close();
            return number;
        }
    }

hidden #2322530

Уровень 41

19 октября 2020, 10:58

не надо использовать File, если об этом не попросили в условии. валидатор очень плохо относится к этому.

Vitalik

Уровень 19

19 октября 2020, 18:31

сделал чуть по другому. file.available засунул сразу в условие if. Минус строчка, так сказать. Чтобы я без вас делал, не знаю. Спасибо огромное, очень и очень помогли.