Question

我在Android Studio中制作了一款应用。

快速摘要：它是一款适合儿童的应用，可在用户启动游戏时呈现随机形状。用户有4个选项可供选择，其中一个形状是正确的形状。然后，用户需要将形状拖放到轮廓中。下面显示了一个例子的图片。

问题是，我需要下面4个形状中的1个来匹配要猜到的形状。我有2套18个形状，第一套是形状轮廓与？内部。

    int[] outlines = new int[] {R.drawable.outline_0, R.drawable.outline_1, R.drawable.outline_2,
            R.drawable.outline_3, R.drawable.outline_4, R.drawable.outline_5, R.drawable.outline_6,
            R.drawable.outline_7, R.drawable.outline_8, R.drawable.outline_9, R.drawable.outline_10,
            R.drawable.outline_11, R.drawable.outline_12, R.drawable.outline_13, R.drawable.outline_14,
            R.drawable.outline_15, R.drawable.outline_16,R.drawable.outline_17};

第二组是带脸的实际彩色形状。

    int[] images = new int[] {R.drawable.img_0, R.drawable.img_1, R.drawable.img_2, R.drawable.img_3, R.drawable.img_4,
    R.drawable.img_5, R.drawable.img_6, R.drawable.img_7, R.drawable.img_8, R.drawable.img_9, R.drawable.img_10,
            R.drawable.img_11, R.drawable.img_12, R.drawable.img_13, R.drawable.img_14, R.drawable.img_15, R.drawable.img_16,
            R.drawable.img_17};

我需要某种功能或陈述，其中底部的4个形状不能与对应于需要猜测的形状的1个形状相同。

注意：outline_0形状对应于img_0，outline_1对应于img_1等。

这是此活动的完整代码。

public class SecondActivity extends AppCompatActivity {

int n;
ImageView shape1, shape2, shape3, shape4, guessShape;
ImageButton exit;
Random rand = new Random();
ImageView[] shapes = new ImageView[4];


@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_second);

    shape1 = (ImageView) findViewById(R.id.shape1);
    shape2 = (ImageView) findViewById(R.id.shape2);
    shape3 = (ImageView) findViewById(R.id.shape3);
    shape4 = (ImageView) findViewById(R.id.shape4);
    guessShape = (ImageView) findViewById(R.id.guessShape);
    shapes[0] = shape1;
    shapes[1] = shape2;
    shapes[2] = shape3;
    shapes[3] = shape4;

    //store all the shapes in an array
    int[] images = new int[] {R.drawable.img_0, R.drawable.img_1, R.drawable.img_2, R.drawable.img_3, R.drawable.img_4,
    R.drawable.img_5, R.drawable.img_6, R.drawable.img_7, R.drawable.img_8, R.drawable.img_9, R.drawable.img_10,
            R.drawable.img_11, R.drawable.img_12, R.drawable.img_13, R.drawable.img_14, R.drawable.img_15, R.drawable.img_16,
            R.drawable.img_17};

    int[] outlines = new int[] {R.drawable.outline_0, R.drawable.outline_1, R.drawable.outline_2,
            R.drawable.outline_3, R.drawable.outline_4, R.drawable.outline_5, R.drawable.outline_6,
            R.drawable.outline_7, R.drawable.outline_8, R.drawable.outline_9, R.drawable.outline_10,
            R.drawable.outline_11, R.drawable.outline_12, R.drawable.outline_13, R.drawable.outline_14,
            R.drawable.outline_15, R.drawable.outline_16,R.drawable.outline_17};

    //generate random number between 0 and image.length
   int img1 = (int) Math.round((Math.random() * images.length));
    int img2 = (int) Math.round((Math.random() * images.length));
    int img3 = (int) Math.round((Math.random() * images.length));
    int img4 = (int) Math.round((Math.random() * images.length));
    int outlineID = (int) Math.round((Math.random() * outlines.length));


    //set the image
    guessShape.setBackgroundResource(outlines[outlineID]);
    shape1.setBackgroundResource(images[img1]);
    shape2.setBackgroundResource(images[img2]);
    shape3.setBackgroundResource(images[img3]);
    shape4.setBackgroundResource(images[img4]);

      //set tags for the imageViews
    guessShape.setTag("RandomImage");
    shape1.setTag("Shape1");
    shape2.setTag("Shape2");
    shape3.setTag("Shape3");
    shape4.setTag("Shape4");

    //1 of the 4 image views needs to match outline of the shape that needs to be guessed
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_0)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_0);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_1)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_1);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_2)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_2);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_3)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_3);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_4)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_4);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_5)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_5);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_6)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_6);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_7)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_7);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_8)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_8);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_9)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_9);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_10)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_10);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_11)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_11);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_12)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_12);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_13)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_13);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_14)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_14);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_15)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_15);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_16)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_16);
    }
    if(guessShape.getBackground().getConstantState().equals(R.drawable.outline_17)){
        int random = new Random().nextInt(shapes.length);
        shapes[random].setBackgroundResource(R.drawable.img_17);
    }
}

}

