Arrays und Bankkonten in Java

Ich bin versucht, zu schreiben eine einfache Bank-Account-Management-Programm, das das folgende tut:

Erstellt ein neues Konto mit Kontonummer und Balance genommen Nutzers gespeichert und in ein array
Wählt ein Konto (aus dem array)
Löscht die ausgewählten Kontos
Zurückziehen und mit der Einzahlung auf ein Konto ausgewählt haben.

Problem: ich verstehe nicht, was meine Fehler sind.

Ich habe versucht, mit verschiedenen Arten von arrays, die für die Kontonummer und balance zu speichern, aber ich habe nicht keine Antwort noch. Ich Suche im web und Stackoverflow für Referenzen, Dokumentationen, einfache Beispiele, aber konnten nicht finden, alle (die, die ich gefunden habe, verwenden einige Befehle und Dinge, die ich noch nicht gelernt, aber ich kann verstehen, wie Sie funktionieren). Ich bin ein Anfänger, ich bin immer noch lernen, und würden uns freuen, einige Ratschläge. Danke!

//Bank account class
public class account {
private int ANumber;
private double balance;

public account(double initialBalance, int accno) {
    balance = initialBalance;
    ANumber = accno;
}

public void deposit (double u_amm){
    balance += u_amm;
}

public double withdraw(double amount) {
    balance -= amount;
    return amount;
}

public double getBalance() {
    return balance;
}
public int getAccount(){
    return ANumber;
}
}

- Und das ist die Haupt-Klasse

 import java.util.Scanner;
 //main class
 public class bankmain {
 public static void main(String[] args) {
    Scanner scan = new Scanner(System.in);
    //Menu starts from here
    Scanner input = new Scanner(System.in);
    System.out.println("Enter the option for the operation you need:");
    System.out.println("****************************************************");
    System.out.println("[ Options: ne - New Account de - Delete Account ]");
    System.out.println("[       dp - Deposit    wi - Withdraw      ]");
    System.out.println("[           se - Select Account ex - Quit      ]");
    System.out.println("****************************************************");
    System.out.print("> ");  //indicator for user input
    String choice = input.next();
    //Options
    while(true){
  if(choice == "ne"){
    int nacc;
    int bal;
    int [][]array = new int[nacc][bal];   //Array for account and balance
    System.out.print("Insert account number: ");
    nacc =input.nextInt(); //-- Input nr for array insertion
    System.out.print("Enter initial balance: ");
    bal=input.nextInt(); //-- Input nr for array insertion
    System.out.println("Current account: " + nacc + " " +  "Balance " + bal);
    break;
  }
  //account selection      
  if(choice.equals("se")){
    System.out.println("Enter number of account to be selected: ");
    //user input for account nr from array
    System.out.println("Account closed.");
  }     
  //close account
  if(choice.equals("de")){
    //array selected for closing
    System.out.println("Account closed.");
  }
  //deposit
  if(choice.equals("dp")){
    System.out.print("Enter amount to deposit:  ");
    double amount = scan.nextDouble();
    if(amount <= 0){
        System.out.println("You must deposit an amount greater than 0.");
    } else {
        System.out.println("You have deposited " + (amount + account.getBalance()));
    }
  }
  //withdrawal     
  if(choice.equals("wi")){
    System.out.print("Enter amount to be withdrawn: ");
    double amount = scan.nextDouble();
        if (amount > account.balance()){ 
            System.out.println("You can't withdraw that amount!");
   } else if (amount <= account.balance()) {
    account.withdraw(amount);
    System.out.println("NewBalance = " + account.getBalance());
   }
   }
//quit 
if(choice == "ex"){
    System.exit(0);
        } 
    }   //end of menu loop
 }//end of main
 } //end of class

