【リファクタリング】口コミが無いGoogleのURLが入力された際にエラーを出さず「口コミの取得に失敗しました」などのアラートを出すようにしたい_0530_MKさん質問事項

pekorinko commented 3 years ago

例）口コミがそもそも存在しないGoogleの検索結果のURL

現状

口コミが存在しないhttps://www.google.com/~が入力された場合以下エラーが起こる


Selenium::WebDriver::Error::TimeoutError (timed out after 30 seconds (no such element: Unable to locate element: {"method":"css selector","selector":".lcorif"}
(Session info: headless chrome=90.0.4430.212))):

lib/my_tools/selenium_tool.rb:10:in initialize' lib/my_tools/place_data_scraper.rb:7:innew' lib/my_tools/place_data_scraper.rb:7:in initialize' app/controllers/results_controller.rb:43:innew' app/controllers/results_controller.rb:43:in `create'

![image](https://user-images.githubusercontent.com/61894818/119823369-b078b180-bf2f-11eb-81ca-34a01ffff680.png)
## エラーの原因
- `selenium_tool.rb`のinitializeメソッドで口コミモーダルのHTMLに存在する`lcorif`クラスが表示されるまで30秒待つ処理を行っているため、口コミが存在しないGoogleのURLの場合は`Selenium::WebDriver::Error::TimeoutError`が起こる

【selenium_tool.rb】 def initialize(url) @url = url options = Selenium::WebDriver::Chrome::Options.new options.add_argument('--headless') @d = Selenium::WebDriver.for :chrome, options: options wait = Selenium::WebDriver::Wait.new(timeout: 30) @d.get(@url) wait.until { @d.find_element(:class_name, 'lcorif').displayed? }←口コミモーダルが表示されるまで30秒待つ end


口コミモーダル全体に割当てられている`lcorif`クラス
![image](https://user-images.githubusercontent.com/61894818/119824415-d81c4980-bf30-11eb-9e95-92bbb8078d7e.png)

pekorinko commented 3 years ago

解決→`results_controller.rb`の以下コードを`begin~rescue`で囲った

begin
        place_data_scraper = MyTools::PlaceDataScraper.new(@url)
        place = place_data_scraper.save_place
        place_id = place.id
        place_data_scraper.save_review(place_id)
        @place = place_data_scraper.save_place
        user_id = session[:user_id]
        check_credibility = MyTools::CheckCredibility.new(place_id)
        @result = check_credibility.credibility(user_id)
        redirect_to result_path(@result)
      rescue StandardError
        redirect_to root_path, notice: '口コミの取得に失敗しました'
      end

解決に至った背景

当初はエラーの原因となっていたlib/my_tools/selenium_tool.rb:10:ininitialize'をbegin~rescue`で囲った
だが、その部分のエラーは免れても、place_data_scraper.rbのsave_placeメソッドでエラーが起こるなど、特定のメソッドやクラスでエラーを免れても、URLを使うクラスやメソッドがある限りエラーが起こり続けると思った

上記の気付きから、「各URLを使用するクラスを呼び出しているresults_controller.rbのcreateメソッドの上記部分をbegin~rescueで囲えば良いのでは？」という発想に至った

MKさんに伺うこと：コントローラーに`begin~rescue`の処理を書くのはrailsの作法的に正しいのか？

【results_controller.rb】
def create
@url = params[:url]

url_filter = MyTools::UrlFilter.new(@url)
@url = url_filter.filter

url_validator = MyTools::UrlValidator.new(@url)

if url_validator.validate
  begin　←【railsの作法的にbegin~rescue処理をコントローラーに書くのは大丈夫なのか？】
    place_data_scraper = MyTools::PlaceDataScraper.new(@url)
    place = place_data_scraper.save_place
    place_id = place.id
    place_data_scraper.save_review(place_id)
    @place = place_data_scraper.save_place
    user_id = session[:user_id]
    check_credibility = MyTools::CheckCredibility.new(place_id)
    @result = check_credibility.credibility(user_id)
    redirect_to result_path(@result)
  rescue StandardError
    redirect_to root_path, alert: '口コミの取得に失敗しました'
  end
else
  redirect_to root_path, alert: '不正なURLです'
end
end

回答

コントローラーにbegin~rescueを書くこと自体は問題ない
アプリケーションコントローラーにbegin~rescueを書いて全画面で共通の
「現在メンテナンス中です」→全部のコントローラーでbegin~rescueを書くのは面倒なのでapplicationコントローラーにまとめて書くことが多い

pekorinko commented 3 years ago

【MKさんに伺うこと_0530】Googleでは無い口コミURLが入力された際の新たに発見した問題点

Gooleの口コミが無いURLを検索した後に改めてGoogleの口コミURLを入力して調査ボタンを押すと「口コミの取得に失敗しました」とアラートが出てしまう「口コミの取得に失敗しました」とアラートが出た後に再度、口コミURLを入力して「調査」ボタンを押すと正常に挙動し、口コミの調査結果を表示してくれる

pekorinko commented 3 years ago

46: place = place_data_scraper.save_place

全部作り終わった後に再考する

解決

→HTMLの構造改善（application.html.erbにCSSをまとめる、_form.html.erbの廃止）をしたら自然にバグが解消された

pekorinko / review_check