【问题标题】:Problems with ArrayList, for loop and if statement - prints line for each item in the listArrayList、for 循环和 if 语句的问题 - 为列表中的每个项目打印行
【发布时间】:2017-05-22 04:52:20
【问题描述】:

在编程方面,我是一个完全的初学者,希望能得到一些帮助。我的程序有一个由用户输入的具有不同属性(例如名称和年龄)的狗对象的 ArrayList (dogRegister)。

该程序应该能够更新狗的年龄并从列表中删除狗,等等。要求用户输入要删除的狗的名字,如果输入的名字在列表中,则删除该狗。如果该名称未包含在列表中,则程序将打印“未找到狗”。年龄命令也是如此,如果包含名字,狗会变大 1 岁。如果不是 - “找不到狗”。

我正在为 if 语句及其条件而苦苦挣扎。我的代码和策略不起作用,因为它会为列表中的每个项目打印“未找到”,直到找到正确的项目。我只希望它打印一次,并且只告诉是否找到了该项目。关于如何解决它的任何想法?

如果您需要查看我的更多代码,请告诉我。

 public void deleteDog(){
    System.out.print("Enter name of dog to delete: ");
    String nameOfDogToDelete = readString(); //returns the nextLine();

    for (Dog dl : dogRegister) {
        if (dl.getName().contains(nameOfDogToDelete)) {
            dogRegister.remove(dl); 
            System.out.println("Dog deleted.");
        }
        else if (!dl.getName().contains(nameOfDogToDelete)) {
            System.out.println("Dog not found.");
        }
    }
}

public void editAge(){
    System.out.print("Enter name of aging dog: "); 
    String nameOfAgingDog = readString(); //returns the nextLine();

        for (Dog a : dogRegister) {
            if (a.getName().contains(nameOfAgingDog)) {
                a.updateAge(); //method returning the dog's age++
                System.out.println("Dog's age updated.");
            }
            else if (!a.getName().contains(nameOfAgingDog)) {
                System.out.println("Dog not found.");
            }
        }
    }

【问题讨论】:

    标签: java if-statement for-loop arraylist contains


    【解决方案1】:

    两个建议:

    1. 删除条目后立即跳出循环,这样就不会检查进一步的迭代。

    2. 如果在此之前您不想打印“未找到”案例,请不要包含“未找到”案例的打印。检查整个列表,如果仍然没有找到该条目,则打印“未找到”。您可以使用布尔变量来跟踪这一点。

    您已经检查了 if 条件,并且通过再次通过相同的检查再次检查 else 条件。所以当找到要删除的狗的名字时,它会删除它,并继续检查循环中的其他迭代。因此,一旦您找到要删除的狗并删除它,就跳出循环。

    public void deleteDog(){
        System.out.print("Enter name of dog to delete: ");
        String nameOfDogToDelete = readString(); //returns the nextLine();
        boolean isFound = false; //To keep track of whether the dog is found or not
        for (Dog dl : dogRegister) {
            if (dl.getName().contains(nameOfDogToDelete)) {
                dogRegister.remove(dl); 
                System.out.println("Dog deleted.");
                isFound = true;
                break;//Break out of the loop as soon as you find it
            }
            // else //no need to check the negation of the above condition again
            //   System.out.println("Dog not found.");
            // }
        }
        if (!isFound)
             System.out.println("Dog not found.");
    }
    

    【讨论】:

    • 耐心点,伙计.. 轻松投反对票.. 我正在输入解释,正如你评论的那样
    • 我撤回了我的反对票,但这仍然无法按 OP 的预期工作,例如考虑如果 nameOfDogToDelete 不是列表中的第一只狗会发生什么...
    • 更新说明
    • @MarkRotteveel:最新更新。抱歉,这里的网速不好。
    • 您可能希望将 if(!isFound) 移出循环。
    【解决方案2】:

    循环时只检查是否找到狗并将找到的布尔字段标记为真。如果没有找到,当循环结束时,打印没有找到

     public void deleteDog(){
        System.out.print("Enter name of dog to delete: ");
        String nameOfDogToDelete = readString(); //returns the nextLine();
        boolean found =false;
    
        for (Dog dl : dogRegister) {
            if (dl.getName().contains(nameOfDogToDelete)) {
                dogRegister.remove(dl); 
                System.out.println("Dog deleted.");
                found =true;
                break;
            }
        }
        if(!found){
         System.out.println("Dog not found.");
        }
    }
    

    【讨论】:

    • 您可能希望在该代码之外包含一些解释。
    • 已添加,代码中有什么不懂的可以随时问。
    • 我不是操作者,我理解你的代码很好,但没有解释的代码通常没有真正的帮助。
    • 是的,你是对的。请检查代码是否适合您:)
    • 感谢您的快速响应,我的代码现在可以使用了! :)
    猜你喜欢
    • 2017-07-19
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2017-07-31
    • 2021-02-02
    • 2020-03-24
    • 1970-01-01
    • 2021-12-24
    相关资源
    最近更新 更多