Bekommst du eine Fehlermeldung? Was passiert und was würden Sie erwarten?
Sie müssen erstellen Sie ein array von 'Konto' Objekte und weisen Sie jedem Konto, um neue Kunden und daher die erforderlichen Methoden für das jeweilige Konto. Auch "switch" - Anweisung anstelle verschachtelter wenn-dann-sonst-Anweisung. Sie sind immer noch das mischen der prozeduralen Programmierung mit der Objekt-orientierten Programmierung.
Behalten Sie im Hinterkopf: Was würde passieren, wenn jemand sich zurückgezogen "-100.00"? Sein Konto aufladen! Dasselbe geht den anderen Weg mit Einzahlung.
Machen, die Klassennamen mit Großbuchstaben beginnen und camel-case z.B. "Konto" oder "BankMain". Sind Sie sicher, dass Sie verwenden möchten if(choice == "ex") diese tests für Referenz-Gleichheit, nicht Wert. Was ist account.balance() ich sehe nicht ein balance-Methode, meinst du getBalance()? Wenn Sie Bearbeiten möchten, und fügen Sie die syntax-Fehler, Sie werden immer die Leute werden eher geneigt sein, zu helfen.

InformationsquelleAutor Azdamus | 2014-08-16

Dein code war falsch auf vielen Ebenen - zunächst versuchen, die Arbeit auf Formatierung und Namensgebung, so dass Sie nicht lernen, bad habbits. Verwenden Sie keine kleinen Briefe für die Benennung von Klassen, versuchen, ganze Wörter, um zu beschreiben, Variablen und Felder, ähnlich wie in Account.class Beispiel:

public class Account  {
    private Integer accountNumber;
    private Double balance;

    public Account(final Integer accountNumber, final Double initialBalance) {
        this.accountNumber = accountNumber;
        balance = initialBalance;
    }

    public Double deposit (double depositAmmount) {
        balance += depositAmmount;
        return balance;
    }

    public Double withdraw(double withdrawAmmount) {
        balance -= withdrawAmmount;
        return balance;
    }

    public Double getBalance() {
        return balance;
    }

    public Integer getAccountNumber() {
        return accountNumber;
    }
}

Auch versuchen die gleiche Formatierung(zB. Leerzeichen nach der Klammer), die in alle code, so dass es einfacher für Sie zu Lesen.

Nun - was falsch war in Ihrer app:

Definieren, die richtige Halterung für Ihre Konten, d.h. HashMap gespeichert werden die Informationen über alle Konten und wird in der Lage sein, Sie zu finden, indem Sie Kontonummer.

HashMap<Integer, Account> accountMap = new HashMap<Integer, Account>();

Diese sollte sich in Ihrem while-Schleife, wie Sie möchten, dass Ihre Benutzer mehrere Aufgaben. Sie konnte ommit die println ' s, aber dann müsste der Benutzer zurück zu der oben auf dem Bildschirm, um die Optionen zu sehen.

System.out.println("Enter the option for the operation you need:");
System.out.println("****************************************************");
System.out.println("[ Options: ne - New Account de - Delete Account ]");
System.out.println("[       dp - Deposit    wi - Withdraw      ]");
System.out.println("[           se - Select Account ex - Quit      ]");
System.out.println("****************************************************");
System.out.print("> ");  //indicator for user input
String choice = input.nextLine();   
System.out.println("Your choice: " + choice);

Sollte man nicht vergleichen von Strings mit '= = ' - operator, da es möglicherweise keine Rückkehr erwarteten Wert. Werfen Sie einen Blick auf: Wie Vergleiche ich strings in Java? oder Was ist der Unterschied zwischen == vs equals() in Java?. Für safty verwenden Sie Objekte equals statt, dh:
```
choice.equals("ne")
```

Gab es keine Stelle, wo du den account erstellt:

Account newAccount = new Account(newAccountNumber, initialBalance);

Sie nicht verwenden Sie if-else, nur if. Es sollte Aussehen, wie das (warum um zu überprüfen, wenn die Wahl ist, 'wi', wenn es bereits in 'ne'):
```
if(choice.equals("ne")) {
    //doStuff
} else if choice.equals("se") {
    //doStuff
} //and so on.
```
Wenn Ihr mit der Java-version >= 7 dann anstelle von if-else können Sie wechseln, um Zeichenketten miteinander zu vergleichen.

