は、なぜ私は文句を言わないそれを解析し、この現在のコードで</p> <p>をintにずさんな変換、二重のためのエラーなしでコンパイルこれは、これまでと（int型）なしで私のクラスのためのコードである境界

Question

のうち、このエラー配列のインデックスを取得していますそれはコンパイルが、あなたのmonthlyFee方法は、この行にIndexOutOfBoundsExceptionをスローします

public class Bank { 


    // instance variables========================================= 
    // setup new array of BankAccount objects and internal null constructor 
    BankAccount [] custAcct = new BankAccount[10]; 

    // data above here 
    //=========================================================== 
    // methods down below 

    // display methods======================================= 

    public void printAccounts(){ 
     for(int i=0; i<custAcct.length; i++) { 
      if (custAcct[i]!=null){ 
       System.out.println(custAcct[i]); 
      } 
     } //end loop 

    } // end method 

    // utility methods============================= 

    public void addAccount(BankAccount acct) { 
     for(int i=0; i<custAcct.length; i++) { 
      if (custAcct[i]==null){ 
       custAcct[i] = acct; 
       break; 
      } 
     } // end loop 

    } // end method 

    public BankAccount getAccount(int acc) { 
     return custAcct[acc]; 
    } // end method 

    public void monthlyFee (double acctVal){ 
     for(int i=0; i<custAcct.length-1; i++) { 
      if (custAcct[i]!=null){ 
       custAcct[i] = custAcct[(int)acctVal]; 
       break; 
      } 
     } // end loop 

    } // end method 

Answer 1

monthlyFee方法で例外エラーなしで実行されません。

custAcct[i] = custAcct[(int)acctVal]; 

acctValの長さがcustAcct（この場合は10）よりも大きいか、0より小さい場合。質問に投稿したコメントから、monthlyFeeと-2.95を呼び出していることがわかります。これはIndexOutOfBoundsExceptionのソースです。

あなたaddAccount方法は、あまりにも、設計上の欠陥が含まれる場合があります。それは、現在コーディングされている方法は、それはそれは、引数として渡されたBankAccountとcustAcct配列内の空のスポットの全てを満たします。あなただけ最初空の位置にアカウントを追加したい場合は、この使用します。私は、これはここから来てする必要があります考えています

public void addAccount(BackAccount acct) { 
    for(int i = 0; i < custAcct.length; i++) { 
     if(custAcct[i] == null) { 
      custAcct[i] = acct; 
      return; 
     } 
    } 
} 

Answer 2

を（おそらくスタックトレースのスニペットを送信するのに役立ちます）

  custAcct[i] = custAcct[(int)acctVal]; 

値が0より小さいか、またはcustAcct.lengthより大きい場合、acctValに渡される値は、Out of bound例外が発生する可能性があります。他の問題は、custAcct [i]が初期化されておらず、ヌルポインタ例外が発生する可能性があります。あなたがスタックトレースを与えるなら、より良く伝えることができます。