数组无意中连续填充在开关盒内

Question

我目前在处理我最近正在做的事情时遇到了问题。为了简短起见，我希望我的程序按以下方式运行：

1. 用户输入一个数字，然后检查该数字是否为有效选项。如果它不是一个有效的选项，则会出现一条错误消息并让主菜单再次出现。

2. 如果选择了一个有效的选项，一个 switch case 会接受选择的选项并运行每个 case 语句中的代码。

3. 一旦代码在 switch-case 部分完成运行，它会循环回到顶部并重新显示主菜单，并且进程 1-3 将继续运行，直到用户选择选项 4 退出程序。

我似乎遇到的问题是，当我进入案例 1 时，它喜欢循环一段不打算循环的代码，并继续向数组中添加一个 Triangle 对象，这是这件作品：

if((side1 == 0 || side2 == 0 || side3 == 0)) 
{
        // Problem with do-while loop begins here
        triangleArray[trianglesAdded] = new Triangle();
        System.out.println("Default triangle created");
        trianglesAdded += 1;
}

其中 triangleArray 是一个大小为 100 的 Triangle 对象数组， trianglesAdded 跟踪实际添加到数组中的三角形的数量，并作为该数组索引的变量。由于它不断将 Triangle 对象添加到此数组中，因此我收到一个错误，因为该数组一旦达到其限制就不能再容纳任何 Triangle 对象，这意味着我也无法循环回到主菜单。

所以我的问题是：我怎样才能让它停止循环通过这部分（因为它应该只发生一次，并且一次只能将一个 Triangle 对象添加到数组中）还跑到可以转一圈，跑完后再次显示主菜单的地方？

完整代码如下：

do 
    {
        while(validMenuOption == false) 
        {
            System.out.println("1. Enter data for a new triangle.");
            System.out.println("2. Print all triangles sorted by area, smallest to largest.");
            System.out.println("3. Print only triangles with a specific color.");
            System.out.println("4. Exit the program.");
            System.out.print("\nPlease enter in a number to select an option: ");
            menuOption = sc.nextInt();
            
            if(menuOption >= 1 && menuOption <= 4) 
            {
                validMenuOption = true;
            }
            
            else 
            {
                System.out.println("Invalid menu choice. Please try again.\n");
                validMenuOption = false;
            }
            
            if(menuOption == 1 && trianglesAdded == triangleArray.length - 1)
            {
                System.out.println("The triangle array has hit the max. You are unable to add more triangles.");
                validMenuOption = false;
            }
        }
        
        switch(menuOption) 
        {
            case 1:
                ... 
                
                // If one, two, or all of the side lengths given are zero and the color of the triangle is 1, creates a default triangle
                // with the sides being 1 and the color being 1
                if((side1 == 0 || side2 == 0 || side3 == 0)) 
                {
                    // Problem begins here
                    triangleArray[trianglesAdded] = new Triangle();
                    System.out.println("Default triangle created");
                    trianglesAdded += 1;
                }
                
                // If side lengths greater than zero were given, creates a triangle with the side lengths and color given
                else 
                {
                    triangleArray[trianglesAdded] = new Triangle(side1, side2, side3, color);
                    System.out.println("Triangle created with specified size and color");
                }
                
                break;
                
            case 2:
                ...
                break;
                
            case 3:
                ...
                break;
                
            case 4:
                ...
                break;
                
            default:
                ...
                break;
        }
    } while(true);

Answer 1

嗯，你有一个错误。因此，当您遇到错误时，您会做每个人都会做的事情：您在脑海中逐行计算程序应该做什么。然后你将它与计算机告诉你它认为你的代码意味着什么进行比较。在您的大脑和计算机不一致的地方，您发现了一个错误。

1. 您的程序说“validMenuOption”为假。
2. 我输入“1”。
3. validMenuOption 设置为 true，然后我们循环“输入 while 块”。
4. vMO 现在为真，所以 while 循环结束，我们继续。
5. 开关运行，我们执行“添加三角形”部分。
6. 外循环循环。
7. validMenuOption 仍然是 true。
8. 所以.. 整个“获取一些输入”while 循环运行零次。

