这…… 要吐槽的地方恐怕有点多

首先你的 CMSet 看起来是作为接口用的，但它并不是个抽象类
然后在 derived 类中，由于你要使用多态特性且试图自己管理裸指针，那么你必然要实现虚析构并满足 rule of three

然后你在计算 key 时试图直接使用 std::hash<T> ，但没有对 T 做任何 constraints/traits ，这会导致实际使用这个类时几乎必然失败（你只用这个 set 来存整数？）

构造和 add 函数缺乏模板转发 （ T&& t; std::forward<T>(t) ） 不过考虑到上一条，也许这个类确实不需要转发

再然后你试图用 atomic 来「以某种想象的方式保证一致性」—— 也是错的，在你的查找函数里有一个链表遍历，在并发条件下无锁遍历===不同步（你有没有考虑……正在附加上去的节点被脱链了？）另外你也忘了读写 count 和 delete 节是两个不同时点，很可能发生 A 读 count==1; B 增加 count 到 2; A 将节点删除 这样的时序。（ count 不可交换，只能 atomic 增减）

析构函数完全没考虑——你是想让使用者保证这个对象一定不会在还有元素的时候销毁？



我的建议是
- 忘记 c++有 new 这个关键字，只有 make_shared 和 make_unique
- 只要函数参数是个模板，就记得想想能不能套 std::forward
- 重修 E 系列的书
- 数据结构中包含链表时，老老实实加锁