必须输入“N”两次才能显示相关信息

Question

我正在尝试编写一些代码，根据用户输入的内容按升序或降序对数字进行排序。当用户输入“Y”时，该程序能够按升序对它们进行排序，但是如果他们输入“N”以降序排序，则用户必须在显示之前将“N”输入两次。我已经发布了下面的语法，所以如果有人想告诉我缺少什么/做错了什么，请随时这样做。

import java.util.*;

public class SortProgram { 
    public static void main(String args[]) { 
        Scanner in = new Scanner(System.in);
        System.out.println("\nHow many numbers do you want sorted?: ");
        int count = in.nextInt();
        int[] array = new int[count];

        for (int i = 0; i < count; i++) {
            System.out.print("Enter number #" + (i+1) + ": ");
            array[i] = in.nextInt();
        }

        System.out.print("\nPress 'Y' to sort numbers in  order, press 'N' to sort numbers in DESCENDING order: ");

        in.nextLine();

        boolean ascending = true;
        boolean descending = false;

        if (in.nextLine().toUpperCase().charAt(0) == 'Y') {
            ascending = true;
        } else if (in.nextLine().toUpperCase().charAt(0) == 'N') {
            descending = true;
            ascending = false;
        }

        for (int i = 0; i < count; i++) {
            for (int j = 0; j < count - 1; j++) {
                if (ascending) {
                    if (array[j] > array[j + 1]) {
                        int temp = array[j];
                        array[j] = array[j + 1];
                        array[j + 1] = temp;
                    }
                } else if (!ascending) {
                    if (array[j] < array[j + 1]) {
                        int temp = array[j];
                        array[j] = array[j + 1];
                        array[j + 1] = temp;
                    }
                }
            }
        }
        System.out.print("\nThe sorted numbers are: ");

        for (int i = 0; i < count; i++) {
            System.out.print(array[i] + " ");
        }
    }
}

Answer 1

您好像在给in.nextLine() 打了两次电话。每次调用时，程序都会从​​用户那里读取另一件事。

默认情况下，您的代码按升序运行。但是，由于您输入了两次，因此您需要输入两次“N”（当它“计数”时 - 即第二次，当您实际使用结果时）才能工作。

解决这个问题很简单：只需在声明 ascending 和 descending 之前删除 in.nextLine()。

顺便说一句，您的用户输入测试代码比它需要的要复杂一些。要正确设置ascending，您只需要：

ascending = in.nextLine().equalsIgnoreCase("y");

它使用equalsIgnoreCase()，相当于你获取第一个字符的方法，将其大写，然后与'Y'进行比较。

这将替换 for 循环之前的 if/elseif 块。

旁注#2：你在for循环中有很多重复的代码。由于唯一不同的是 if 块中的条件，因此您可以只使用布尔表达式而不是内部 if/else 语句：

    for (int i = 0; i < count; i++) {
        for (int j = 0; j < count - 1; j++) {
            boolean swap = ascending
                    ? array[j] > array[j + 1]
                    : array[j] < array[j + 1];
            if (swap) {
                int temp = array[j];
                array[j] = array[j + 1];
                array[j + 1] = temp;
            }
        }
    }