变量分配问题

时间:2014-12-17 09:06:12

标签: java

我正在创建一个项目,允许用户通过银行执行您通常可以执行的操作(创建,删除帐户,取款,存款,打印交易)。

我的问题是打印交易。我可以打印一些东西,但错误的值正在出现。例如,当您创建银行帐户时,您需要输入一个(5.00)的期初余额。当我打印交易时,什么都没有出现。因此,我将钱存入同一帐户(10.00),当我打印时,(5.00)从之前出现,(10.00)出现在transactionDate中。但是,当我查看帐户余额时,这一切都会增加到它显示的内容。

有人可以帮我解决问题并解决问题吗?

package mainsample;
import java.util.*;

public class Main {


    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);

        BankProcess bankProcess = new BankProcess();
        TransactionProcess transactionProcess = new TransactionProcess();


    Bank bank = new Bank();
    BankAccount bankAccount = new BankAccount();

    int input;
    int selection;

    while(true) {
      System.out.println(""); 
      System.out.println("######## MENU ########");
      System.out.println("[1] Create an account");
      System.out.println("[2] Print all existing accounts");
      System.out.println("[3] Delete an account");
      System.out.println("[4] Deposit");
      System.out.println("[5] Withdraw");
      System.out.println("[6] Print transactions");
      System.out.println("[0] Exit");      
      System.out.println("######################");
      System.out.println("Please choose one of the following: ");
      selection = scan.nextInt();

      switch (selection) {

        case 0:
          System.out.println("Exit Successful");
          System.exit(0);
            break;

        case 1:
          System.out.println("'[1] Create an account' has been selected.");
          System.out.print("Account Id: ");
          int accountId = scan.nextInt();
          scan.nextLine();

          System.out.print("Holder Name: ");
          String holderName = scan.nextLine();

          System.out.print("Holder Address: ");
          String holderAddress = scan.nextLine();

          System.out.print("Opening Balance: ");
          double openingBalance = scan.nextDouble();

          System.out.print("Open Date: ");

          String openDate = scan.next();

          bankAccount = new BankAccount(accountId, holderName, openingBalance, holderAddress, openDate);
          bank.setAccounts(bankProcess.openNewAccount(bank.getAccounts(), bankAccount));

          System.out.println("Successfully Added.");
          break;      

        case 2:
          System.out.println("'[2] Display all existing accounts' has been selected");
          System.out.println("-----------------------------------------------------");
            bank.getAccounts().forEach((i,b)->System.out.println(b));
          System.out.println("-----------------------------------------------------");
            break;

        case 3:
            System.out.println("[3] Delete an account has been selected");
            System.out.println("Enter the account ID: ");
            int accountNo = scan.nextInt();
            bankAccount = bank.getAccount(accountNo); // get bankAccount from account id
            bank.removeAccounts(bankProcess.removeAccount(bank.getAccounts(), bankAccount));
            System.out.println("Account has been deleted.");
      break;

        case 4:
            System.out.println("[4] Deposit has been selected");
            System.out.println("Enter account ID: ");
            int accountNumber =  scan.nextInt(); 

            System.out.println("Enter deposit amount: ");
            double depositAmount = scan.nextDouble();           

            transactionProcess.deposit(bankAccount, depositAmount);


            break;

        case 5:
            System.out.println("[5] Withdraw has been selected");
            System.out.println("Enter account ID: ");
            int accountNu =  scan.nextInt(); 

            System.out.println("Enter withdraw amount: ");
            double withdrawAmount = scan.nextDouble();           

            transactionProcess.withdraw (bankAccount, withdrawAmount);           

            break;           


        case 6:
            System.out.println("[6] Print Transaction has been selected");
            System.out.println("Enter account ID: ");
            int accountN = scan.nextInt();
            bankAccount = bank.getAccount(accountN);
            for (Transaction transaction: bankAccount.getTransactions()) {
            // print transaction information ...
            System.out.println(transaction.toString());

}



            break;






        default:
          System.out.println("Your choice was not valid!");

      }

    }
  }
}
package mainsample;

/**
 *
 * @author Khalid
 */
public class Transaction {


  private String transactionType;
  private double transactionAmount;
  private int transactionDate;


  public Transaction() {}

  public Transaction( String transactionType, double transactionAmount, int transactionDate) {

    this.transactionType = transactionType;
    this.transactionAmount = transactionAmount;
    this.transactionDate = transactionDate;

  }

  public int getTransactionDate() {
      return transactionDate;
  }

