Chapter 2:Dialog project "force close" error

ichthus · April 28th, 2011, 10:33 PM

Hi, i am a 25 year software engineer-retired. i wrote most of my code in the 80s-90s...fortran, pascal, basic, and macro-32. now i am learning java with the eclipse ide for android os.

After getting several "The application DialogActivity has stopped unexpectedly. Please try again." with a "force close" option, i downloaded and executed the source code from your web site. i get the same results when i click the "Click to display a dialog box."

any ideas?
thanks

ichthus · April 29th, 2011, 12:54 PM

forgot to say that the downloaded code was programmed to show a progress dialog and it worked fine. i modified the showDialog(1) to be like the display a dialog activity showDialog(0) and it then fails.

ichthus

mart1826 · May 11th, 2011, 07:15 AM

Just to say that I am getting the same error. Any help would be appreciated.

Martin

sediverse · May 16th, 2011, 10:35 AM

Ah, it's crashing there because when you change it to showDialog(0) , right after it returns the AlertDialog, it's trying to call _progressDialog which hasn't been initialized yet (because that gets initialized in case 1 of onCreateDialog).

Some simple ducttape:

In the initial variable declarations, right after

Code:

    private Handler _progressHandler;

Shove in:

Code:

    //DUCTTAPE
    private boolean inProgressMode = false;

For the onCreate():

Code:

    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        
        Button btn = (Button) findViewById(R.id.btn_dialog);        
        btn.setOnClickListener(new View.OnClickListener() {
            public void onClick(View v) {
                showDialog(0);
                //DUCTTAPE Conditional
                if (inProgressMode) {
                    _progress = 0;
                    _progressDialog.setProgress(0);
                    _progressHandler.sendEmptyMessage(0);
                } //End the DUCTTAPE
            }
        });

And then in onCreateDialog, right after "case 1:", put

Code:

    inProgressMode = true; //DUCTTAPE Trigger

sediverse · May 16th, 2011, 11:16 AM

Bah, while my previous ducttape does work, it's inefficient.

Thus, rather than mess with all that, this makes a better fix:

Code:

                //DUCTTAPE Conditional
                if (_progressDialog != null) {
                    _progress = 0;
                    _progressDialog.setProgress(0);
                    _progressHandler.sendEmptyMessage(0);
                } //End the DUCTTAPE

No sense in making a new variable when there's already one to be used.