ruby-dev

こんにちは、なかむら(う)です。

In message "[ruby-dev:38939] Re: String#valid_encoding?にオプションが欲しい"
    on Jul.29,2009 22:12:41, <fuj@rabbix.jp> wrote:
> なるせさんのパッチだと、encodingの変更に失敗したときに
> ascii-8bitに強制変更するようになっています。
> これだと驚き最小の法則にひっかかると思いました。
> encodingの変更に失敗したときはそのまま元のstringを返すのが
> いいのではないでしょうか？
> もしくはnilを返すか。

まず、指定のencodingに変更できない場合に強制的にASCII-8BITに
設定されるのはよくないと私も思います。
ダメな場合は元のencodingのまま無変更とすることを推します。

次に、戻り値ですが、assume_encodingは破壊的メソッドなわけです
が、現状のrubyの破壊的メソッドの多くは、結局破壊的操作が行わ
れなかった場合はnilを返すようになっているはずです。
なので、変更できた場合はselfを、できなかった場合はnilを返すこ
とを推します。


ところで。

変更に失敗した場合に例外を投げるという仕様も考えられると思う
のですが、どうでしょうか。

ユースケース的には、たぶん元のencodingはASCII-8BITで、指定の
encodingに変更できない場合はそのまま処理を続行するのだろうと
思います。
とすると、例外を投げるよりは上記の仕様の方が使い勝手がいいの
だろうと思うわけですが、仕様としては、例外を投げる方がなんと
なく綺麗な感じはします。

# どっちにすべきかは判断できないので投げっぱなしです。
# すみません。


それでは。
-- 
U.Nakamura <usa@garbagecollect.jp>

Thread

Prev Next

In This Thread

Prev Next

