Project

General

Profile

Actions

Feature #11267

closed

CSV::Row#field と CSV::Table#[] の引数に Range を使えるようにした

Added by takkanm (三村 益隆) almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Target version:
-
[ruby-dev:49091]

Description

CSV を使用している際、n 行目から m 行目までを取得したいというケースがあり、Array と同じ気持ちで Range オブジェクトを渡したところ動作しないことに気づきました。

row = CSV::Row.new(%w(A B C), [1, 2, 3])
row[1..2] #=> nil

CSV::Row でのアクセスでも Range でアクセスできると嬉しいと思い実装してみました。
この修正で以下のような振る舞いになります。

row[1..2] #=> [2, 3]

同様に CSV::Table#[] でも Range を許すようにしています。


Files

csv_field_allow_range.patch (1.93 KB) csv_field_allow_range.patch takkanm (三村 益隆), 06/16/2015 01:12 PM

Updated by matz (Yukihiro Matsumoto) almost 9 years ago

いいんじゃないでしょうか。コードレビュー後、問題なければコミットしてください。
中田さん?

Matz.

Updated by hsbt (Hiroshi SHIBATA) almost 9 years ago

  • Status changed from Open to Assigned
  • Assignee set to hsbt (Hiroshi SHIBATA)

やっておきます

Actions #3

Updated by hsbt (Hiroshi SHIBATA) almost 9 years ago

  • Status changed from Assigned to Closed

Applied in changeset r50929.


Updated by takkanm (三村 益隆) almost 9 years ago

ありがとうございました !

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0