因此，我正在尝试制作一个具有ImageView的应用程序，该应用程序显示图像列表中的随机图像。还有两个按钮，根据图像，需要按下正确的按钮。这样就可以继续，并且您会得到分数，直到您得到错误的答案为止。
我在异步方法中使用过，以在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进行比较的条件。
谢谢
最佳答案
您有多个名为imageId的变量。这是正在发生的事情：
1）您在Game类中声明了一个名为imageId的变量，并带有以下行：
int imageId = (int) (Math.random() * cardGallery.length);
这是代码示例中的第六行。 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;
}