Все работает, но не проходит валидатор.

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

Обсуждается

Понятия не имею, в чем проблема. Вроде бы все работает идеально.

В задании 11 мы захардкодили номер кредитной карточки с пином, с которыми разрешим работать нашему банкомату.
Но юзеров может быть много. Не будем же мы их всех хардкодить! Если понадобится добавить еще одного пользователя, то придется передеплоить наше приложение. Есть решение этой проблемы.

Смотри, добавился новый пакет resources, в котором мы будем хранить наши ресурсные файлы.
В этом пакете есть файл verifiedCards.properties, в котором заданы карточки с пинами.

1. В LoginCommand добавь поле private ResourceBundle validCreditCards.
При объявлении инициализируй это поле данными из файла verifiedCards.properties.
Почитай в инете, как это делается для ResourceBundle.
Важно: путь к ресурсу verifiedCards.properties строй динамически, для этого используй у класса CashMachine метод getPackage()
2. Замени хардкоженные данные кредитной карточки и пина на проверку наличия данных в ресурсе verifiedCards.properties.

Требования:

LoginCommand должен содержать приватное поле ResourceBundle validCreditCards.
Поле validCreditCards должно быть проинициализировано из файла verifiedCards.properties.
Используй проверку кредитной карточки и пина через verifiedCards.properties.

package com.javarush.task.task26.task2613; import com.javarush.task.task26.task2613.command.CommandExecutor; import com.javarush.task.task26.task2613.exception.InterruptOperationException; import java.util.Locale; public class CashMachine { public static void main(String[] args) { try { Locale.setDefault(Locale.ENGLISH); Operation operation = null; CommandExecutor.execute(Operation.LOGIN); do { operation = ConsoleHelper.askOperation(); CommandExecutor.execute(operation); } while (!(operation == Operation.EXIT)); } catch (InterruptOperationException e) { ConsoleHelper.writeMessage("Bye!"); } } }

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

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

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

Roman Pryshchepa Java Developer

Expert

17 января 2018, 16:00

Нашел ошибку: private ResourceBundle validCreditCards = ResourceBundle.getBundle( CashMachine.class.getPackage().getName() + ".resources.VerifiedCards", new Locale("ru", "RU")); старое решение тоже проходит после исправления этого "бага" ))

Roman Pryshchepa Java Developer

Expert

17 января 2018, 15:10

У тебя проверка карты и проверка пина на соответствие шаблону в разных циклах - зачем? Почему проверка карты и пина на валидность не в цикле? Попробуй сорвместить эти проверки в одном цикле.

Славиша Мишич

Уровень 40

17 января 2018, 15:21

Сделал в 2 цикла, чтобы при ошибочном номере карты не спрашивать ПИН код. Меньше работы для пользователя. Хорошо, сейчас сделаю в одном цикле.

Славиша Мишич

Уровень 40

17 января 2018, 15:37

Сделал все в одном цикле так: public void execute() throws InterruptOperationException { String[] input = new String[2]; while (true) { ConsoleHelper.writeMessage("Enter card number (12 digits):"); input[0] = ConsoleHelper.readString().trim(); ConsoleHelper.writeMessage("Enter PIN code (4 digits):"); input[1] = ConsoleHelper.readString().trim(); if (!isNumeric(input[0]) || (input[0].length() != 12) || !isNumeric(input[1]) || (input[1].length() != 4)) { System.out.println("Wrong card number or PIN code! Try again!"); } else { if (validCreditCards.containsKey(input[0]) && validCreditCards.getString(input[0]).equals(input[1])){ ConsoleHelper.writeMessage("Login is successful!"); break; }else { ConsoleHelper.writeMessage("Wrong card number or PIN!"); } } } } но все одно и тоже. Все работает, валидатор не проходит.