ruby 配列で数字を+1ずつ増やしたい

Question

vod1 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[n])
 @vod1 =   vod1.map{|a| a.attribute('alt')}

vod1の最後のnの部分を+1ずつ増えるようにしたいのですが、どのように書けばいいのでしょうか？

 <% if @arrays.present? %>
<% @arrays.each do |array| %>
    <br>
    <div><%= array["title"] %></div>
    <br>
    <div><%= image_tag 'https://image.tmdb.org/t/p/w500'+array["poster_path"] %></div>　　　　　
    <br>
    <div><%= array["overview"] %></div>
    <br>
    <div><%= array["release_date"] %></div>
   
   
   <% if @vods.present? %>
   
　　<p><%= @vod1 %></p> 
　　<% if @vod2.present? %>
　　<p><%= @vod2 %></p>
　　<% end %>
　　<% if @vod3.present? %>
　　<p><%= @vod3 %></p>
　　<% end %>
　　<% if @vod4.present? %>
　　<p><%= @vod4 %></p>
　　<% end %>
　　
　　
    <% end %>
    
    
<% end %>

Answer 1

n = 10
(1..n).each { |i|
  vod1 = ....div[i])
}

こういうことですかね。

Comments

[pokerStars]

controller
n = 10
(1..n).each { |i|
vod1 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[i]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[1]/div/a/div/img')
vod2 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[i]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[2]/div/a/div/img')
vod3 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[i]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[3]/div/a/div/img')
vod4 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[i]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[4]/div/a/div/img')
}

doc = Nokogiri::HTML.parse(driver.execute_script('return document.documentElement.innerHTML'))

# 検索結果のタイトルを表示
doc.xpath('/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]').each do |node|
@vod1 = vod1.map{|a| a.attribute('alt')}
@vod2 = vod2.map{|a| a.attribute('alt')}
@vod3 = vod3.map{|a| a.attribute('alt')}
@vod4 = vod4.map{|a| a.attribute('alt')}

