(位置ゲー)コルーチンのwhile内部で、前回処理の最後に変数へ代入した位置情報が上書きされてしまう。

Question

前提

UnityでiPhone用の位置情報アプリを作成しています。
後述のコルーチンで3秒ごとの移動距離を取得したく色々試したのですが、うまくいきません。

実現したいこと

・コルーチンで3秒ごとに端末緯度経度を用いて移動距離を算出したい。
・while内で新しく位置情報を取得した際にoldLocationが上書きされないようにしたい。

発生している問題・エラーメッセージ

・恐らくwhile内下部でoldLocationを更新した後、whileループの最初に戻ってから3秒後に現在の端末位置を更新した際に、計算処理を行う前にoldLocationも上書きされてしまっているのだと思います。
計算処理を行なっても移動距離が0mになってしまっています。

該当のソースコード

//目標距離までの残り距離
public static int remainingDistance = 1000;
//前回while処理時の位置情報
private Location oldLocation;

〜中略〜

private IEnumerator GetDistance()
    {
        var cachedWait = new WaitForSeconds(3);

        LocationInfo locationInfo = Input.location.lastData;

        oldLocation = new Location(locationInfo.latitude, locationInfo.longitude);
        //実機テスト用デバッグテキスト
        debug1.text = string.Format("3秒前緯度{0},経度{1}", oldLocation.Latitude, oldLocation.Longitude);

        while (remainingDistance >= 0)
        {
            yield return cachedWait;

            LocationInfo locationInfo1 = Input.location.lastData;
            //現在の端末の緯度経度           
            Location deviceLocation = new Location(locationInfo1.latitude, locationInfo1.longitude);

            debug2.text = string.Format("現在緯度{0},経度{1}", deviceLocation.Latitude, deviceLocation.Longitude);

            //移動距離(メートル)　二点間の緯度経度から距離を計算する処理
            int distance = (int)NaviMath.LatlngDistance(deviceLocation, oldLocation) * 1000;

            text2.text = string.Format("{0}", distance);

            remainingDistance -= distance;

            text.text = string.Format("残り{0}m", remainingDistance);
            //更新
            oldLocation = deviceLocation;        
            
        }

        //ここは無視してください
        text2.text = string.Format("");
        arrivalpanel.SetActive(true);
        yield break;

    }

試したこと

試しにstartメソッド内でInput.location.lastDataを使ってoldLocationに端末位置を代入したら、処理一回で移動距離に1400万mほど加算されてしまいました。こちらの原因は分かりませんでした。
実在地の緯度経度で試算を行った際にはおおよそ正確な値が出たので計算処理は間違っていないはずです。

Answer 1

上書き云々よりも
キャストと乗算はキャストのほうが優先順位が高いです

例えば(int)0.5*1000は0です、500ではありません

int distance = (int)(NaviMath.LatlngDistance(deviceLocation, oldLocation) * 1000);

Comments

[YA87]

回答ありがとうございます。
ここ数日間のモヤモヤが解消しました。本当に助かります。

[ozwk]

oldLocationが上書きされているという仮説を立てたところまでは素晴らしいので
次は仮説を検証してみましょう

[YA87]

今思えばdeviceLocationを取得した後、計算を行う前に一度yield returnで中断などすればテキストの挙動から本当に上書きされているのかが確認できたかもしれません…
今回は指摘いただいた箇所を、もともとのdouble型で計算してからint型へ変換するよう書き換えた結果、想像していた処理を実装することができました。
今後は検証もしっかり行うよう心掛けます。回答ありがとうございました。