位置登録処理の改善 #749

元々の位置情報取得処理だと、誤差が大きくなりやすいので改善

改善後の位置登録処理
1. 10秒間位置情報を継続取得
2. 誤差が300メートル以下なった段階で、位置情報を登録
3. 10秒経っても誤差が300メートル以下にならなかったら、最後に取得した位置情報を登録
4. apiでエラーが発生するか、10秒経って位置情報が取れなければエラー表示

誤差の許容範囲は実際に位置登録して調整予定

#log_user_locationに誤差情報を格納カラム追加
ALTER TABLE log_user_location ADD accuracy float DEFAULT '0' AFTER latitude;

参考
https://youcune.com/2012/01/12/221628/

@kouho @iwamoto
レビューお願いします。

デバッグユーザで本番テスト後に全ユーザに解放予定です。
httpだと位置登録できない可能性があります。
その場合は、デバッグツールでロケーションを変更してください。

!スクリーンショット 2021-04-20 19

@wada
誤差が300メートル以上の状態の処理を見ようとしているのですが、
Main-Location.tpl291行目の position.coords.accuracy < limit_meter を 500<limit_meter
に変更しても297行目のgeo_success(position) に入らず、278行目のgeo_success(position) に入ってしまいます。
どのように確認したら良いでしょうか？

@kouho
それぞれのsuccessコールバックの呼び出しタイミングについて以下の通りです。

L278: setTimeoutのgeo_success(position);
accuracyが指定値(300)未満にならず、10秒過ぎるとこちらのsuccessが呼ばれます

L297: watchPositionのgeo_success(position);
accuracyが指定値(300)未満になると、こちらのsuccessが呼ばれます

300メートル以上は上のsuccess、300メートル未満は下のsuccessが呼び出されるので、コメントにある挙動であっています。
> 297行目のgeo_success(position) に入らず、278行目のgeo_success(position) に入ってしまいます。

@wada
なるほど、了解です。1~4に加えてlog/error_location.txtにエラーログが出力されているのを確認しました。
誤差の許容範囲については判断がつきませんが、それ以外については問題ないかと思います。

@wada
レビューしました、現状では問題ないと思います。