c# - 使用 String.Split 的替代方法

Question

下面的代码在一个由逗号分隔的电子邮件组成的字符串中查找，如果它不在结果集合中，则添加一个电子邮件，该集合也是字符串类型。

string [] oEmails=orderEmails.Split(',');
string[] partEmails= part[Constants.Emails].ToString().Split(','); 
foreach(string email in oEmails)
{
  if(!partEmails.Contains(email))
  {
      part[Constants.Emails] += "," + email;
  }
}

这是编写此逻辑的最佳方法吗？我工作得很好，但我想知道是否有一种方法可以将这个逻辑整合到一个 lambda 表达式中？

困扰我的是我没有对这些数组做任何事情，而是拆分输入字符串。它们没有其他用途。

Answer 1

part[Constants.Emails] =
 string.Join(",", part[Constants.Emails].Split(',').Union(orderEmails.Split(',')).ToArray());

除此之外，您以错误的方式存储电子邮件。不要将它们存储为逗号分隔的字符串，将它们存储为列表。这样您就不必在每次修改“集合”时解析它们。

Answer 2

您可以使用 aHashSet而不是单个 largestring吗？如果列表中尚不存在值，这只会将值添加到列表中。例如

part[Constants.Emails] = new HashSet<string>();
foreach (var email in oEmails)
    part[Constants.Emails].Add(email);

如果您需要更轻松地与一个大字符串进行转换，您可以创建一个类，例如

class Emails : HashSet<string>
{
    public Emails(string concatenatedList)
        : base(concatenatedList.Split(','))
    {
    }
    public override string ToString()
    {
        return string.Join(",", this);
    }
}

Answer 3

part[Constant.Emails] += oEmails.Where(email => !partEmails.Contains(email))
                                .Aggregate(b, (current, email)
                                              => current + ("," + email));

首先，这会选择尚未出现的电子邮件（第一行），然后将它们聚合成一个由几,mail部分组成的字符串（第二行和第三行）。然后它将它添加回字符串（第一行）。

Answer 4

这种方式part[Constants.Emails].Split(',')通过直接在字符串中执行查找来避免您：

string[] oEmails = orderEmails.Split(',');
string partEmails = part[Constants.Emails];

foreach (string email in oEmails)
{
    bool index = partEmails.IndexOf(email);
    bool isAlreadyPresent = index != -1 && 
        (index == 0 || partEmails[index - 1] == ',') &&
        (index + email.Length == partEmails.Length || partEmails[index + 1] == ',');

    if (!isAlreadyPresent)
    {
        partEmails += "," + email;
    }
}

part[Constants.Emails] = partEmails;

Answer 5

试试这个：

part[Constants.Emails] = string.Join( ",", ( 
    from e in (part[Constants.Emails].ToString() + "," +  orderEmails).Split(',')
    where !string.IsNullOrEmpty( e )
    select e 
).Distinct().ToArray() );

它的可读性较差，但可能运行得更快。

编辑：大多数其他解决方案（包括问题）使用字符串连接。StringBuilder 对此要好得多。

编辑：将此数据存储为列表或哈希表比将生成的数组/列表连接成逗号分隔的字符串并每次都将其解析出来要好得多，如果您的一个字符串有合法的答案，请不要说得到错误的答案逗号在里面。

Answer 6

您应该能够像这样整合逻辑：

oEmails.Where(partEmails.Contains).Select(e => partEmails[Constants.Emails] += string.Format(",{0}", e));

Answer 7

也许你可以试试

var result = oEmails.Select((x, i) => oEmails[i] == partEmails[i] ? part[Constants.Emails] :
             partEmails.Contains(x) ? part[Constants.Emails] : -1).ToArray();