每次猜后有限制的随机数猜谜游戏

Question

我在做一个数字猜谜游戏：

• 计算机在间隔内生成一个数字
• 我试着猜测它，并收到一个回复，不管它是高于/低于我的猜测，还是等于我的猜测，我赢了
• 有一个可以猜测的间隔，还有一个猜测尝试限制

public String guess(int guess)
{
    
    int lowerBound = 0;
    int upperBound = 99;
    
    Set<Integer> lowerGuesses = new TreeSet<>();
    Set<Integer> higherGuesses = new TreeSet<>();

    if (gameOver) {
        return "The game is over.";
    }
    
    if (guess < 0 || guess > 99) {
        return "The guess is out of bounds.";
    }
        
    if (guessCount < maxGuessCount) {
            
        if (guess < secretNumber) {       
            if (lowerGuesses.contains(guess)) {
                return "The guess doesn't make sense.";
            }
            else {
            guessCount++;
            lowerBound = guess;
            lowerGuesses.add(guess);
            
            return "The random number is larger."; 
            }
        }

        if (guess > secretNumber) {
            if (higherGuesses.contains(guess)) {
                return "The guess doesn't make sense.";
            }
            else {
            guessCount++;
            upperBound = guess;
            higherGuesses.add(guess);
            
            return "The random number is smaller.";
            }
        }
        
        if (lowerGuesses.contains(guess)) {
                return "The guess doesn't make sense.";
        }
        
        if (higherGuesses.contains(guess)) {
                return "The guess doesn't make sense.";
        }
        
    }
    
    if (guess < lowerBound || guess > upperBound) {
        return "The guess doesn't make sense.";
    }
    
    if (guessCount == maxGuessCount) {
        gameOver = true;
        victorious = false;
        return "Ran out of guess attempts.";
    }
    
    guessCount++;
    gameOver = true;
    victorious = true; 
    return "You won.";
    }

Answer 1

首先，为了避免混淆，让我们重命名该方法，以确保它的名称与其参数不完全匹配，因此它应该是这样的：

public String makeGuess(int guess)

避免用完全相同的名称命名相同名称空间中的不同实体（局部变量存在于不同的方法或参数中，出于初始化的目的，它们与数据成员具有相似的名称是一种例外）。从现在开始，您将调用该方法为MakeGuess(25)。

现在，到实际问题。你的假设不正确。你假设你需要跟踪过去的间隔。事实并非如此。你可以改变间隔的边缘。另外，你的代码是多余的，我建议你重构它。最后，您总是将上界、局部界以及较高和较低的猜测初始化为局部变量，因此它们永远不会被跟踪。相反，您需要执行以下简单措施才能使其发挥作用：

protected int lowerBound = 0;
protected int higherBound = 99;
protected int lb = 0;
protected int hb = 99;
protected int limit = 5;
protected int guessCount = 0;
protected int randomizedNumber; //Initialize this somewhere

public void initialize() {
    lb = lowerBound;
    hb = higherBound;
    guessCount = 0;
}

public String makeGuess(int guess) {
    if (++guessCount >= limit) return "The game is over.";
    else if ((lb > guess) || (hb < guess)) return "The guess doesn't make sense";
    else if (randomizedNumber == guess) return "You won.";
    else if (guess < randomizedNumber) {
        hb = guess;
        return "The random number is smaller.";
    } else {
        lb = guess;
        return "The random number is larger.";
    }
}

注意：我不喜欢把逻辑和输出层混在一起，我在上面的方法中这样做的原因是你已经提到你是一个初学者，我的意图是让这个答案对刚刚开始编程并非常困惑的人来说是可以理解的。为了实际解决方案的目的，您应该返回一个状态，并在不同的层处理该状态，并执行所需的控制台/UI操作。现在我将不详细说明，因为它也超出了范围，但现在，请使用上面的解决方案取得一些成功，但随后您应该明确地研究您需要如何编写代码，因为这几乎与使代码工作同样重要。