Gab es keine Benachrichtigung über die falsche option:

} else {
    System.out.println("Wrong option chosen.");
}

Sie nicht schließen Sie Ihren Scanner-Objekt. Dies spielt wirklich keine Rolle hier, da es in der main-Methode und wird in der Nähe, aber es ist eine Frage des guten habbits, um immer in der Nähe Ihrer streams, Daten, Quellen, etc wenn Sie fertig sind es zu benutzen:
```
input.close(); 
```

Also den ganzen code kann wie folgt Aussehen:

import java.util.HashMap;
import java.util.Scanner;
//main class
public class BankAccount {

    public static void main(String[] args) {

        Scanner input = new Scanner(System.in);
        HashMap<Integer, Account> accountMap = new HashMap<Integer, Account>(); 

         //Options
        while(true) {

            System.out.println("Enter the option for the operation you need:");
            System.out.println("****************************************************");
            System.out.println("[ Options: ne - New Account de - Delete Account ]");
            System.out.println("[       dp - Deposit    wi - Withdraw      ]");
            System.out.println("[           se - Select Account ex - Quit      ]");
            System.out.println("****************************************************");
            System.out.print("> ");  //indicator for user input

            String choice = input.next();   
            System.out.println("Your choice: " + choice);

            if(choice.equals("ne")) {
                Integer newAccountNumber;
                Double initialBalance;
                Account newAccount;

                //Array for account and balance
                System.out.print("Insert account number: ");
                newAccountNumber = input.nextInt(); //-- Input nr for array insertion
                System.out.print("Enter initial balance: ");
                initialBalance=input.nextDouble(); //-- Input nr for array insertion
                newAccount = new Account(newAccountNumber, initialBalance);
                accountMap.put(newAccountNumber, newAccount);
                System.out.println("New Account " + newAccountNumber + " created with balance: " + initialBalance);
            }
            //select account
            else if(choice.equals("se")) {
                System.out.println("Enter number of account to be selected: ");
                Integer accountToGetNumber = input.nextInt();
                Account returnedAccount = accountMap.get(accountToGetNumber);
                if (returnedAccount != null)
                {
                    System.out.println("Account open. Current balance: " + returnedAccount.getBalance());
                }
                else
                {
                    //user input for account nr from array
                    System.out.println("Account does not exist.");
                }
            }
            //close account
            else if(choice.equals("de"))
            {
                System.out.println("Enter number of account to be selected: ");
                Integer accountToDeleteNumber = input.nextInt();
                Account removedAccount = accountMap.remove(accountToDeleteNumber);
                if (removedAccount != null)
                {
                    System.out.println("Account " + removedAccount.getAccountNumber() + " has been closed with balance: " + removedAccount.getBalance());
                }
                else
                {
                    //user input for account nr from array
                    System.out.println("Account does not exist.");
                }
            }
            //deposit
            else if(choice.equals("dp")) {
                System.out.println("Enter number of account to deposit: ");
                Integer accountToDeposit = input.nextInt();
                System.out.print("Enter amount to deposit:  ");
                double amount = input.nextDouble();
                if(amount <= 0){
                    System.out.println("You must deposit an amount greater than 0.");
                } else {
                    accountMap.get(accountToDeposit).deposit(amount);
                    System.out.println("You have deposited " + (amount));
                    System.out.println("Current balance " + accountMap.get(accountToDeposit).getBalance());
                }
            }
            //withdrawal     
            else if(choice.equals("wi")) {
                System.out.println("Enter number of account to withdraw: ");
                Integer accountToWithdraw = input.nextInt();
                System.out.print("Enter amount to withdraw:  ");
                double amount = input.nextDouble();
                if(amount <= 0) {
                    System.out.println("You must deposit an amount greater than 0.");
                } else {
                    accountMap.get(accountToWithdraw).withdraw(amount);
                    System.out.println("You have deposited " + (amount));
                    System.out.println("Current balance " + accountMap.get(accountToWithdraw).getBalance());
                }
            }
            //quit 
            else if(choice.equals("ex")) {
                break;
            } else {
                System.out.println("Wrong option.");
            } //end of if
        } //end of loop

        input.close();
    } //end of main
 } //end of class

