[前][次][番号順一覧][スレッド一覧]

ruby-reference-manual:3574

From: "sho-h (Sho Hashimoto)" <sho-h@n...>
Date: Tue, 26 Feb 2013 17:19:04 +0900
Subject: [ruby-reference-manual:3574] [るりまプロジェクト - Bug #7962][Closed] Array#bsearch、Range#bsearchのレビュー


Issue #7962 has been updated by sho-h (Sho Hashimoto).

Status changed from Open to Closed
% Done changed from 0 to 100

ありがとうございます!!どれも正しいと思ったのでコミットしました。(サンプルはary[i]を返す訳じゃないので修正前の説明はおかしかったですね

----------------------------------------
Bug #7962: Array#bsearch、Range#bsearchのレビュー

https://bugs.ruby-lang.org/issues/7962#change-37087

Author: authorNari (Narihiro Nakamura)
Status: Closed
Priority: Normal
Assignee: sho-h (Sho Hashimoto)
Category: doc:review
Target version: 
reporter: 
ruby_version: 


nariです。

Array#bsearch、Range#bsearchのドキュメントを読みました。
http://doc.ruby-lang.org/ja/2.0.0/method/Array/i/bsearch.html

find-minimumモードの方はすごく分かりやすいのですが、find-anyモードがどうも最初ピンとこないんですよね(本家のRDocも同様ですが…)。
そもそも仕様がわかりづらいといえばそうなのかもですけど…。

ということで、find-anyモードの方を少しだけ修正した版のパッチを作りました。

修正点としては


* Array
** 求める条件に対するブロックパラメータの関係性を言葉で示したほうがよさそう
** i から j *未満* であることを明示化
* Range
** v はブロックパラメータだと思います(間違ってたら適宜訂正してもらえると助かります)
** 求める条件に対するブロックパラメータの関係性を言葉で示したほうがよさそう

と言ったところです。よさそうであれば取り込んでもらえると助かります。


-- 
http://bugs.ruby-lang.org/

--
ML: ruby-reference-manual@m...
Info: http://QuickML.com/

[前][次][番号順一覧][スレッド一覧]