如何在遍历数组时删除元素

Question

我有一个包含其他对象列表的对象，其中包含另一个对象列表。很像 DTO。 在这些对象中，有另一个类型的对象列表，如果该对象包含特定的字符串，我想在其中删除该元素。

例如

CustomerDTO customerDTO = CustomerTransformer.transformCustomer(custom);
List<InfoDTO> info = customerDTO.getInfo();
info(infoDTO -> {
    List<MoreDTO> d = infoDTO.getMore();
    for (MoreDTO moreDTO : d) {
        List<ChannelDTO> channels = MoreDTO.getChannels();
        for (ChannelDTO cdto : contentChannels) {
            if ("apple".equals(cdto.getId())) {
                moreDTO.getChannels().remove(cdto);
            }
        }
    }

});

但这看起来确实效率不高，并且可能会更改空指针？有没有更好的写法？使用流和在流中使用过滤器会有帮助吗？

我是 Java 新手。

Answer 1

在迭代/循环时不能从列表中删除元素，会有一个ConcurrentModificationException

为了避免这种情况，您可以在这里使用 Java 8 过滤器，以这种方式 -

contentChannels = contentChannels
 .stream()
 .filter(cdto->(!"apple".equals(cdto.getId())))
 .collect(Collectors.toList());

另一个选项是使用removeIf()

Answer 2

上面的代码不起作用请参考@JB Nizet 评论，

Triggering a ConcurrentModificationException

本质上，ConcurrentModificationException 用于在我们迭代的内容被修改时快速失败。让我们通过一个简单的测试来证明这一点：

List<Integer> integers = newArrayList(1, 2, 3);

for (Integer integer : integers) {
    integers.remove(1);
}

所以你可以使用forEach和removeIf来避免ConcurrentModificationException

List<InfoDTO> info = customerDTO.getInfo();
info.forEach(more->more.getMore().forEach(channelDto->channelDto.getChannels().removeIf(cdto->"apple".equals(cdto.getId()))));

或者如果你只有List<ChannelDTO>

List<ChannelDTO> channels = MoreDTO.getChannels();
channels.removeIf(cdto->"apple".equals(cdto.getId()));