正規表現.subのエラー

Question

###前提・実現したいこと

投稿されたyoutubeの動画一覧ページを作るために
投稿されたyoutubeのURLからcontroller上で動画IDを抜き出して
viewファイルに渡したいと思っています。

カラムの値
https://www.youtube.com/watch?v=RV0-bmVs_ic

抜き出したいのは
RV0-bmVs_ic
の部分です。

※投稿されたURLは
Moviesモデルで持っているimageというカラムに格納されています

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

undefined local variable or method `youtube_url' for #MoviesController:0x007fd8302b5448 Did you mean? youtube_urls

youtube_urls = Movie.select("image")
youtube_urls.each do |youtube_url|
end
youtube_element = youtube_url.sub("https://www.youtube.com/watch?v=", "") ←エラーの箇所

###該当のソースコード

youtube_urls = Movie.select("image")
youtube_urls.each do |youtube_url|
end
youtube_element = youtube_url.sub("https://www.youtube.com/watch?v=", "")

###試したこと

eachの後にbinding.pry
をしてURLが抜き出せていることは確認できました。

subでURLを置き換える方法をいろいろ調べたのですが
正しいやり方がわからず。。。

教えていただきたいです。
よろしくお願い

###補足情報(言語/FW/ツール等のバージョンなど)
よろしくお願いいたします。

Answer 1

youtube_urls = Movie.select("image") 
youtube_urls.each do |youtube_url| 
end 
youtube_element = youtube_url.sub("https://www.youtube.com/watch?v=", "") 

は

youtube_urls = Movie.select("image") 
youtube_urls.each do |youtube_url| 
youtube_element = youtube_url.sub("https://www.youtube.com/watch?v=", "") 
# ここにyoutube_elementを使った処理を記述する（viewファイルに渡すとか？）
end 

の間違いでしょうか？
少なくとも上の記述ではeachの中で何もしてません。
eachループを使う時は、each文とendの間にやりたい処理を記述しましょう。

Comments

[tallryo]

コメントありがとうございます！
そもそもeachの中身が書けていなかったですね・・・

教えていただいた方法で解決しました！