Ruby：二つのcsvファイルの差分データを配列に格納したい。

Question

前提・実現したいこと

Ruby初心者です。

前提・実現したいこと
Rubyで二つのcsvファイル比較して、新しく追加されたレコードのみを配列に格納したい。

【問題点】
新しく追加された配列のデータの要素を配列に格納する機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

`block in <main>': undefined method `<' for nil:NilClass (NoMethodError)

該当のソースコード

#encoding: SJIS
#lib
require 'csv'
require 'diff/lcs'

#var_num
checkonlynewdiff = Array.new()
newolddiff = Array.new()

puts "start..."

#read csvfile(old&new)
data1 = CSV.read "csvtestold.csv"
data2 = CSV.read "csvtestnew.csv"

#diff_old&new read_csv_data
diffs = Diff::LCS.diff(data1, data2)

diffs.each do |diff|
  diff.each do |line|
    newolddiff << line.to_a
  end
end

#purpose:read only diff_new_add_data
newolddiff.each_index{|idx|
  if newdiff[idx][0] == "+"
    #happen to error message
    checkonlynewdiff << newolddiff[idx][3]
  end
}
puts "end..."

試したこと

メソッドが存在しないのがエラー原因と書かれているので、@newdiff[idx][3].to_aにして試したが駄目でした。

補足情報（FW/ツールのバージョンなど）

Ruby：2.5.2
Ruby library install：gem install diff-lcs
OS：Windows10
csvファイル
csvtestold.csv------
name,id,date
sho,220,2018/1/5
akiho,221,2018/1/6
sss, 222,2018/1/7

csvtestnew.csv------
name,id,date
akiho,221,2018/1/6
sss, 222,2018/1/7
kdo,223,2018/1/8
shi,224,2018/1/9
skd,225,2018/1/10

参考ＵＲＬ
(rubyの変数について)
https://qiita.com/nakamaru/items/1339303487fe8a876825

(Class: Diff::LCS::Change)
https://www.rubydoc.info/gems/diff-lcs/1.2.5/Diff/LCS/Change

Answer 1

変数の使い方が間違えてる云々は割愛して...
typoがかなり多くて云々も割愛して...

いくつか...

