带有数组列表的“if”和“else”

Question

我正在 Android Studio 上开发游戏。游戏有 12 个块和 12 个符号（在本例中为“0”个零）。当您按下新游戏按钮时，将启动第一级。总共有12个级别。(curLevel)。几秒钟后，屏幕上会随机显示一个符号，然后消失。（根据每个级别输入的数量增加1，例如，级别1：必须猜测1个符号，级别2：必须猜测2个符号）直到级别12。到目前为止，它的工作没有问题。现在我需要解决我的代码来接收用户的输入，检查是否与显示的正确块预览匹配。如果是，则进入下一个级别（重复清除块的循环，显示随机符号，再次清除，等待用户输入）直到第 12 级。如果不是，则显示失败消息并再次显示新游戏按钮。我创建了这个 Arraylist 但它不能正常工作，现在它的条件总是负面的（即使它按下了正确的块）“失败！再试一次！” 它正在显示的消息。我运行调试器并显示： List tempList = new ArrayList<> ();tempList:size 12for (int i = 0; i < curLevel; i++) { curLevel size:13tempList.add ( buttons.get ( i ) ); buttons size:12if (tempList.contains ( v.getTag () )) { tempList size 12Its not matching the condition. –这是值：static int MAXLEVELS = 12;        static String SUCCESS_SYMBOL = "0";curLevel = 1;        curGuess = 0;有什么建议可以让它工作吗？甚至让这段代码更整洁？private void buttonLogic(View v) {        List<Button> tempList = new ArrayList<> ();        for (int i = 0; i <= curLevel; i++) {            if (i!= MAXLEVELS)                tempList.add ( buttons.get ( i ) );            else                break;        }        if (tempList.contains ( v.getTag () )) {            curGuess++;            checkWin ();        } else {            lostGame ();        }    }    private void lostGame () {        Toast.makeText(this, "Fail! Try Again", Toast.LENGTH_SHORT).show();        disableButtons();        b_new.setVisibility(View.VISIBLE);    }        private void checkWin() {            if (curGuess == curLevel) {                disableButtons ();                if (curLevel == 12) {                    Toast.makeText ( this, "Congratulations! You Won", Toast.LENGTH_SHORT ).show ();                    b_new.setVisibility ( View.VISIBLE );                } else {                    curGuess = 0;                    curLevel++;                    generateButtons ( curLevel );                }            }        }还有一个来自之前代码的图像。干杯。

Answer 1

ArrayList<Button> tempList = new ArrayList<>();改为使用，因为否则您将缺少方法（尽管将变量从ArrayListto转换为List不会引起任何投诉）。

该if情况可能是(i+1) < MAXLEVELS因为，i是0基础，curLevel是1基础。

最简单的可能是初始化curLevel = 0并仅将其显示为curLevel + 1...否则您将始终必须考虑业务逻辑中的+1/-1偏移量，我认为这是“无用的复杂性”...以便游戏开始在 level 0，同时 level1显示在 GUI 上。

Answer 2

看起来您正在尝试为buttons已执行点击事件的按钮查找可用的任何匹配项。

为此，我建议您使用 找到完全匹配viewId，您可以更改您的方法，如下所示。

private void buttonLogic(View v) {

    boolean hasAnyMatch = false;

    // Assumption is curLevel will always be less than MAXLEVELS

    for (int i = 0; i <= curLevel; i++) {

        if (v.getId() == buttons.get(i).getId()) {

            hasAnyMatch = true;

            break;

        }

    }

    if (hasAnyMatch) {

        curGuess++;

        checkWin ();

    } else {

        lostGame ();

    }

}