按钮不执行正确的命令[编辑]
Posted
技术标签:
【中文标题】按钮不执行正确的命令[编辑]【英文标题】：Buttons don't execute correct commands [EDIT] 【发布时间】：2015-05-12 20:20:56 【问题描述】：所以我正在尝试制作一个应用程序，其中有一个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。 谢谢
【问题讨论】：
发布您的代码，以便我们弄清楚。 也请看：How do I ask a good question? 刚刚添加。如果它看起来凌乱且写得不好，请道歉。我正在尝试掌握一些东西。 @hofmeister 抱歉，下次会记住的 【参考方案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;
【讨论】：
哇，谢谢！这成功了！是的，我知道在这两个部分中调用 imageId 是问题所在（我在代码之后对此进行了评论）。我试着做你正在做的事情，但在 doInBackground 部分声明了 newImageId = imageId （错误的方式）。再次感谢！！以上是关于按钮不执行正确的命令[编辑]的主要内容，如果未能解决你的问题，请参考以下文章