hirodesu85 / GIFTech2024_backend

0 stars 0 forks source link

fix: nearで検索して1件も見つからなかった場合無限ループになってしまうので修正 #30

Closed ryichk closed 4 months ago

ryichk commented 4 months ago

すみません、目的地提案APIでnearで検索して1件もヒットしないと無限ループになってしまうので修正しました 🙏

→ nearの場合はmiddleとfarと違って検索範囲を変えないためです

ryichk commented 4 months ago

nearの場合はmiddleとfarと違って検索範囲を変えないためです

実装内容自体はありがたいですし良いと思うのですが、どのようなユースケースを想定してこれを実装しようと思ったのかは少し確認させていただきたいです 👀

というのも「使い過ぎて近場は行き尽くしてしまった」といったケースの想定ならこのままで良いと思うのですが、もし「ド田舎などそもそもの店舗数が少ない場所で利用する」などであればmiddleまでは同様の処理が必要だからです 🤔

@Yoshino-Yukitaro コメントありがとうございます! デモでは起こる確率はほぼないかなと思ったのですが、テストしていて見つけてしまい、無限ループになるとサーバーへの負荷が心配なので念のため修正した次第です! 🙏 (テストする時間によって周辺のサウナが見つからず..(営業時間中のサウナのみ検索するため)) おっしゃる通り、ケースとしてはmiddleまで考慮する方がより安全なのですが、明日のデモでは不要かなと思います 🙏

修正した箇所は、「使い過ぎて近場は行き尽くしてしまった」ケースではなく、その前段階の「近場に一つも該当する条件の場所がなかった場合」になります。