Как сделать валидацию пользователя?

Question

[Amir Kenesbay]

У меня есть метод validateUser и он должен выбрасывать исключение AccessDeniedException ну который сам создал. И я написал таким образом:

public static void validateUser(User user) throws AccessDeniedException{
        if(user.getAge()<18){
            throw new AccessDeniedException("Access Denied");
        }else{
            System.out.println("Age access granted");
        }
    }

И вопрос в том что как его вывести в методе main, ну допустим вводим логин и пароль юзера и если он меньше 18ти то должен вывести исключение. Ну я создал класс юзер с конструктором и геттерами. Весь код внизу(Не обращайте внимание на другие методы)

public static void main(String[] args) throws UserNotFoundException, AccessDeniedException{
        Scanner scanner = new Scanner(System.in);

        System.out.println("Введите логин: ");
        String login = scanner.nextLine();
        System.out.println("Введите пароль: ");
        String password = scanner.nextLine();

        //TODO Проверить логин и пароль
//        User user = getUserByLoginAndPassword(login, password);
//        System.out.println("User: " + user.getLogin() + " - " + user.getEmail());
//        System.out.println("Access granted");
        //TODO Вызвать методы валидации пользователя

    }
    
    public static User[] getUsers(){
        User user1 = new User("john", "[e-mail]", "pass", 24);
        User user2 = new User("mike", "[e-mail]", "qwerty", 22);
        User user3 = new User("steve", "[e-mail]", "asdfg", 23);
        return new User[]{user1, user2, user3};
    }

    public static User getUserByLoginAndPassword(String login, String password) throws UserNotFoundException{
        User[] users = getUsersList();
        for (User user : users) {
            if(login.equals(user.getLogin()) && password.equals(user.getPassword())){
                return user;
            }
        }
        throw new UserNotFoundException("User not found: " + login);
    }

    private static User[] getUsersList() {
        return getUsers();
    }


    public static void validateUser(User user) throws AccessDeniedException{
        if(user.getAge()<18){
            throw new AccessDeniedException("Access Denied");
        }else{
            System.out.println("Age access granted");
        }
    }

Answer 1

[gsaw]

Непонятный вопрос конечно, но наверное так

try {
    validateUser()
    System.out.println("Проходите")
} catch( AccessDeniedException e) {
    System.out.println("Не положено!")
}

Comments

[Amir Kenesbay]

уже сделал таким образом

public static void main(String[] args) throws UserNotFoundException, AccessDeniedException{
        Scanner scanner = new Scanner(System.in);

        System.out.println("Введите логин: ");
        String login = scanner.nextLine();
        System.out.println("Введите пароль: ");
        String password = scanner.nextLine();

        //TODO Проверить логин и пароль
        User user = getUserByLoginAndPassword(login, password);
        System.out.println("User: " + user.getLogin() + " - " + user.getEmail());
        System.out.println("Access granted");
        //TODO Вызвать методы валидации пользователя
        validateUser(user);
    }

    public static User[] getUsers(){
        User user1 = new User("john", "[e-mail]", "pass", 17);
        User user2 = new User("mike", "[e-mail]", "qwerty", 18);
        User user3 = new User("steve", "[e-mail]", "asdfg", 23);
        return new User[]{user1, user2, user3};
    }

    public static User getUserByLoginAndPassword(String login, String password) throws UserNotFoundException{
        User[] users = getUsersList();
        for (User user : users) {
            if(login.equals(user.getLogin()) && password.equals(user.getPassword())){
                return user;
            }
        }
        throw new UserNotFoundException("User not found: " + login);
    }

    private static User[] getUsersList() {
        return getUsers();
    }

    public static void validateUser(User user) throws AccessDeniedException{

        if(user.getAge()<18){
            throw new AccessDeniedException("Access Denied");
        }else{
            getUsers();
        }
    }

[Jacen11]

Amir Kenesbay,
у вас же выведется
Access granted
Exception in thread "main" java.nio.file.AccessDeniedException: Access Denied

что это хрень?

[Amir Kenesbay]

Jacen11,

//TODO Проверить логин и пароль
        User user = getUserByLoginAndPassword(login, password);
        System.out.println("User: " + user.getLogin() + " - " + user.getEmail());

        //TODO Вызвать методы валидации пользователя
        validateUser(user);
        System.out.println("Access granted");

чуть ниже поместил System.out.println("Access granted");

[Jacen11]

Amir Kenesbay,
у вас же все равно стек трейс вылезет. Когда такое случается в настоящей разработке над таким смеются. И это может иметь очень печальные последствия. Не стоит так делать

[Amir Kenesbay]

Jacen11, а как можно сделать по другому?

[Jacen11]

Amir Kenesbay,
вам выше написали. Обернуть в трай кетч и выводить какое то сообщение.

для теста я немного сократил код

в мейне

try {
            User user = getUserByLoginAndPassword(login, password);
            System.out.println("Access granted");
        } catch( AccessDeniedException e) {
            System.out.println("Access Denied");
        }

public static User getUserByLoginAndPassword(String login, String password) throws AccessDeniedException {
        User[] users = getUsersList();

        User user = users[0];
        if (user.getAge() < 18) {
            throw new AccessDeniedException("age less 18");
        } else {
            getUsers();
        }
        return user;
    }

в сообщение об ошибке пишут причину ошибки. И в данном случае это возраст. На самом деле нужно свой эксепшин делать на такое

Answer 2

[Денис Загаевский]

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

Метод валидации должен возвращать код ошибки валидации (в простейшем случае - true/false, в более сложном - enum { OK, AGE_DENIED, PERMISSION_DENIED, ....}, в сложном - объект с дополнительными полями(в джаву как раз sealed class'ы завезли).
Выше по стеку(в твоём случае - в main) должна быть проверка и уже реакция, принты и тд. Сам метод валидации не должен ничего печатать.

Не надо хранить ничего вот в статике. Это дурной тон, и неудобно. Максимально быстро уходи от статики как-то так:

class App {
    public static void main (args){
         new App(args).run();
    }

    private  void run(args) { 
          //.... твой код
    }
}

Comments

[Amir Kenesbay]

Ок я понял, просто это учебная задачка и там над готовыми статистическими методами и над готовыми заранее созданными исключениями нужно было это все сделать. А так спасибо за совет, ты там использовал enum? А как с помощью него реализовать?

[Денис Загаевский]

Amir Kenesbay, меняешь возвращаемый тип на енам, возвращаешь нужное значение, и потом проверяешь,что вернулось.

[Amir Kenesbay]

Денис Загаевский, окей попробую сделать