例外がスローされ、さらにコードが実行される

Question

状況に応じて - メインメソッドでは、オブジェクトをチェックする子メソッドが呼び出され、この子メソッドで例外がスローされます（オブジェクトの1つリストのNULLに）。しかし、メインメソッドのコードはまだ実行され続けています！ コード例：この点で

@Transactional 
public boolean addCompany(List<Company> companies, List<Address> addresses) throws Exception{ 
    checkAddress(addresses); 
    try{ 
     for(int i = 0; i < companies.size(); i++){ 
      if(findCompany(companies.get(i).getId()) == null && !isExistsCompany(companies.get(i))){ 
       companies.get(i).setAddress(addresses.get(i)); 
       this.em.persist(companies.get(i)); 
      } 
     } 
    }catch(Exception e){ 
     return false; 
    } 
    return true; 
} 

public void checkAddress(List<Address> addresses) throws Exception{ 
    try{ 
     if(addresses == null) 
      throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid parameter: list is null"); 
     for(Address a : addresses) 
      if(a == null) 
       throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid list item: object is null"); 
    }catch(Exception e){ 
     e.printStackTrace(); 
    } 
} 

、いくつかの質問が生じた： を - コードが停止しない理由は？ - オプションとして、checkAddressメソッドの型をvoidからbooleanに変更し、true/falseを処理するmainメソッドで状況を脱出する必要がありますか？ - フロントエンドでこのようなエラーを正しく処理する方法 - テキストをフロントエンドに送信するか、コード500を処理するだけですか？その場合は、バックエンドで例外を生成する理由を開発プロセスで助けてください。それをうまく処理するには？ アドバイスをお願いします。 ありがとうございます。

Answer 1

はこれを試してみてください：スレッドに関するすべてのものです

public void checkAddress(List<Address> addresses) throws Exception{ 
     if(addresses == null) 
      throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid parameter: list is null"); 
     for(Address a : addresses) 
      if(a == null) 
       throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid list item: object is null"); 
    } 

} 

何？クレイジーコード。

誰もが最初にNullインスタンスをリストに追加することを許可しないでください。

私はそれをこのように書くかもしれません：

public void checkAddresses(List<Address> addresses) { 
    if (addresses == null) throw new IllegalArgumentException("Address List cannot be null"); 
    for (Address a : addresses) { 
     if (a == null) throw new IllegalArgumentException("Address cannot be null"); 
    } 
} 

Answer 2

e.printStackTrace(); 

この行はあなたのコードが継続ではなく、失敗する原因、例外を抑制することができます。それは例外がスローされているように見えるかもしれないスタックトレースを出力しますが、それ以上の行は得られません。

ほとんどの場合、printStackTrace()を使用したくない場合は、意図した例外を適切に処理するか、例外をメソッドの呼び出し元に伝播させるだけです。

Answer 3

checkAddress()メソッド内からtry .. catchブロックを削除する必要があります。こうすると、checkAddress()の内部からスローされた例外がすべてその呼び出し元に伝播します。

addCompany()メソッドでは、checkAddress()メソッドをtrycatchの中に呼び出して例外を処理します。

checkAddress()が例外をスローすると、コード実行がcatchブロックにジャンプします。

Answer 4

あなたはJavaランタイムがそれを扱う考慮Exceptionを再スローしていないときは、Exception（複数可）キャッチされています。プログラムの実行が停止することを期待した場合は、Exceptionが発信者に伝播する必要があります。例えば、

} catch(Exception e) { 
    e.printStackTrace(); 
    throw e; // <-- re-throw the Exception 
} 

またはのようなものにcheckAddress変更

} catch(Exception e) { 
    e.printStackTrace(); 
} 

に単にそれからExceptionを自動的に呼び出し側にスローされ、完全にtryとcatchを削除します。また、Java 8以降ではStreamを使用できます。同様に、

public void checkAddress(List<Address> addresses) throws Exception { 
    if (addresses == null) { 
     StackTraceElement[] ste = Thread.currentThread().getStackTrace(); 
     throw new Exception(ste[2].getClassName() + "." 
       + ste[2].getMethodName() + "." + ste[1].getMethodName() 
       + ": Invalid parameter: list is null"); 
    } 
    if (addresses.stream().anyMatch(a -> a == null)) { 
     StackTraceElement[] ste = Thread.currentThread().getStackTrace(); 
     throw new Exception(ste[2].getClassName() + "." 
       + ste[2].getMethodName() + "." + ste[1].getMethodName() 
       + ": Invalid list item: object is null"); 
    } 
}