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

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) 您在 Game 类中声明一个名为 imageId 的变量，其中包含以下行：

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

这是代码示例的第 6 行。 Game 类的每个实例都有一个名为 imageId 的成员，该成员在创建实例时（即创建活动时）被分配一个随机值。

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

那么你的两个选择是：

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

• doInBackground 运行，并修改 imageId
• 然后用户在 onPostExecute 运行之前按下按钮。将显示旧图像，但已设置新的 imageId。
• 然后将运行 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 类中的字段：

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