[ruby-dev:50486] [Ruby trunk Bug#14482][Closed] headerが重複したCSVをheaders:trueでparseした場合、row.to_h[key] と row[key] の結果が一致しない

[kou@cozmixng•org]

Issue #14482 has been updated by kou (Kouhei Sutou).

Status changed from Open to Closed
Assignee set to kou (Kouhei Sutou)

後方互換性がなくなってしまうのでどうするかなぁと考えてみたんですが、もともと重複していた場合はなにを返すかわからない、という仕様だったので変わっても大丈夫かなぁと言う気持ちになったので、同じになるようにしました。

----------------------------------------
Bug #14482: headerが重複したCSVをheaders:trueでparseした場合、row.to_h[key] と row[key] の結果が一致しない
https://bugs.ruby-lang.org/issues/14482#change-70654

* Author: tompng (tomoya ishida)
* Status: Closed
* Priority: Normal
* Assignee: kou (Kouhei Sutou)
* Target version: 
* ruby -v: ruby 2.5.0p0 (2017-12-25 revision 61468) [x86_64-darwin16]
* Backport: 2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN
----------------------------------------
1行目に重複したkeyがあるcsvファイルをheaders: trueでparseした場合、
row[key]とrow.to_h[key]が異なる値を返します。

~~~ruby
require 'csv'
invalid_header_csv = "id,name,name\n1,ichiro,ICHIRO\n2,jack,JACK"
rows = CSV.parse invalid_header_csv, headers: true
puts rows[0]['name'] #=> ichiro
puts rows[0].to_h['name'] #=> ICHIRO
~~~

ドキュメントには

~~~
to_hash -> Hash
自身をシンプルなハッシュに変換します。
フィールドの順序は無視されます。 重複したフィールドは削除されます。
~~~
とありますが、削除されるフィールドとrow[key]で無視されるフィールドは一致している方が良いのではないかと思います。

~~~ruby
class CSV::Row
  def to_h
    headers.uniq.map { |key| [key, self[key]] }.to_h
  end
  alias_method :to_hash, :to_h
end
~~~




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