既に検索されたことがある施設の場合は、検索される度に新たなPlaceのレコードに上書きする

pekorinko commented 3 years ago

現状とこれからやること（その理由）

現時点ではPlaceが既に存在している場合、一番最初に検索された際のPlaceを返す仕組みとなっている
そのため同じ施設を再度検索した際に万が一住所が変わっている、Googleの星評価が前回検索した時から変わっていた場合にGoogleの口コミ情報と本サービスが算出した情報との間に乖離が生まれる
現状、過去に検索されたことがある施設を検索した際は、Googleの平均星評価が取れない状況にある
関連するissue #65

pekorinko commented 3 years ago

試したこと

place_data_scraper.rbのsave_placeメソッドを以下に書き換え
- lrdは一意である必要があるため過去に検索された施設であっても更新しない
- @result = @scrape_process.fetch_placeの内容としてはselenium_tool.rbのfetch_placeメソッドの返り値であり、具体的内容としてはhash = { place_name: facility_name, address: address, star_ave: star_ave }が返ってくる
- もし既に検索されたことがある施設の場合はPlaceのレコードをlrd以外（place_name, address, star_ave）、最新にアップデートする
- 初めて検索される施設の場合は新たにPlaceのレコードを作成する
```
def save_place
uri = URI.parse(@url)
@lrd = uri.fragment.split('=')[1].split(',')[0]
place = Place.find_by(lrd: @lrd)

@result = @scrape_process.fetch_place
if place.present?
return(
  place.update(
    place_name: @result[:place_name],
    address: @result[:address],
    star_ave: @result[:star_ave],
  )
)
end

create_place
end
```

def create_place Place.create( lrd: @lrd, place_name: @result[:place_name], address: @result[:address], star_ave: @result[:star_ave], ) end

## 上記を試した結果のエラー（既に検索したことがある施設で実施した）
- エラー内容：results_controller.rbの４６行目`place_id = place.id`は定義されていないと怒られている
- 恐らくこのエラーが出ている原因は`place_data_scraper.rb`の`save_palce`メソッドの以下コードが原因（`place.present?`がtrue=Placeが既に存在する）際は`place_name`,`address`,`star_ave`を更新する）
- 上記箇所を原因として考えた理由としてはエラーメッセージで`NoMethodError (undefined method `id' for true:TrueClass):`となっている

【エラーの原因となっていると思しき箇所】→place_data_scraper.rbのsave_palce` if place.present? return( place.update( place_name: @result[:place_name], address: @result[:address], star_ave: @result[:star_ave], ) ) end

【エラーの内容】 NoMethodError (undefined method `id' for true:TrueClass):

app/controllers/results_controller.rb:46:in `create'


![image](https://user-images.githubusercontent.com/61894818/118734760-fff91680-b879-11eb-9368-f8d8bbaa6e5b.png)

## 既に存在する施設の場合はレコードを上書きするコードを書く際に参考にした記事
- Railsドキュメント データベースを更新 https://railsdoc.com/page/model_update
## 新たに検索する施設に関しては上記のようにコードを書き換えたとしても問題なく動く
![image](https://user-images.githubusercontent.com/61894818/118735815-2f108780-b87c-11eb-924c-84df7d416a5c.png)

pekorinko commented 3 years ago

place_name, address, star_aveのupdateはrails cでやればちゃんと上書きされていること確認済み【レコードを作る】【既にあるlrdのplace_name, address, star_aveを書き換えた】 place.update(place_name:"米屋",address:"藤沢",star_ave:2.0)

pekorinko commented 3 years ago

NoMethodError (undefined method `id' for true:TrueClass):の原因

results_controller.rbのcreateメソッドの以下部分においてplaceをputsした結果trueが返ってきた

本来であればPlaceのレコードが返ってくるはずなのにtrueが返ってきてしまっているのが本エラーの原因

【`results_controller.rb`のcreateメソッド】
place = place_data_scraper.save_place
  puts "#{place}======================="
  place_id = place.id

【本来であればPlaceのレコードが返ってくるはずなのに`true`が返ってきている】
true=======================

解決

既に存在する施設の場合はPlaceのレコードをアップデート、そのアップデートしたPlaceのレコードを戻り値とするようにした 【place_data_scraper.rb save_placeメソッド　改善前】

if place.present?
    return(
      place.update(
        place_name: @result[:place_name],
        address: @result[:address],
        star_ave: @result[:star_ave],
      )
    )
  end

【place_data_scraper.rb save_placeメソッド　改善後】

if place.present?
    place.update(
      place_name: @result[:place_name],
      address: @result[:address],
      star_ave: @result[:star_ave],
    )
    return place
  end

今回のエラーを通しての気付き

updateメソッドはレコードの更新を行う役割を担っているのであり、更新後のレコードを返すことまではしない

pekorinko / review_check