別の方法String.Splitを使用する

Question

以下のコードは、コンマで区切られた電子メールで構成された文字列を検索し、string型の結果コレクションにない場合は電子メールを追加します。

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

これは、このロジックを書き込むための最良の方法ですか？私はうまくいきますが、このロジックをラムダ式に統合する方法があるのだろうかと疑問に思っていましたか？

私はこれらの配列で何もしていませんが、入力文字列を分割しています。彼らは他の目的を果たしていません。

Answer 1

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

さらに、電子メールを間違った方法で保存しています。それらをコンマ区切りの文字列として格納しないで、リストとして格納します。そうすれば、 "コレクション"を変更するたびにそれらを解析する必要はありません。

Answer 2

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

まず、これはそれが少数,mail部（第3行目）の文字列にそれらを集約し、まだ存在していない電子メール（最初の行）を選択します。その後、それを文字列（最初の行）に追加します。

Answer 3

この方法では、文字列内で直接検索を実行することによって、あなたの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 4

は、あなたの代わりに単一の大きなstringのHashSetを使用することはできますか？リストに値が存在しない場合は、リストに値が追加されます。例えば。

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 5

これを試してみてください：

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

それは読みにくくだが、おそらくはるかに高速に実行されます。

EDIT：他のほとんどのソリューション（質問を含む）は、文字列連結を使用します。 StringBuilderはこれにはるかに優れています。

EDIT：リストまたはハッシュテーブルとして、このデータを格納するは、カンマ区切りの文字列に結果の配列/リストをconcatinatingとアウトバックするたびに、それを解析するよりもはるかに優れている、1つが間違った答えを得るには何も言いませんあなたの弦の中に正当なカンマが入っていました。

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();