値の受け渡しについて

Question

正の整数を入力し残高以内なら引き出すという動作をさせたいのですが、残高以内の数を入力しても残高を超えているという結果が出てしまいました。
文字数の制限上、引き出しの部分のみのコードになります。
他に必要なコード等があればご指摘お願いします。

import java.util.Hashtable;

public class ExtendedBank{
    private Hashtable<String,Account>customer; // 口座リスト
    private int balance; // 残高格納用

    public ExtendedBank(){ // 口座リスト初期化
        customer = new Hashtable<String,Account>();
    }
    public int withdraw(String name,int amount){ // 引き出し
        Account myaccount = customer.get(name);
        balance = myaccount.showBalance();
        if(myaccount != null){
            if(amount <= 0)
                return -3;
            else if(amount > balance)
                return -1;
            else{
                myaccount.withdraw(amount);
                return 0;
            }
        }else return -7;
    }   
    public int showBalance(String name){ // 残高照会
        Account myaccount = customer.get(name);
        if(customer.get(name) != null) return myaccount.showBalance();
        else return -7;
    }

    // ExtendedBank要素
    // 引き出し
    public int withdraw(String name,String amount){  // name:口座名
        int result;
        int w_result;
        try{
            result = Integer.parseInt(amount);
        }catch(NumberFormatException e){
            if(showBalance(name) == -7)
                return -7;
            else
                return -4;
        }
        result = Integer.parseInt(amount);
        if(result <= 0)
            return -3;
        else if(result > balance)
            return -1;
        else{
            w_result=withdraw(name,result);
            return w_result;
        }
    }
}

Answer 1

当てずっぽうです。

withdraw(String, String) 内で、ExtendedBank#balance が未初期化で使われているように見えます。そもそも、この関数は

        try {
            return withdraw(name, Integer.parseInt(amount));
        } catch (NumberFormatException e) {
            ....

だけで良いんじゃないでしょうか。

Comments

[Mequitazine]

正常に動作しました。ですが、なぜこの部分のみで正常に動作するのかが理解できません。
自分のコードで書いてあった下の部分はなぜ必要ないのでしょうか。

[int32_t]

バランスチェックやアカウントからのwithdraw()は withdraw(String, int) の中で行われているので、withdraw(String, String) ではその結果だけ利用すればよいです。