在 HashMap 中使用 ComputeIfAbsent

Question

我已经阅读过类似的帖子，但这是使用 computeIfAbsent 函数的正确方法吗？cookieMap 是一个 HashMap，响应是一个对象，其中包含所有标题、cookie、响应、状态代码等...

cookieMap.computeIfAbsent("Varlink", varLink -> {

  if (responses.getCookie("VARLINK").length() < 1) {

    throw new ProviderException("Varlink not present in response, check response status!!!");

  }

  return responses.getCookie("VARLINK");

});

我需要向 cookieMap 添加多个这样的键。我最初的想法是将所有内容都放在 If 条件中，但由于某些限制，我们不应该嵌套 if-else 条件（我猜 Code Reviewer 对 Clean Code 这本书太认真了）

Answer 1

如果responses和cookieMap是两个不同的数据源，那么您的代码段是正确的。唯一的问题是调用cookieMap::getCookie两次，这可能会使用有人在评论中建议的变量来解决。我会使用以下方法缩短整个表达式Optional：cookieMap.computeIfAbsent("Varlink", v -> {&nbsp; &nbsp; Optional.of(respones.getCookie("VARLINK"))&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Gets a cookie&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .filter(c -> c.length() >= 1)&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;// Filters the length&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; .orElseThrow(() -> new ProviderException("...")); // Returns only if present});