Question

我一直在寻找一种简短易读的方法来从hashmap中删除条目。具体来说，这是我的方法：

Map<String, HashMap<String, Long>> kitcooldowns = new HashMap<String, HashMap<String, Long>>();


// METHOD TO REMOVE EXPIRED COOLDOWNS FROM HASHMAP

final long currentime = System.currentTimeMillis();
final HashMap<String, HashMap<String, Long>> tmp = new HashMap<String, HashMap<String,Long>>();
for (final Entry<String, HashMap<String, Long>> kits : kitcooldowns.entrySet()) {
    final HashMap<String, Long> newitems = new HashMap<String, Long>();
    for (final Entry<String, Long> item : kits.getValue().entrySet()) {
        if (item.getValue() + getCooldownTime(item.getKey()) > currentime) {
            newitems.put(item.getKey(), item.getValue());
        }
    }
    if (newitems.isEmpty() == false) tmp.put(kits.getKey(), newitems);
}

kitcooldowns = tmp;


private long getCooldownTime(final String type) {
    switch (type) {
    case "CooldownX":
        return 3600000;
    case "CooldownY":
        return 1800000;
    default:
        return 0L;
    }
}

为简化这一点，这是主要结构：

MAP<NAME OF PLAYER, MAP<TYPE OF COOLDOWN, TIME WHEN USED>>

如果特定的冷却时间已过，则播放器将从Hashmap中删除。现在，这对我来说似乎是一个混乱的解决方案，我相信有一个更好的解决方案。

编辑： 我的问题是，如果Java 8有一个高效且干净的方法（如迭代器），它为大多数长方法提供了大量新的单行解决方案。

Answer 1

无需创建单独的地图。如果您遍历地图的entrySet()或values()，则可以使用Iterator#remove()。

for (Iterator<Entry<String, Long>> iter = kitcooldowns.entrySet().iterator(); iter.hasNext();) {
  Entry<String, Long> entry = iter.next();
  if (entry.getValue() + getCooldownTime(entry.getKey()) > currentime) {
    iter.remove();
  }
}

OP希望知道：

Java 8没有任何单行解决方案吗？

当然，但是我强烈建议你不要因为可以而把所有内容都写成单行。请记住，未来的开发人员存在代码 read ，而不是尽可能简洁地编写。此外，使用Iterator#remove()的代码将使用更少的内存，因为它不必制作地图的副本。使用较少内存的代码最终也会更快，因为较少的内存使用会导致较少的GC（这会花费CPU时间）和较少的CPU缓存未命中。

那说：

kitcooldowns = kitcooldowns.entrySet().stream()
  .filter(entry -> entry.getValue() + getCooldownTime(entry.getKey()) <= currentime)
  .collect(Collectors.toMap(Entry::getKey, Entry::getValue));

Answer 2

你可以简单地使用这个Java单行代码：

    final long currentime = System.currentTimeMillis();
    kitcooldowns.entrySet().removeIf(entry -> entry.getValue().entrySet()
    .removeIf(entry2 -> entry2.getValue() + getCooldownTime(entry2.getKey()) 
    < currentime) && entry.getValue().isEmpty());

从hashmap中删除特定条目的简短方法

2 个答案: