Я создал приложение о банкомате с размером учетной записи массива 10, но по какой-то странной причине вход принимает только 0 (первый идентификатор), но не другие 9 идентификаторов для других идентификаторов, он всегда показывает ошибку недопустимого идентификатора сообщение. Я часами искал ошибку.

Те, которые связаны с проверкой идентификаторов, будут этими двумя;

public static boolean hasID(Account acc[], int id){ //this is to check whether the id exist or not.
    for(int i=0; i<acc.length; i++){
        if(id == acc[i].getID()){
            return true;
        }
    }
    return false;
}

public static int gID(Account[] acc){  //this is the login method to initiate the hasID method.
    Scanner sc = new Scanner(System.in);
    int id=0;
    boolean valid = false;
    while(!valid){
        System.out.println("Enter ID: ");
        id = sc.nextInt();

        if(!hasID(acc, id)){
            System.out.println("YOUR ID IS INVALID.");
        } else{
            valid = true;
        }
    }
    return id;
}

public Account(int mID, double mBalance/*, double mInterestRate*/){
    this.id = getID();
    this.balance = getBalance();
    //this.interestRate = getInterestRate();
}

public static Account getAccount(Account acc[], int id){
    for(int i = 0; i<acc.length; i++){
        if(id == acc[i].getID()){
            return acc[i];
        }
    }
    return null;
}

И вот мой основной:

public static void main(String[] args) {

    Scanner sc = new Scanner(System.in);
    Account acc[] = new Account[10];
    for(int i = 0; i<acc.length; i++){
        acc[i] = new Account(i, 100.0);
    }

    int a = 0;
    int id = gID(acc);

    while(a != 4){
        Account ac = getAccount(acc,id);
        System.out.println("1: Check Balance");
        System.out.println("2: Withdraw");
        System.out.println("3: Deposit");
        System.out.println("4: Exit");

        System.out.println("Enter your choice: ");
        a = sc.nextInt();
        switch(a){
        case 1:
            System.out.println("Your Balance is: RM " + ac.getBalance());
            break;

        case 2:
            System.out.println("Amount to withdraw: RM ");
            ac.withdraw(sc.nextDouble());
            break;

        case 3:
            System.out.println("Amount to deposit: RM ");
            ac.deposit(sc.nextDouble());
            break;

        case 4:
            id = gID(acc);
            a = 0;
            break;

        default:
            System.out.println("Invalid input!");
        }
    }

}

Изменить: конструктор учетной записи, insterestrate не используется. Установщик и получатель для идентификатора и баланса:

public int getID(){
    return id;
}

public void setID(int mID){
    id = mID;
}

public double getBalance(){
    return balance;
}

public void setBalance(double mBalance){
    balance = mBalance;
}

Спасибо за ваше время.

1
HeatKai C 15 Ноя 2017 в 21:04

1 ответ

Лучший ответ
public Account(int mID, double mBalance/*, double mInterestRate*/){
    this.id = getID();
    this.balance = getBalance();
    //this.interestRate = getInterestRate();
}

Разве вы не должны использовать

 public Account(int mID, double mBalance/*, double mInterestRate*/){
    this.id = mID;
    this.balance = mBalance;
    //this.interestRate = getInterestRate();
 } 

Геттеры и сеттеры являются частью ЗАКРЫТИЯ ПОЛЕЙ, но для лучшего взаимодействия с пользователем я бы предложил вам следующий код

public Account(/*you dont need a parameter*/){
   this.id = setIdByScannedValue();
   this.balance = setBalanceByScannedValue();
   //this.interestRate = setInterestRateByScannedValue();
}

private void setIdByScannedValue(){
    Scanner sc = new Scanner(System.in);
    boolean provided = false;
    while(!provided)
    try{
        System.out.print("provide id: ");
        this.id = sc.nextInt();
    }catch(NumberFormatException e){
        System.out.println("you must provide an integer id format !!!");
    }finally{
        provided = true;
    }
}

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

1
Mohammed Housseyn Taleb 15 Ноя 2017 в 18:52