La mejor manera de eliminar el último carácter de una cadena creada con stringbuilder

Tengo lo siguiente

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

El problema con esto es que lo estoy usando en un bucle y habrá una coma de prueba. ¿Cuál es la mejor forma de eliminar la coma al final?

¿Tengo que cambiar los datos a una cadena de la subcadena?

La forma más sencilla y eficaz es ejecutar este comando:

data.Length--;

al hacer esto, mueve el puntero (es decir, el último índice) hacia atrás un carácter, pero no cambia la mutabilidad del objeto. De hecho, StringBuilderes mejor borrar a Lengthtambién (pero en realidad use el Clear()método para mayor claridad porque así es como se ve su implementación):

data.Length = 0;

de nuevo, porque no cambia la tabla de asignación. Piense en ello como si dijera que ya no quiero reconocer estos bytes. Ahora, incluso al llamar ToString(), no reconocerá nada más allá de su Length, bueno, no puede. Es un objeto mutable que asigna más espacio del que le proporcionas, simplemente está construido de esta manera.

Solo usa

string.Join(",", yourCollection)

De esta manera no necesitas el StringBuildery el bucle.

Adición larga sobre el caso asincrónico. A partir de 2019, no es una configuración rara cuando los datos llegan de forma asincrónica.

En caso de que sus datos estén en una recopilación asíncrona, no hay string.Joinsobrecarga IAsyncEnumerable<T>. Pero es fácil crear uno manualmente, pirateando el código destring.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();
        }
    }
}

Si los datos llegan de forma asincrónica pero la interfaz IAsyncEnumerable<T>no es compatible (como se menciona en los comentarios SqlDataReader), es relativamente fácil envolver los datos en 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]);
}

y usarlo:

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

Para poder usarlo Select, necesitará usar nuget package System.Interactive.Async. Aquí puede encontrar un ejemplo compilable.

Utilice lo siguiente después del ciclo.

.TrimEnd(',')

o simplemente cambia a

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

Qué tal esto..

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

Prefiero manipular la longitud del constructor de cadenas:

data.Length = data.Length - 1;

Te recomiendo que cambies tu algoritmo de bucle:

• Agregue la coma no DESPUÉS del elemento, sino ANTES
• Use una variable booleana, que comience con falso, suprima la primera coma
• Establezca esta variable booleana en verdadera después de probarla

Debe usar el string.Joinmétodo para convertir una colección de elementos en una cadena delimitada por comas. Se asegurará de que no haya comas al principio o al final, así como también garantizará que la cadena se construya de manera eficiente (sin cadenas intermedias innecesarias).

Sí, conviértalo en una cadena una vez que finalice el ciclo:

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

Tienes dos opciones. El primero es un Removemétodo de uso muy fácil , es bastante efectivo. La segunda forma es usar ToStringcon índice inicial y índice final ( documentación de MSDN )

Pregunta SO similar aquí.

Me gustó el uso de un método de extensión StringBuilder.

Eliminar último método

La forma más sencilla sería utilizar el método Join ():

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

Si realmente necesita StringBuilder, recorte la coma final después del bucle:

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