Excluir último caractere de string

Estou recuperando muitas informações de uma lista, vinculadas a um banco de dados e desejo criar uma sequência de grupos, para alguém conectado ao site.

Eu uso isso para testar, mas isso não é dinâmico, por isso é muito ruim:

string strgroupids = "6";

Eu quero usar isso agora. Mas a string retornada é algo como1,2,3,4,5,

groupIds.ForEach((g) =>
{
    strgroupids = strgroupids  + g.ToString() + ",";
    strgroupids.TrimEnd(',');
});

strgroupids.TrimEnd(new char[] { ',' });

Eu quero excluir o ,após o, 5mas definitivamente não está funcionando.

strgroupids = strgroupids.Remove(strgroupids.Length - 1);

MSDN:

String.Remove (Int32):

Exclui todos os caracteres dessa sequência, começando em uma posição especificada e continuando até a última posição

Que tal fazer dessa maneira

strgroupids = string.Join( ",", groupIds );

Um muito mais limpo.

Anexará todos os elementos internos groupIdscom um ','entre cada, mas não colocará um ','no final.

Strings em c # são imutáveis. Quando você faz o seu código strgroupids.TrimEnd(',');ou strgroupids.TrimEnd(new char[] { ',' });a strgroupidsstring não é modificada .

Você precisa fazer algo parecido strgroupids = strgroupids.TrimEnd(',');.

Para citar aqui :

As strings são imutáveis ​​- o conteúdo de um objeto de string não pode ser alterado após a criação do objeto, embora a sintaxe faça com que pareça que você pode fazer isso. Por exemplo, quando você escreve esse código, o compilador realmente cria um novo objeto de seqüência de caracteres para armazenar a nova sequência de caracteres e esse novo objeto é atribuído a b. A sequência "h" é qualificada para a coleta de lixo.

Adicione um método de extensão.

public static string RemoveLast(this string text, string character)
{
    if(text.Length < 1) return text;
    return text.Remove(text.ToString().LastIndexOf(character), character.Length);
}

então use:

yourString.RemoveLast(",");

Remove as vírgulas finais:

while (strgroupids.EndsWith(","))
    strgroupids = strgroupids.Substring(0, strgroupids.Length - 1);

Isso é inverso, você escreveu o código que adiciona a vírgula em primeiro lugar. Você deve usar string.Join(",",g), assumindo que gé a string[]. Dê um nome melhor do que gtambém!

Como alternativa à adição de vírgula para cada item, você pode apenas usar String.

var strgroupids = String.Join(",",  groupIds);

Isso adicionará o separador ("," nesta instância) entre cada elemento na matriz.

string strgroupids = string.Empty;

groupIds.ForEach(g =>
{
    strgroupids = strgroupids + g.ToString() + ",";
});

strgroupids = strgroupids.Substring(0, strgroupids.Length - 1);

Observe que o uso ForEachaqui é normalmente considerado "errado" (leia, por exemplo, http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Usando alguns LINQ:

string strgroupids = groupIds.Aggregate(string.Empty, (p, q) => p + q + ',');
strgroupids = strgroupids.Substring(0, str1.Length - 1);

Sem substring final:

string strgroupids = groupIds.Aggregate(string.Empty, (p, q) => (p != string.Empty ? p + "," + q : q.ToString()));

Adicional à solução da sll: É melhor aparar a corda caso haja alguns espaços em branco no final.

strgroupids = strgroupids.Remove(strgroupids.Trim().Length - 1);

string.Joiné melhor, mas se você realmente quer um LINQ ForEach:

var strgroupids = string.Empty;

groupIds.ForEach(g =>
{
    if(strgroupids != string.Empty){
        strgroupids += ",";
    }

    strgroupids += g;
});

Algumas notas:

• string.Joine foreachsão melhores do que essa abordagem, muito mais lenta
• Não é necessário remover o último, ,pois ele nunca é anexado
• O operador de incremento ( +=) é útil para anexar a strings
• .ToString() é desnecessário, pois é chamado automaticamente ao concatenar não-strings
• Ao manusear cordas grandes, StringBuilderdeve ser considerado em vez de concatenar as cordas