删除 ArrayList 的最后一个元素

Question

我是 Java 新手，一个多星期以来一直在尝试解决一个练习，但我不知道自己做错了什么。

我需要删除 ArrayList 的最后一个元素，在本例中为整数。

问题是当我运行测试时，它仍然返回旧值。

public static void removeLastOccurrence(int x, ArrayList<Integer> list) {
    if (list != null && !list.isEmpty()) {
       list.remove(list.size()-1);
    }
}

我也尝试过使用 list.remove(list.lastIndexOf(x));

但是当我运行这个测试时它仍然返回相同的列表。

public class UTest{
    @Test
    public void testMultipleLast() {
        ArrayList<Integer> input = new ArrayList<Integer>(asList(1,1,3,5,7,1,5,9,1));
        ArrayList<Integer> result = new ArrayList<Integer>(asList(1,1,3,5,7,1,5,9));
        Solution.removeLastOccurence(1, input);
        assertEquals(result, input);
    }
}

如果有人可以帮助并告诉我我缺少什么，那就太好了，因为我觉得我只是缺少一小部分拼图，这让我感到非常沮丧。

Answer 1

您的测试应该如下所示。在原始帖子的测试代码中，您实际上并没有调用您要测试的方法。

public class UTest
{
  @Test
  public void testMultipleLast() {
     ArrayList<Integer> input = new ArrayList<Integer>(asList(1,1,3,5,7,1,5,9,1));
     ArrayList<Integer> result = new ArrayList<Integer>(asList(1,1,3,5,7,1,5,9));

     // int x = ?
     ArrayList<Integer> actual = SomeClass.removeLastOccurrence(x, input)
     assertEquals(result, actual);
  }
 }

removeLastOccurrence() 方法可以做到以下几点

if(list != null && !list.isEmpty()){
    list.remove(list.size() - 1);
}

Answer 2

这是因为您没有删除任何元素。

list.get(list.size()-1);

不删除元素。

使用

list.remove(list.size()-1)

改为。

Answer 3

根据Java ArrayList API 和get(int index) 方法，您只需获取ArrayList 中index 位置的元素。 这是您正在寻找的方法：

public static void removeLastOccurrence(int x, ArrayList<Integer> list) {
    if (list != null && !list.isEmpty()) {
        list.remove(list.size()-1);
    }
}

Answer 4

你必须使用：

list.remove(list.size()-1);

并返回您的新列表，以便您可以使用：

public static ArrayList<Integer> removeLastOccurrence(int x, ArrayList<Integer> list) {
    if (list != null && !list.isEmpty()) {
       list.remove(list.size()-1);
    }
    return list;
}

Answer 5

如果你将你的列表作为参数传递给方法，它就会变成一个局部变量。因此，您不会从input 列表中删除元素，而只是从局部变量list 中删除元素。解决方案是 return 您方法中的本地列表或使用相同代码直接从您的“输入”列表中 remove 元素。您原始方法中的 x 参数是不必要的。