NoMethodError (undefined method `<<' for nil:NilClass)

このエラーの発生は変数の使い方が間違っているというのに起因しています。

diffs = Diff::LCS.diff(@@data1, @@data2)

この部分は正常に動けば

#=> [[["-", 13, "s"], ["+", 13, "a"], ["+", 14, "k"], ["+", 15, "i"]], [["+", 21, "1"], ["+", 22, ","], ["+", 23, "2"]], [["+", 25, "1"], ["+", 26, "8"], ["+", 27, "/"], ["+", 28, "1"], ["+", 29, "/"], ["+", 30, "6"], ["+", 31, "\n"], ["+", 32, "s"], ["+", 33, "s"], ["+", 34, "s"]], [["+", 37, "2"], ["+", 38, "2"], ["+", 39, ","], ["+", 40, "2"]], [["-", 28, "5"], ["+", 47, "7"]], [["-", 30, "a"]], [["-", 32, "i"], ["-", 33, "h"], ["+", 50, "d"]], [["-", 38, "1"], ["+", 55, "3"]], [["-", 47, "6"], ["+", 64, "8"]], [["-", 50, "s"], ["-", 51, "s"], ["+", 67, "h"], ["+", 68, "i"]], [["+", 72, "4"], ["+", 73, ","]], [["+", 75, "0"], ["+", 76, "1"], ["+", 77, "8"], ["+", 78, "/"], ["+", 79, "1"], ["+", 80, "/"], ["+", 81, "9"], ["+", 82, "\n"], ["+", 83, "s"], ["+", 84, "k"], ["+", 85, "d"]], [["+", 88, "2"], ["+", 89, "5"], ["+", 90, ","], ["+", 91, "2"]], [["-", 64, "7"], ["+", 98, "1"], ["+", 99, "0"]]]

こんな感じで返って来ると思いますが、これは期待する値なのでしょうか...

diffs.each do |diff|
diff.each do |line|
@@newolddiff << line.to_a
end
end

わざわざArrayにする必要を感じません。

以上を踏まえて少し書いてみました。
できればコピペして解決〜なんてのは避けて欲しいです。
もしこれが望み通りの結果であれば、理解できていなかった部分を調べ直して経験値としていただければ幸いです。

require 'csv'
require 'diff/lcs'

checkonlynewdiff = []

data1 = <<~EOS
          name,id,date
          sho,220,2018/1/5
          akiho,221,2018/1/6
          sss,222,2018/1/7
        EOS
data2 = <<~EOS
          name,id,date
          akiho,221,2018/1/6
          sss,222,2018/1/7
          kdo,223,2018/1/8
          shi,224,2018/1/9
          skd,225,2018/1/10
        EOS

diffs = Diff::LCS.diff(data1.split("\n"), data2.split("\n"))
#=> [[["-", 1, "sho,220,2018/1/5"]], [["+", 3, "kdo,223,2018/1/8"], ["+", 4, "shi,224,2018/1/9"], ["+", 5, "skd,225,2018/1/10"]]]

diffs.flatten.each do |diff|
  checkonlynewdiff << diff.element if diff.adding?
end

p checkonlynewdiff
#=> ["kdo,223,2018/1/8", "shi,224,2018/1/9", "skd,225,2018/1/10"]

#ちなみに最後の部分は一発で取ることもできる
diffs.flatten.select(&:adding?).map(&:element)
#=> ["kdo,223,2018/1/8", "shi,224,2018/1/9", "skd,225,2018/1/10"]

Comments

[sss_kdo]

指摘ありがとうございます。

期待する値は新しく追加されたデータの部分のみなので、
>#=> ["kdo,223,2018/1/8", "shi,224,2018/1/9", "skd,225,2018/1/10"]を抽出したかったです。

質問なのですが、実行すると
>diffs.each do |diff|
> checkonlynewdiff << diff.element if diff.adding?
>end
の箇所で、以下のエラーメッセージが返ってきました。
=>undefined method `adding?' for [["-", 1, "sho,220,2018/1/5"]]:Array (NoMethodError)
buta_bottiさんの環境では動作できていますか？

[退会済みユーザー]

最後の指摘箇所、Arrayに変更する必要を云々の部分の違いだと思います。
sss_kdoさんのコードではまだ配列に変換している処理が残っているんじゃないですかね？

[退会済みユーザー]

あぁ、すみません僕のミスでした。
.flattenが抜けてますね...
修正しておきます。

[sss_kdo]

修正ありがとうございます。正常に動作しました。
僕の理解が追い付いてなくて何が駄目なのかわかっていなかったです。=>flattenを勉強します。

もう一点質問したいです。
diff/lcs標準ライブラリではないのですが、buta_bottiさんはよく使われているのですか？

[退会済みユーザー]

flattenよりまず変数の使い方を学ぶべきです。
今回のエラーも変数の使い方がまずいが故に起きたものですし...
flattenなんてメソッドは知らなくてもたいして困らないので優先度は低いですよ。

diff/lcs は今日初めて使いました。

[sss_kdo]

わかりました。
まずは基本的な箇所から勉強していきます。

diff/lcs初めて使うのに、対応してくれてありがとうございます。

Answer 2

唐突に、@checkdiffという変数が出てきていますが、これは何ですか？もしかして、@checkonlynewdiffの間違いでは？

あと、インスタンス変数が間違った使われ方をしています。

Comments

[otn]

@newolddiff[idx][3]
も
@newolddiff[idx][2]
の間違いでしょうね。

[sss_kdo]

指摘ありがとうございます。
@checkonlynewdiffに直しました。

インスタンス変数はインスタンス生成時に使うんですね！
じゃあここでは、クラス変数に直した方がいいってことですよね？

[sss_kdo]

@newolddiff[idx][3]は合ってると思うんですけどね。
今、外出中なので、家に帰ったら試してみます！

[otn]

> じゃあここでは、クラス変数に直した方がいいってことですよね？

いえ。普通のローカル変数を使います。

[sss_kdo]

勉強不足ですみません。

基本的なところから勉強しなおしていきます。