如何从“不可修改”地图中删除空列表或空列表

Question

我有一张不可修改的地图 (Map<String, Object>)。这包含一个键值对，其中值应该是一个列表，但对于特定键，值是一个空列表。

如何从地图中删除该空列表并返回？

我已经使用谓词来完成它，该谓词检查值是否是集合的实例，然后检查 Collections.isNotEmpty(..) 是否。

我想知道有没有比这更好的方法？ （在 Java 8 中）

public class StudentTest {

    Predicate<Object> isCollectionNotEmpty = input -> {
        if (input instanceof Collection) {
            return CommonCollectionUtils.isNotEmpty((Collection<?>)input);
        }
        return true;
    };
    Predicate<Object> isObjectNotNull = input -> Objects.nonNull(input);

    public static void main(String[] args) {

        StudentTest s = new StudentTest();
        final Map<String, Object> map1 = new HashMap<>();
        map1.put("student_batch_size", 200);
        map1.put("student_avg_height", 172);
        map1.put("Student_names", Collections.emptyList());

        final Map<String, Object> finalMap = s.getFinalMap(Collections.unmodifiableMap(map1));

        System.out.println(finalMap);
    }

    private Map<String, Object> getFinalMap(final Map<String, Object> inputMap) {
        final Map<String, Object> resultMap = new HashMap<>();
        //inputMap.values().remove(Collections.emptyList());
        resultMap.putAll(inputMap.entrySet().stream()
            .filter(entry -> isObjectNotNull.and(isCollectionNotEmpty).test(entry.getValue()))
        .collect(Collectors.toMap(e -> e.getKey(), e -> e.getValue())));
        return resultMap;
    }

}

预期输出：

{student_avg_height=172, student_batch_size=200}

Answer 1

Collection#removeIf 可能是一个不错的选择。

private Map<String, Object> getFinalMap(final Map<String, Object> inputMap) {
    final Map<String, Object> resultMap = new HashMap<>(inputMap);
    resultMap.entrySet().removeIf(e -> e.getValue() instanceof Collection && ((Collection) e.getValue()).isEmpty());

    return resultMap;
}

如果基于流的方法对您更有吸引力

private Map<String, Object> getFinalMap(final Map<String, Object> inputMap) {
    return inputMap.entrySet()
        .stream()
        .filter(e -> !(e.getValue() instanceof Collection && ((Collection) e.getValue()).isEmpty()))
        .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
}

---

更新

如果你想过滤掉空对象而不考虑它们的类型，这里对上面提到的方法做一些细微的改动

1) 删除如果

resultMap.entrySet()
    .removeIf(e -> Objects.isNull(e.getValue()) || 
                   (e.getValue() instanceof Collection && ((Collection) e.getValue()).isEmpty());

2) 基于流

.filter(e -> Objects.nonNull(e.getValue()))
.filter(e -> !(e.getValue() instanceof Collection && ((Collection) e.getValue()).isEmpty()))

Answer 2

我认为您使用Object 作为Map 中的值的方法是有问题的。您不能完全控制类型安全。我会考虑用反映单个学生批次属性的对象替换学生 Map。

class StudentBatch {
   int size;
   int avgHeight;
   Collection<String> names;

   boolean hasValidNames() {
       return names != null && !names.isEmpty();
   }
}

如您所见，我已将 hasValidNames 谓词放在对象上，使其可重用并允许简化您尝试优化的条件。

  List<StudentBatch> batchList = new ArrayList<>();
  batchList.stream().filter(StudentBatch::hasValidNames).collect(Collectors.toList());

Answer 3

instanceof 也检查 no nullity，因此无需显式设置它，并且您无需创建新的 HashMap 即使您流式传输的 Map 不可修改，因为流不会修改它直接但创建一个新地图：

final Map<String, Object> finalMap =
     map
    .entry()
    .stream()
    .filter(e-> !isEmptyCollection(e))
    .collect(toMap(Entry::getKey, Entry::getValue);


public void boolean isEmptyCollection(Entry<String, Object> entry){     
    return entry.getValue() instanceof Collection && ((Collection<?>) entry.getValue()).isEmpty();
}