检查地图是否包含任何空值或键的最佳方法是什么？

Question

我有一张地图，其中存储了一组问题和答案。不能有空答案的问题或空问题的答案。下面是我如何初始化我的地图。

Optional<Map<String, String>> questionsAndAnswers();

我正在通过以下方式验证问题/答案的空字符串。

questionsAndAnswers().ifPresent(questionsAndAnswers -> {
        if (questionsAndAnswers.isEmpty()) {
            throw new IllegalArgumentException("questions cannot be empty if present");
        } else if (questionsAndAnswers.keySet().contains("") || questionsAndAnswers.values().contains("")) {
            throw new IllegalArgumentException("Questions or answers cannot be empty");
        }
    });

有没有更好的方法来实现这一点？任何建议将不胜感激。

更新：我没有特别需要在两个不同的条件下完成这些检查。

Answer 1

你可以试试下面

    questionsAndAnswers().ifPresent(qna -> {
        if (qna.isEmpty() || qna.containsKey("") || qna.containsValue("")) {
            throw new IllegalArgumentException("Error Message Comes Here");
        }
    });

Answer 2

questionsAndAnswers().ifPresent(ConcurrentHashMap::new); 如果有任何空键/值，则抛出 NullPointerException

Answer 3

你抛出了两个不同的异常，所以空地图和地图的空条目之间肯定有区别。

第二个是否应该检查，键或值是否为空或空：

questionsAndAnswers().ifPresent(pMap -> {
            if(pMap.isEmpty())
                throw new IllegalArgumentException();
            
            if(pMap.keySet().stream().anyMatch(pKey -> pKey == null || pKey.trim().isEmpty())
                    || pMap.values().stream().anyMatch(pValue -> pValue == null || pValue.trim().isEmpty()))
                throw new IllegalArgumentException();
        });

编辑： 对于字符串，您可以使用 Apache Commons：https://stackoverflow.com/a/14721575/11050826

Answer 4

首先：您上面显示的代码包含一些语法错误。可能您的 IDE 或编译器会帮助您解决这个问题。

您的地图存储键/值对，其中键或值是问题，另一个是答案。您要求的是不能存在空键或空值。

您可以遍历地图并查找具有空值的条目。你将如何处理它们仍然取决于你......

for (Map.Entry<String, String> entry: questionsAndAnswers) {
    if (entry.getValue()==null || entry.getValue().isBlank()) {
        // here do what you need to do with an emty value
        // removing that from the map may be thrilling while you loop over it
    }
}

一旦解决了，您可能需要关心空问题或空问题。文档说“将键映射到值的对象。映射不能包含重复的键；每个键最多可以映射到一个值。”这意味着最多有一个空问题。要找到它，您只需这样做

questionsAndAnswers.get(null); // unlikely to reveal anything at all
questionsAndAnswers.get("");   // there you go, now put this answer again with the
                               // right question. 
questionsAndAnswers.remove();  // finally remove the empty question from the map