C# と VB.NET の質問掲示板

2010/09/24(Fri) 15:47:46 編集(投稿者)

普段は.NET(VB,C#)で開発をしています。
Javaの初歩的な基礎知識はありますが、業務でJavaを使用したことはありません。
今回Java（Android)で作業しているのですが、件名について質問いたします。

件名の通り、Javaの検査例外の処理が面倒です。

たとえばファイルをコピーする処理の場合、finallyのbr.close(),bw.close()でも検査例外が出るのでtry〜catchで囲まないとコンパイルエラーになります。

private void fileCopy(InputStream is , String filename ){
    BufferedReader br = null;
    OutputStream os = null;
    BufferedWriter bw = null;
    try { 
        br = new BufferedReader(new InputStreamReader(is));    
        os = openFileOutput(filename, MODE_PRIVATE);   
        bw = new BufferedWriter(new OutputStreamWriter(os));   
   
        String str;      
         while((str = br.readLine()) != null){      
            bw.append(str +"\n");     
        } 
         bw.flush();          
    } catch (FileNotFoundException e) {
        e.printStackTrace();
    } catch (IOException e) {
        e.printStackTrace();
    } finally {   
        try {
            br.close();
            bw.close();   	   
        } catch (IOException e) {
            e.printStackTrace();
        }          
    }  
}

fileCopyメソッドを呼び出す側では、ファイルの存在チェックを行ってからこのメソッドを呼び出しているので
FilNotFoundExceptionがもし発生しても何もしないですし、IOExceptionが発生しても何もできないです。
（.Netでは例外を出すコストもあるので、事前に検査できるものは検査しましょうって言われていたと思います。
なのでファイルの存在チェックは必ず行っています。その上で発生するFileNotFoundExceptionはキャッチしていません。）

.NETでは処理できる例外のみをキャッチして処理し、それ以外はApplication.ThreadExceptionで処理しています。
Javaでも検査例外を実行時例外でラップし再スローして、UncaughtExceptionHandlerで処理しようと思っています。

質問1
finallyをスッキリ書きたいのですが、以下のようなtry〜catchを2重にするサンプルがありました。
このような書き方は一般的ではないのでしょうか？
（あまりみたことがないけれど、他にスッキリ書ける方法がわかりません）
private void fileCopy(InputStream is , String filename ){
    BufferedReader br = null;
    OutputStream os = null;
    BufferedWriter bw = null;
    try { 
        try {
            br = new BufferedReader(new InputStreamReader(is));    
            os = openFileOutput(filename, MODE_PRIVATE);   
            bw = new BufferedWriter(new OutputStreamWriter(os));   
   
            String str;      
            while((str = br.readLine()) != null){      
                bw.append(str +"\n");     
            } 
            bw.flush();   
        } finally {   
            br.close();
            bw.close();   	   
        }  
    } catch (FileNotFoundException e) {
        e.printStackTrace();
    } catch (IOException e) {
        e.printStackTrace();
    }
}

質問2
実行時例外でラップし再スローするにあたって、以下のようなラップ例外を一つ一つ書かなければいけないのでしょうか？
その際のパッケージはどうするのがいいでしょうか？Exceptionパッケージに全部つめこんでも問題ないでしょうか？
FileNotFoundRuntimeException
IORuntimeException

質問3
Javaと.Netでは例外の取り扱いが違うのは理解しているつもりですが
.Netに合わせたようなこのような方法はおかしいですか？

通りすがりさん
返信ありがとうございます。

以下のようなサイトを見つけました。

サルでもわかる 逆引きデザインパターン > 第4章 逆引きカタログ　その他 > 実行時例外を標準的に使う 
http://www.nulab.co.jp/designPatterns/designPatterns4/designPatterns4-2.html
私も同じ考えです。
（検査例外をまったく使用しないわけではありません）




> 自分ならはExceptionを継承した独自例外を定義します
> （例外を捕捉しているか、コンパイル時にチェックされるため）
> 
> １．メソッドのthrows 節に独自例外を記載
> 　例外のcatchを行いExceptionを継承した独自例外を生成してthrowする
> ２．利用側メソッドのthrows 節に独自例外を記載
> ３．最終的にはUncaughtExceptionHandlerで捕捉する
> 
> 独自例外は１つでもかまわないと思います
> 　独自例外に原因となった例外やメッセージを保持させる方式をとる
> 

