использование функции получения и установки для учетной записи? - PullRequest
Новая
       26

использование функции получения и установки для учетной записи?

0 голосов
/ 08 марта 2019

Я делаю программу, которая включает создание учетной записи, которую мне нужно создать, чтобы она сканировала определенные данные для выполнения назначенной команды.подходит ли для него функция получения и установки? 

public class Account {

    //data
    private int userId;
    private String password;
    private char type;

    public Account(int userId, String password, char type) {
        this.userId = userId;
        this.password = password;
        this.type = type;
    }

    public int getUserId() {
        return userId;
    }

    public void setUserId(int userId) {
        this.userId = userId;
    }

    public String getPassword() {
        return password;
    }

    public void setPassword(String password) {
        this.password = password;
    }

    public char getType() {
        return type;
    }

    public void setType(char type) {
        this.type = type;
    }


    //methods
    public boolean verifyLogin(int usrid , String pass)
    {
      if((usrid == userId) & (pass == password)){
          return true;
      }  
      else{
        return false;
    }
}

Ответы [ 2 ]

2 голосов
/ 08 марта 2019

Ваши геттеры и сеттеры отлично выглядят для доступа к данным этого класса.

С чем нужно быть очень осторожным, так это с тем, как вы проверяете правильность пароля.

В вашей реализациивы используете pass == password для сравнения двух строк.Это НЕ правильно, и вам лучше использовать pass.equals(password).

1 голос
/ 08 марта 2019

Выглядит нормально, но вам нужно переосмыслить, если для некоторых значений нужны сеттеры.В примере это не очень распространенный случай, когда UserID как-то изменится.Если вы хотите сохранить его постоянным, сеттер не нужен.Установите его один раз в конструкторе.

Кроме того, вы можете посмотреть аннотацию Lombok Project и @Getter & @Setter.Это минимизирует ваш код до 3 строк.

...