[#38724] [ANN] Ruby開発会議のお知らせ — "Yugui (Yuki Sonoda)" <yugui@...>

[#38725] [Bug #1720] [NaN] == [NaN] が true になる — tadayoshi funaba <redmine@...>

[#38726] [Bug #1721] NaN.divmod(NaN) が例外にならない — tadayoshi funaba <redmine@...>

[#38728] open-uri の content_type — Kazuhiro NISHIYAMA <zn@...>

[#38731] FreeBSD で ruby-mecab のライブラリ参照の不具合 — "KISHIMOTO, Makoto" <ksmakoto@...4u.or.jp>

[#38738] [Bug #1720] [NaN] == [NaN] が true になる — tadayoshi funaba <redmine@...>

[#38740] [Backport #1329] Ruby 1.8.8 で "p eval('0' + '+0.1'*n)" が Segmentation Fault を発生させる — Shyouhei Urabe <redmine@...>

[#38745] A proposal for announcing a release of ruby to ruby distributors — "Yugui (Yuki Sonoda)" <yugui@...>

[#38756] rb_obj_untrust, rb_obj_untrusted, rb_obj_trust — Hidetoshi NAGAI <nagai@...>

[#38760] [Bug #1763] SecurityError on an Exception object because of modification of instance variable #__ThrowState__ — Hidetoshi Nagai <redmine@...>

[#38761] Re: [ruby-cvs:31084] Ruby:r23862 (trunk): * io.c (argf_rewind): need to rewind $. and ARGF.lineno. — "Yugui (Yuki Sonoda)" <yugui@...>

[#38762] Re: [ruby-cvs:31110] Ruby:r23892 (trunk): * rational.c (float_to_r): always returns rational. — "Yugui (Yuki Sonoda)" <yugui@...>

[#38766] [Bug #1768] json で Kernel に追加されるメソッドは private であるべき — okkez _ <redmine@...>

[#38767] [BUG:trunk] eval with binding, without location — "Yugui (Yuki Sonoda)" <yugui@...>

[#38769] RCLASS_M_TBL() の構造の変更 — SASADA Koichi <ko1@...>

[#38770] [Bug #1694](Closed) Tempfile.open{} の返り値 — Yuki Sonoda <redmine@...>

[#38771] [Bug #1720](Assigned) [NaN] == [NaN] が true になる — Yuki Sonoda <redmine@...>

[#38777] [Bug #1768](Assigned) json で Kernel に追加されるメソッドは private であるべき — Yui NARUSE <redmine@...>

[#38782] [Bug:trunk] Re: [ruby-cvs:31281] Ruby:r24063 (trunk): * ext/tk/extconf.rb: New strategy for searching Tcl/Tk libraries. — "U.Nakamura" <usa@...>

[#38786] [Bug #1701](Closed) Regexp#match の pos 引数がマルチバイト文字に対応していない — Yui NARUSE <redmine@...>

[#38815] Ruby会議懇親会でリジェクトされてしまった forwardable.rb につ いて — keiju@... (Keiju ISHITSUKA)

[#38820] [Bug #1787] URI ライブラリの正規表現パターン — drry <redmine@...>

[#38821] セキュリティモデルのドキュメント — Shugo Maeda <shugo@...>

[#38822] [Bug #1787] URI ライブラリの正規表現パターン — drry <redmine@...>

[#38823] [Bug #1787] URI ライブラリの正規表現パターン — drry <redmine@...>

[#38824] [Bug #1788] ある種のメソッドをalias_methodしようとすると落ちる — Akira Matsuda <redmine@...>

[#38826] [Bug #1789] TCPSocket#read returns untainted string — Masahiro Tomita <redmine@...>

[#38828] [Bug #1790] [patch] fix error message in Addrinfo#family_addrinfo — Nobuhiro IMAI <redmine@...>

[#38829] [Bug #1791] r24205,r23960,r23892,r23871 をバックポートする — tadayoshi funaba <redmine@...>

[#38830] [Bug #1792] Fixnum#& 等が、Rational などを受けつける — tadayoshi funaba <redmine@...>

[#38834] 無名クラスを定数に代入する前後の、その内側のクラスの名前 — wanabe <s.wanabe@...>

[#38835] DelegateClass元のクラスのprotectedメソッドが呼び出せない — Akira Matsuda <ronnie@...>

[#38836] ext/tk/extconf.rb creates a file in $srcdir — "U.Nakamura" <usa@...>

[#38843] 複素数リテラルについて — Yukihiro Matsumoto <matz@...>

[#38848] [Bug #1803] rdocがサイズ0のファイルのバイナリ判定に失敗する — Sakuro OZAWA <redmine@...>

[#38849] [Bug #1804] ext/socket/mkconstants.rb doesn't recognize enum'ed constants — Usaku NAKAMURA <redmine@...>

[#38850] Rational#hash — Tadayoshi Funaba <tadf@...>

[#38877] Re: [Backport #1183] Adding support for the new hash literal syntax — Urabe Shyouhei <shyouhei@...>

[#38882] [Bug #1807] r+ オープンしたファイルが MinGW の拡張ライブラリで扱えない — Kimura Yoshihiro <redmine@...>

[#38887] ext/tk/extconf.rb should call find_library in all paths — "U.Nakamura" <usa@...>

[#38900] rb_eval_string_protect and encoding — Masaki Suketa <masaki.suketa@...>

[#38903] [Backport #1817] bug of ARGF.eof? — Nobuyoshi Nakada <redmine@...>

[#38912] String#valid_encoding?にオプションが欲しい — Fujioka <fuj@...>

[#38914] [Bug #1819] Ruby-1.9.1を使用しDB（MySQL）接続時にエラー — Ryouhei Saita 斉田 <redmine@...>

[#38924] thread switch hook for RubyCocoa — Nobuyoshi Nakada <nobu@...>

[#38929] [Bug #1826] json で無駄な private 指定がある — okkez _ <redmine@...>

[#38932] Enumerator#peek — Tanaka Akira <akr@...>

[#38936] [Bug #1826](Closed) json で無駄な private 指定がある — Yui NARUSE <redmine@...>

[#38938] Re: [ruby-list:46234] Re: irbでの式展開中の動作について — keiju@... (石塚圭樹)

[#38942] Test failure (test_block.rb) in Ruby 1.9.2 preview1 on FreeBSD 6.0 — Hiroshi Ichikawa <gimite@...>

[#38961] [Bug #1837] Test failure (test_block.rb) in Ruby 1.9.2 preview1 on FreeBSD 6.0 — Hiroshi Ichikawa <redmine@...>

[#38962] [Feature #1840] Hash に要素追加して self を返すメソッド — Kimura Yoshihiro <redmine@...>

[#38968] [Bug #1842] mspec で SIGABRT — Kazuhiro NISHIYAMA <redmine@...>

[#38971] [Bug #1848] Net::SSH hangs — Shyouhei Urabe <redmine@...>

[ruby-dev:38941] Re: String#valid_encoding?にオプションが欲しい

Thread

In This Thread

[#38760] [Bug #1763] SecurityError on an Exception object because of modification of instance variable #ThrowState — Hidetoshi Nagai <redmine@...>

[#38815] Ruby会議懇親会でリジェクトされてしまった forwardable.rb について — keiju@... (Keiju ISHITSUKA)

[#38820] [Bug #1787] URI ライブラリの正規表現パターン — drry 　 <redmine@...>

[#38822] [Bug #1787] URI ライブラリの正規表現パターン — drry 　 <redmine@...>

[#38823] [Bug #1787] URI ライブラリの正規表現パターン — drry 　 <redmine@...>