Der beste Weg, um das letzte Zeichen aus einer mit Stringbuilder erstellten Zeichenfolge zu entfernen

Ich habe folgendes

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

Das Problem dabei ist, dass ich es in einer Schleife verwende und es ein Komma gibt. Was ist der beste Weg, um das nachfolgende Komma zu entfernen?

Muss ich Daten in eine Zeichenfolge ändern, die Teilzeichenfolge enthält?

Der einfachste und effizienteste Weg besteht darin, diesen Befehl auszuführen:

data.Length--;

Auf diese Weise verschieben Sie den Zeiger (dh den letzten Index) um ein Zeichen zurück, ändern jedoch nicht die Veränderlichkeit des Objekts. Tatsächlich ist das Löschen von a StringBuilderam besten auch mit Length(verwenden Sie die Clear()Methode jedoch aus Gründen der Klarheit, da die Implementierung so aussieht):

data.Length = 0;

wieder, weil es die Zuordnungstabelle nicht ändert. Stellen Sie sich vor, Sie möchten diese Bytes nicht mehr erkennen. Selbst wenn er anruft ToString(), erkennt er nichts mehr als das , was Lengther kann. Es ist ein veränderliches Objekt, das mehr Speicherplatz zuweist als Sie bereitstellen. Es ist einfach so aufgebaut.

Benutz einfach

string.Join(",", yourCollection)

Auf diese Weise benötigen Sie die StringBuilderund die Schleife nicht.

Langer Zusatz zum asynchronen Fall. Ab 2019 ist es keine seltene Einrichtung, wenn die Daten asynchron kommen.

Wenn sich Ihre Daten in einer asynchronen Sammlung befinden, erfolgt keine string.JoinÜberlastung IAsyncEnumerable<T>. Es ist jedoch einfach, einen manuell zu erstellen und den Code zustring.Join hacken von :

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

Wenn die Daten asynchron kommen, die Schnittstelle IAsyncEnumerable<T>jedoch nicht unterstützt wird (wie in den Kommentaren erwähnt SqlDataReader), ist es relativ einfach, die Daten in Folgendes zu verpacken 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]);
}

und benutze es:

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

Um verwenden zu können Select, müssen Sie das Nuget-Paket verwenden System.Interactive.Async. Hier finden Sie ein kompilierbares Beispiel.

Verwenden Sie nach der Schleife Folgendes.

.TrimEnd(',')

oder einfach zu wechseln

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

Wie wäre es damit..

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

Ich bevorzuge es, die Länge des Stringbuilders zu manipulieren:

data.Length = data.Length - 1;

Ich empfehle Ihnen, Ihren Schleifenalgorithmus zu ändern:

• Fügen Sie das Komma nicht NACH dem Element hinzu, sondern VOR
• Verwenden Sie eine boolesche Variable, die mit false beginnt. Unterdrücken Sie das erste Komma
• Setzen Sie diese boolesche Variable nach dem Testen auf true

Sie sollten die string.JoinMethode verwenden, um eine Sammlung von Elementen in eine durch Kommas getrennte Zeichenfolge umzuwandeln. Dadurch wird sichergestellt, dass kein führendes oder nachfolgendes Komma vorhanden ist, und es wird sichergestellt, dass die Zeichenfolge effizient aufgebaut ist (ohne unnötige Zwischenzeichenfolgen).

Ja, konvertieren Sie es nach Abschluss der Schleife in einen String:

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

Sie haben zwei Möglichkeiten. Erstens ist eine sehr einfach zu bedienenRemove Methode, die sehr effektiv ist. Die zweite Möglichkeit ist die Verwendung ToStringmit Startindex und Endindex ( MSDN-Dokumentation ).

Ähnliche SO Frage hier.

Mir hat die Verwendung einer StringBuilder-Erweiterungsmethode gefallen.

RemoveLast-Methode

Am einfachsten wäre es, die Join () -Methode zu verwenden:

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

Wenn Sie den StringBuilder wirklich benötigen, schneiden Sie das Endkomma nach der Schleife ab:

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