Melhor maneira de remover o último caractere de uma string construída com stringbuilder

Eu tenho o seguinte

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

O problema com isso é que estou usando-o em um loop e haverá uma vírgula de teste. Qual é a melhor maneira de remover a vírgula final?

Eu tenho que alterar os dados para uma string a substring dela?

A maneira mais simples e eficiente é executar este comando:

data.Length--;

ao fazer isso, você move o ponteiro (isto é, o último índice) um caractere para trás, mas não altera a mutabilidade do objeto. Na verdade, StringBuilderé melhor limpar um Lengthtambém (mas use o Clear()método para maior clareza, porque é assim que sua implementação se parece):

data.Length = 0;

novamente, porque não muda a tabela de alocação. Pense nisso como dizer: não quero mais reconhecer esses bytes. Agora, mesmo quando chama ToString(), ele não reconhece nada além de seu Length, bem, ele não pode. É um objeto mutável que aloca mais espaço do que o fornecido por você; ele é simplesmente construído dessa maneira.

Apenas use

string.Join(",", yourCollection)

Dessa forma, você não precisa do StringBuildere do loop.

Longa adição sobre o caso assíncrono. A partir de 2019, não é uma configuração rara quando os dados chegam de forma assíncrona.

No caso de seus dados estarem em coleta assíncrona, não há string.Joinsobrecarga IAsyncEnumerable<T>. Mas é fácil criar um manualmente, hackeando o 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();
        }
    }
}

Se os dados estão chegando de forma assíncrona, mas a interface IAsyncEnumerable<T>não é compatível (como o mencionado nos comentários SqlDataReader), é relativamente fácil agrupar os dados em 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 usá-lo:

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

Para usar Select, você precisará usar o pacote nuget System.Interactive.Async. Aqui você pode encontrar um exemplo compilável.

Use o seguinte após o loop.

.TrimEnd(',')

ou simplesmente mude para

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

Que tal agora..

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

Eu prefiro manipular o comprimento do construtor de cordas:

data.Length = data.Length - 1;

Eu recomendo, você muda seu algoritmo de loop:

• Adicione a vírgula não DEPOIS do item, mas ANTES
• Use uma variável booleana, que começa com false, suprima a primeira vírgula
• Defina esta variável booleana como verdadeira após testá-la

Você deve usar o string.Joinmétodo para transformar uma coleção de itens em uma string delimitada por vírgulas. Isso garantirá que não haja vírgulas à esquerda ou à direita, bem como garantirá que a string seja construída de forma eficiente (sem strings intermediárias desnecessárias).

Sim, converta-o em uma string assim que o loop for concluído:

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

Você tem duas opções. O primeiro é o Removemétodo de uso muito fácil , é bastante eficaz. A segunda maneira é usar ToStringcom índice inicial e índice final ( documentação MSDN )

Pergunta semelhante de SO aqui.

Eu gostei de usar um método de extensão StringBuilder.

Método RemoveLast

A maneira mais simples seria usar o método Join ():

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

Se você realmente precisa do StringBuilder, apare a vírgula final após o loop:

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