大家好，

我进行了一些分析，发现在一次添加多个源时存在以下性能瓶颈。

该代码旨在每次生成一个唯一的 ID，但他们并不确信 ID 每次都是唯一的，这就是为什么使用 While 循环重新生成新 ID 的原因。

问题 1：新对象是一个循环，但这是性能瓶颈的原因，被认为是非常非常糟糕的编码风格。
问题2：不需要生成一个随机数，乘以10000再乘以10，这可以简化为%100000。

使用 C#，大多数时候您可以忘记释放内存，考虑到这种情况，如果您在 C++ 中编写完全相同的代码，您会在一开始就注意到内存问题（或内存泄漏检测工具）。

建议：

1.使用GUID作为种子，默认随机性不好。可以参考这个帖子：https : //helloacm.com/c-randomness-using-guid/

如果您使用GUID作为种子，则几乎不会发生问题。

2. 使用伪随机算法，例如如果您是性能极客的话。可以参考这个帖子：

https://codingforspeed.com/using-faster-psudo-random-generator-xorshift/

​。

• 回复 - -

我对这段代码非常熟悉——我添加了一个 JIRA 任务来重写它！根据我的记忆，这里的“性能”问题实际上来自糟糕的设计而不是语言选择。如果你真的想要一个唯一的 ID 来标识对象，那么 GUID 正是应该使用的类型。

“碰撞风险”是如此之低，以至于被其他因素所抵消。（见http://blogs.msdn.com/b/oldnewthing/archive/2008/06/27/8659071.aspx）

如果您想要一个可以以较少的数字/字符显示在屏幕上的“id”，那么我将简单地实现一个专门的数字分配器类。目前随机选择一个数字然后检查它是否已经被使用的方法是我能想象的最糟糕的实现方式！

这并不是否认，对于某些高性能热点或算法，C++（或某些其他语言）不会胜过 C#，但我总是先看算法/设计。

• 回复 - - -

是的，我完全同意你的看法。设计/算法是首先要改进的。“过早优化是万恶之源”，我们需要通过分析器找出性能损失的根本原因。

结束！