  public void setTransactionDate (int transactionDate) {
      this.transactionDate = transactionDate;
  }


  public String getTransactionType() {
    return transactionType;
  }

  public void setTransactionType(String transactionType) {
    this.transactionType = transactionType;
  }

  public double getTransactionAmount() {
    return transactionAmount;
  }

  public void setTransactionAmount(double transactionAmount) {
    this.transactionAmount = transactionAmount;
  }

  //Override the toString() method of String ? 
  public String toString() {
    return "\nTransaction Amount : "+ this.transactionAmount +
           "\nTransaction Type : " + this.transactionType +
           "\nTransaction Date:  " + this.transactionDate;

  }

}
package mainsample;
/**
 *
 * @author Khalid
 */
public class TransactionProcess {

    public void deposit(BankAccount bankAccount, double depositAmount) {
    //Get the CurrentBalance
    double currentBalance = bankAccount.getCurrentBalance();

    //First Argument : set the Id of transaction
    //Second Argument : set the Type of Transaction
    //Third Argument : set the TransactionAmount 
    //Fourth Argument : set the Balance Before the transaction (for record purposes)
    Transaction transaction = new Transaction("Deposit", currentBalance, (int) depositAmount);

    if (depositAmount <= 0) {
      System.out.println("Amount to be deposited should be positive");
    } else {
      //Set the updated or transacted balance of bankAccount.
      bankAccount.setCurrentBalance(currentBalance + depositAmount);
      //then set the MoneyAfterTransaction

      bankAccount.addTransaction(transaction);    // adds a transaction to the bank account
      System.out.println(depositAmount + " has been deposited.");
    }

  }

  // Explanation same as above
  public void withdraw(BankAccount bankAccount, double withdrawAmount) {
    double currentBalance = bankAccount.getCurrentBalance();
    Transaction transaction = new Transaction("Withdraw", currentBalance, (int) withdrawAmount);

    if (withdrawAmount <= 0) {
      System.out.println("Amount to be withdrawn should be positive");
    } else {
      if (currentBalance < withdrawAmount) {
        System.out.println("Insufficient balance");
      } else {
        bankAccount.setCurrentBalance(currentBalance - withdrawAmount);        
        bankAccount.addTransaction(transaction);    // adds a transaction to the bank account
        System.out.println(withdrawAmount + " has been withdrawed,");
      }
    }
  }
}

/*
 * To change this license header, choose License Headers in Project Properties.
 * To change this template file, choose Tools | Templates
 * and open the template in the editor.
 */
package mainsample;
import java.util.*;
/**
 *
 * @author Khalid
 */
public class BankAccount {
    private int accountId;
    private String holderName;
    private String holderAddress;
    private String openDate;
    private double currentBalance;

    private List<Transaction> transactions = new ArrayList<Transaction>();

    //Provide Blank Constructor
    public BankAccount(){}

    //Constructor with an arguments.
    public BankAccount(int accountNum, String holderNam,double currentBalance, String holderAdd,String openDate) {
            this.accountId = accountNum;
            this.holderName = holderNam;
            this.holderAddress = holderAdd;
            this.openDate = openDate;
            this.currentBalance = currentBalance;
    }

    // Always Provide Setter and Getters
    public int getAccountId() {
        return accountId;
    }
    public void setAccountId(int accountId) {
        
         this.accountId = accountId;                  
    }
    public String getHolderName() {
        return holderName;
    }
    public void setHolderName(String holderName) {
        this.holderName = holderName;
    }
    public String getHolderAddress() {
        return holderAddress;
    }
    public void setHolderAddress(String holderAddress) {
        this.holderAddress = holderAddress;
    }
    public String getOpenDate() {
        return openDate;
    }
    public void setOpenDate(String openDate) {
        this.openDate = openDate;
    }
    
    public double getCurrentBalance() {
        return currentBalance;
    }
    public void setCurrentBalance(double currentBalance) {
        this.currentBalance = currentBalance;
    }

    public List<Transaction> getTransactions() {
        return transactions;
    }

    public void setTransactions(List<Transaction> transactions) {
        this.transactions = transactions;
    }
    
    public void addTransaction(Transaction transaction){
      if(transactions.size() >= 6){  // test if the list has 6 or more transactions saved 
      transactions.remove(0);     // if so, then remove the first (it's the oldest)
     }
     transactions.add(transaction); // the new transaction is always added, no matter how many other transactions there are already in the list
     }