Gibt es immer noch viel zu verbessern, d.h. die input-Validierung - aber diese sollten für den Anfang.

Ich danke Ihnen sehr für Ihre Zeit zu erklären, meine Fehler und geben Sie mir diese tollen Ratschläge. Ich nicht wusste, dass ich hatte, um das Scanner-Eingabe, weil ich wusste, er schließt sich mit der main-Methode, wie du Sie erwähnt hast, aber ich werde von nun an. Ich versuche, mich zu verbessern und ich Schätze, was jeder sagte. Bezüglich der Formatierung, ich werde üben, sahen viele Versionen der Formatierung mit 2 oder 4 idents. Auch, vielen Dank für die Referenzen, die Sie haben eine viel bessere Beschreibung als das, was ich habe und gelernt.
Froh, dass ich helfen konnte.

InformationsquelleAutor Michał Schielmann

Du geschrieben hast den obigen code, aber es hat viele Fehler. Sie müssen lernen, die Die Grundlagen der Java-Programmierung.

Habe ich verändert Ihr Programm zu führen unterhalb Operationen :-

Neues Konto erstellen.
Wählen Sie vorhandenes Konto.
Einzahlung.
Zurückziehen Betrag.
Anzeigen-balance.
Konto löschen.
Beenden.

Account.java :

/**
 * This class performs bank operations
*/
public class Account {
    private int accNumber;
    private double balance;

    public Account(double initialBalance, int accNo) {
        balance = initialBalance;
        accNumber = accNo;
    }

    public void deposit(double amount) {
        balance += amount;
    }

    public double withdraw(double amount) {
        balance -= amount;
        return amount;
    }

    public double getBalance() {
        return balance;
    }

    public int getAccNumber() {
        return accNumber;
    }
}

BankMain.java :

public class BankMain {
    private static double amount;
    private static ArrayList<Account> accountList = new ArrayList<>();
    private static Account selectedAccount;
    private static boolean flag = false;

    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);
        //Menu starts from here
        Scanner input = new Scanner(System.in);
        System.out.println("Enter the option for the operation you need:");
        System.out
                    .println("****************************************************");
        System.out
                .println("[ Options: new - New Account del - Delete Account ]");
        System.out
                .println("[       dp - Deposit    wi - Withdraw  bal - Check balance    ]");
        System.out
                .println("[           se - Select Account exit - Quit      ]");
        System.out
                .println("****************************************************");
        Account account = null;
        while (true) {
            System.out.println("> "); //indicator for user input
            String choice = input.next();
            //Options
            switch (choice) {
                case "new":
                    //Create new account
                    int accNo = 0;
                    int bal = 0;
                    System.out.println("Enter account number : ");
                    accNo = input.nextInt();
                    System.out.println("Enter initial balance: ");
                    bal = input.nextInt();
                    System.out.println("Current account: " + accNo + " "
                        + "Balance " + bal);
                    account = new Account(bal, accNo);
                    accountList.add(account);
                    break;
                case "se":
                    //select account
                    System.out
                        .println("Enter account number for further operations : ");
                    int selectedAcc = scan.nextInt();
                    System.out.println("Selected account : " + selectedAcc);
                    for (Object object : accountList) {
                        selectedAccount = (Account) object;
                        if (selectedAccount.getAccNumber() == selectedAcc) {
                            flag = true;
                            break;
                        } else {
                            flag = false;
                        }
                    }
                    if (!flag) {
                        System.out.println("Account doesn't exists.");
                    }
                    if (accountList.size() == 0) {
                        System.out.println("Zero account exists.");
                    }
                    break;
                case "del":
                    //close account
                    System.out
                        .println("Enter account number for further operations : ");
                    int selectedAcc1 = scan.nextInt();
                    System.out.println("Selected account : " + selectedAcc1);
                    Iterator<Account> iterator = accountList.iterator();
                    while (iterator.hasNext()) {
                        selectedAccount = (Account) iterator.next();
                        if (selectedAccount.getAccNumber() == selectedAcc1) {
                            iterator.remove();
                            flag = true;
                            break;
                        }
                    }
                    if (!flag) {
                        System.out.println("Account doesn't exists.");
                    }
                    System.out.println("Account " + selectedAcc1 + " closed.");
                    break;
                case "dp":
                    //Deposit amount
                    System.out.println("Enter amount to deposit :  ");
                    amount = scan.nextDouble();
                    if (amount <= 0) {
                        System.out
                            .println("You must deposit an amount greater than 0.");
                    } else {
                        if (flag) {
                            selectedAccount.deposit(amount);
                            System.out.println("You have deposited " + amount
                                + ". Total balance : "
                                + (selectedAccount.getBalance()));
                        } else {
                            System.out.println("Please select account number.");
                        }
                    }
                    break;
                case "wi":
                    //Withdraw amount
                    System.out.println("Enter amount to be withdrawn: ");
                    amount = scan.nextDouble();
                    if (amount > account.getBalance() && amount <= 0) {
                        System.out.println("You can't withdraw that amount!");
                    } else if (amount <= selectedAccount.getBalance()) {
                        if (flag) {
                            selectedAccount.withdraw(amount);
                            System.out.println("You have withdraw : " + amount
                                + " NewBalance : "
                                + selectedAccount.getBalance());
                        } else {
                            System.out.println("Please select account number.");
                        }
                    }
                    break;
                case "bal":
                    //check balance in selected account
                    if (flag) {
                        System.out.println("Your current account balance : "
                            + selectedAccount.getBalance());
                    } else {
                        System.out.println("Please select account number.");
                    }
                    break;
                case "exit":
                default:
                    //quit
                    System.out.println("Thank You. Visit Again!");
                    flag = false;
                    input.close();
                    scan.close();
                    System.exit(0);
                    break;
            }
        } //end of menu loop
    }//end of main
} //end of class

