Eliminar el último carácter de la cadena

Estoy recuperando mucha información en una lista, vinculada a una base de datos y quiero crear una cadena de grupos, para alguien que esté conectado al sitio web.

Lo uso para probar pero esto no es dinámico, por lo que es realmente malo:

string strgroupids = "6";

Quiero usar esto ahora. Pero la cadena devuelta es algo así como1,2,3,4,5,

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

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

Quiero eliminar ,después, 5pero definitivamente no funciona.

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

MSDN:

String.Remove (Int32):

Elimina todos los caracteres de esta cadena que comienzan en una posición específica y continúan hasta la última posición

¿Qué hay de hacerlo de esta manera?

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

Un montón más limpio.

Agregará todos los elementos dentro groupIdscon un ','entre cada uno, pero no pondrá un ','al final.

Las cadenas en c # son inmutables. Cuando en su código lo hace strgroupids.TrimEnd(',');o strgroupids.TrimEnd(new char[] { ',' });la strgroupidscadena no se modifica .

Necesitas hacer algo como en su strgroupids = strgroupids.TrimEnd(',');lugar.

Para citar desde aquí :

Las cadenas son inmutables: el contenido de un objeto de cadena no se puede cambiar una vez creado el objeto, aunque la sintaxis hace que parezca que puede hacerlo. Por ejemplo, cuando escribe este código, el compilador realmente crea un nuevo objeto de cadena para contener la nueva secuencia de caracteres, y ese nuevo objeto se asigna a b. La cadena "h" es elegible para la recolección de basura.

Agregar un método de extensión.

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

luego use:

yourString.RemoveLast(",");

Elimina cualquier coma final:

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

Sin embargo, esto es al revés, usted escribió el código que agrega la coma en primer lugar. En su string.Join(",",g)lugar, debe usar , suponiendo que ges a string[]. ¡Dale un mejor nombre que gtambién!

Como alternativa a agregar una coma para cada elemento, puede usar String.Join:

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

Esto agregará el separador ("," en este caso) entre cada elemento de la matriz.

string strgroupids = string.Empty;

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

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

Tenga en cuenta que el uso de ForEachaquí normalmente se considera "incorrecto" (lea, por ejemplo, http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Usando algunos LINQ:

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

Sin subcadena final:

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

Adicional a la solución de sll: es mejor recortar la cadena en caso de que haya algunos espacios en blanco al final.

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

string.Joines mejor, pero si realmente quieres un LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Algunas notas:

• string.Joiny foreachson ambos mejores que este enfoque, mucho más lento
• No es necesario eliminar el último ,ya que nunca se adjunta
• El operador de incremento ( +=) es útil para agregar cadenas
• .ToString() es innecesario ya que se llama automáticamente al concatenar sin cadenas
• Al manejar cadenas grandes, se StringBuilderdebe considerar en lugar de concatenar cadenas