Javaの銀行口座プログラムのユーザーの入金履歴と出金履歴の表示の仕方について

Question

前提・実現したいこと

Javaを用いて簡単な銀行口座のプログラムを作っています。ユーザーがDepositを選択するとユーザーの好きな額を入金し、withdrawalを選択すると好きな額を引き下ろし、output balanceを選択するとその時の口座に入っているお金を表示し、Output all depositsを選択するとその時までの入金履歴を表示し、Output all withdrawalsを選択するとその時までの出金履歴を表示します。現在、入金履歴と出金履歴の仕方がよく分からなくこちらで質問させて頂きました。
下記のコードでdeposit.lengthとwithdrawal.lengthが０だと思うので入金履歴と出金履歴が表示されないのだと思います。そしてcountDpとcountWdのカウントの仕方が間違えていると思います。正しくするにはどのようにすれば良いでしょうか？教えて頂きたいです。
このプログラムの問題文にarrayを使えと書いてあるのでarrayを使う前提でお願いします。

該当のソースコード

import java.util.Scanner;
import java.io.*;
import java.io.FileNotFoundException;
import java.io.PrintWriter;

class Account {
    private float balance;

    private int countDp;
    private int countWd;

    float[] deposit = new float[10];
    float[] withdrawal = new float[10];

    Scanner input = new Scanner(System.in);

    public Account() {
        balance = 0;
        countDp = 0;
        countWd = 0;
    }

    private float[] enhanceArray(float[] array) {
        float[] newArray = new float[array.length + 10];
        for (int i = 0; i < array.length; i++) {
            newArray[i] = array[i];
        }
        return newArray;
    }


    public float depositing() {
        System.out.println("Enter the amount you would like to deposit: ");

        float depositAmount = input.nextFloat();

        if (countDp == deposit.length) {
            deposit = enhanceArray(deposit);
        }
        deposit[countDp] = depositAmount;
        balance += depositAmount;

        countDp += 1;
        return balance;
    }

    public float withdrawing() {
        System.out.println("Enter the amount you would like to withdraw: ");

        float withdrawalAmount = input.nextFloat();

        if (balance >= withdrawalAmount) {
            if (countWd == deposit.length) {
                withdrawal = enhanceArray(withdrawal);
            }
            withdrawal[countDp] = withdrawalAmount;
            balance -= withdrawalAmount;
            countWd += 1;
        } else {
            System.out.println("Less balance, transaction failed.");
        }

        return balance;
    }

    public float balanceOfAccount() {
        return balance;
    }
                         
    public void outputAllDeposits() {

        System.out.println("Outputting all your deposits:");

        for (int i = 0; i < countDp; i++) {
            deposit[i] = input.nextFloat();

            if(deposit[i] >= 0.0f) {
                System.out.println("$" + deposit[i]);
            }
        }
    }

    public void outputAllWithdraws() {
        System.out.println("Outputting all your withdrawals:");

        for (int i = 0; i < countWd; i++) {
            withdrawal[i] = input.nextFloat();

            if(withdrawal[i] >= 0.0f) {
                System.out.println("$" + withdrawal[i]);
            }
        }
    }

    public void outputToTextFile() {
        try {
            PrintWriter pw = new PrintWriter("bankAcount.txt");

            pw.println("Your balance: " + balanceOfAccount());

            pw.println("Outputting all your withdrawals:");

            for (int i = 0; i < withdrawal.length; i++) {
                withdrawal[i] = input.nextFloat();

                if(withdrawal[i] >= 0.0f) {
                    pw.println(withdrawal[i]);
                }
            }

            pw.println("Outputting all your deposits:");
            
            for (int i = 0; i < deposit.length; i++) {
                deposit[i] = input.nextFloat();

                if(deposit[i] >= 0.0f) {
                    pw.println(deposit[i]);
                }
            }

            pw.close();
        } catch (IOException ex) {
            ex.printStackTrace();
        }
    }
}

public class Main {

    public static void main(String[] args) throws Exception{
        int choice = 0;
        Account account = new Account();

        Scanner in = new Scanner(System.in);


        do {
            System.out.println("**************************************\n" +
                    "* Bank Account Program:              *\n" +
                    "*    Enter # to run program or Quit  *\n" +
                    "*    1)  Make a Deposit              *\n" +
                    "*    2)  Make a Withdrawal           *\n" +
                    "*    3) Output balance               *\n" +
                    "*    4) Output all deposits          *\n" +
                    "*    5) Output all withdrawals       *\n" +
                    "*    6) Quit                         *\n" +
                    "**************************************\n");

            choice = in.nextInt();

            switch (choice) {
                case 1:
                    account.depositing();
                    break;
                case 2:
                    account.withdrawing();
                    break;
                case 3:
                    System.out.println("Your balance is " + account.balanceOfAccount());

                    break;
                case 4:

                    account.outputAllDeposits();
                    break;
                case 5:
                    account.outputAllWithdraws();
                    break;

                case 6:
                    System.out.println("Thank you for using this Bank.");
                    break;

                    default:
                        break;
            }
        } while(choice != 6);

        account.outputToTextFile();
    }
}

Answer 1

履歴を出力する以前の話で、そもそも履歴の記録ができていません。
履歴を記録すると思われる配列は長さが0のままノータッチなので、何も記録できません。
なおカウント自体は間違っていないのですが、それも使われないままになっています。

配列を使って履歴を記録していくことになるのですが、配列はListのように長さを自在に変えることができません。次のように対処します。

• 適当な長さの配列を作っておき、そこに履歴を記録していく
• カウンターは、記録数の管理及び記録する位置の管理に使う
• 長さが足りなくなったら長さを適当に大きくした配列を作り、内容をコピーしたものを新たに使う

とりあえず配列は次のように長さ10で初期化しておきます。

    float[] deposit = new float[10];
    float[] withdrawal = new float[10];

足りなくなった場合に使う、長さを10足してコピーするメソッドも用意しておきます。

private float[] enhanceArray(float[] array) {
    float[] newArray = new float[array.length + 10];
    for (int i = 0; i < array.length; i++) {
        newArray[i] = array[i];
    }
    return newArray;
}

それで、例えば入金のメソッドは次のようにします。

public float depositing() {
    System.out.println("Enter the amount you would like to deposit: ");
    float depositAmount = input.nextFloat();

    if (countDp == deposit.length) {
        deposit = enhanceArray(deposit);
    }
    deposit[countDp] = depositAmount;
    balance += depositAmount;

    countDp += 1;
    return balance;
}

出力時は、配列の長さまで出力しようとすると、履歴の入っていない要素まで使うことになるので、入金だったらcountDp未満までforでまわせばいいです。

Comments

[amareno]

詳しい回答ありがとうございます。swordoneさんの回答を参考にコードを書いてみたのですがまだ入金履歴と出金履歴が表示されず困っています

public void outputAllDeposits() {

System.out.println("Outputting all your deposits:");

for (int i = 0; i < countDp; i++) {
deposit[i] = input.nextFloat();

if(deposit[i] >= 0.0f) {
System.out.println("$" + deposit[i]);
}
}
}

上記のコードのdeposit[i] = input.nextFloat();の部分、input.nextFloat()ではなく別の何かにするべきでしょうか？

[swordone]

むしろそれいらないのでは？

[amareno]

すいません、気がつきませんでした。ありがとうございます。