Злобный валидатор не пропускает редактирование строки.

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

Решен

вывод валидатора: "При запуске программы с параметрами "-u id productName price quantity" должна обновится информация о товаре в файле." строка с новыми параметрами становится на то же самое место, лишних строк не добавляется, не понимаю :(

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; /* Прайсы 2 */ import java.io.*; import java.util.ArrayList; public class Solution { private static ArrayList<Integer> idList = new ArrayList<>(); private static ArrayList<String> dataList = new ArrayList<>(); private static String fileName; private static int id; private static String productName; private static Double price; private static int quantity; public static void main(String[] args) throws IOException { readFromFile(); if (args.length > 0) { String key = args[0]; id = Integer.parseInt(args[1]); if (key.equals("-u")) { productName = args[2]; price = Double.valueOf(args[3]); quantity = Integer.parseInt(args[4]); updateData(); } else if (key.equals("-d")) { deleteData(); } } writeToFile(); } private static void readFromFile() throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); fileName = reader.readLine(); BufferedReader fileReader = new BufferedReader(new FileReader(fileName)); String lineFromFile; String dataFromFile; int idFromFile; while (fileReader.ready()) { lineFromFile = fileReader.readLine(); idFromFile = Integer.parseInt(lineFromFile.substring(0, 8).trim()); dataFromFile = lineFromFile.substring(8); idList.add(idFromFile); dataList.add(dataFromFile); } fileReader.close(); } private static void updateData() { for (int i = 0; i < idList.size(); i++) { if (idList.get(i) == id) { String newProductName = String.format("%-30s", productName); String newPrice = String.format("%-8s", price); String newQuantity = String.format("%-4s", quantity); String newData = newProductName + newPrice + newQuantity; dataList.set(i, newData); } } } private static void deleteData() { for (int i = 0; i < idList.size(); i++) { if (idList.get(i) == id) { idList.remove(i); dataList.remove(i); } } } private static void writeToFile() throws IOException { BufferedWriter fileWriter = new BufferedWriter(new FileWriter(fileName)); for (int i = 0; i < idList.size(); i++) { String newID = String.format("%-8s", idList.get(i)); String newData = dataList.get(i); String newFileLine = newID + newData; fileWriter.write(newFileLine); if (i != idList.size() - 1) fileWriter.newLine(); } fileWriter.close(); } }

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

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

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

JustinianJudge в Mega City One

Master

30 июля 2019, 18:08

решение

Сложно даже сказать отчего, вроде отрабатывает нормально. Пару различий которые я заметил (не знаю влияют ли на валидацию), твой код не резистентен к некорректным данным - то есть он позволит записать 1289 аааааааааааааааааааааааааааааааааааааааааааааааааааааааааааааа 11111111 и тд. Второй момент который я заметил, в конце строки будут пробелы, если например количество одноразрядное число, то в примере в условиях будет конец файла, у тебя будет три пробела. В чем проблема сложно сказать, попробуй прошерстить секцию Обсуждение и вопросы (в которых не проходило по этому пункту). А я традиционно пару замечаний/предложений дам. Модульная система, хелпер методы, хорошие имена, тут пятерик. Но вот это:

if (i != idList.size() - 1) fileWriter.newLine();

фигурные скобки мы можем себе позволить не ставить (для IF, для циклов ставим всегда) только в низкоуровнеых/математических алгоритмах и где общепринято, например переопределение equals. Далее, статические поля класса. Поле класса это его значимая характеристика. Имя у собаки. Рост человека, количество денег на счету и тд. В этом случае у тебя это сугубо внутренние временные переменные, которые не должны быть выведены в поля класса. Я понимаю что ты так сделал, чтобы обеспечить cutting through использование переменных, между методами и слоями, но так не делают. По хорошему , в этой задаче был бы дополнительный класс Product, который имел бы поля id, name, price, quantity. Мы бы просто выгружали инфу в List<Product>, сохраняли бы с параметров новый product, и было бы что-то из серии: save(product), delete(id). Это можно решить по разному, я приведу тебе пример твоего же кода и алгоритма, но БЕЗ статических переменных. Изучи этот вариант, это один из способов как работают без статиков, когда есть много методов использующих совместные ресурсы. В подавляющем большинстве случаев, без статических временных переменных можно обойтись, просто продумай дизайн.

JustinianJudge в Mega City One

Master

30 июля 2019, 18:08

полезный

public class Solution {
    public static void main(String[] args) throws IOException {
        String fileName = getFileNameFromConsole();
        List<String> products = readFromFile(fileName);

        if (args.length == 0) {
            return;
        }

        String key = args[0];
        int id = Integer.parseInt(args[1]);

        if (key.equals("-u")) {
            updateProduct(args, products);
        } else if (key.equals("-d")) {
            deleteProduct(id, products);
        }

        writeToFile(fileName, products);
    }


    private static List<String> readFromFile(String fileName) throws IOException {
        BufferedReader fileReader = new BufferedReader(new FileReader(fileName));
        List<String> result = new ArrayList<>();

        while (fileReader.ready()) {
            String line = fileReader.readLine();
            result.add(line);
        }
        fileReader.close();

        return result;
    }

    private static void updateProduct(String[] inputParams, List<String> products) {
        int idForUpdate = Integer.parseInt(inputParams[1]);
        for (int i = 0; i < products.size(); i++) {
            String product = products.get(i);
            int productId = Integer.parseInt(product.substring(0, 8).trim());

            if (productId == idForUpdate) {
                String updatedProduct = prepareStringForUpdate(inputParams);

                products.set(i, updatedProduct);
            }
        }
    }

    private static String prepareStringForUpdate(String[] inputParams) {
        String id = String.format("%-8s", inputParams[1]);
        String name = String.format("%-30s", inputParams[2]);
        String price = String.format("%-8s", inputParams[3]);
        String quantity = String.format("%-4s", inputParams[4]);
        return id + name + price + quantity;
    }

JustinianJudge в Mega City One

Master

30 июля 2019, 18:08

полезный

private static void deleteProduct(int id, List<String> products) {
        for (int i = 0; i < products.size(); i++) {
            String product = products.get(i);
            int productId = Integer.parseInt(product.substring(0, 8).trim());
            if (productId == id) {
                products.remove(i);
            }
        }
    }

    private static void writeToFile(String fileName, List<String> products) throws IOException {
        BufferedWriter fileWriter = new BufferedWriter(new FileWriter(fileName));
        for (int i = 0; i < products.size(); i++) {
            fileWriter.write(products.get(i));
            if (i != products.size() - 1) {
                fileWriter.newLine();
            }
        }
        fileWriter.close();
    }

    private static String getFileNameFromConsole() {
        String fileName = "";
        try (BufferedReader reader = new BufferedReader(new InputStreamReader(System.in))) {
            fileName = reader.readLine();
        } catch (IOException e) {
        }
        return fileName;
    }
}