为什么循环没有中断？答案

【问题标题】：Why the loop is not breaking?为什么循环没有中断？
【发布时间】：2020-02-17 06:09:04
【问题描述】：

我的目标是打印出给定的坐标，如果它们是有效的格式。我有一个辅助函数 isValid 来检查字符串是否是坐标，并且 positionQuery 询问用户输入，如果 isValid 为假 positionQuery 给出错误并再次询问坐标，如果为真则打印出坐标.但是，我的布尔值或循环有问题，因为我没有得到任何输出。

public static boolean validCoordinate(String coordinate, int dimension) {
        String [] alphabet = new String[dimension];
        for(int i = 0; i < dimension; i++) {
            alphabet[i] = Character.toString((char)(65 + i));
        }
        List<String> list = Arrays.asList(alphabet);
        int [] numbers = new int [dimension];
        for(int i = 0; i < dimension; i++){
            numbers[i] = 1 + i;
        }
            if(list.contains(Character.toString(coordinate.charAt(0)))) {
                for(int j = 0; j < dimension; j++) {
                    if ((coordinate.substring(1)).contains(Integer.toString(numbers[j]))) {
                        return true;
                    }
                }
            }

        return false;
    }



    public static String[] positionQuery(int dim, Scanner test_in) {
        Scanner scanner = new Scanner(System.in);
        System.out.println("Provide origin and destination coordinates.");
        System.out.println("Enter two positions between A1-H8:");
        while(true) {
            String line = scanner.nextLine();
            String[] coordinates = line.split(" ");
            if(coordinates.length == 2) {
                String origin = coordinates[0];
                String dest = coordinates[1];
                if(validCoordinate(origin, dim) && validCoordinate(dest,dim)) {
                    System.out.println(coordinates);
                }
            }
            else {
                System.out.println("ERROR: Please enter valid coordinate pair separated by space.");
            }

        }
    }

如果我能看到我做错了什么，那就太棒了:)

【问题讨论】：

那是因为if(validCoordinate(origin, dim) && validCoordinate(dest,dim)) 总是假的
这能回答你的问题吗？ How to break this while(true) loop?

标签： java loops while-loop

【解决方案1】：

你的方法被命名为validCoordinate（不是isValid）。而且，该方法的代码太多了。检查是否有两个字符，检查两者是否在各自的范围内。没有循环，没有集合，没有数组。喜欢，

public static boolean validCoordinate(String coordinate, int dim) {
    if (coordinate.length() == 2) {
        char a = coordinate.charAt(0);
        char b = coordinate.charAt(1);
        return a >= 'A' && a <= ('A' + dim) && b >= '1' && b <= ('1' + dim);
    }
    return false;
}

而且，System.out.println(coordinates); 不是您打印数组的方式。试试

System.out.println(Arrays.toString(coordinates));

并且请停止asking和question。并阅读How to create a Minimal, Reproducible Example。

【讨论】：