このようにしてみたのですが、undefined local variable or method `vod1'
こういったエラーになってしまうのですが、書き方がおかしいのでしょうか？

[dice142]

vod1がeach内でしか使えない変数になっているのだと思います。
どういう使い方をしたいのかわからないのですが、ループ内で完結させるかviewにわたすなら@vod1などを配列にするのが良いかと。

[pokerStars]

上記のコードは全てcontrollerでの記述で、div[i]の数字を変えて

view
<% if @arrays.present? %>
<% @arrays.each do |array| %>
<br>
<div><%= array["title"] %></div>
<br>
<div><%= image_tag 'https://image.tmdb.org/t/p/w500'+array["poster_path"] %></div>　　　　　
<br>
<div><%= array["overview"] %></div>
<br>
<div><%= array["release_date"] %></div>

<% if @vods.present? %>

　　<p><%= @vod1 %></p>
　　<% if @vod2.present? %>
　　<p><%= @vod2 %></p>
　　<% end %>
　　<% if @vod3.present? %>
　　<p><%= @vod3 %></p>
　　<% end %>
　　<% if @vod4.present? %>
　　<p><%= @vod4 %></p>
　　<% end %>

　　<% else %>
　　<p>配信しているvodはありません!!</p>
<% end %>
<% end %>
<% end %>

表示する情報を変えたいと思っています。

[pokerStars]

viewは上記の記述です

[pokerStars]

元は、
vod1 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[1]/div/a/div/img')

としていてdiv[1]しか取得できないので数字を配列ごとにdiv[2],div[3]と変えたいと思っています。

[dice142]

配列ではなく、同じ変数を使っている限りview側で表示を変えることはできないと思うのですが。
controller側でループで数値を変えても、同じ変数に入れてる限りは最後の値しか格納されません。
view側を見る限りループさせているわけでもなさそうですし、どう表示したいのかがいまいち伝わってないです。

[pokerStars]

理想の表示は、作品ごとにみれるvodが違うので表示を変えたいのですが、
例. 現在　検索名「ゴジラ」
作品名「ゴジラ」
vod[amazonプライム]　　

作品名「ゴジラ2」
vod[amazonプライム]　　

作品名「ゴジラ3」
vod[amazonプライム]　　

理想
作品名「ゴジラ」
vod[amazonプライム]　　

作品名「ゴジラ2」
vod[hulu]　　

作品名「ゴジラ3」
vod[Netflix]　　

こういう風にしたいのですが、全部amazonプライムになってしまうという状態です。

[pokerStars]

そのためにdivの数字を1ずつ足していけば解決するのですが、どのようにすればいいかわからないという感じです

[dice142]

作品名とvodとやらはviewにある@array毎に違うということですよね？
なのにcontroller側で@vod1などのように設定したら@arrayに関係ないのは当然ではないでしょうか？

やるのであれば@arrayのように配列で同じ順で格納するとか@arrayの各要素に追加するとかになります。

[pokerStars]

そうですね！
@vod1がおかしいのでしょうか？
@vod1はseleniumで指定したxpathのalt属性を表示するという処理を入れているのですが、いまいちどうすればいいかわかりません。

view をいじるのでしょうか？

[dice142]

配列が | A | B | C | D | と4作品分あるとして、
vod1は | X | と1つ分しか格納できないので、いくらcontroller側で
void1 = ...div[i]...
としても1作品分しか格納できません。
A作品のvod、B作品のvod...というようにしたいのなら
vod1も | X | Y | Z | W | のように配列で格納するか
元のarrayに | A,X | B,Y | C,Z | D,W | のようにしないとダメです。

[pokerStars]

controller

def index

require 'uri'
require 'net/http'
require 'json'

@name = params[:name]
a = URI.encode("https://api.themoviedb.org/3/search/movie?api_key=!!&language=ja&query="[e-mail]_s)
url =URI(a)

http = Net::HTTP.new(url.host, url.port)
http.use_ssl = true
http.verify_mode = OpenSSL::SSL::VERIFY_NONE

request = Net::HTTP::Get.new(url)

request.body = "{}"

response = http.request(request)

hash= JSON.parse(response.body)

@arrays = hash["results"]

require 'selenium-webdriver'
require 'nokogiri'

options = Selenium::WebDriver::Chrome::Options.new
options.add_argument('--headless')
options.add_argument('--disable-gpu')

driver = Selenium::WebDriver.for :chrome, options: options

driver.navigate.to 'https://www.justwatch.com/jp/検索?q=' [e-mail]_s
wait = Selenium::WebDriver::Wait.new(timeout: 300)
wait.until { driver.find_element(xpath: '//div[2]/filter-bar/div[2]/div[2]/div[3]/div[2]/div',) }

vod1 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[1]/div/a/div/img')
vod2 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[2]/div/a/div/img')
vod3 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[3]/div/a/div/img')
vod4 = driver.find_elements(:xpath, '/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]/div[4]/div/a/div/img')

doc = Nokogiri::HTML.parse(driver.execute_script('return document.documentElement.innerHTML'))

# 検索結果のタイトルを表示
doc.xpath('/html/body/div[2]/filter-bar/ng-transclude/core-list/div/div/div[1]/search-result-entry/div/div[2]/div[2]/div/div[1]/div/div/div/div/div[2]/div[1]/div[2]').each do |node|
@vod1 = vod1.map{|a| a.attribute('alt')}
@vod2 = vod2.map{|a| a.attribute('alt')}
@vod3 = vod3.map{|a| a.attribute('alt')}
@vod4 = vod4.map{|a| a.attribute('alt')}
@vods = [@vod1, @vod2, @vod3, @vod4]
end
driver.quit

end

view

<p><b>Hulu, U-NEXT,dtv,GYAO!,Amazonprime,Netflixの中から配信中のvodを見つけられます！</b></p>

<%= form_tag({action: :index}, {method: :get}) do %>
<br>
<%= label_tag :name, '作品名' %>
<%= text_field_tag :name, @name %>

<%= submit_tag '検索', data: { disable_with: '送信中'} %> ※作品名がヒットしない場合、英語入力も試して見てください！！
<% end %>

<% if @arrays.present? %>
<% @arrays.each do |array| %>
<br>
<div><%= array["title"] %></div>
<br>
<div><%= image_tag 'https://image.tmdb.org/t/p/w500'+array["poster_path"] %></div>　　　　　
<br>
<div><%= array["overview"] %></div>
<br>
<div><%= array["release_date"] %></div>

<% if @vods.present? %>

　　<p><%= @vod1 %></p>
　　<% if @vod2.present? %>
　　<p><%= @vod2 %></p>
　　<% end %>
　　<% if @vod3.present? %>
　　<p><%= @vod3 %></p>
　　<% end %>
　　<% if @vod4.present? %>
　　<p><%= @vod4 %></p>
　　<% end %>

　　<% else %>
　　<p>配信しているvodはありません!!</p>
<% end %>

<% end %>

[pokerStars]

全てを表示するとこんな感じなのですが、
aaa = @array.@vod1 こんな感じでまとめるという意味でしょうか？

[dice142]

Rubyの配列や連想配列を復習したほうが良い気がします。

[pokerStars]

配列の復習をしたのですが、@arraysに別の配列を入れ込むことができません
@arrays = hash["results"]これと、
vod1 = -div[1]-
vod2 =--div[1]-
vod3 =--div[1]-
vod4 =--div[1]-

vods = [vod1,vod2,vod3,vod4]

これを繋げて、まとめて出力すればいいということですよね？

[pokerStars]

num = 0
n = num + 1

div[n]みたいにして足していきたいです

[dice142]

@arraysの各要素は連想配列だと思うので、その連想配列に「vod」を追加すると
view側でarray['vod']といった感じで出力できるようになります。

もしくは@arraysと同じ順番のvod用配列を用意すれば
([e-mail]).each { |i|
# @arrays[i]["title"]
# @vod_array[i]
}
で出力できるようになります。

[dice142]

そもそも@arraysと@vod1～@vod4の関係性がわからないので噛み合ってない気もします。

[pokerStars]

@arraysがapiで取得した配列で
@vod1はseleniumのスクレイピングで取得したものになります。

ありがとうございます！
一度やってみます