【rails】DBに正しく値が入ってくれない

Question

nokogiriでwebスクレイピングした値をDBに保存するというコードをrakeファイルに記述しております。

Prescraping
```はモデルクラス名です。

```ここに言語を入力
#scrap.rake
 doc = Nokogiri::HTML.parse(html,nil,'Shift_JIS')
 
  doc.search("table a").each do |nodesets|
　name = nodesets.inner_text 
　feed = Prescraping.new(:name  => name)
　feed.save
end

 doc.search("table b").each do |nodesets|
 number = nodesets.inner_text 
　feed = Prescraping.new(:number  => number)
　feed.save
end

table aの値にはテーブル内でa hrefのリンクタグで囲っている名前のテキスト(複数あります)を取ってきており、
table bの値には同じくテーブル内にあるbタグの中にある数が書いてあるテキスト(こちらもnameの数だけあります)を取ってきているとします。

理想としては

id      name        number
1       tanaka          15
2        satoh           18
3       watanabe      20
4       katoh        　　　 23

といった形でDBで保存したいのですが現状、上のコードですと

id      name        number
1       tanaka         
2        satoh           
3       watanabe      
4       katoh         
5　　　　　　　　　　　　                 15
6                              18
7                              20
8                              23

といったようにsearch(table a)の値を全てレコードに保存した後に、numberカラムにレコードが保存されている形となっております、上のように綺麗にレコードに値を保存していくにはrakeファイルをどのように書き換えれば良いでしょうか？
よろしくお願いいたします。

Answer 1

ここでは、「name, number 2つの情報を統合する」という点を考える必要があるのですね

それぞれを一度配列として保存することで、そのあとの統合する処理は実現できそうです

# 仮に [:a, :b, :c] が入るとする
names =
  doc.search('table a').map do |nodesets|
    nodesets.inner_text
  end

# 仮に [1, 2, 3] が入るとする
numbers =
  doc.search('table b').map do |nodesets|
    nodesets.inner_text
  end

# transpose するので、配列は [[:a, 1], [:b, 2], [:c, 3]] に変換される
[names, numbers].transpose.each do |name, number|
  feed = Prescraping.new(name: name, number: number)
  feed.save
end

今回の例だと nodesets.inner_text の値を取り出すという単純なことをしているので、その部分は次のように書くことができるかもしれません

names = doc.search('table a').map(&:inner_text)

何か参考になれば幸いです

Links

• オブジェクト指向スクリプト言語 Ruby リファレンスマニュアル (Ruby 2.5.0)
• instance method Array#transpose (Ruby 2.5.0)
• module Enumerable#map (Ruby 2.5.0)
• プロを目指す人のためのRuby入門 言語仕様からテスト駆動開発・デバッグ技法まで Software Design plus | 伊藤 淳一 | コンピュータ・IT | Kindleストア | Amazon

Comments

[widget11]

ありがとうございます！
二つのカラムを統合するという考えがなく、且つネットで探してても見つからなかったので非常に嬉しいです！