Question

我创建了一个应用程序，点击按钮说[next]。我正在调用一种方法，这种方法的工作是生成5个随机数并随机设置为5个按钮，点击每个按钮它将随机选择一个小的（2秒）音频文件并播放它。如果我点击[next]按钮超过8次，我的整个应用程序将重新启动。

如果一次又一次地调用nextChangeAdd();方法，我的应用程序会在多次10次函数调用后停止响应。请给我一些替代？ logcat也没有显示任何错误

import java.util.ArrayList;
import java.util.List;
import java.util.Random;

import android.app.Activity;
import android.content.Context;
import android.graphics.Typeface;
import android.media.MediaPlayer;
import android.os.Bundle;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.TextView;


public class Addition extends Activity{

    TextView addNo1, addNo2;
    Button option1, option2, option3, option4, next;
    Random randomGenerator;
    Integer randomInt1, randomInt2;
    int flag = 0, setans;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        // TODO Auto-generated method stub
        super.onCreate(savedInstanceState);
        setContentView(R.layout.additionactivity);
        addNo1 = (TextView) findViewById(R.id.addNOOne);
        addNo2 = (TextView) findViewById(R.id.addNOTwo);
        option1 = (Button) findViewById(R.id.addoption1);
        option2 = (Button) findViewById(R.id.addoption2);
        option3 = (Button) findViewById(R.id.addoption3);
        option4 = (Button) findViewById(R.id.addoption4);
        next = (Button) findViewById(R.id.addNextBtn);
        next.setOnClickListener(new OnClickListener() {
            @Override
            public void onClick(View v) {
                nextChangeAdd();
            }
        });
        randomGenerator = new Random();
        Typeface tf = Typeface.createFromAsset(getAssets(), "fonts/StefanieDots.ttf");
        addNo1.setTypeface(tf);
        nextChangeAdd();

    }

    public void nextChangeAdd() {
        randomInt1 = randomGenerator.nextInt(10);
        randomInt2 = randomGenerator.nextInt(10);
        setans = randomGenerator.nextInt(5);
        if (setans == 0) {
            setans++;
        }
        int ans = randomInt1 + randomInt2;
        String addans = Integer.toString(ans);
        addNo1.setText(randomInt1.toString());
        addNo2.setText(randomInt2.toString());
        List<Integer> generated = new ArrayList<Integer>();
        for (int i = 0; i < 3; i++) {
            while (true) {
                Integer next = randomGenerator.nextInt(19) + 1;
                if (!generated.contains(ans)) {
                    generated.add(next);
                    break;
                }
            }
        }

        switch (setans) {
        case 1:
            flag = 1;
            option1.setText(addans);
            option1.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //rightAns();
                }
            });

            break;
        case 2:
            flag = 2;
            option2.setText(addans);
            option2.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //rightAns();
                }
            });

            break;
        case 3:
            flag = 3;
            option3.setText(addans);
            option3.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //rightAns();
                }
            });
            break;
        case 4:
            flag = 4;
            option4.setText(addans);
            option4.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //rightAns();
                }
            });
                        break;
        }
        String a2 = generated.get(0).toString();
        String a3 = generated.get(1).toString();
        String a4 = generated.get(2).toString();

        if (flag == 1) {
            option2.setText(a2);
            option2.setOnClickListener(new OnClickListener() {

                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option3.setText(a3);
            option3.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option4.setText(a4);
            option4.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
        }
        if (flag == 2) {
            option1.setText(a2);
            option2.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option3.setText(a3);
            option3.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option4.setText(a4);
            option4.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
        }
        if (flag == 3) {
            option2.setText(a2);
            option2.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option1.setText(a3);
            option1.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option4.setText(a4);
            option4.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
        }

        if (flag == 4) {
            option2.setText(a2);
            option2.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option3.setText(a3);
            option3.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
            option1.setText(a4);
            option1.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    //wrongAns();
                }
            });
        }
    }



}

Answer 1

代码中没有任何内容需要很长时间才能执行，因此AsyncTask不是答案。

实际上，这是一个逻辑错误，使得以下内容成为无限循环：

 while (true) {
     Integer next = randomGenerator.nextInt(19) + 1;
     if (!generated.contains(ans)) {
         generated.add(next);
         break;
     }
 }

如果发生这种情况，生成ans并将其插入列表中，循环将永远不会结束。

Answer 2

Use AsyncTask

异步任务由称为Params，Progress和Result的三种泛型类型定义。

Params ：Params用于向异步任务发送参数。

进度：进度用于在任务进行过程中将数据从异步任务发送到UI线程（主线程）。

结果：结果用于在执行后将数据从异步任务发送到UI线程。

异步任务分四步完成，如下所示：

onPreExecute ：启动任务时在UI线程上执行的回调方法。

doInBackground ：在非UI线程上执行的回调方法，在onPreExecute方法之后调用（如果存在）。

onProgressUpdate ：在UI线程上执行的回调方法，由sendProgress（）从doInBackground（）方法调用。

onPostExecute ：在UI线程上执行的回调方法，在任务完成后调用。

应用程序停止响应，主线程做了这么多工作..？

2 个答案: