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

[theworldsong]

举例：

test(Map<String, String> m) {

}

现在要在 test 内遍历 m。

但是：

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

老铁们，有办法吗

[reeco]

外部 remove 了跟你内部怎么遍历没有冲突啊，至于数据同步的问题就是楼上说的谁调用谁负责

[miao1007]

最开始应该上写时复制

[lovedebug]

所有调用 test 函数的地方强制使用 Collections.unmodifiedXX

[Ginsai]

既然只是传入 Map，外部能随时修改的话对 test()方法操作本身就已经不安全了。。如果没法将传入 Map 的类型线程安全的，那么就在 test()里面做 try catch，设置遍历失败尝试次数，异常捕获之后继续遍历处理。

[laxenade]

就和 spinlock 的原理一样不断拷贝咯，直到没有异常为止。

[fengdianxun]

用 kotlin 的只读 map 呢

[hearfish]

并发的环境下多线程共用一个不知道是不是线程安全的 Map，而且还不知道别的线程是怎么用它的？

[MoHen9]

把 map 改为 ConcurrentHashMap 呢，如果不允许，那么在传过来的时候就应该是一个拷贝的 map，而不是原始的 map

[mifly]

这样的代码应该要避免，考虑看看是不是可以用 blockquene 替代 map

[assiadamo]

上面的 boywang 貌似对不可变集合有不同的观点，我为了避免遍历中对原始集合有增改的这个问题，一般都不会去操作原始集合，而是直接做一个新的集合然后赋值，直接切换引用，这个是绝对线程安全的，而且遍历时切换也对遍历毫无影响

[D3EP]

@assiadamo 你这说的不就是 copyonwrite 的集合么？都是现成的线程安全的集合

[D3EP]

多线程条件下，不用线程安全的集合，我也是醉了

[ilaipi]

感觉你这既然外部可以随时修改这个 map，是不是这个 map 是个常量？那是不是不需要传进来？统一的地方去维护这个 map 然后上锁？

[assiadamo]

@D3EP copyonwrite 帮你省掉了切换引用这步操作，让你可以直接 add remove，但需要原始集合声明为 copyonwrite

[TommyLemon]

序列化再反序列化就行了。
可以用 fastjson( https://github.com/topics/fastjson)

[ColinWang]

Copy-On-Write 正解

[ZSeptember]

设计有问题，在多个地方都会写的不应该传参。
改用共享的，然后用线程安全的。

[lihongjie0209]

你这个接口有问题, 一个接口连自己的参数都无法处于一个确定的状态, 那你怎么写业务逻辑?

[passerbytiny]

引用以下前面的回复

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

# 37 你这个接口有问题, 一个接口连自己的参数都无法处于一个确定的状态, 那你怎么写业务逻辑?

如果你这个方法的参数类型是 java.util.Map ，又没有对外部调用做任何附加规范限制，那就意味着该方法声明了它处理的是不安全的 Map，出错是正常的，需要外部系统确认自己处理 Map 的线程安全问题。如果你需要这个方法自己考虑同步问题，那么它至少需要在规范上限制传入的参数类型是线程同步的或者不可变的。

从你的描述看，调用方非常明确的告诉你传入的是一个没有线程同步还被多线程共享的 java.util.Map 对象。你赶紧把这个 BUG 提交上去吧，技术层面解决不了。

解释一下：只有多线程共享、并且线程不安全的 Map 对象，才有可能出现这边正在处理着，那边 put/remove 的情况。

[luozic]

这种多线程请求还不同步的，脑袋被踢了？ 暂时可以抽象为，只有一个线程有取数据和移除数据的操作，其他线程都从提交到这个线程处理。