Saya memiliki yang berikut ini

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

Masalahnya adalah saya menggunakannya dalam satu lingkaran dan akan ada koma uji coba. Apa cara terbaik untuk menghilangkan tanda koma?

Apakah saya harus mengubah data menjadi string substring itu?

Cara termudah dan paling efisien adalah dengan melakukan perintah ini:

data.Length--;

dengan melakukan ini Anda memindahkan penunjuk (yaitu indeks terakhir) ke belakang satu karakter tetapi Anda tidak mengubah mutabilitas objek. Faktanya, membersihkan a StringBuilderpaling baik dilakukan dengan Lengthjuga (tetapi sebenarnya menggunakan Clear()metode untuk kejelasan sebagai gantinya karena seperti itulah implementasinya):

data.Length = 0;

sekali lagi, karena tidak mengubah tabel alokasi. Anggap saja seperti mengatakan, saya tidak ingin mengenali byte ini lagi. Sekarang, bahkan saat menelepon ToString(), ia tidak akan mengenali apa pun yang melewatinya Length, yah, ia tidak bisa. Ini adalah objek yang bisa berubah yang mengalokasikan lebih banyak ruang daripada yang Anda sediakan, itu dibuat dengan cara ini.

Gunakan saja

string.Join(",", yourCollection)

Dengan cara ini Anda tidak membutuhkan StringBuilderdan loop.

Penambahan panjang tentang kasus asinkron. Pada 2019, bukan hal yang jarang terjadi ketika data datang secara tidak sinkron.

Jika data Anda berada dalam koleksi asinkron, tidak ada string.Joinpengambilan berlebihan IAsyncEnumerable<T>. Tetapi mudah untuk membuatnya secara manual, meretas kode daristring.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();
        }
    }
}

Jika data datang secara asinkron tetapi antarmukanya IAsyncEnumerable<T>tidak didukung (seperti yang disebutkan di komentar SqlDataReader), relatif mudah untuk menggabungkan data menjadi 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]);
}

dan gunakan itu:

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

Untuk menggunakan Select, Anda harus menggunakan paket nuget System.Interactive.Async. Di sini Anda dapat menemukan contoh kompilasi.

Gunakan yang berikut ini setelah pengulangan.

.TrimEnd(',')

atau cukup ubah menjadi

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

Bagaimana dengan ini..

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

Saya lebih suka memanipulasi panjang stringbuilder:

data.Length = data.Length - 1;

Saya sarankan, Anda mengubah algoritma loop Anda:

• Tambahkan koma bukan SETELAH item, tetapi SEBELUM
• Gunakan variabel boolean, yang dimulai dengan false, jangan tekan koma pertama
• Setel variabel boolean ini ke true setelah mengujinya

Anda harus menggunakan string.Joinmetode ini untuk mengubah kumpulan item menjadi string yang dipisahkan koma. Ini akan memastikan bahwa tidak ada koma di depan atau di belakang, serta memastikan string dibangun secara efisien (tanpa string perantara yang tidak perlu).

Ya, ubah menjadi string setelah loop selesai:

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

Anda punya dua pilihan. Yang pertama adalah Removemetode penggunaan yang sangat mudah , cukup efektif. Cara kedua adalah menggunakan ToStringindeks awal dan indeks akhir ( dokumentasi MSDN )

Pertanyaan SO serupa di sini.

Saya suka menggunakan metode ekstensi StringBuilder.

Metode RemoveLast

Cara paling sederhana adalah dengan menggunakan metode Join ():

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

Jika Anda benar-benar membutuhkan StringBuilder, potong koma akhir setelah pengulangan:

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

Kena kau!!

Sebagian besar jawaban di utas ini tidak akan berfungsi jika Anda menggunakan AppendLineseperti di bawah ini:

var builder = new StringBuilder();
builder.AppendLine("One,");
builder.Length--; // Won't work
Console.Write(builder.ToString());

builder = new StringBuilder();
builder.AppendLine("One,");
builder.Length += -1; // Won't work
Console.Write(builder.ToString());

builder = new StringBuilder();
builder.AppendLine("One,");
Console.Write(builder.TrimEnd(',')); // Won't work

Fiddle Me

MENGAPA??? @ (& ** (& @ !!

Masalahnya sederhana tapi butuh beberapa saat untuk mengetahuinya: Karena ada 2 karakter yang tidak terlihat di akhir CRdan LF(Carriage Return dan Line Feed). Oleh karena itu, Anda perlu menghilangkan 3 karakter terakhir:

var builder = new StringBuilder();
builder.AppendLine("One,");
builder.Length -= 3; // This will work
Console.WriteLine(builder.ToString());

Kesimpulannya

Gunakan Length--atau Length -= 1jika metode terakhir yang Anda panggil adalah Append. Gunakan Length =- 3jika Anda metode terakhir yang Anda panggil AppendLine.