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）删除＆＃34; int＆＃34;来自doInBackground中的变量声明，因此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;
}

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

1 个答案: