オールドスクールを書くための最も効率的な方法は何ですか:
StringBuilder sb = new StringBuilder();
if (strings.Count > 0)
{
foreach (string s in strings)
{
sb.Append(s + ", ");
}
sb.Remove(sb.Length - 2, 2);
}
return sb.ToString();
... LINQで?
オールドスクールを書くための最も効率的な方法は何ですか:
StringBuilder sb = new StringBuilder();
if (strings.Count > 0)
{
foreach (string s in strings)
{
sb.Append(s + ", ");
}
sb.Remove(sb.Length - 2, 2);
}
return sb.ToString();
... LINQで?
回答:
この回答はAggregate
、質問で要求されているLINQ()の使用法を示しており、日常的な使用を目的としていません。これはaを使用しないため、StringBuilder
非常に長いシーケンスに対しては恐ろしいパフォーマンスになります。String.Join
他の回答に示されている通常のコードの使用
次のような集約クエリを使用します。
string[] words = { "one", "two", "three" };
var res = words.Aggregate(
"", // start with empty string to handle empty list case.
(current, next) => current + ", " + next);
Console.WriteLine(res);
これは出力します:
、 一二三
集約は、値のコレクションを取り、スカラー値を返す関数です。T-SQLの例には、min、max、sumがあります。VBとC#はどちらもアグリゲートをサポートしています。VBとC#はどちらも、拡張メソッドとして集計をサポートしています。ドット表記を使用すると、IEnumerableオブジェクトのメソッドを呼び出すだけです。
集計クエリはすぐに実行されることに注意してください。
詳細情報-MSDN:集計クエリ
CodeMonkeyKingによって提案されたコメントAggregate
を使用してバリアントを使用したい場合は、通常のコードとほぼ同じで、多数のオブジェクトのパフォーマンスが向上します。StringBuilder
String.Join
var res = words.Aggregate(
new StringBuilder(),
(current, next) => current.Append(current.Length == 0? "" : ", ").Append(next))
.ToString();
""
ため、で使用される最初の値current
は空の文字列です。したがって、1つ以上の要素の場合、常に,
文字列の先頭に移動します。
return string.Join(", ", strings.ToArray());
.Net 4では、それを受け入れるための新しいオーバーロードがstring.Join
ありますIEnumerable<string>
。コードは次のようになります。
return string.Join(", ", strings);
Linqを使用する理由
string[] s = {"foo", "bar", "baz"};
Console.WriteLine(String.Join(", ", s));
これは完璧に機能し、IEnumerable<string>
私の知る限りすべてを受け入れます。Aggregate
ここには何も必要ありません。
String.Join(",", s.ToArray())
ただし、古いバージョンでも引き続き使用できます。
Aggregate拡張メソッドを見ましたか?
var sa = (new[] { "yabba", "dabba", "doo" }).Aggregate((a,b) => a + "," + b);
私のコードの実際の例:
return selected.Select(query => query.Name).Aggregate((a, b) => a + ", " + b);
クエリは、文字列であるNameプロパティを持つオブジェクトであり、選択したリストにあるすべてのクエリの名前をコンマで区切って表示します。
他の回答と同様の質問で対処された問題(つまり、AggregateとConcatenateが0要素で失敗する)を確認した後、Join / Linqを組み合わせたアプローチを次に示します。
string Result = String.Join(",", split.Select(s => s.Name));
または(s
文字列でない場合)
string Result = String.Join(",", split.Select(s => s.ToString()));
StringBuilder
実装するために追加のオブジェクト(例:)を(手動で)作成する必要はありませんそしてもちろん、Joinは他のアプローチ(for
、foreach
)に潜入することがある厄介な最後のカンマを処理します。そのため、最初にLinqソリューションを探していました。
.Select()
すると、この操作中に各要素を簡単に変更できるため、この答えが好きです。たとえば、各アイテムを同じような文字でラッピングするstring Result = String.Join(",", split.Select(s => "'" + s + "'"));
あなたは使用することができますStringBuilder
にAggregate
:
List<string> strings = new List<string>() { "one", "two", "three" };
StringBuilder sb = strings
.Select(s => s)
.Aggregate(new StringBuilder(), (ag, n) => ag.Append(n).Append(", "));
if (sb.Length > 0) { sb.Remove(sb.Length - 2, 2); }
Console.WriteLine(sb.ToString());
(これSelect
は、より多くのLINQを実行できることを示すためだけにあります。)
new[] {"one", "two", "three"}.Aggregate(new StringBuilder(), (sb, s) =>{if (sb.Length > 0) sb.Append(", ");sb.Append(s);return sb;}).ToString();
if (length > 0)
linqのをチェックせずに取り出すことで、貴重なクロックサイクルを節約できます。
new[] {"", "one", "two", "three"}.Aggregate(new StringBuilder(), (sb, s) => (String.IsNullOrEmpty(sb.ToString())) ? sb.Append(s) : sb.Append(", ").Append(s)).ToString();
StringBuilderと3000要素を超えるSelect&Aggregateケースのクイックパフォーマンスデータ:
単体テスト-期間(秒)
LINQ_StringBuilder-0.0036644
LINQ_Select.Aggregate-1.8012535
[TestMethod()]
public void LINQ_StringBuilder()
{
IList<int> ints = new List<int>();
for (int i = 0; i < 3000;i++ )
{
ints.Add(i);
}
StringBuilder idString = new StringBuilder();
foreach (int id in ints)
{
idString.Append(id + ", ");
}
}
[TestMethod()]
public void LINQ_SELECT()
{
IList<int> ints = new List<int>();
for (int i = 0; i < 3000; i++)
{
ints.Add(i);
}
string ids = ints.Select(query => query.ToString())
.Aggregate((a, b) => a + ", " + b);
}
私は常に拡張メソッドを使用します:
public static string JoinAsString<T>(this IEnumerable<T> input, string seperator)
{
var ar = input.Select(i => i.ToString()).ToArray();
return string.Join(seperator, ar);
}
string.Join
.netでは4はすでにIEnumerable<T>
任意の任意のものをとることができT
ます。
「超クールなLINQの方法」では、LINQが拡張メソッドを使用して関数型プログラミングをより美しくする方法について話しているかもしれません。つまり、入れ子になっているのではなく、視覚的に線形な方法で(次々に)関数を連鎖できる構文上の砂糖です。例えば:
int totalEven = Enumerable.Sum(Enumerable.Where(myInts, i => i % 2 == 0));
このように書くことができます:
int totalEven = myInts.Where(i => i % 2 == 0).Sum();
2番目の例の方が読みやすいことがわかります。また、インデントの問題や式の最後に表示されるLispyの閉じかっこを減らして、関数を追加する方法も確認できます。
他の多くの回答String.Join
は、読むのが最も速いか、最も簡単なので、それが進むべき道であると述べています。しかし、「超クールなLINQ方法」の私の解釈を採用すると、答えは使用String.Join
することですが、視覚的に楽しい方法で関数をチェーンできるようにするLINQスタイルの拡張メソッドにラップされます。したがって、書きたい場合は、次のsa.Concatenate(", ")
ようなものを作成する必要があります。
public static class EnumerableStringExtensions
{
public static string Concatenate(this IEnumerable<string> strings, string separator)
{
return String.Join(separator, strings);
}
}
これにより、(少なくともアルゴリズムの複雑さに関して)直接呼び出しと同じくらいのパフォーマンスのコードが提供され、特にブロック内の他のコードが連鎖関数スタイルを使用している場合は、場合によってはコードが読みやすくなります(コンテキストによって異なります)。 。
私は少しごまかして、コメント内に貼り付けるのではなく、ここでのすべての最高のものをまとめているように見える新しい答えを捨てます。
だからあなたはこれを1行にすることができます:
List<string> strings = new List<string>() { "one", "two", "three" };
string concat = strings
.Aggregate(new StringBuilder("\a"),
(current, next) => current.Append(", ").Append(next))
.ToString()
.Replace("\a, ",string.Empty);
編集:最初に空の列挙型を確認するか.Replace("\a",string.Empty);
、式の最後にを追加します。私は少し賢くなりすぎたのではないかと思います。
@ a.friendからの答えは少しパフォーマンスが高いかもしれませんが、Removeと比較してReplaceが内部で何を行うかわかりません。\ aで終わる文字列を連結したい場合、他の唯一の警告は、セパレーターを失うことになります...私はそうは思わないでしょう。その場合は、他にファンシーキャラクターを選択できます。
LINQとstring.join()
非常に効果的に組み合わせることができます。ここでは、文字列からアイテムを削除しています。これを行うにはもっと良い方法がありますが、ここにあります:
filterset = String.Join(",",
filterset.Split(',')
.Where(f => mycomplicatedMatch(f,paramToMatch))
);
ここにはたくさんの選択肢があります。LINQとStringBuilderを使用して、次のようなパフォーマンスを得ることができます。
StringBuilder builder = new StringBuilder();
List<string> MyList = new List<string>() {"one","two","three"};
MyList.ForEach(w => builder.Append(builder.Length > 0 ? ", " + w : w));
return builder.ToString();
builder.Length > 0
ForEachでチェックせず、ForEachの後の最初のコンマを削除する方が速いでしょう
linqを使用してIISログファイルを解析するときに、次のようにすばやくダーティにしました。200万行を試行するとメモリ不足エラーが発生しましたが、@ 100万行(15秒)でかなりうまくいきました。
static void Main(string[] args)
{
Debug.WriteLine(DateTime.Now.ToString() + " entering main");
// USED THIS DOS COMMAND TO GET ALL THE DAILY FILES INTO A SINGLE FILE: copy *.log target.log
string[] lines = File.ReadAllLines(@"C:\Log File Analysis\12-8 E5.log");
Debug.WriteLine(lines.Count().ToString());
string[] a = lines.Where(x => !x.StartsWith("#Software:") &&
!x.StartsWith("#Version:") &&
!x.StartsWith("#Date:") &&
!x.StartsWith("#Fields:") &&
!x.Contains("_vti_") &&
!x.Contains("/c$") &&
!x.Contains("/favicon.ico") &&
!x.Contains("/ - 80")
).ToArray();
Debug.WriteLine(a.Count().ToString());
string[] b = a
.Select(l => l.Split(' '))
.Select(words => string.Join(",", words))
.ToArray()
;
System.IO.File.WriteAllLines(@"C:\Log File Analysis\12-8 E5.csv", b);
Debug.WriteLine(DateTime.Now.ToString() + " leaving main");
}
私がlinqを使用した本当の理由は、以前必要だったDistinct()のためでした:
string[] b = a
.Select(l => l.Split(' '))
.Where(l => l.Length > 11)
.Select(words => string.Format("{0},{1}",
words[6].ToUpper(), // virtual dir / service
words[10]) // client ip
).Distinct().ToArray()
;
私はこれについて少し前にブログに書いています、私が継ぎ目を行ったことはまさにあなたが探しているものと同じです:
http://ondevelopment.blogspot.com/2009/02/string-concatenation-made-easy.html
ブログの投稿では、IEnumerableで機能し、Concatenateという名前の拡張メソッドを実装する方法について説明しています。これにより、次のような記述が可能になります。
var sequence = new string[] { "foo", "bar" };
string result = sequence.Concatenate();
または次のようなより複雑なもの:
var methodNames = typeof(IFoo).GetMethods().Select(x => x.Name);
string result = methodNames.Concatenate(", ");