0

Всем привет! Я изучаю Java и в пишу программку для сканера штрихкода.

Программа составлена так, что при вводе в консоль пользователем определенных команд происходит соответствующее действие.

Так например, при вводе в консоль слова "Склад" пользователю в консоль выводится список продуктов, которые есть в магазине.

Реализовано это так: Есть файл Warehouse.txt. В него предварительно заносится информация о продуктах в таком формате: Артикул, Наименование, Цена -> 486414, Хлеб, 40

Программа считывает информацию из файла, создает объекты класса Product и заносит данные в ArrayList. Код метода ниже:

public static void ListForToday(ArrayList<Product> products) throws IOException {
    BufferedReader storageReader = new BufferedReader(new FileReader ("C:\\Users\\Home PC\\IdeaProjects\\Barcode_reader\\src\\Warehouse.txt"));
    String mainLine = storageReader.readLine();
    while (true){
        String line = storageReader.readLine();
        if (line == null) break;
        String [] cells = line.split("\\s*,\\s*");
        int article = Integer.parseInt(cells[0]);
        String name = cells[1];
        int price = Integer.parseInt(cells[2]);
        products.add(new Product(article, name, price));
    }
}

Код, когда вводим слово "Склад" (находится в методе main):

if (stringScanner.equals("Склад") || stringScanner.equals("склад")) {
    System.out.println("Сегодня в продаже:");
    ListForToday(today);
    for (Product product : today) {
        System.out.println(product);
    }
}

И вроде как даже этот ужас работает, НО!

Если пользователь введет слово "Склад" ещё раз, то в ArrayList снова добавятся продукты из файла. И соответственно так при каждом вводе.

Как мне запретить запись из файла в ArrayList после того, как в него внесены данные в первый раз?

Не совсем понимаю как исправить этот баг.

Заранее благодарю!

4
  • if(today.isEmpty()) { ListForToday(today); }?
    – ЮрийСПб
    26 дек 2021 в 20:33
  • Не совсем понял, сорри
    – Alex_Lynx
    26 дек 2021 в 20:47
  • Имеется в виду - проверять, нуст ли список и только в этом случае его наполнять. Это одно из множества возможных вариантов решения расплавчато поставленной задачи.
    – ЮрийСПб
    26 дек 2021 в 20:55
  • Если вам нужна уникальная коллекция, то вам нужно множество а не аррэй 26 дек 2021 в 21:33

2 ответа 2

1

Проблема состоит в том, что пользователь может ввести команду инициализации списка несколько раз, для её решения есть разные способы:

  1. принимать команду "склад" только один раз при запуске метода main - вне цикла
  2. проверять, пустой ли список перед обработкой команды "склад" -- можно дополнительно предложить пользователю варианты действий, если список непустой: прочитать заново, добавить в существующий список (как сделано сейчас), проигнорировать:
if ("склад".equalsIgnoreCase(stringScanner)) {
    if (!today.isEmpty()) {
        System.out.println("Список товаров непустой, 1) обновить список 2) добавить 3) отмена");
        int option = Integer.parseInt(scanner.nextLine());
        if (option < 3) {
            if (option == 1) today.clear();

            ListForToday(today);
        }
    }
}
  1. Можно ввести какой-то флажок, чтобы запомнить, что список уже был проинициализирован -- но такой подход наименее дружелюбный, так как не имеет смысла давать возможность какого-то действия, а затем возвращать ошибку, что оно запрещено.
boolean initialized = false;
if ("склад".equalsIgnoreCase(stringScanner)) {
    if (!initialized) {
        ListForToday(today);
        initialized = true;
    } else {
        System.out.println("Список уже был инициализирован");
    }
}
1
  • Спасибо большое! Проработаю все предложенные варианты в качестве обучения
    – Alex_Lynx
    28 дек 2021 в 18:53
1

Чтобы не добавлять заново те же объекты, можно спрашивать у List существует ли добавляемый экземпляр объекта. Для сравнения можно использовать лишь поле "Артикул".

 public static void ListForToday(ArrayList<Product> products) throws IOException {
    BufferedReader storageReader = new BufferedReader(new FileReader("C:\\Users\\Home PC\\IdeaProjects\\Barcode_reader\\src\\Warehouse.txt"));
    String mainLine = storageReader.readLine();
    // New
    ArrayList<Integer> articleList = new ArrayList<String>();
    if (products.size() != 0) {
        for (Product x : products) {
            articleList.add(x.getArticle());
            // Используйте инкапсуляцию! (гетеры и сетеры)
        }
    }
    //New
    while (true) {
        String line = storageReader.readLine();
        if (line == null) break;
        String[] cells = line.split("\\s*,\\s*");
        int article = Integer.parseInt(cells[0]);
        String name = cells[1];
        int price = Integer.parseInt(cells[2]);

        if (articleList.size() != 0) {
            articleList.contains(article) ? System.out.println("Объект уже существует")
                    :
                    products.add(new Product(article, name, price));
        }

    }
}

Честно признаюсь, я не знаю как будет работать анбоксинг в этой ситуации, но я думаю, должно сработать. Лучше хранить вашу коллекцию не в списке, а в множестве Set, переопределив поля equals и hashcode.

1
  • Спасибо большое!
    – Alex_Lynx
    28 дек 2021 в 19:00

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.