stringbuilderで構築された文字列から最後の文字を削除する最良の方法

私は以下を持っています

data.AppendFormat("{0},",dataToAppend);

これの問題は、それをループで使用していて、試行錯誤のカンマがあることです。末尾のコンマを削除する最良の方法は何ですか？

データを文字列、部分文字列に変更する必要がありますか？

最も簡単で最も効率的な方法は、次のコマンドを実行することです。

data.Length--;

これを行うことにより、ポインタ（つまり、最後のインデックス）を1文字戻しますが、オブジェクトの変更可能性は変更しません。実際、aのクリアStringBuilderはLengthも同様にです（ただしClear()、実装がどのようなものであるかを明確にするために、実際にはメソッドを使用してください）。

data.Length = 0;

割り当てテーブルを変更しないためです。このようなバイトを認識したくないというようなことを考えてみてください。さて、を呼び出してもToString()、それ以降は何も認識しLengthません。これは、提供するものより多くのスペースを割り当てる可変オブジェクトであり、この方法で単純に構築されます。

使うだけ

string.Join(",", yourCollection)

この方法StringBuilderでは、ループは必要ありません。

非同期のケースに関する長い追加。2019年の時点では、データが非同期で送信される場合のセットアップはまれではありません。

データが非同期コレクションにある場合、string.Joinオーバーロードの取得はありませんIAsyncEnumerable<T>。しかし、手動で作成して、次のコードをstring.Joinハッキングするのは簡単です。

public static class StringEx
{
    public static async Task<string> JoinAsync<T>(string separator, IAsyncEnumerable<T> seq)
    {
        if (seq == null)
            throw new ArgumentNullException(nameof(seq));

        await using (var en = seq.GetAsyncEnumerator())
        {
            if (!await en.MoveNextAsync())
                return string.Empty;

            string firstString = en.Current?.ToString();

            if (!await en.MoveNextAsync())
                return firstString ?? string.Empty;

            // Null separator and values are handled by the StringBuilder
            var sb = new StringBuilder(256);
            sb.Append(firstString);

            do
            {
                var currentValue = en.Current;
                sb.Append(separator);
                if (currentValue != null)
                    sb.Append(currentValue);
            }
            while (await en.MoveNextAsync());
            return sb.ToString();
        }
    }
}

データが非同期で送られてくるが、インターフェースIAsyncEnumerable<T>がサポートされていない場合（コメントで述べたようにSqlDataReader）、データをにラップするのは比較的簡単IAsyncEnumerable<T>です：

async IAsyncEnumerable<(object first, object second, object product)> ExtractData(
        SqlDataReader reader)
{
    while (await reader.ReadAsync())
        yield return (reader[0], reader[1], reader[2]);
}

そしてそれを使う：

Task<string> Stringify(SqlDataReader reader) =>
    StringEx.JoinAsync(
        ", ",
        ExtractData(reader).Select(x => $"{x.first} * {x.second} = {x.product}"));

を使用するにはSelect、nugetパッケージを使用する必要がありますSystem.Interactive.Async。ここにコンパイル可能な例があります。

ループの後、次を使用します。

.TrimEnd(',')

または単にに変更

string commaSeparatedList = input.Aggregate((a, x) => a + ", " + x)

これはどう..

string str = "The quick brown fox jumps over the lazy dog,";
StringBuilder sb = new StringBuilder(str);
sb.Remove(str.Length - 1, 1);

私は文字列ビルダーの長さを操作することを好みます：

data.Length = data.Length - 1;

ループアルゴリズムを変更することをお勧めします。

• 項目の後ではなく前にコンマを追加します
• falseで始まるブール変数を使用して、最初のコンマを抑制します
• テスト後にこのブール変数をtrueに設定します

string.Joinメソッドを使用して、項目のコレクションをカンマ区切りの文字列に変換する必要があります。これにより、先頭または末尾にコンマがなくなり、文字列が効率的に構築されます（不要な中間文字列なしで）。

はい、ループが完了したら、文字列に変換します。

String str = data.ToString().TrimEnd(',');

2つのオプションがあります。最初のものは非常に簡単な使用Remove方法で、非常に効果的です。2番目の方法はToString、開始インデックスと終了インデックスで使用することです（MSDNドキュメント）

ここで同様のSOの質問。

StringBuilder拡張メソッドを使用するのが好きでした。

RemoveLastメソッド

最も簡単な方法は、Join（）メソッドを使用することです。

public static void Trail()
{
    var list = new List<string> { "lala", "lulu", "lele" };
    var data = string.Join(",", list);
}

StringBuilderが本当に必要な場合は、ループの後に終了コンマをトリミングします。

data.ToString().TrimEnd(',');