DirectorySearcher循环内存泄漏问题：PropertiesToLoad设置引发的性能矛盾

嘿，这个坑我之前踩过！用DirectorySearcher循环处理大量对象时，PropertiesToLoad的设置方式确实会同时影响内存和性能，咱们来理清楚问题根源和解决办法：

为什么会出现这种矛盾？ #

• 每次循环设置PropertiesToLoad：你相当于每次都创建了新的属性集合，DirectorySearcher底层会为这个新集合绑定COM资源，循环5000次后这些资源无法被GC及时回收，直接导致内存暴增。但好处是每次只加载你指定的属性，搜索自然快。
• 初始阶段一次性设置：复用同一个属性集合避免了重复创建资源，内存没问题，但如果这个集合包含了所有可能需要的属性（或者不是当前搜索最优的属性列表），搜索时会加载多余数据，拖慢速度。

最优解决方案：复用属性集合，按需调整 #

核心思路是避免每次循环创建新的StringCollection或字符串数组，而是复用同一个集合对象，根据每次搜索的需求动态调整内容：

场景1：所有循环需要加载的属性相同 #

如果每次搜索需要的属性固定，直接把集合初始化放在循环外，每次循环赋值给DirectorySearcher即可：

// 提前初始化好需要的属性列表，放在循环外部
var fixedProperties = new string[] { "samaccountname", "displayname", "mail" };

foreach (var targetObject in your5000Objects)
{
    using (var searcher = new DirectorySearcher())
    {
        // 复用同一个属性数组，避免重复创建资源
        searcher.PropertiesToLoad.AddRange(fixedProperties);
        // 设置搜索条件、范围等其他参数
        searcher.Filter = $"(&(objectClass=user)(cn={targetObject.Name}))";
        
        var searchResult = searcher.FindOne();
        // 处理搜索结果...
    }
}

场景2：每次循环需要的属性不同 #

如果不同对象需要加载的属性不一样，那就复用一个空的StringCollection，每次循环前清空再添加当前需要的属性：

// 初始化一个空的属性集合，放在循环外部
var dynamicProperties = new StringCollection();

foreach (var targetObject in your5000Objects)
{
    dynamicProperties.Clear();
    // 根据当前对象的需求添加属性
    dynamicProperties.Add("samaccountname");
    if (targetObject.NeedEmail)
    {
        dynamicProperties.Add("mail");
    }
    if (targetObject.NeedPhone)
    {
        dynamicProperties.Add("telephonenumber");
    }

    using (var searcher = new DirectorySearcher())
    {
        searcher.PropertiesToLoad = dynamicProperties;
        searcher.Filter = $"(&(objectClass=user)(cn={targetObject.Name}))";
        
        var searchResult = searcher.FindOne();
        // 处理搜索结果...
    }
}

额外优化建议 #

• 尽量缩小SearchScope（比如设为SearchScope.OneLevel而非默认的Subtree），减少搜索范围提升速度。
• 如果搜索结果可以缓存，对于重复的目标对象直接复用缓存结果，避免重复搜索。

内容的提问来源于stack exchange，提问作者Deeeyewhy