Il modo migliore per rimuovere l'ultimo carattere da una stringa creata con stringbuilder

Ho il seguente

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

Il problema con questo è che lo sto usando in un ciclo e ci sarà una virgola di prova. Qual è il modo migliore per rimuovere la virgola finale?

Devo modificare i dati in una stringa la sottostringa?

Il modo più semplice ed efficiente è eseguire questo comando:

data.Length--;

in questo modo sposti il ​​puntatore (cioè l'ultimo indice) indietro di un carattere ma non modifichi la mutabilità dell'oggetto. In effetti, anche la cancellazione di a StringBuilderè meglio farlo Length(ma in realtà usa il Clear()metodo per chiarezza invece perché è così che appare la sua implementazione):

data.Length = 0;

di nuovo, perché non cambia la tabella di allocazione. Pensa come dire, non voglio più riconoscere questi byte. Ora, anche quando chiama ToString(), non riconoscerà nulla oltre il suo Length, beh, non può. È un oggetto mutevole che alloca più spazio di quello che gli fornisci, è semplicemente costruito in questo modo.

Basta usare

string.Join(",", yourCollection)

In questo modo non avrai bisogno StringBuilderdel ciclo e.

Aggiunta lunga sul caso asincrono. A partire dal 2019, non è raro che i dati arrivino in modo asincrono.

Nel caso in cui i tuoi dati siano in una raccolta asincrona, non si verifica alcun string.Joinsovraccarico IAsyncEnumerable<T>. Ma è facile crearne uno manualmente, hackerando il codice dastring.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();
        }
    }
}

Se i dati arrivano in modo asincrono ma l'interfaccia IAsyncEnumerable<T>non è supportata (come menzionato nei commenti SqlDataReader), è relativamente facile racchiudere i dati in un 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]);
}

e usalo:

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

Per Selectpoterlo utilizzare, dovrai utilizzare il pacchetto nuget System.Interactive.Async. Qui puoi trovare un esempio compilabile.

Utilizzare quanto segue dopo il ciclo.

.TrimEnd(',')

o semplicemente passare a

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

Cosa ne pensi di questo..

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

Preferisco manipolare la lunghezza dello stringbuilder:

data.Length = data.Length - 1;

Mi raccomando, cambia il tuo algoritmo di loop:

• Aggiungi la virgola non DOPO l'elemento, ma PRIMA
• Usa una variabile booleana, che inizia con false, sopprimi la prima virgola
• Imposta questa variabile booleana su true dopo averla testata

È necessario utilizzare il string.Joinmetodo per trasformare una raccolta di elementi in una stringa delimitata da virgole. Garantirà che non ci siano virgole iniziali o finali, oltre a garantire che la stringa sia costruita in modo efficiente (senza stringhe intermedie non necessarie).

Sì, convertilo in una stringa una volta terminato il ciclo:

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

Hai due opzioni. Il primo è un Removemetodo di utilizzo molto semplice ed è abbastanza efficace. Il secondo modo è usare ToStringcon l'indice iniziale e l'indice finale ( documentazione MSDN )

Domanda SO simile qui.

Mi è piaciuto l'utilizzo di un metodo di estensione StringBuilder.

Metodo RemoveLast

Il modo più semplice sarebbe usare il metodo Join ():

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

Se hai davvero bisogno di StringBuilder, taglia la virgola finale dopo il ciclo:

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