java - Java 猜颜色游戏

Question

我正在尝试用 Java 创建一个猜色游戏。我创建了一个循环，但我不知道如何创建正确的答案。每次都是不正确的。有人可以帮忙找出我的代码中的问题是什么吗？预先感谢您。

import java.util.*;
public class Guess_The_Color
{
  public static void main(String[] args)
  {
    Scanner cin = new Scanner(System.in);
    Random ran = new Random();

    String sAgain;
    String sGuess;
    int iColor;

    do
    {
      iColor = ran.nextInt(5);
      String sColor;

      if (iColor == 0)
        sColor = "Red";
      else if (iColor == 1)
        sColor = "Blue";
      else if (iColor == 2)
        sColor = "Green";
      else if (iColor == 3)
        sColor = "Yellow";
      else if (iColor == 4)
        sColor = "Orange";
      else
        sColor = "Pink";

      System.out.println("Guess what color I am thinking of... ");
      sGuess = cin.next();

      if (sGuess.toUpperCase().equals(iColor))
      {
        System.out.println("Correct!" + sColor);
        sAgain = "n";
      }
      else
      {
        System.out.println("Incorrect, would you like to try again? (y/n)?" + iColor);
      sAgain = cin.next();
      }

    }
      while (sAgain.equals("y"));
    }
  }

Answer 1

您正在将字符串与行中的整数进行比较

sGuess.toUpperCase().equals(iColor)

这每次都是假的。确保仅比较具有相同类型的变量。我认为您的意思是与 sColor 而不是 iColor 进行比较。

sGuess.toUpperCase().equals(sColor.toUpperCase()) //making them both uppercase or lowercase will work here. 
//You want to make sure both are lowercase or both are uppercase

或者(Maarten Bodewes 的建议):

sGuess.equalsIgnoreCase(sColor)