Habe ich getestet, code & es funktioniert Prima.

Ausgabe :

Enter the option for the operation you need:
****************************************************
[ Options: new - New Account del - Delete Account ]
[       dp - Deposit    wi - Withdraw  bal - Check balance    ]
[           se - Select Account exit - Quit      ]
****************************************************
> new
Enter account number : 101
Enter initial balance: 10000
Current account: 101 Balance 10000
> new
Enter account number : 102
Enter initial balance: 25000
Current account: 102 Balance 25000
> se
Enter account number for further operations : 103
Selected account : 103
Account doesn't exists.
> se
Enter account number for further operations : 101
Selected account : 101
> bal
Your current account balance : 10000.0
> dp
Enter amount to deposit :  2500
You have deposited 2500.0. Total balance : 12500.0
> wi
Enter amount to be withdrawn: 500
You have withdraw : 500.0 NewBalance : 12000.0
> se
Enter account number for further operations : 102
Selected account : 102
> bal
Your current account balance : 25000.0
> del
Enter account number for further operations : 101
Selected account : 101
Account 101 closed.
> se
Enter account number for further operations : 101
Selected account : 101
Account doesn't exists.
> exit
Thank You. Visit Again!

Hi! Danke für die Hilfe, ja der code funktioniert Super, aber es ist das erste mal, dass ich diese Art von boolean verwendet für Bedingungen. Ich verstehe, was er tut und wie er es tut. Meine Frage ist jetzt: Ist es eine gute Praxis, diese Art von boolean in der Zukunft für die IF/ELSE-oder WHILE-Anweisungen?
Sie sind einer anderen Frage in dieser Frage, die hat keine Sorge mit der ersten Frage. Ich kann nur Antworten geben, aber es ist schlechte Angewohnheit, zu Fragen, eine andere Frage in bestehende Frage. Besser du stellst eine neue Frage, damit bekommst du mehr Antworten, die & mehr Erklärung & Klärung über die Verwendung von boolean in bedingten Anweisungen.

InformationsquelleAutor OO7

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.