我在这里做错了什么？
为什么我的if语句不能确保下面的一个随机形状对应于要猜测的形状？
如何确保底部的4个形状不同？

解决我的问题的任何提示都会很棒。谢谢！：）

Answer 1

没有日志，堆栈跟踪或输出很难确切地知道出错的地方。你可以做几件事来修复这个错误，或者可能会阻止另一个错误。

设置变量

让我们将guessShape.getBackground（）。getConstantState（）保存为变量。现在我不知道这是什么类型，所以我将它称为现在的对象。请使用正确的类型进行更新。

Object currentBackground = guessShape.getBackground().getConstantState();

If-Else If

目前，您正在使用if语句，后跟更多if语句。这具有每次测试第一个而不是第二个而不是第三个的效果。相反，我们只想匹配其中的每一个。这是一个tutorial。

让我们改变一下：

if(currentBackground.equals(R.drawable.outline_0){
    ...
}
if(currentBackgorund.equals(R.drawable.outline_1){
    ...
} ....

对此：

if(currentBackground.equals(R.drawable.outline_0){
    ...
} else if(currentBackgorund.equals(R.drawable.outline_1){
    ...
} ....

<强>否则

现在我们知道它只会匹配一次，我们希望确保找到它。我们希望在刚刚完成的else语句末尾捕获if-else if。

.... } else if (currentBackground.equals(R.drawable.outline_17){
       ... 
} else {
     // How do you want to handle if the background did not equal any of your images? 
}

调试

现在我们有了一些重大改进。你需要追踪它出错的地方。一种有用的方法是将内容打印到控制台。在NetBeans或Eclipse等IDE中有更强大的调试方法，但System.out.println现在很棒！

诀窍是知道你可能出错的地方。我看到了几个“失败点”。这些是您出错的风险较高的区域。这将有助于检查该值是否是您认为的那个值。

随机数字。找出数字是什么。它们是否在正确的范围内？这也可以帮助您调试它应该具有的形状。

  //generate random number between 0 and image.length
int img1 = (int) Math.round((Math.random() * images.length));
int img2 = (int) Math.round((Math.random() * images.length));
int img3 = (int) Math.round((Math.random() * images.length));
int img4 = (int) Math.round((Math.random() * images.length));
int outlineID = (int) Math.round((Math.random() * outlines.length));

// Print these to know what they are, especially outlineID.

您可以在每个if语句中放置打印件，以查看哪个（包括其他）被捕获以及原因。你可以确保它是你想要的那个。

<强>结论

尝试一下。它可能无法捕捉到你的错误，但至少你会对你的设置有一个具体的想法，如果它的捕获。这将帮助您彻底调试。如果大纲每次进入正确的if语句，那么我们就知道if语句中存在问题！调试可能不是第一次发现问题，但确实缩小了范围。

非匹配图像答案

我正在编辑我的答案以包含此内容。我很确定我发现了你的问题。

在您的代码中：

//generate random number between 0 and image.length
int img1 = (int) Math.round((Math.random() * images.length));
int img2 = (int) Math.round((Math.random() * images.length));
int img3 = (int) Math.round((Math.random() * images.length));
int img4 = (int) Math.round((Math.random() * images.length));
int outlineID = (int) Math.round((Math.random() * outlines.length));

您正在为图像创建4个随机图像，但您还为轮廓创建随机图像。这意味着您可以获得图像1,2,3,4，然后在轮廓上获得图像17的轮廓！这将在你的if语句中匹配，但它会为你提供17而不是你想要的图像。

在这种情况下，您希望获得4个随机数，然后从您的大纲中选择。这是一种方法（注意：有更好/更简单的方法，但我想确保它在你当前的水平。你很快就会到达那里！）

//generate random number between 0 and image.length
int img1 = (int) Math.round((Math.random() * images.length));
int img2 = (int) Math.round((Math.random() * images.length));
int img3 = (int) Math.round((Math.random() * images.length));
int img4 = (int) Math.round((Math.random() * images.length));

int whichImg = (int) Math.round((Math.random() * 4));

if(whichImg == 1){
   whichImg = img1;
} else if(whichImg == 2){
   whichImg = img2;
} else if(whichImg == 3){
   whichImg = img3;
} else {
   whichImg = img4;
}

int outlineID = outlines[whichImg];

这给你一个随机数1-4。然后，您可以使用保存正确的图像到Im。然后使用它在轮廓数组中获得相同的数字。

旁注您可能还没有学过它们，但for loop将是一个很好的方法，可以将代码从所有这些if语句中删除到只有一个。 :)如果您不熟悉它们，请在尝试之前先修复您的错误。 Here是教程的链接，以防您想要查看它。

为什么我的if语句不起作用？

1 个答案:

非匹配图像答案