I push a web service call in my activity to a thread (shown below).

Question

0

Asked: May 22, 20262026-05-22T18:21:43+00:00 2026-05-22T18:21:43+00:00

I push a web service call in my activity to a thread (shown below).

0

I push a web service call in my activity to a thread (shown below). The first time I do this in the activity it works fine (gets the text from my edittext and loads the service to get lat/lng data)

But when I click the back button (emulator) and try to fire off this thread a second time it blows up after the .start(); in my click handler. What might I be doing wrong here? thanks

private Thread getLocationByZip = new Thread() {
    public void run() {
        try {
            EditText filterText = (EditText) findViewById(R.id.zipcode);
            Editable zip = filterText.getText();

            LocationLookupService locationLookupService = new LocationLookupService();
            selectedLocation = locationLookupService.getLocationByZip(zip.toString());

            locationHandler.post(launchFindWithLocationInfo);
        } catch (Exception e) {
        }
    }
};

private Runnable launchFindWithLocationInfo = new Runnable() {
    @Override
    public void run() {
        try {
            Intent abc = new Intent(LocationLookup.this, FindWithLocation.class);
            startActivity(abc);
        } catch (Exception e) {

        }
    }
};

public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.location);

    locationHandler = new Handler();
    findViewById(R.id.findbyzip).setOnClickListener(new View.OnClickListener() {

        @Override
        public void onClick(View view) {
            getLocationByZip.start();
        }
    });
}

Update

After the great advice I went with an AsyncTask so if anyone finds this going forward the above thread/handler model looks something like the below as an asynctask

private class LocationLookupTask extends AsyncTask<String, Void, Location> {
    private ProgressDialog dialog;

    @Override
    protected void onPreExecute() {
        this.dialog = ProgressDialog.show(LocationLookup.this, "", "Loading...");
    }

    @Override
    protected Location doInBackground(String... zips) {
        Location selectedLocation = null;
        for (String zip : zips) {
            LocationLookupService locationLookupService = new LocationLookupService();
            selectedLocation = locationLookupService.getLocationByZip(zip);
        }
        return selectedLocation;
    }

    @Override
    protected void onPostExecute(Location location) {
        this.dialog.dismiss();

        ((AppDelegate) getApplicationContext()).setSelectedLocation(location);
        Intent abc = new Intent(LocationLookup.this, FindWithLocation.class);
        startActivity(abc);
    }
}

Now to call this in the onclick you would do this

findViewById(R.id.findbyzip).setOnClickListener(new View.OnClickListener() {

            @Override
            public void onClick(View view) {
                EditText filterText = (EditText) findViewById(R.id.zipcode);
                Editable zip = filterText.getText();

                LocationLookupTask task = new LocationLookupTask();
                task.execute(new String[]{zip.toString()});
            }
        });