Java : 方法接受一个 Map 参数，需要对这个 map 做遍历。怎样才是安全的？

[theworldsong]

举例：

test(Map<String, String> m) {

}

现在要在 test 内遍历 m。

但是：

• m 是外部传入的参数，外部随时可能异步 remove 元素，导致你遍历出错。
• 你不能修改外部业务，因为极为复杂。因此你只能在 test 方法内发挥。
• 拷贝？拷贝的过程用到遍历，同样会出错。
• 使用 SynchronizedMap ？ SynchronizedMap 使用的仍然是这个 m 的引用，外部不受限制。

老铁们，有办法吗

[af463419014]

入参 m 的类型改成 ImmutableMap

[yinzhili]

com.google.common.collect.ImmutableMap

[xuhaoyangx]

你这问题 让我想起了 Java 形参和实参

删除操作用 Iterator 去做

[z3jjlzt]

final 修饰入参

[johnj]

Collections.unmodifiableMap() 包一下

[johnj]

我说的不对

[hand515]

@z3jjlzt #4 这个不对，参数的 final 搞错了

[linlinismine]

copy 一份

[psuwgipgf]

不明白怎么解决，关注一下。

[aa6563679]

@linlinismine 可能 copy 中被改了

[crayygy]

8l 的方法是可以的，在传入之前先从原 collection 中复制一份，然后再传入复制出来的对象

[elgae]

加个锁，遍历时独占

[sagaxu]

Map 只是个接口，你需要一个支持并发读写的实现

[talen666]

把声明 Map 的地方改成线程安全的 Map

[psuwgipgf]

@talen666 方法可以，不过他说的业务复杂用的地方也多，这样会影响整个系统的速度吧？？

[oaix]

重试

[boywang004]

只要调用 iterator()都会有机会上抛 ConcurrentModifyException，先做保护性拷贝，拷贝期间肯定会调用 iterator()，只要捕获 CME 重新拷贝，直到某次拷贝时没有改动成功拷出来为止……

上面说的加锁包 unmodifiableMap 啥的……=__=b

[micean]

只能在 test 里面操作的话根本做不到

[ysweics]

明确一下问题，具体的需求是怎么样，比如这个 map 开始传递过来的时候只有 10 个 kv ,然后你在便利的时候，remove 两个 kv，你需要的结果是开始的 10 个 kv,还是剩余的 8 个 kv

[zzorzz]

只是在 test 遍历，没有新启动一个新线程进行遍历的话，应该无须考虑同步问题，谁调用谁负责（方法调用者在自己的线程去考虑和别的线程同步）