modelをdebugしたい

Question

Rails歴２ヶ月ほどの、ど素人なのですが、今回初めてリファクタリングをしたいと思っています。ですが、なかなか思う様に動いてくれず足踏みしています。オブジェクト指向やリーダブルコードなどの本は一式読んだのですが、実際にリファクタリングをするとなると、どうも上手く動きません。以下には、今まで試してみたことや実現したいことなどを記載しますので、ぜひお力添えをいただければと思います。

前提条件

Rails: 5.2.3
OS: Mac
デバック方法: コードの中に**「byebug」**を埋め込む。

※このモデルはスクレイピング用のモデルです。モデルに処理を全て書き、「whenever(gem)」を使ってモデルに対して定期実行を行います。（つまり、メソッドをViewに書く想定はしていません）

実現したいこと

**「モデルファイルの、メソッドの処理をデバッグ」**したい。

#app/models/clone.rb

class Clone < ApplicationRecord

  def self.set_url_for_scraping(url)
    web_url = 'url'
    unparsed_page = HTTParty.get(web_url)
    parsed_page = Nokogiri::HTML(unparsed_page)
    #ここに「byebug」を埋め込んで、ターミナルでデバッグしたい
  end

試したこと

①デバック用にModelファイルを作成。（中身は同じ）
②メソッドの中に「byebug」を埋め込む。

#scrape.rb(Gemfileと同じ階層)
class Clone < ApplicationRecord

  def self.set_url_for_scraping(url)
    web_url = 'url'
    unparsed_page = HTTParty.get(web_url)
    parsed_page = Nokogiri::HTML(unparsed_page)
    byebug #デバッグのため
  end

③の１: ターミナルで**「$ rails r scrape.rb」**を叩く
→Running via Spring preloader in process .....(ただただ走り抜けた)

③の２: ターミナルで**「$ rails r scrape.rb.set_url_for_scraping」**を叩く
→undefined local variable or method `scrape' for main:Object（未定義になる）

わからないこと

①Modelをデバックするために一時的に作るファイルは、どの階層に置くのか。
（自分はGemfileと同じ階層に置きました）

②byebugを埋め込んだメソッドを、指定して実行するコマンドはどんなものか。
(自分は③の1、③の2のように実行して失敗しました)

以上のようなことが解決できず、質問させていただきました。
もしかすると、Modelファイルのメソッドをデバックするという発想自体、間違っているのかもしれませんが、ぜひそのあたりも未熟な自分にご教授いただければ幸いです。

Comments

[m.ts10806]

回答依頼いただきましたがRuby未経験につき、対応できませんので、あしからず…

[Tsuyoponpon]

失礼いたしました。

ご返信ありがとうございます！

Answer 1

warabimochiさんが書かれているとおりですが、さらに「デバッグ」と「テスト」というものをその位置づけやコストなどを比較して考えてみると、今後にも役立つのではないかと思います。

Comments

[Tsuyoponpon]

10tm84　さん

アドバイスありがとうございます！
おっしゃる通り、位置付けやコストなどはほとんど学べていませんでしたので、これを機会に学んでみたいと思います！

貴重なご意見、ありがとうございました！

Answer 2

いろんな方法がありますが、各自がいろんな方法を模索しないでよいように app/models/xxx.rb であれば test/models/xxx_test.rb のファイル名で、一時的ではなく永続的なテストを書くのが Rails の慣習になっています。
今回の app/models/clone.rb であれば test/models/clone_test.rb のファイル名になります。
そのなかで Clone.set_url_for_scraping の動作確認を行うコードを書けばよいです。

それよりも一時的なものであれば、コマンドラインで rails r 'Clone.set_url_for_scraping(...)' のようにすればよいです。

app/models/clone.rb のデバッグをしたいのに、対象が異なる同じ内容のものを別の場所に作って実行しようとする意図はよく理解できませんでした。
Modelファイルのメソッドをデバックする発想は何も間違っていません。

Comments

[Tsuyoponpon]

warabimochi さん

ご回答ありがとうございます！
技術的なことからRailsの慣習についても教えていただき、非常に勉強になりました！

おかげさまで学習が再開できそうですので、Railsの慣習やデバッグについて深く勉強したいと思います！
つきましては、ベストアンサーに選ばせていただき、重ねてお礼申し上げます。