遍历AD用户列表时遇空邮箱抛异常记录后如何继续循环？

我懂你的痛点——现在的代码一碰到邮箱为空的用户就抛出异常，直接终止了整个循环，剩下的用户根本没机会处理。咱们调整一下逻辑，既能把错误记录到数据库，又能让循环继续跑下去，最后返回正常的用户列表。

核心思路 #

问题出在直接抛出未捕获的异常上，一旦抛出，程序就会停止执行。正确的做法是：

• 先检查邮箱是否为空/Null，优先处理错误情况
• 记录错误到数据库后，用continue跳过当前用户，继续下一个迭代
• 如果业务需要上层感知错误，可以收集所有错误，循环结束后统一抛出

修改后的代码示例 #

// 可选：收集所有错误信息，用于后续汇总
var syncErrors = new List<string>();

foreach (var user in adUsers)
{
    // 第一步：优先检查邮箱是否为空
    if (string.IsNullOrEmpty(user.Mail))
    {
        var errorMessage = $"User {user.GivenName} {user.Surname} cannot sync to Quill due to missing email address";
        
        // 替换成你实际的数据库错误记录逻辑
        LogErrorToDatabase(errorMessage);
        
        // 收集错误（可选）
        syncErrors.Add(errorMessage);
        
        // 跳过当前用户，继续处理下一个
        continue;
    }

    // 第二步：邮箱正常时，处理用户添加逻辑
    if (!users.Keys.Contains(user.Mail) && user.Mail.EndsWith("@warburgpincus.com"))
    {
        // 简化角色判断，比嵌套三元运算符更易读
        RoleTypes userRole = RoleTypes.User;
        if (adminMembers.Any(x => x.Mail == user.Mail))
        {
            userRole = RoleTypes.AdminUser;
        }
        else if (readOnlyMembers.Any(x => x.Mail == user.Mail))
        {
            userRole = RoleTypes.ReadOnlyUser;
        }

        // 添加到用户字典
        users.Add(user.Mail, new User 
        { 
            FirstName = user.GivenName, 
            LastName = user.Surname, 
            Username = user.UserPrincipalName, 
            Email = user.Mail, 
            IsActive = true, 
            Role = userRole 
        });
    }
}

// 可选：循环结束后统一抛出异常，通知上层有错误（不影响循环执行）
if (syncErrors.Any())
{
    throw new ApplicationException($"Sync finished with {syncErrors.Count} errors: {string.Join("; ", syncErrors)}");
}

return users;

关键改进点 #

1. 错误处理前置：把邮箱为空的判断放在最前面，避免后续逻辑因为空邮箱出现额外错误
2. 用continue替代直接抛异常：确保循环不会中断，所有用户都能被处理
3. 代码可读性优化：把嵌套的三元运算符拆成if-else结构，后续维护更轻松
4. 错误汇总（可选）：如果需要让调用方感知到错误，循环结束后统一抛出汇总异常，既不中断流程，又能传递错误信息

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