Question

I’m having trouble getting the GPS’s onLocationChanged to run on a different thread. I understand how to manage UI thread when I’m calling a function but with the GPS, I don’t actively call the function.

My intent is to have a light flash every time the GPS receives a reading. I have put this function in a Runnable. I passed this function to a class that implements LocationListener. Then in the main class, I started a new thread that calls requestLocationUpdates. I was hoping that onLocationChanged of the LocationListener would run in a different thread, post to the callback and make the necessary UI effects in the UI thread. Unfortunately, the program crashes every time it tries to call requestLocationUpdates. What’s the proper way of doing it?

Right now it looks something like this

Main class:

final Runnable changeLight = new Runnable(){
        public void run(){
            // do stuff
        }
    };

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.capture);

        status = (ImageView) findViewById(R.id.status);
        database = new DatabaseManager(this);

        new Thread(){
            public void run(){
                location = (LocationManager) getSystemService(LOCATION_SERVICE);
                listener = new GPSManager(database, changeLight, light, status);
                location.requestLocationUpdates("gps", 10000L, 0, listener);
            }
        }.start();
    }

LocationListener class:

public void onLocationChanged(Location location) {
        if (location.getAccuracy() <= 32.0){
            light = lightColors.Green;
            status.post(callback);
            database.insertData(location);
        }
        else{
            light = lightColors.Yellow;
            status.post(callback);
        }
    }

The exception says Can’t create handler inside thread that has not called Looper.prepare()

Answer 1

The thread where you call location.requestLocationUpdates must have a Looper ( see here ).
You can use HandlerThread instead of Thread.