您可能想先重置 validMenuOption，或者您希望它是一个 do {} while (); 循环。

Answer 2

我会完全删除 validMenuOption 变量并将其移动到单独的函数中，如下所示：

public int getMenuOption() {
        Scanner sc = new Scanner(System.in);
        int menuOption = 0;

        System.out.println("1. Enter data for a new triangle.");
        System.out.println("2. Print all triangles sorted by area, smallest to largest.");
        System.out.println("3. Print only triangles with a specific color.");
        System.out.println("4. Exit the program.");
        System.out.print("\nPlease enter in a number to select an option: ");
        
        menuOption = sc.nextInt();

        if(menuOption < 1 || menuOption > 4) {
            return 0;
        } else {
            return menuOption;
        }
    }

See how I return 0 when option is out of range and otherwise I return menuOption.

现在在您的main 或您运行switch 语句的任何地方执行以下操作：

        boolean run = true;
        int[] triangleArray = new int[100];
        int trianglesAdded = 0;

        while (run) {
            menuOption = getMenuOption();
            switch(menuOption) {
                case 0:
                    System.out.println("Invalid menu choice. Please try again.\n");
                    break;
                case 1:
                    if(trianglesAdded == triangleArray.length - 1) {
                        System.out.println("The triangle array has hit the max. You are unable to add more triangles.");
                        run = false;
                    } else {
                        triangleArray[trianglesAdded] = 1; // or whatevery number or object you want
                        System.out.println("Default triangle created");
                        trianglesAdded += 1;
                    }
                    break;
                default:
                    break;
            }
        }

为了简单起见，我将triangleArray 设为int 数组。 我还添加了一个boolean 变量run，它在设置为true 时运行，并在输入while 时从getMenuOption() 函数中获取menuOption 值。

现在如果它得到0，它会打印菜单选项为假breaks 并返回到循环的开头，然后再次返回到getMenuOption()。如果您输入1，它会输入case 1。

你提到的问题：

我收到一个错误，因为一旦达到限制，数组就不能再容纳更多的 Triangle 对象了

通过添加condition 来解决在像我一样添加它之前检查长度。

要退出它，您需要添加另一个case，它将run 设置为false：

case 4:
     run = false;

等等

这里是repl 供您立即测试。我已将数组的大小更改为2，因此您可以看到它在达到限制时停止。

Answer 3

你可以试试下面的代码

while(true){

            System.out.println("1. Enter data for a new triangle.");
            System.out.println("2. Print all triangles sorted by area, smallest to largest.");
            System.out.println("3. Print only triangles with a specific color.");
            System.out.println("4. Exit the program.");
            System.out.print("\nPlease enter in a number to select an option: ");
            menuOption = sc.nextInt();

                switch(menuOption){
                case 1:
                    if(trianglesAdded == triangleArray.length - 1)
                        {
                            System.out.println("The triangle array has hit the max. You are unable to add more triangles.");
                            return;
                        } 
                                    
                    // If one, two, or all of the side lengths given are zero and the color of the triangle is 1, creates a default triangle
                    // with the sides being 1 and the color being 1
                    if((side1 == 0 || side2 == 0 || side3 == 0)) 
                    {
                        // Problem begins here
                        triangleArray[trianglesAdded] = new Triangle();
                        System.out.println("Default triangle created");
                        trianglesAdded += 1;
                    }
                    
                    // If side lengths greater than zero were given, creates a triangle with the side lengths and color given
                    else 
                    {
                        triangleArray[trianglesAdded] = new Triangle(side1, side2, side3, color);
                        System.out.println("Triangle created with specified size and color");
                    }
                    
                    break;
                    
                case 2:
                    ...
                    break;
                    
                case 3:
                    ...
                    break;
                    
                case 4:
                    ...
                    break;
                    
                default:
                    System.out.println("Invalid menu choice. Please try again.\n");
                    break;
            }

        }