rubyで、二重配列を使って表を作り、列を入れ替えたいです

Question

rubyで、家計簿を加工するコードを書きたいです。
もともと家計簿の支出部分を、
1002 サンディ 食品 生鮮食品 1274 現金
1018 サンディ 食品 生鮮食品 2967 現金
1108 サンディ 食品 生鮮食品 2895 現金

というように、
月日 購入先 大分類 小分類 金額 決済手段
というように、タブ区切りの六つの項目で記録していました。
これを、

1. 月日の前に西暦年をつける

2.並び順を
年月日 金額 決済手段 購入場所 大分類 小分類
に並べ替える。
という加工をしたいです。

# 行頭に半角数字4つが並んでいれば、その前に西暦年を追加
open("kakeibo_new1.txt", "w") do |f|
    IO.foreach("kakeibo_ex.rb") do |line|
      line.gsub!(/^(\d{4})/){'2016'+$1 }
      f.puts line
    end
  end
# 各行をタブで分割し、二重配列にする
a=[]
open("kakeibo_new2.txt", "w") do |f|
    IO.foreach("kakeibo_new1.txt") do |line|
      b=line.split(/\t/)
      a.push(b)
    end
  end
#　以下のswapメソッドはHayato OKUMOTO @falcon8823さん作成のもの
class Array
  def swap!(a,b)
    raise ArgumentError unless a.between?(0, self.count-1) && b.between?(0, self.count-1)
    self[a],self[b]=self[b],self[a]
    self
  end
  def swap(a,b)
    self.dup.swap!(a,b)
  end
end

a.each do |b|
  swap!(b[1],b[4])
end

まず、1はできました。
2については、まずいったんデータ全体を入れ子配列にし、子配列の要素を入れ替えるという形で列の入れ替えをしようと思いました。
そこで、配列の要素の入れ替えについては、ネット上で公開されていた、Arrayクラスにメソッドを追加するという形でやろうとしたのですが、
そのメソッドを使っての実行段階で、「メソッドが定義されていない」というエラーが出ます。
swapメソッドはネットで公開されていたものを丸写ししています。

追記
教えていただいたことを参考に、完成させたのが以下のコードです。

# 行頭に半角数字4つが並んでいれば、その前に西暦年を追加
open("kakeibo_new1.txt", "w") do |f|
    IO.foreach("kakeibo_ex.rb") do |line|
      line.gsub!(/^(\d{4})/){'2016'+$1 }
      f.puts line
    end
  end
# 各行をタブで分割し、二重配列にする
a=[]
open("kakeibo_new2.txt", "w") do |f|
    IO.foreach("kakeibo_new1.txt") do |line|
      line.chomp!
      b=line.split(/\t/)
      a.push(b)
    end
  end
#　以下のswapメソッドはHayato OKUMOTO @falcon8823さん作成のもの
class Array
  def swap!(a,b)
    raise ArgumentError unless a.between?(0, self.count-1) && b.between?(0, self.count-1)
    self[a],self[b]=self[b],self[a]
    self
  end
  def swap(a,b)
    self.dup.swap!(a,b)
  end
end
p a

a.each do |b|
  b.swap!(1,4)
  b.swap!(2,5)
  b.swap!(3,4)
  b.swap!(4,5)
end

open("kakeibo_new3.txt", "w") do |f|
  a.each do |b| 
      f.puts b.join("\t")
  end
end

katoyさんのようにやったほうが明らかに効率出来ですが、最初に作りかけたものも完成させたかったので。
その際に、12行目に
line.chomp!
も入れました。
unexpected /n
というエラーメッセージが出て、何が原因かなかなかわからなかったのですが、ここの改行を取ったらちゃんと動くようになりました。

Answer 1

a.rb

infos = []

File.open('kakeibo.txt', 'rt') do |f|
  f.each_line do |line|
    row = line.split
    infos << {
      date: "2016#{row[0]}",
      item: row[1],
      type: row[2],
      sub_type: row[3],
      price: row[4],
      payment: row[5]
    }
  end
end

open('kakeibo_new.txt', 'w') do |f|
  infos.each do |info|
    row = [
      info[:date], info[:price], info[:payment],
      info[:item], info[:type], info[:sub_type]
    ]
    f.puts row.join("\t")
  end
end

実行例

Comments

[otakurounin]

ありがとうございます。
このように子配列のかわりにハッシュを使うと、何が何だかわかりやすくなりますね。

Answer 2

まず質問の部分では、メソッドの使い方を間違えています。Arrayのメソッドで引数は添え字なので、

a.each do |b|
  b.swap!(1,4)
end

ですね。意味を理解しないと使えません。
わざわざメソッドを呼ぶほどでもないので、

a.each do |b|
  b[1],b[4] = b[4],b[1]
end

でいいと思います。

あとは、大きな改善事項としては、
・いちいちファイルに書かなくていい
・CSVライブラリの使用を検討（CSVという名前だがタブ区切りも当然使える）

細かい改善事項はたくさんあります。

Comments

[otakurounin]

ありがとうございます。
確かに、よく見るとこのswapメソッドでは引数は添え字ですね。
気が付きませんでした。
それに、確かにメソッドを作らなくても配列要素の交換はできました。

Answer 3

こういう時はエラーメッセージを翻訳しないでそのままコピペしたほうが良いです。
「何にメソッドがない」と出ているはず。

swap!(b[1],b[4])　ではなく　b.swap!(1,4) では？

Comments

[otakurounin]

ありがとうございます。