[ruby-list:23181] Re: http.rb will change

高橋征義です。

Minero Aoki <[e-mail]>さん：
> > > > また、HTTPResponse オブジェクトに body を含めれば、例外発生時には、こ
> > > > の HTTPResponse オブジェクトを利用できるような方法を考えれば、ヘッダ情
> > > > 報、ボディともに利用できて、便利そうな気もします。
> > > 
> > > それはもはや無理でしょう。
> 
> > これは無理でも、$!.body とかで取り出す、ということはできないん
> > でしょうか？
> 
> $!.data.body でいいですか？ それなら簡単です。

えーと、混乱させてしまってとっても申し訳ないんですが、今さらながら
青山さん提案の、 「HTTPResponseそのものにBodyの内容を入れる」、
というのが良いような気がしています。もちろん互換性は気にしては
いるのですが、あまりこだわりすぎて、改善を躊躇しすぎるのもまずい
んじゃないかと。

# 週末から「リファクタリング」を読んでるんで、リファクタ猿と
# 化してるような気もしますが(汗;＞自分

青山さん提案の利点としては、

・「GETが返すもの」が一つにまとまっていて分かりやすい。
・PerlのLWPでもUserAgentが返すResponseはhead、body込みに
  なっているなど、似たような前例もある。
・$!で渡すときにも便利。
・res, data = hoge()、という多重代入のメソッドはRubyの標準
  ライブラリにもさほどあるわけではなく、さほど分かりやすい
  とは言えない。

といったことが挙げられます。

で、懸念材料である「APIの変更」ですが、いきなり今までの
やり方ができなくなるのはさすがに問題だと思います。ので、
移行期間を設けて、それまでは res, dataの形での取得を残した
まま、res.body という形でも取得、ということにするのは
どうでしょう。

ということでパッチまで書いてみました。あ、いや、さすがに
いきなりこれを取り込んでほしい、ということではなく、こういう
風にした方がいいんじゃないかと思うんだけどどうでしょう、
という提案・説明の資料のつもりです(_o_)


net/http.rb を使っている他の方の意見もぜひ聞いてみたいです。


--- http.rb.org	Mon Jun 12 23:33:03 2000
+++ http.rb	Mon Jun 12 23:35:10 2000
@@ -171,6 +171,7 @@
 
     def get( path, u_header = nil, dest = nil, &block )
       resp = get2( path, u_header ) {|f| dest = f.body( dest, &block ) }
+      resp.body = dest
       resp.value
       return resp, dest
     end
@@ -352,9 +353,11 @@
       super( code_type, code, msg )
       @data = {}
       @http_body_exist = bexist
+      @body = nil
     end
 
     attr_reader :http_body_exist
+    attr_reader :body
 
     def []( key )
       @data[ key.downcase ]



高橋征義 (TAKAHASHI Masayoshi)       Email:[e-mail]