按钮不执行正确的命令 [编辑]

Question

所以我正在尝试制作一个应用程序，其中有一个 ImageView 显示图像列表中的随机图像。还有两个按钮，根据图像，需要按下正确的按钮。这一直持续下去，你会得到分数，直到你得到一个不正确的答案。

我在 Async 方法中使用随机显示 ImageView 中的图像，并在我的按钮上设置条件。但是，当我运行应用程序时，该条件仅适用于显示的第一张图像。之后，无论显示什么图像，按钮条件都会像显示第一张图像一样工作。

这是代码

public class Game extends ActionBarActivity {

static TextView timeDisplay;
int[] cardGallery =  {R.drawable.tile0, R.drawable.tile1, R.drawable.tile2, R.drawable.tile3, R.drawable.tile4, R.drawable.tile5, R.drawable.tile6, R.drawable.tile7, R.drawable.tile8, R.drawable.tile9};
int score = 0;
int imageId = (int) (Math.random() * cardGallery.length);

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

    ImageView cardImageView = (ImageView) findViewById(R.id.cardImage);
    ImageButton bigButton = (ImageButton) findViewById(R.id.upButton);
    ImageButton smallButton = (ImageButton)findViewById(R.id.downButton);
    final TextView scoreDisplay = (TextView)findViewById(R.id.scoreMessage);

    timeDisplay = (TextView) findViewById(R.id.timerMessage);
    cardImageView.setImageResource(cardGallery[imageId]);
    scoreDisplay.setText("Current score: " + score);

   .
   .
   .

    bigButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            /* may need to implement switch and cases for each button
            switch (cardGallery[imageId]){
                case R.drawable.tile0:
                    GameTimer.onFinish();
            }*/

            if (imageId == 0) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 1) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 2) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 3) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 4) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 5) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 6) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 7) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 8) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 9) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
    }});

        smallButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            if (imageId == 0) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 1) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 2) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 3) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 4) {
                GameTimer.start();
                scoreDisplay.setText("Current score: " + ++score);
                new CardAsyncTask().execute();
            }
            else if (imageId == 5) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 6) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 7) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 8) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else if (imageId == 9) {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
            else {
                GameTimer.cancel();
                GameTimer.onFinish();
            }
        }
    });




}





class CardAsyncTask extends AsyncTask<Integer, Void, Integer> {

    @Override
    protected Integer doInBackground(Integer... params) {
        int imageId = (int) (Math.random() * cardGallery.length);
        return imageId;
    }

    @Override
    protected void onPostExecute(Integer imageId) {
        ImageView cardImageView = (ImageView) findViewById(R.id.cardImage);
        cardImageView.invalidate();
        cardImageView.setImageResource(cardGallery[imageId]);
    }

}

}

我认为这是因为我已将旧的和新的 imageId 值声明为相同，但如果我将它们更改为 imageId 和 newimageId，这将改变我的代码中要求值的条件与 imageId 进行比较。谢谢

Answer 1

您有多个名为 imageId 的变量。这是正在发生的事情：

1) 您在游戏 class 中声明了一个名为 imageId 的变量，其中包含以下行：

int imageId = (int) (Math.random() * cardGallery.length);

这是您的代码示例中的第 6 行。您的游戏 class 的每个实例都会有一个名为 imageId 的成员，该成员会在创建实例时（即创建您的 activity 时）分配一个随机值。

2) 在 CardAsyncTask class 的 doInBackground 方法中声明一个局部变量，也命名为 imageId，并为其分配一个随机值。由于您已经声明了一个局部变量，因此为其赋值不会影响存储在您的游戏 class 中的 imageId 变量。然后从该方法返回 doInBackground 中的 imageId，并由 AsyncTask 作为参数传递给 onPostExecute 方法，您可以在其中使用它来更新图像视图。

你的两个选择是：

a) 从 doInBackground 内的变量声明中删除 "int"，这样 doInBackground 任务将修改 Game class 实例中的 imageId，而不是在函数内创建局部变量。然而，这是一个坏主意。因为 doInBackground 运行ning 在后台线程上，所以可能会发生以下事件序列：

doInBackground 运行s，并修改imageId
用户然后在 onPostExecute 运行之前按下按钮。将显示旧图像，但已设置新图像ID。
然后 onPostExecute 将运行，将图像设置为新图像。

诚然，用户可能不会注意到它是乱序发生的，但是在使用异步进程时最好考虑这些事情。

b) 更好的选择是在 onPostExecute 中更改图像时将 Game 实例的 imageId 字段设置为新的 imageId。您在该函数中还有一个名为 imageId 的局部变量，这是传入的参数。我的建议是：

在doInBackground中，将imageId重命名为newImageId:

protected Integer doInBackground(Integer... params) {
    int newImageId = (int) (Math.random() * cardGallery.length);
    return newImageId;
}

然后在onPostExecute中，将新的图片ID赋值给Game中的字段class:

protected void onPostExecute(Integer newImageId) {
    ImageView cardImageView = (ImageView) findViewById(R.id.cardImage);
    cardImageView.invalidate();
    cardImageView.setImageResource(cardGallery[newImageId]);
    imageId = newImageId;
}

按钮不执行正确的命令 [编辑]

Buttons don't execute correct commands [EDIT]

java

android

button

android-asynctask

android-studio