IllegalArgumentException正しく機能していませんJavaで正しく

Question

不正な例外を投げることは私には新しく、その中にいくつかのポインタが必要です。私はイベントで発生または損失の金額を決定するコードを作っていましたが、テスト中にコードに2つのエラーが発生しました。最初に入力したものに関係なく、IllegalArgumentExceptionエラーメッセージ "例外スレッド"メイン "java.lang.IllegalArgumentException：選択された文字は無効です.T、D、またはEでなければなりません。指定されたデータTは無視されます。私はメッセージが正しい文字を使用していると言っているのはなぜか分かりません。

2番目の問題は、自分のプログラムで実際に表示方法に値（0以外の値）が表示されていないことでした。私は正しいアイデアがあるように感じたので、別の場所でこれを間違ってやったかもしれないと思った。必要に応じてコード全体を添付しましたが、エラーが発生したと思われるコメントを残しました。

public class EdmondEventManager2 { 

     /**Purpose of code is to determine the amount of money generated or lost in an event 
     * @param args the command line arguments 
     */ 
     public static void main(String[] args) { 
      EdmondEventClass object= new EdmondEventClass(); 

      object.addNewValue(amountType(),validateAmount()); 
      object.displayResults(); 
     } 

     /*static method that will read and validate amount type T-ticket sales, D-Donation, E-expenses */ 
     public static char amountType(){ 
      Scanner keyboard= new Scanner (System.in); 
      System.out.println("Please select the amount Type: T-Ticket Sales," 
         + " D-Donations, E-Expenses"); 
      char amtType=keyboard.next().charAt(0); 
      amtType=Character.toUpperCase(amtType); 

      switch(amtType) 
      { 
       case 'T' &'t': 
        break; 
       case 'D'& 'd' : 
        break; 
       case 'E' &'e' : 
        break; 
      } 

      if (amtType !='T'&& amtType!='D'&& amtType!='E'&&amtType!='t' 
        &&amtType!='d'&&amtType!='e'){ 
       do {  
        System.out.println("Choice invalid. Please select T, D, or E"); 
        System.out.println("Enter amount type: T, D, or E."); 
        amtType=keyboard.next().charAt(0); 
       }while (amtType!='T'&&amtType!='D'&&amtType!='E'); 
       return amtType= Character.toUpperCase(amtType); 
      } 

      return amtType= Character.toUpperCase(amtType); 
     } 

     /*static method that will read and validate the amount. I believe te first issue is here */ 
     public static double validateAmount(){ 
     Scanner keyboard= new Scanner (System.in); 
     System.out.println("Please enter the amount in dollars"); 
     double amount=keyboard.nextInt(); 

     if (amount<=0){ 
      do {  
       System.out.println("Amount must be a positive number!" 
        + " Try Again"); 
       System.out.println("Please enter the amount in dollars"); 
       amount=keyboard.nextInt(); 
      }while (amount<=0); 
       return amount; 
     } 

     return amount;     
     } 

     public static class EdmondEventClass{ 
     private int T; 
     private int D; 
     private int E; 

     //constructors 
     public EdmondEventClass(){  
      T=0; 
      D=0; 
      E=0;  
     } 

     //getters 
     public int getTotalIncomeSale() { 
      return this.T; 
     } 

     public int getTotalDonatins() { 
      return this.D; 
     } 

     public int getTotalExpenses(){ 
      return this.E; 
     } 

     /*Instance method that will add a new value to one of the totals 
     *@char amtType- One of the letters the user must chose. T-ticket sales, 
      D-Donation, E-expenses 
     *@ double amount- the amount for the chosen amtType.  
     */ 
     public double addNewValue (char amtType, double amount) { 

      if (amount<=0) { 
       throw new IllegalArgumentException("Amount must be positive and " 
       + "non-zero. The given data " +amount+ " will be ignored."); 
      } 
      else if (amtType !=T&& amtType!=D&& amtType!=E) { 
       throw new IllegalArgumentException ("The letter chosen is invalid." 
        + "Must be T, D, or E. The given data " +amtType+ 
        " will be ignored"); 
      } 

      return amount + amtType; 
     } 

     //Will display the outcome of the event. I believe the second issue is here. 
     public double displayResults(){ 
      System.out.println ("Event Overall Outcome:"); 
      System.out.println ("Total ticket sales:  "+T); 
      System.out.println ("Total donations:  "+D+ " +"); 
      System.out.println ("      -------"); 
      System.out.println ("Total income: "); 
      System.out.println ("Total expense:   "+E+ " -"); 
      System.out.println ("      -------"); 
      System.out.println ("Event profits: "); 
      String numberString=String.format ("%8.2f"); 
      return T+D;  
     } 
    } 
} 

Answer 1

あなたのパラメータは文字ですが、int型に対してそれをチェックしているように見える、とこれだけ意味がありませんaddNewValue方法。 charsとintをintと比較する必要があります。

サイド推奨：コードフォーマットは非常にであり、オプションではなく、注意深く行う必要があることを理解してください。ここで最善の助けが必要な場合は、できるだけ読みやすく理解しやすいようにコードを作成してください。また、変数名を始めるために小文字を使用することを含むJava命名規則を学び、従うことが望ましいでしょう。