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

■No47006 (こあら さん) に返信
>>なんか変だな、と思うのですが。
> 
> そうですね。私も変だと思います。というか、フレームワークの設計ミスでは・・・

お。
同志が。

> みたいに、WebRequest.GetResponseの戻り値とWebException.Responseが同じインスタンスを指す実装って
> できないんでしたっけ？

できるんですが…。

InnerMethod( arg, arg, ... ) {
  try {
    response = request.GetReponse;
    return response;
  } finally {
    if (request.status = error) throw new webexception(response)
  }
}

returnされたreponseを使う側では普通はUsingを使いますよね。

OuterMethod {
  try
    Using (response = SomeMethodReturnsResponse( args, args )) {
      '正常系
    }
  } catch ex as WebException {
    if ( ex.Response.Status == XX ) {  #ここでObjectDisposedException
      MessageBos("some error occurred.")
    }
    using ( stream = ex.Response.GetReponseStream ) { #ここでObjectDisposedExceptionやIOException
      ReadErrorMessage(stream)
    }
  }
}

という感じで、
普通に使うことができなくなってしまいます。

Usingを使わずにFinallyで囲んで、WebExceptionのキャッチを他の例外のキャッチと毎回確実に別にすればいいのですが…。

やってられない。

こんにちは。はずしてたらスルーしてください。

れいさんの、

> WebRequest/Responseを自分で実装しているのですが、

とか、

> ライブラリを作る場合は困ります。

のところがどんな状況か想像できていないのと、

> より一般に。
> ExceptionにIDisposableなメンバを入れたい場合に、どう入れたらいいのでしょう？
> ExceptionにIDisposableなメンバが入っている場合に、どう廃棄したらいいのでしょう？

の話でもないのですが、

static class WebRequestExtensions
{
    public static WebResponse GetResponse(
        this WebRequest webRequest,
        out WebException webExceptionWithoutRespose)
    {
        try
        {
            webExceptionWithoutRespose = null;
            return webRequest.GetResponse();
        }
        catch (WebException e)
        {
            if (e.Response != null)
            {
                webExceptionWithoutRespose =
                    new WebException(e.Message, e.Status);
                return e.Response;
            }
            throw;
        }
    }
}

としておいて、

MyMethod()
{
    var request = WebRequest.Create("http://www.google.co.jp/");

    WebException webExceptionWithoutRespose;
    using (var response = request.GetResponse(out webExceptionWithoutRespose))
    {
        using (var stream = response.GetResponseStream())
        {
            using (var reader = new StreamReader(stream))
            {
                richTextBox1.ForeColor =
                    webExceptionWithoutRespose == null
                    ? Color.Blue : Color.Red;
                richTextBox1.Text = reader.ReadToEnd();
            }
        }
    }
}

のように使うのはどうですか？

■No47045 (渋木宏明(ひどり) さん) に返信
> >>ExceptionがIDisposableを実装していて、
> >>tryを出るとDisposeされる仕様だとよかったのに。
> 
> それはそれで、再スローと相性が良くないような。

そうなんですよね。
再スローされるExceptionなのにIDisposableがメンバって、
失敗作な雰囲気が感じられます。

■No47048 (ななし さん) に返信
> こんにちは。はずしてたらスルーしてください。
>略
> のように使うのはどうですか？

外していません。
こういった使い方をすればusingできちんとReponseを廃棄することができますね。

でもこれだと異常系がusingの中に入ってしまっています。
構造化されていません。
「ファイルAとファイルBの内容を連結してCに書く」という単純な作業の場合、

