为什么有时这不起作用？

Question

TL;DR 我有很多按钮，我正在交换它们的图像。出于某种原因，我的代码只能在某些手机上运行，​​而不能在其他手机上运行。

我的应用使用以下代码来比较图像按钮上的图像：

onCreate:

redsquare = getResources().getDrawable(R.drawable.redsquare);
bitred = ((BitmapDrawable) redsquare).getBitmap();

onClick （v 是点击的按钮）

ClickGround = v.getBackground(); //Get the background of button clicked
//the bitmap background of the button clicked
BitClick = ((BitmapDrawable) ClickGround).getBitmap(); 

然后，稍后在 onClick 中，我通过执行以下操作检查用户是否点击了 redSquare：

if (BitClick == bitred) { //Make sure it is the red square that user clicked
}

我已经在模拟器和华为手机上测试过，效果很好。当我在我的另一部手机（lg g3）上测试它时，if 语句没有通过。为什么结果不一样？ 我的手机中的图像是否被弄乱了？

Answer 1

首先，Resources.getDrawable(int) 是deprecated。它可能与您的问题无关，但您应该解决它。

如果您使用== 比较位图，您就是在比较对象身份。如果== 测试给你false，那意味着你在比较不同的对象。这是毫无疑问的。

您的 sn-ps 没有提供足够的上下文来确定，但这里有一些可能性：

1. 代码中的某些内容导致将不同的值分配给 bitred。

2. 两个sn-ps代码中的bitred标识符不表示同一个Java变量。

3. 您认为用作红色背景的“红色”位图始终是相同对象的假设是无效的。

假设您已经消除了上面的 1. 和 2.，并专注于 3. 这怎么会发生？好吧，我们看不到相关代码（您可以在其中随机交换按钮图像），但我可以想到几种可能性：

• 您可能会反复从Resources 获取位图。
• 您调用切换位图的方法可能是创建/设置副本。
• 您为获取单击按钮的位图而调用的方法可能会返回一个副本。

由于上述每个操作都可能依赖于行为不同的 API 实现（因为 javadoc 允许这样做），因此您的应用程序的行为可能取决于平台。

---

那么解决办法是什么？

假设，如果您可以找出导致使用不同位图对象的原因，您可能会解决它。然而，虽然您的代码仍然依赖于未指定行为，但它有可能会在其他手机上中断...

更好的解决方案是更改您的应用程序，以便不依赖使用 == 来比较 Bitmap 对象。例如：

• 为每个按钮关联一个标签。

   button1.setTag("one");
   button2.setTag("two");
  

• 创建一个HashMap，将按钮标签映射到该按钮的当前颜色。 HashMap 是应用“模型”状态的一部分。

  Map colors = new HashMap();
  ...
  colors.put("one", "red");
  colors.put("two", "blue");
  

• 当您更改按钮的图像位图时，对地图进行相应的更新。

  // After swap the images of button1 and button2 ...
  String c1 = colors.get("one");
  String c2 = colors.get("two");
  colors.put("one", c2);
  colors.put("two", c1);
  

• 在 onClick 方法中，使用地图查找按钮的当前颜色，而不是尝试通过比较 Bitmap 对象来找出它。

  if (colors.get(v.getTag()).equals("red")) {
      // it is red!
  }
  

（请注意，这与 Biju Parvathy 的建议很接近，但他没有明确说明如何处理按钮颜色/图像的变化。）

Answer 2

您可以，我正在对您的比较代码进行一些小改动，以使其适用于所有设备。

Bitmap BitRed = ((BitmapDrawable)getResources().getDrawable(R.drawable.redsquare)).getBitmap();

Bitmap BitClick = ((BitmapDrawable) v.getBackground()).getBitmap();

if (BitClick.sameAs(BitRed)) 
{ 
    //Your Button with Red Square Clicked
}

更多详情请参考SameAs函数。 希望对您有所帮助。

Answer 3

首先Resources.getDrawable(int) 已弃用，因此您肯定会在一些较新的设备中遇到此类问题。更好的方法是为您要用于该特定按钮的每个可绘制对象使用按钮标签。例如，假设您有 4 个按钮，即按钮 1、按钮 2、按钮 3 和按钮 4。现在假设他们都有一些默认的背景。

让我们说，

button1.setBackgroundResource(R.drawable.default1);
button1.setTag(R.drawable.default1); // set the tag for button same as the drawable id

对所有 4 个按钮执行相同操作

button4.setBackgroundResource(R.drawable.default4);
button4.setTag(R.drawable.default4); // set the tag for button same as the drawable id

现在每次更改按钮的背景/图像时，您也需要更新标签，类似这样

button1.setBackgroundResource(R.drawable.redimage);
button1.setTag(R.drawable.redimage); // set the tag for button same as the drawable id

在您的按钮onClick() 上，您必须简单地使用开关盒来区分不同的标签

@Override
public void onClick(View view) {
    switch (view.getId()) {
        case R.id.button1:
        switch(button1.getTag()){
            case R.drawable.default1:
            // do whatever action you want to perform
            break;
            case R.drawable.redimage:
            // do whatever action you want to perform
            break;
        }
        break;
        // similarly you can do the same for rest of the buttons.
    }
}

希望这会有所帮助。