    public String toString(){
        return "\nAccount number: " + accountId + 
                "\nHolder's name: " + holderName + 
                "\nHolder's address: " + holderAddress + 
                "\nOpen Date: " + openDate + 
                "\nCurrent balance: " + currentBalance;
    }

   
}

package mainsample;
import java.util.*;
/**
 *
 * @author Khalid
 */
public class Bank {
    
    private TreeMap<Integer,BankAccount> bankAccounts = new TreeMap<Integer,BankAccount>();

 
    
    public TreeMap<Integer, BankAccount> getAccounts() {
        return bankAccounts;
    }
    public void setAccounts(TreeMap<Integer, BankAccount> accounts) {
        this.bankAccounts = accounts;
    }
    
     public BankAccount getAccount(Integer accountNumber){
     return bankAccounts.get(accountNumber);
     }

    
    public void removeAccounts(TreeMap<Integer, BankAccount> accounts) {
        this.bankAccounts = accounts;
    }

   
}

package mainsample;
import java.util.*;
/**
 *
 * @author Khalid
 */
public class BankProcess {
    // return the  Updated list of BankAccounts
    public TreeMap<Integer,BankAccount> openNewAccount(TreeMap<Integer,BankAccount> bankAccounts,BankAccount bankAccount) {
        //Get the List of existing bank Accounts then add the new BankAccount to it.
        bankAccounts.put(bankAccount.getAccountId(), bankAccount);        
        return bankAccounts;
    }
    
    public TreeMap<Integer,BankAccount> removeAccount(TreeMap<Integer,BankAccount> bankAccounts,BankAccount bankAccount) {
      bankAccounts.remove(bankAccount.getAccountId(), bankAccount);
      return bankAccounts;  
        
    }
    

    

}

2 个答案:

答案 0 :(得分:0)

首先你在Transaction constrctor

中犯了一个错误
public Transaction( String transactionType, double transactionAmount, int transactionDate) {

        this.transactionType = transactionType;
        this.transactionAmount = transactionAmount;
        this.transactionDate = transactionDate;

    }

但你这样分配

Transaction transaction = new Transaction("Deposit", currentBalance, (int) depositAmount);

你的论点顺序在这里是完全错误的。你为什么要分配存款金额而不是日期。它应该像

Transaction transaction = new Transaction("Deposit", (int) depositAmount,date );

此外,当您创建新帐户时,您还没有将该金额添加到交易中。您正在使用此代码

创建新帐户
bankAccount = new BankAccount(accountId, holderName, openingBalance, holderAddress, openDate);
bank.setAccounts(bankProcess.openNewAccount(bank.getAccounts(), bankAccount));

但您尚未将此初始金额添加到交易中。它应该包含在bankAccount

的构造函数中
Transaction transaction = new Transaction("Deposit", 0, (int) depositAmount);

if (depositAmount <= 0) {
  System.out.println("Amount to be deposited should be positive");
} else {
  //Set the updated or transacted balance of bankAccount.
  bankAccount.setCurrentBalance(currentBalance + depositAmount);
  //then set the MoneyAfterTransaction

  bankAccount.addTransaction(transaction);    // adds a transaction to the bank account
  System.out.println(depositAmount + " has been deposited.");
}

现在我得到了正确的输出。当您创建帐户时,您询问了日期。当你存款时,你没有问过约会。因此,您无法指定存款或取款的交易日期

Transaction Amount : 5.0 Transaction Type : Deposit Transaction Date:  4

Transaction Amount : 10.0 Transaction Type : Deposit Transaction Date: 10

答案 1 :(得分:-1)

你没有打电话:

bankAccount = bank.getAccount(accountN);

将您的银行帐户指向第4点(存款)和5(撤回)。

FIX:

   case 4:
        System.out.println("[4] Deposit has been selected");
        System.out.println("Enter account ID: ");
        int accountNumber =  scan.nextInt(); 

        System.out.println("Enter deposit amount: ");
        double depositAmount = scan.nextDouble();           
  --->  bankAccount = bank.getAccount(accountN);
        transactionProcess.deposit(bankAccount, depositAmount);


        break;

    case 5:
        System.out.println("[5] Withdraw has been selected");
        System.out.println("Enter account ID: ");
        int accountNu =  scan.nextInt(); 

        System.out.println("Enter withdraw amount: ");
        double withdrawAmount = scan.nextDouble();           
  --->  bankAccount = bank.getAccount(accountN);
        transactionProcess.withdraw (bankAccount, withdrawAmount);           

        break;