如何改进从用户配置文件对象获取过滤数据的方法？

Question

我将以下代码组合在一起，以从用户的个人资料数据中获取一些详细信息。 它可以工作，但不太漂亮。

我有一个保存在用户配置文件中的 UserPreferences 对象。它具有关于是否需要 SMS 警报消息以及警报类型的属性。短信将作为电子邮件发送。创建警报时，我想要获取应收到该警报类型通知的地址的逗号分隔字符串。

用户拥有手机号码和运营商。运营商的 SMS 电子邮件格式类似于 “{number}@ {domain}，我将这些字段替换为用户的号码和运营商的域。

正如我所说，这确实有效，但是我不关心它是两种方法或者它看起来有多混乱是否有更紧凑的方式将其编写为一种方法？

public static string GetSMSAlertSubscribers(int alertTypeId) {
  // end result I want is like: "[email protected],[email protected]"
  return String.Join(",", GetSMSAlertSubscribersEnumerable(alertTypeId));
}

public static IEnumerable<string> GetSMSAlertSubscribersEnumerable(int alertTypeId) {
  var prefs = Membership.GetAllUsers()
                        .Cast<MembershipUser>()
                        .Where(u => WebProfile.GetProfile(u.UserName).Preferences.SendAlertsToSMS.Equals(true)
                                 && WebProfile.GetProfile(u.UserName).Preferences.SMSAlertTypeIds.Contains(alertTypeId))
                        .Select(u => WebProfile.GetProfile(u.UserName).Preferences);

  var carriers = new AlertRepository().FindAllMobileCarriers().ToList();

  foreach (UserPreferences p in prefs) {
    yield return carriers.Where(c => c.Id.Equals(p.MobileCarrierId))
                         .Select(c => c.SMSEmailAddressFormat.Replace("{domain}", c.SMSEmailAddressDomain).Replace("{number}", p.MobileNumber))
                         .SingleOrDefault();
  }
}

我会这样称呼它：

var emailTo = GetSMSAlertSubscribers(3);

---

>更新：我重构了我的“prefs”查询，以减少 GetProfile() 的重复使用，如下所示：

var prefs = Membership.GetAllUsers()
              .OfType<MembershipUser>()
              .Select(u => WebProfile.GetProfile(u.UserName).Preferences)
              .Where(p => p.SendAlertsToSMS && p.SMSAlertTypeIds.Contains(alertTypeId));

Answer 1

var q = from x in
            (from p in prefs
            from c in carriers
            where p.MobileCarrierId == c.Id
            select new
            {
                c.SMSEmailAddressFormat,
                c.SMSEmailAddressDomain,
                p.MobileNumber
            }).Distinct()
        select x.SMSEmailAddressFormat
            .Replace("{domain}", x.SMSEmailAddressDomain)
            .Replace("{number}", x.MobileNumber);

return String.Join(", ", q);

---

var q = from p in prefs
        from c in carriers
        where p.MobileCarrierId == c.Id
        select c.SMSEmailAddressFormat
           .Replace("{domain}", c.SMSEmailAddressDomain)
           .Replace("{number}", p.MobileNumber);

return String.Join(", ", q.Distinct());

Answer 2

嗯，这样怎么样：

var prefs = Membership
                .GetAllUsers()
                .OfType<MembershipUser>()
                .Select(u => WebProfile.GetProfile(u.UserName).Preferences)
                .Where(p => SendAlertToSMS.Equals(true) && SMSAlertTypeIds.Contains(alertTypeId));

var carriers = new AlertRepository().FindAllMobileCarriers().ToList();

foreach (UserPreferences p in prefs) 
{
    var carrier = carriers.FindOrDefault(c => c.Id.Equals(p.MobileCarrierId));
    yield return PopulateAddress(c, p);
}

然后在 PopulateAddress 中进行替换。现在对我来说看起来不那么混乱了。将所有内容塞进一个 Linq 查询中不一定是最好的做法。