Always getting same speed from GPS location - Kotlin

Question

I'm trying to get the user's speed 10 times in three seconds when a function is called, and then calculate the sum of all the gotten values. I have the global variable:

lateinit var fusedLocationProviderClient : FusedLocationProviderClient

Which I then initialize in the onCreate method like this:

fusedLocationProviderClient = LocationServices.getFusedLocationProviderClient(this)

And then this is the code where I try to calculate the sum of the speeds:

val task = fusedLocationProviderClient.lastLocation
var sumSpeed = 0F
task.addOnSuccessListener {
    if (it != null) {
        for (i in 1..10) {
            Thread.sleep(300)
            sumSpeed += it.speed
        }
    }

    current_gps_speed_tv.text = "Total is: $sumSpeed"

    if (sumSpeed < 25) {
        ...
    }
}

The result of sumSpeed always remains the same (34m/s), which is 10 times it.speed, which is also the same every time (3.4m/s).

I've searched for other answers (like this one) but what I'm using seems to be different. Why does this happen? Thank you.

user_506796 · Answer

This won't work for at least a few different reasons.

You're trying to poll it more than once per second. I think the fastest rate position is updated is at best once per second, and it's going to depend on device.
You are simply adding the same value to the sum 10 times and sleeping the thread pointlessly. Calling sleep pauses execution of your code. It does not magically change the value of it to some new value polled from the GPS.
Calling Thread.sleep on the main thread will freeze your app.
You are polling lastLocation, which is some location already known from the last time the GPS was used. It only reports a single value that comes from the past. It cannot report multiple new values. Your success listener is called only one time, because either the last location is available or it's not.

To get multiple locations over a period of time, you need to use requestLocationUpdates instead of lastLocation. See here for the documentation.

Edit, to expand on 2 based on your comment:

Imagine this code:

val x = 5
var sum = 0
for (i in 0..9) {
    sum = sum + 5
}
println(sum)

This adds the value of x to sum 10 times, so the result will be 50.

Now consider this code:

val x = 5
var sum = 0
for (i in 0..9) {
    Thread.sleep(100)
    sum = sum + 5
}
println(sum)

This code has the exact same result. It just takes a lot longer to return because it keeps pausing to sleep. Sleeping has no effect on the value of x. x is always the same value. The same is true of it in your code. There is only ever one instance of it in your function.

Always getting same speed from GPS location - Kotlin

1 Answers