Question

in this app, the user logs in and their credentials are checked against a server.

The user could be waiting a few seconds, depending on how fast the phone can open a data connection if at all. I need dialog box saying “please wait” or “verifying credentials” or something a long those lines after the user clicks log in.

Desired visual order: press log in -> “please wait” dialog is show in this same activity -> when result comes in from server a new activity is loaded (or error is thrown)

Current visual order: press log in -> user waits as if the app is frozen -> new activity is loaded

I’m trying to do this threading thing with AsyncTask but I’m just not getting it right now!

class Progressor extends AsyncTask<Void, Void, Void> {

ProgressDialog dialog;
    protected void onPreExecute(){
        dialog = ProgressDialog.show(Login.this, "Logging In", 
                "Verifying Credentials, Please wait...", true);
    }

Then in my oncreate method I had all of the other logic like user clicking the button and stuff, but I’ve since moved that into the AsyncTask method’s doInBackGround function

  /* When the Login Button is clicked: */
        Button loginButton = (Button) findViewById(R.id.loginButton);
        loginButton.setOnClickListener(new OnClickListener() {            


            public void onClick(View v) {

                Progressor showMe = new Progressor();
                showMe.onPreExecute();
                showMe.doInBackground(null);
                showMe.onPostExecute();

and onPostExecute simply dismisses the dialog box

Why doesn’t this work and how should it be re-arranged. What variable should I be passing into the showMe.doInBackGround() function, it is void. In debugging it never goes in here

    @Override
    protected Void doInBackground(Void... arg0) {

Answer 1

That isn’t how you use an AsyncTask, have a look at the documentation. Once you have created a new instance of your task, just call execute(), not the individual methods:

Progressor showMe = new Progressor();
showMe.execute();