すいません。よくわからないです。
ExExceptionはExceptionを継承した独自例外で書きなおしてみました。
public class MainActivity extends Activity {
	
　　　　　> ２．利用側メソッドのthrows 節に独自例外を記載
　　　　throws節が書けません。
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		//レイアウトの読み込み
		this.setContentView(R.layout.main);
		//filesフォルダにtest.txtが存在しなければ、res/raw/test.txtをコピーする。
		String filename = this.getFilesDir().getAbsolutePath() + "/test.txt";
		File file = new File(filename);
		if ( file.exists() == false ) {
			Resources res = this.getResources();
			try {
				fileCopy(res.openRawResource(R.raw.test),filename);
　　			} catch (ExException e) {
			　　　//何もできないので、エラー画面へ遷移する。
			}			
		}	
	}
	
　　　　　> １．メソッドのthrows 節に独自例外を記載
　　　　　> 　例外のcatchを行いExceptionを継承した独自例外を生成してthrowする
　　　　　finally句がすっきり書けません。
	private void fileCopy(InputStream is , String filename ) throws ExException{
		BufferedReader br = null;
		OutputStream os = null;
		BufferedWriter bw = null;
		try { 
			try {
	        	br = new BufferedReader(new InputStreamReader(is));    
	            os = openFileOutput(filename, MODE_PRIVATE);   
	            bw = new BufferedWriter(new OutputStreamWriter(os));   
	   
	            String str;      
	            while((str = br.readLine()) != null){      
	            	bw.append(str +"\n");     
	            } 
	            bw.flush();   
			} finally {   
        		br.close();
				bw.close();   	   
			}  
		} catch (FileNotFoundException e) {
			throw new ExException(e);
		} catch (IOException e) {
			throw new ExException(e);
		}
	}
}

2010/09/27(Mon) 19:11:45 編集(投稿者)
2010/09/27(Mon) 18:52:28 編集(投稿者)

■No53808 (yan さん) に返信
> 通りすがりさん
> 返信ありがとうございます。
> 
> 以下のようなサイトを見つけました。
> 
> サルでもわかる 逆引きデザインパターン > 第4章 逆引きカタログ　その他 > 実行時例外を標準的に使う 
> http://www.nulab.co.jp/designPatterns/designPatterns4/designPatterns4-2.html
> 私も同じ考えです。
> （検査例外をまったく使用しないわけではありません）
> 
> 
> すいません。よくわからないです。
> ExExceptionはExceptionを継承した独自例外で書きなおしてみました。
> public class MainActivity extends Activity {
> 	
> 　　　　　> ２．利用側メソッドのthrows 節に独自例外を記載
> 　　　　throws節が書けません。
> 	protected void onCreate(Bundle savedInstanceState) {
> 		super.onCreate(savedInstanceState);
> 		//レイアウトの読み込み
> 		this.setContentView(R.layout.main);
> 		//filesフォルダにtest.txtが存在しなければ、res/raw/test.txtをコピーする。
> 		String filename = this.getFilesDir().getAbsolutePath() + "/test.txt";
> 		File file = new File(filename);
> 		if ( file.exists() == false ) {
> 			Resources res = this.getResources();
> 			try {
> 				fileCopy(res.openRawResource(R.raw.test),filename);
> 　　			} catch (ExException e) {
> 			　　　//何もできないので、エラー画面へ遷移する。
> 			}			
> 		}	
> 	}

継承したメソッドを使用しているため、throws節が書けないということでしょうか？
であれば、RuntimeExceptionもありと思います
だだし、コール側で捕捉しわすれると、プログラムが即時停止します


最初の投稿でUncaughtExceptionHandlerで処理するつもりとありますが
どのクラスで実装するのでしょうか？

■AndroidのActivityクラスのonCreateメソッドなら下記でよいのでは？

 fileCopyメソッドにthrows IOExceptionを追記
　＝＞fileCopyメソッドをコールする側でcatchするコードが強制される

public class MainActivity extends Activity
  //
protected void onCreate(Bundle savedInstanceState) {
  try {
      //
      fileCopy(res.openRawResource(R.raw.test),filename);
  } catch (Exception e) {
      //何もできないので、エラー画面へ遷移する。
  }
}

private void fileCopy(InputStream is , String filename ) throws IOException
  try{
    //ファイルコピー処理
  } finally { 
    //
    if (br != null) br.close();
    if (bw != null) bw.close(); 
  }
}