java.lang.StackOverflowErrorを解決したい。

Question

学校の課題なのですが、預金をする際に整数を入力した際にこのエラーが出ました。
調べて自分でコードを見直してみたり、書き換えたりなどしているのですが解決の兆しが見えなかったので質問投稿させていただきます。
至らぬ点が多いため、コード不足や補足事項必要でしたらご指摘よろしくお願いいたします。
文字数の制限のため、預金処理以降のコードしか載せられませんでした。

Comments

[int32_t]

ExtendedBank のコードを開示してください。サーブレットのコードはたぶん不要です。

[Mequitazine]

サーブレットのコードを削除して、追記しますね。

[jimbe]

HashTable や -1 ～ -7 等のマジックナンバー、 get/set のネーミングルールを用いない等、何時の教材を使い回しをしているのやらと言った感じです。

何を変えて良くて何を変えてはいけないのでしょうか。

[int32_t]

解決後でも、コードやスクリーンショットを削除しないでください。質問と回答が他者から見て意味不明になってしまいます。

[Mequitazine]

すみません。

Answer 1

処理結果を示す戻りを無くして例外にしたり。(一応戻りの値は例外内に code として保存)

package teratail_java.q_c9mdeundv32wxi;

import java.util.HashMap;

import teratail_java.q_c9mdeundv32wxi.Account.AmountException;

public class ExtendedBank {
  /** 口座例外 */
  public static class AccountException extends Exception {
    public final int code;
    public AccountException(String message, int code) {
      super(message+" (code="+code+")");
      this.code = code;
    }
  }

  /** 口座マップ */
  private HashMap<String,Account> customer = new HashMap<>();

  /**
   * 口座オブジェクトを取り出す
   * @param name 口座名
   * @return Account オブジェクト
   * @throws AccountException　口座が無かった場合
   */
  private Account getAccount(String name) throws AccountException {
    Account account = customer.get(name);
    if(account == null) throw new AccountException(name+" NOT exists.", -7);
    return account;
  }

  /**
   * 金額文字列数値化
   * @param amount 金額
   * @throws AmountException 金額が処理出来ない場合
   */
  private int parseAmount(String amount) throws AmountException {
    try {
      return Integer.parseInt(amount);
    } catch(NumberFormatException e) {
      throw new AmountException(amount, -4, e);
    }
  }

  /**
   * 口座開設
   * @param name 口座名
   * @throws AccountException 指定口座が既に有った場合
   */
  public void open(String name) throws AccountException {
    if(customer.containsKey(name)) throw new AccountException(name+" exists.", -7);
    customer.put(name, new Account(name));
  }

  /**
   * 口座解約
   * @param name 口座名
   * @throws AccountException 指定口座が無かった又は残高が残っている場合
   */
  public void close(String name) throws AccountException {
    Account account = getAccount(name);
    if(account.showBalance() != 0) throw new AccountException(name+" has balance.", -1);
    customer.remove(name);
  }

  /**
   * 預金
   * @param name 口座名
   * @param amount 金額
   * @throws AccountException 指定口座が無かった場合
   * @throws AmountException 金額が処理出来ない場合
   */
  public void deposit(String name, String amount) throws AccountException, AmountException {
    getAccount(name).deposit(parseAmount(amount));
  }

  /**
   * 預金
   * @param name 口座名
   * @param amount 金額
   * @throws AccountException 指定口座が無かった場合
   * @throws AmountException 金額が処理出来ない場合
   */
  public void deposit(String name, int amount) throws AccountException, AmountException {
    getAccount(name).deposit(amount);
  }

  /**
   * 引き出し
   * @param name 口座名
   * @param amount 金額
   * @throws AccountException 指定口座が無かった場合
   * @throws AmountException 金額が処理出来ない場合
   */
  public void withdraw(String name, String amount) throws AccountException, AmountException {
    getAccount(name).withdraw(parseAmount(amount));
  }

  /**
   * 引き出し
   * @param name 口座名
   * @param amount 金額
   * @throws AccountException 指定口座が無かった場合
   * @throws AmountException 金額が処理出来ない場合
   */
  public void withdraw(String name, int amount) throws AccountException, AmountException {
    getAccount(name).withdraw(amount);
  }

  /**
   * 残高照会
   * @param name 口座名
   * @return 預金残高
   * @throws AccountException 指定口座が無かった場合
   */
  public int showBalance(String name) throws AccountException {
    return getAccount(name).showBalance();
  }
}

package teratail_java.q_c9mdeundv32wxi;

public class Account {
  /** 金額例外 */
  public static class AmountException extends Exception {
    public final int code;
    public AmountException(int amount, int code) {
      super("amount="+amount+" (code="+code+")");
      this.code = code;
    }
    public AmountException(String amount, int code, Throwable cause) {
      super("amount='"+amount+"' (code="+code+")", cause);
      this.code = code;
    }
  }

  /** 口座名 */
  public final String name;
  /** 残高 */
  private int balance;

  /**
   * 口座
   * @param name 口座名
   */
  public Account(String name) {
    this.name = name;
    balance = 0;
  }

  /**
   * 入金
   * @param amount 金額
   * @throws AmountException 金額が処理できない値の場合
   */
  public void deposit(int amount) throws AmountException {
    if(amount <= 0) throw new AmountException(amount, -3);
    balance += amount;
  }

  /**
   * 出金
   * @param amount 金額
   * @throws AmountException 金額が処理できない値の場合
   */
  public void withdraw(int amount) throws AmountException {
    if(amount <= 0 || balance < amount) throw new AmountException(amount, amount<=0?-3:-1);
    balance -= amount;
  }

  /**
   * 残高参照
   * @return 残高
   */
  public int showBalance() {
    return balance;
  }
}

Answer 2

パッと見、withdraw()の再帰呼び出しが無限ループになって落ちてそうですね。

Answer 3

    public int deposit(String name,String amount){ // name:口座名
            ...
            d_result=deposit(name,amount);

amount のパーズ結果が正のとき、まったく同じ引数で自分自身を呼び出していて、無限再帰になっています。

d_result = deposit(name, result); としたかったんじゃないでしょうか。

Comments

[Mequitazine]

指摘されて見てみると、amountではおかしいことに気付けました。
本当にありがとうございます！値の受け渡しもうまくいき正常に動かせました！

[javahack]

無限再帰はdeposit()の方でしたか。
エラー文をちゃんと読んでなかった。恥ずかしい。。。

[int32_t]

javahack さんの言うとおり、withdraw(String, String) も同じような問題がありますね。

[Mequitazine]

withdrawでも同じく発生しました。
お恥ずかしいのですが、さっきの解答のように
w_result=withdraw(name,result); に直して実行してみたところ、残高以下の入力でもうまく引き出しが行えなえず詰まってしまったのですが、コードでおかしいなと感じた部分は他にはなかったでしょうか。

[int32_t]

入力、期待される結果、実際の結果を添えて別の質問を立ててください。

[jimbe]

そもそも再帰は必要無いのでは・・・と思ったらオーバーロードしたメソッドの呼び出しだったんですね。失礼しました。