質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.81%

スタックオーバーフローの原因箇所がわかりません。

受付中

回答 0

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 1,091

score 0

前提・実現したいこと

Javaで銀行システムを実装しています。

JDBCを使用してデータベースへ接続しSQLによるデータベースへのアクセスと実行結果の受け取りを行います。

getメソッドは引数で指定された口座情報をデータベースから読み出し、Accountクラスのオブジェクトを作成し戻り値として返します。口座が存在しない場合はnullを返します。

以上の機能を実装中にエラーメッセージが発生しました。

発生している問題・エラーメッセージ

java.lang.StackOverflowError

該当のソースコード

import java.sql.DriverManager;
import java.sql.Connection;
import java.sql.Statement;
import java.sql.ResultSet;
import java.sql.SQLException;

public class DBtable{
    private String user = "";
    private String password = "";
    private String db_url = "";

 public Account get(String name){
        String myName;//読みだす口座名
        int myBalance;//読みだした残高を入れる変数
        Connection conn = null;//データベースへの接続
        Statement stmt = null;//SQL文の実行用オブジェクト
        try{
            //データベースからデータを読み出す
            conn = DriverManager.getConnection(db_url, user, password);
            stmt = conn.createStatement();

            String sql = "SELECT * FROM customer WHERE name ='"+name+"'";
            ResultSet rs = stmt.executeQuery(sql);
            if(rs.next()){
                    myBalance = rs.getInt("balance");//変数への残高の格納
            }
            rs.close();
        }catch(SQLException e){
            return null;//エラーは表示せずnullを返す
        }finally{
            try{
                if(stmt != null) {
                    stmt.close();
                }
                if(conn != null) {
                    stmt.close();
                }
            }catch(SQLException e){
                return null;//エラーは表示せずnullを返す
            }
        }
//後処理 戻り値となるAccountオブジェクトの生成
        Account myAccount = new Account(name);
        myAccount.deposit(myBalance);
        return myAccount;
    }
public class Bank{
    private DBtable customer;//口座リスト

    public Bank(){//口座リストを初期化
        customer = new DBtable();
    }

    public int open(String name){//口座開設
        Account x = customer.get(name);
        if(x == null){
            Account myName = new Account(name);
            customer.put(name,myName);//口座開設
            int balance = customer.get(name).showBalance();
            return 0;
        }else{
            return -7;

        }
    }

試したこと

Hashtableクラスを使用した場合は問題なく動きました。
DBtableクラスへ変更するとエラーが起きてしまいます。
DBtableクラスのgetメソッドをBankクラスで呼び出しています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正の依頼

  • koushi3192

    2021/06/24 11:42

    ありがとうございます。
    ソースコードを変更してみました。つたない質問で申し訳ありません。

    キャンセル

  • momon-ga

    2021/06/24 12:11 編集

    myBalanceが未初期化の可能性でコンパイルエラーになると思うのだけど・・・
    正しいソースコードを載せないと回答できない

    キャンセル

  • dameo

    2021/07/15 08:58

    コードは読んでいません。

    スレッドごとにネストして呼び出せるメソッドの最大回数が決まっており、それを決めるのがスタックサイズです。スタックサイズが溢れてしまうと、StackOverflowErrorが発生します。例えば以下のようなコードで起こすことができます。

    class StackoverflowSample {
    private static void recursive(int count) {
    if (count > 0) {
    recursive(count - 1);
    }
    }
    public static void main(String[] args) {
    for (int i = 1; true; ++i) {
    System.out.println(i);
    recursive(i);
    }
    }
    }

    OracleのJavaVMでは、-Xssオプションなどでこのスタックサイズを調整することができます。現在値を知る方法は
    (unix系)java -XX:+PrintFlagsFinal -version | grep ThreadStackSize
    (windows系)java -XX:+PrintFlagsFinal -version | findstr ThreadStackSize
    などです。

    現在値を倍にするなどして頻度が減るのであれば、値の調整で済む可能性がありますが、スレッドごとの使用メモリは増えるので別の問題が発生する可能性があります。頻度が減らないのであれば、再帰呼出が無限に実行されている可能性があり、その場合はコードの修正が必要になると思います。

    いずれにしても、スタックトレースは出ていると思うので、解析調査は比較的容易だと思うのですが...。
    ※お返事を頂いてもこちらからはもうお返事を返せません。あしからず。

    キャンセル

まだ回答がついていません

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.81%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る