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

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

Обсуждается

package com.javarush.task.task18.task1827;

import java.io.*;
import java.util.ArrayList;
import java.util.Arrays;

/*
Прайсы
*/

public class Solution {
    public static void main(String[] args) throws Exception {


// Создание переменных и присвоение им параметров


        if (args.length!=0) {


        int id;
        String key = args[0];
        String productName = String.join(" ", Arrays.copyOfRange(args, 1,args.length-2));
        String price = args[args.length-2];
        String quantity = args[args.length-1];


// Создание переменной Названия продукта нужного формата


        productName = productName.length() > 30 ? productName.substring(0, 30) : productName;
        productName = String.format("%-30s", productName);


// Создание переменной Цены нужного формата


        price = price.length() > 8 ? price.substring(0, 8) : price;
        price = String.format("%-8s", price);

// Создание переменной Количества нужного формата


        quantity = quantity.length() > 4 ? quantity.substring(0, 4) : quantity;
        quantity = String.format("%-4s", quantity);


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

//        String path = "1";


        FileReader fileReader = new FileReader(path);
        FileWriter fileWriter = new FileWriter(path, true);


// Поиск последнего ID в файле

        ArrayList<Character> arrayList = new ArrayList();

        int charCode;
        while((charCode = fileReader.read()) != -1) {
            arrayList.add((char)charCode);
        }

        String idString="";

        for (int i = 0; i < 8; i++) {
            if(arrayList.size()>49) {
                idString += arrayList.get(arrayList.size() - 50 + i);
            } else if (arrayList.size()==49) {
                idString = "1";
            }  else{
                idString = "0";
            }
        }

// Присвоение нового ID

        id = Integer.parseInt(idString.trim())+1; // Прибавляем +1


// Создание переменной ID нужного формата

        String idStr = (String.valueOf(id)).length() > 8 ? (String.valueOf(id)).substring(0, 8) : (String.valueOf(id));
        idStr = String.format("%-8s", idStr);


// Запись в фаил


        fileWriter.write("\n");
        fileWriter.write(idStr+productName+price+quantity);


        fileReader.close();
        fileWriter.flush();
        fileWriter.close();
        }
    }

}

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.Arrays; /* Прайсы */ public class Solution { public static void main(String[] args) throws Exception { // Создание переменных и присвоение им параметров if (args.length!=0) { int id; String key = args[0]; String productName = String.join(" ", Arrays.copyOfRange(args, 1,args.length-2)); String price = args[args.length-2]; String quantity = args[args.length-1]; // Создание переменной Названия продукта нужного формата productName = productName.length() > 30 ? productName.substring(0, 30) : productName; productName = String.format("%-30s", productName); // Создание переменной Цены нужного формата price = price.length() > 8 ? price.substring(0, 8) : price; price = String.format("%-8s", price); // Создание переменной Количества нужного формата quantity = quantity.length() > 4 ? quantity.substring(0, 4) : quantity; quantity = String.format("%-4s", quantity); BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(System.in)); String path = bufferedReader.readLine(); // String path = "1"; FileReader fileReader = new FileReader(path); FileWriter fileWriter = new FileWriter(path, true); // Поиск последнего ID в файле ArrayList<Character> arrayList = new ArrayList(); int charCode; while((charCode = fileReader.read()) != -1) { arrayList.add((char)charCode); } String idString=""; for (int i = 0; i < 8; i++) { if(arrayList.size()>49) { idString += arrayList.get(arrayList.size() - 50 + i); } else if (arrayList.size()==49) { idString = "1"; } else{ idString = "0"; } } // Присвоение нового ID id = Integer.parseInt(idString.trim())+1; // Прибавляем +1 // Создание переменной ID нужного формата String idStr = (String.valueOf(id)).length() > 8 ? (String.valueOf(id)).substring(0, 8) : (String.valueOf(id)); idStr = String.format("%-8s", idStr); // Запись в фаил fileWriter.write("\n"); fileWriter.write(idStr+productName+price+quantity); fileReader.close(); fileWriter.flush(); fileWriter.close(); } } }

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

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

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

Anonymous #2372013

Уровень 26

13 октября 2021, 09:43

В аррэйлист я бы добавил try with resources, а затем создал бы отдельный геттер для объекта product, в котором он бы хранил id, name, price, quantity, а затем возвращал бы их параметры : ArrayList<Character> arrayList = new ArrayList<>(); try(BufferedReader fileReader = new BufferedReader(new FileReader(fileName))){ while (fileReader.ready()){ Product product = getProduct(fileReader.readLine()); arrayList.add(product); } } /* в конце вашего кода пишем геттер для объекта типа Product c конструктором типа String */ public static Product getProduct(String string){ String id string.substring(0, 8).trim(); String name= string.substring(8, 38).trim(); String price = string.substring(38, 46).trim(); String quantit = string.substring(46, 50).trim(); return new Product(Integer.parseInt(id), name, price, quantity); } }

Vit

Уровень 33

12 октября 2021, 18:19

Как же греют душу комментарии в коде👍. Валидатор пишет: "Товар должен иметь следующий id, после максимального, найденного в файле."

Ksenia VolkovaJava Developer в DXC

Master

12 октября 2021, 19:58

Многие гуру программирования считают, что комментарии - скорее признак плохого кода, ну или по-крайней мере мусор. Чистый и продуманный код объясняет сам себя и в комментариях не нуждается. Есть хорошая книжка на эту тему - "Чистый код" Роберта Мартина.

Ksenia VolkovaJava Developer в DXC

Master

12 октября 2021, 17:51

В чем смысл этого цикла?

for (int i = 0; i < 8; i++) {
    if(arrayList.size()>49) {
        idString += arrayList.get(arrayList.size() - 50 + i);
    } else if (arrayList.size()==49) {
        idString = "1";
    }  else{
        idString = "0";
    }
}

Максимальный id - необязательно последний.

Evgeny Krasnoperov

Уровень 24

Expert

13 октября 2021, 08:46

Поиск крайнего id и если фаил пуст присвоить 1 ну как я понял основная ошибка нужен не крайний ID а максимальный