try {
  using (var responseA = requestA.GetResponse(out webExceptionWithoutResposeA)) {
    using (var streamA = responseA.GetResponseStream()) {
      if ( webExceptionWithoutResposeA != null ) {
        using (var responseB = requestB.GetResponse(out webExceptionWithoutResposeB)) {
          using (var streamB = responseB.GetResponseStream()) {
            if ( webExceptionWithoutResposeA != null ) {
              //ここに正常系の処理
            } else {
              //ここに異常系の処理
              //異常系のstreamBはここでしか使えないので、その処理が必要。
            }
          }
        }
      } else {
        //ここにも異常系の処理
        //異常系のstreamAはここでしか使えないので、その処理が必要。
      }
    }
} catch {
  //ここにも異常系の処理
}

となって、異常系が分散していて見にくくなります。

上記例だと一つのMethodの中なのでまだ耐えられますが、
異常系を処理したいのが違う階層のメソッドだとするとかなり大変になります。

なんかひどく醜いので
生産性だとか意思疎通だとか教育だとか
そういった大人な経験のある方々はどうしてるのかと。

これを他人に見せたら、
「なんで例外をキャッチしてパラメーターとして返してるの？それってデザインに反してない？」
とか
「なんで正常系と異常系が同じ構造(using)内で条件分岐してるの？構造化例外処理しないの？」
とか、
なりそうなわけです。

「処理できない例外は上に投げていい」というのが構造化例外処理なのに、WebExceptionだけは「処理できなきゃダメ」というのが問題だと思うのです。
だとすると、ななしさんのように、メソッドの返り値にして毎回確認するべきで、例外を投げるという実装はおかしいと思うのですが。
WebExceptionを投げる実装にはどんな利点があるのかわかりません。

もうひとつ問題なのが、こういった処理がIDisposableなメンバをもつExceptionでいつも起こるのに、
発生しうる例外をすべて列挙するのが無理なことです。
ドキュメントを全部漁って、そういうExceptionのリストでも作ればいいんですが、面倒でまだできていません。


「処理しなきゃいけない例外」がいつどこで起きるかわからないなんて！

いつどこで起きるかわからない「絶対に処理しなきゃいけない仕事」だって、
風邪ひきましたって言って上司に任せることができるのに…


考え始めてもう一週間くらいになるので、とりあえず解決したことにします。
WebRequest/Responseは構造化例外処理できない。
ななしさんのように使うべし、ということで。

解決としますが、なにか知っていること、考えたこと、アイデアなどがあったら教えてください。
よろしくお願いします。

GetしたらfinallyでCloseしろパターン？
WebRequestにCloseResponseメソッドを持たせるのはダメでしょうか？


try
    Using (response = WebRequest.GetResponse()) {
      '正常系
    }
} catch ex as WebException {
    if ( ex.Response.Status == XX ) {
        MessageBos("some error occurred.")
    }
    using ( stream = ex.Response.GetReponseStream ) {
        ReadErrorMessage(stream)
    }
} finally {
    WebRequest.CloseResponse()   <-- if (this.Response) Response.Close のようなメソッド
}

こんにちは。

■No47074 (れい さん) に返信
> 外していません。
> こういった使い方をすればusingできちんとReponseを廃棄することができますね。

よかったです。
異常時にも stream を返しちゃえば、忘れずに破棄しやすくできる思いました。

> でもこれだと異常系がusingの中に入ってしまっています。
> 構造化されていません。
> 「ファイルAとファイルBの内容を連結してCに書く」という単純な作業の場合、
> ・・・
> となって、異常系が分散していて見にくくなります。

れいさんのコード、少し見やすくしてみました。↓

try
{
    var requestA = WebRequest.Create("http://www.google.co.jp/");

    WebException webExceptionWithoutRespose;
    using (var responseA = requestA.GetResponse(out webExceptionWithoutRespose))
    using (var streamA = responseA.GetResponseStream())
    {
        if (webExceptionWithoutRespose != null)
        {
            webExceptionWithoutRespose.Data.Add(
                "エラー内容",
                "streamA から取得したレスポンス");
            throw webExceptionWithoutRespose;
        }

        var requestB = WebRequest.Create("http://www.google.co.jp/");

        using (var responseB = requestB.GetResponse(out webExceptionWithoutRespose))
        using (var streamB = responseB.GetResponseStream())
        {
            if (webExceptionWithoutRespose != null)
            {
                webExceptionWithoutRespose.Data.Add(
                    "エラー内容",
                    "streamB から取得したレスポンス");
                throw webExceptionWithoutRespose;
            }

            // A と B の内容を連結して C に書くなどの正常系の処理
        }
    }
}
catch
{
    // ここで異常系の処理
}

これだと正常系と異常系がそれなりに分けられてると思いますし、「異常系を処理したいのが違う階層のメソッド」でも大丈夫じゃないですか？
昨日、Data.Add のあたりも拡張メソッドの中に入れちゃおうかと思ったりもしてたんですが、いらない人もいるということなのでやめてました。
ところで、Stream の読み込み中に Response 付きの WebException が発生したりなんてことはないですよね？ 昨日の返信後、そこを心配してました。

> WebExceptionを投げる実装にはどんな利点があるのかわかりません。

確かに。

> もうひとつ問題なのが、こういった処理がIDisposableなメンバをもつExceptionでいつも起こるのに、
> 発生しうる例外をすべて列挙するのが無理なことです。
> ドキュメントを全部漁って、そういうExceptionのリストでも作ればいいんですが、面倒でまだできていません。

Exception から派生しているクラスのプロパティもしくはメソッドの戻り値に、IDisposable を実装しているものって、他にあるんですかね。
プログラムで調べればすぐわかりそうに思いますけど、私だとコード書くのに時間かかっちゃいます。


■No47076 (こあら さん) に返信
> GetしたらfinallyでCloseしろパターン？
> WebRequestにCloseResponseメソッドを持たせるのはダメでしょうか？

この具体的な実装を考えてみたんですが、難しそうに思いました。