Question

我正在尝试编写一个程序，要求用户输入一个字母（R，G，B），然后输出五个结果。连续不能有2个字母。当我输入第三个字母并且双字母检查不起作用时，我得到indexoutofbounds。

package absolutejava;

import java.util.Scanner;
import java.util.*;

public class RGB {

    public static void main(String[] args) {
        Scanner kb = new Scanner(System.in);
        int count = 0;
        boolean isColor = false;
        String finalString = "";
        int i = 0;
        int j = 1;

        String temp = "";
        for (count = 0; count < 5;) {
            System.out.println("Enter a color. Use R for red, G for green, and B for blue.");
            temp = kb.nextLine();

            if ((temp.equals("R") || temp.equals("G") || temp.equals("B"))) {
                isColor = true;
                temp += temp;

            } else {
                isColor = false;
                System.out.println("Invald Color, please choose again");
            }

            if (isColor == true && j < 6 && i < 5) {
                count++;
                if (temp.length() > 2 && temp.length() <= 5 && finalString.substring(i, j).equals(temp.substring(i - 1, j - 1))) {
                    System.out.println("Two colors cannot be next to each other ");
                    isColor = false;
                    count--;

                } else if (temp.length() == 5) {
                    finalString = finalString + temp.substring(i);
                    //debugging line
                    System.out.println("i " + i + "j " + j + "count " + count + " " + finalString + " length " + temp.length());
                    i++;
                    j++;
                } else {
                    finalString = finalString + temp.substring(i, j);
                    //debugging line
                    System.out.println("i " + i + "j " + j + "count " + count + " " + finalString + " length " + temp.length());
                    i++;
                    j++;
                }
            }
        }

        System.out.println(finalString);
    }
}

Answer 1

以下行肯定是错误的：

temp += temp;

每次迭代都会用当前输入替换temp，所以这没有任何效果。即使不是的情况，你只需要向自己添加相同的字符串 - 例如“A”将成为“AA。”

我认为你的意思是

finalString += temp;

或者那种效果。

一般来说，您似乎在某些地方混淆了temp和final。

还有一件事：不要明确地与true和false进行比较，这是不必要的，通常被认为是不好的风格。

卡住。索引超出范围。无法弄清楚为什么

1 个答案: