C#のout修飾子経由のOutOfMemory発生ポイント。

Question

C#でout修飾子で変数を別関数に投げ、呼んだ関数でファイル読み込みし、読み込みデータを関数内でオブジェクトを作らずにout修飾子変数に代入して処理終了する場合、呼び出し元側でもOutOfMemoryは出るでしょうか？
出るとすれば呼び出し先なら、呼び出し元にcatchのOutOfMemoryは置かなくても？と思っているところです。
いつなったかによる、などありましたら、詳細を頂けると助かります。
例）
class class1{
public void main(){
string data;
read(out data);
}
public void read(out string data){
data = File.WriteAllText("input.txt");
}
}

naktak · Accepted Answer

どこでキャッチするかだけであって、スタックトレース見れば発生元まで追えると思います。

ログ情報で『○○関数開始』と出力後、read()内で当該例外になった時、
例外をmain()内でキャッチし、エラーメッセージおよびスタックトレースをログ出力すればいいのでは。
それは別に誤った情報にはなりませんよね。

個別にやるべきことがない場合、階層が深いメソッド内で例外をキャッチすると、
その都度必要な実装を書かなければならないことになり、煩雑になるかと思います。

何が何でも様々な例外においてその場ですべてキャッチし、個別に例外処理をするんだ！
という思想ならそれまでですが。

先に最も正解かと思う実装を記しますと、そもそも下層ではキャッチしない、となります。
class class1{
public void main(){
try {
string data;
read(out data);
} catch(OutOfMemoryException e) {
  // OutOfMemoryException共通の処理
  // Console.WriteLine(e.Message + Environment.NewLine + e.StackTrace);
} catch(Exception e) {
  // 共通の例外処理
  // Console.WriteLine(e.Message + Environment.NewLine + e.StackTrace);
}
}
public void read(out string data){
data = File.WriteAllText("input.txt");
}
}

OutOfMemoryException自体、いつどこで発生するか分からないという想定外の例外かと思います。
それをわざわざありとあらゆるところでキャッチするというのは明らかに非効率でしょう。
よって、ほとんどの場合でこれが正解かと思います。

例えばファイルがオープンできなかったとか、そういう業務的な例外は、その都度キャッチして、更に上位に例外を返す方法が正解かと思います。
（下記コードは例にあるコードに付け加えただけなのでテキトーです）

class class1{
public void main(){
try {
string data;
read(out data);
} catch(OutOfMemoryException e) {
  // OutOfMemoryException共通の処理
  // Console.WriteLine(e.Message + Environment.NewLine + e.StackTrace);
} catch(Exception e) {
  // 共通の例外処理
  // Console.WriteLine(e.Message + Environment.NewLine + e.StackTrace);
}
}
public void read(out string data){
try {
data = File.WriteAllText("input.txt");
} catch(FileNotFoundException e) {
  // ファイルが開けなかった際の個別処理
  throw;
}
}
}

===================================
以下、パターンごとの考察となります。

１．
OutOfMemoryExceptionのキャッチをどっちにも入れるという方がどうかと思います。
class class1{
public void main(){
try {
string data;
read(out data);
} catch(OutOfMemoryException e) {
  // なんか処理
}
}
public void read(out string data){
try {
data = File.WriteAllText("input.txt");
} catch(OutOfMemoryException e) {
  // ログ出力処理
}
}
}

例えばこのような実装があった時、read()で例外が発生したとしましょう。
例外が発生したことをmain()は理解しませんので、そのまま処理を継続してしまいますよね。
絶対そこでめんどくさいことをしなければならなくなります。
エラー情報も戻り値を持つだとか、実際は不必要な実装など。
このような実装をすると、main()では、あくまでもmain()内でOutOfMemoryExceptionが発生した時しか例外をキャッチしなくなります。

２．
ではread()をこうしてみましょうか。
public void read(out string data){
try {
data = File.WriteAllText("input.txt");
} catch(OutOfMemoryException e) {
// なんか固有のログ出力
throw e;
}
}
}
こうすると今度は、read()内で発生した例外を、新たな例外情報として呼び元に返すことになります。
よって、スタックトレースとしては、例外が発生した直前のトレース情報しか得られなくなります。
これは、メソッドの階層が深くなればなるほど、どこで発生したのが分からなくなります。

３．
個別の処理が必要で、どっちにも入れる場合、最初に述べた通り、以下のようにして、スタック情報をすべて有している例外を最上位でキャッチすべきかと思います。
その場合、最上位は、OutOfMemoryExceptionだからどうのとかってあまりないかと思います。
（どうしても他の例外と処理を分ける必要があるならしてもいいですが）
class class1{
public void main(){
try {
string data;
read(out data);
} catch(Exception e) {
  // 共通の例外処理
}
}
public void read(out string data){
try {
data = File.WriteAllText("input.txt");
} catch(OutOfMemoryException e) {
  // なんか固有のログ出力
  throw;
}
}
}

でもこの場合、OutMemoryException自体、いつ発生するかもわからないのですから、ありとあらゆる場所でOutOfMemoryExceptionをキャッチしてログ出力しなければならないというとても煩雑なことになるかと思います。

Tacosan · Answer

一応確認です.

本当に WriteAllText でいいんでしょうか?

naktak · Answer

例外処理を一切書いてないのだから、最終的な例外をスローする場所は呼び出し元に帰結します。

どのメソッドにコードを書いているから、どのメソッドで発生するとかじゃないです。
呼び先メソッドで例外が発生して、そのメソッドでキャッチしてないなら呼び元までスローされます。

C#のout修飾子経由のOutOfMemory発生ポイント。

どこでキャッチするかだけであって、スタックトレース見れば発生元まで追えると思います。

一応確認です.

例外処理を一切書いてないのだから、最終的な例外をスローする場所は呼び出し元に帰結します。

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング