如何迭代对象的 ArrayList 并在迭代时访问对象所属的类的方法？

Question

我有一个类Employee 和该类中的一个方法getName()，它返回员工的姓名。每个员工都有一些属性（姓名、dob、职称、薪水），我将它们存储在一个 ArrayList 中。

现在我想编写一个方法deleteEmployee(String, ArrayList<Employee>)，它将作为参数给出的字符串与员工姓名进行比较，如果找到任何匹配项，它将删除该/那些员工的全部详细信息 我已经尝试如下，但我遇到了错误，我不知道如何解决......

 public void deleteEmployee (String str,ArrayList <Employee> arr)
 {
    Employee obj2;

    for (int i = 0; i < arr.size(); i++)
    {
        if(str.equalsIgnoreCase(arr.get(obj2[i].getName())))
            arr.remove(obj2[i]);
        break;
    }
 }

Answer 1

你可能有一个 ConcurrentModificationException 或类似的东西。 那是因为你不能修改你正在迭代的数组，除非使用迭代器

你应该试试这样的

Iterator<Employee> it = arr.iterator();
Employee currentEmployee = null;
while(it.hasNext()){
   currentEmployee = it.next();
   if(currentEmployee.getName().equals(obj2.getName()){
      it.remove();
   }
}

为了帮助您将来解决问题，请同时发布您的错误和堆栈跟踪。

Answer 2

如果您只有一个与名称匹配的项目，您的方法将有效。 但是，您的错误在这里：

if(str.equalsIgnoreCase(arr.get(obj2[i].getName())))

相反，您需要从数组中获取对象，然后在结果上调用 getName：

if(str.equalsIgnoreCase(arr.get(i).getName()))

您还需要在if 块中使用您的break：if(...) { arr.remove(i); break; }

在您的实施中当然还有其他可以改进的地方。

Answer 3

据我所知，您的代码存在一些问题。

首先，使用break 不是很好的做法。尤其是在你用过的地方使用它。在您的代码中，if 语句处理后，for 循环结束。不管if 语句的结果如何，for 循环都不会迭代。因此，除非您只想检查所需员工是否在索引 0 处，否则这是不可取的。

for (int i = 0; i < arr.size(); i++)

这是第二个问题。由于您使用 break 语句，它在理想情况下运行良好，但如果所需的员工不在 ArrayList 中，您将获得 IndexOutOfBoundsException 因为在最后一次迭代中，您将尝试获得ArrayList 大小的索引，不存在。相反，你的 for 循环应该是

for (int i = 0; i < arr.size() - 1; i++)

这样，如果员工不在场，循环将在到达 ArrayList 的最后一个索引时停止，该索引比大小小 1。

另一个更直接的问题是这一行：

if(str.equalsIgnoreCase(arr.get(obj2[i].getName())))

在这一行中，您使用 Employee 对象作为参数调用 arr.get，而您应该使用 ArrayList 的索引。

if(str.equalsIgnoreCase(arr.get(i).getName()))

通过此修改，您调用 ArrayList 的索引，该索引将返回一个 Employee 对象。然后，您在该对象上调用 getName()，它会返回字符串，该字符串将与 str 进行比较，就像您打算做的那样。

最后一个问题是删除对象的行。

arr.remove(obj2[i]);

再次，您似乎试图访问 Employee 对象的索引，而不是 ArrayList。将此行更改为：

arr.remove(i);

会更有效。这样，您将删除数组的索引。您可以使用一个对象作为 remove() 方法的参数，但您首先需要将 i 的对象分配给 obj2，而且整个过程要麻烦得多。 因此，将所有内容放在一起，新的固定循环将是

for (int i = 0; i < arr.size() - 1; i++){
   if(str.equalsIgnoreCase(arr.get(i).getName())){
      arr.remove(i);
   }
}

假设只有一名员工具有给定名称，这不会有问题，但请注意，在遍历列表时删除 ArrayLists 中的对象存在问题。例如，如果您删除i 处的对象，则ArrayList 中它之后的对象都将向下移动一个索引。然后，如果您再次迭代，您将访问列表中i+1 的对象。但是，i+1 的 ACTUAL 对象是最初位于 i+2 的对象。因此，如果您删除 ArrayList 中的一个对象，您将跳过在下一次迭代中检查紧随其后的对象。 一个简单的解决方案是通过 ArrayList 迭代 BACKWARDS，这样当从列表中删除一个对象时，所有向下移动一个索引的对象都已经被检查过。对此还有其他更合适的解决方案，但向后迭代是一种简单的解决方法。