Supprimer le dernier caractère de la chaîne

Je récupère beaucoup d'informations dans une liste, liée à une base de données et je veux créer une chaîne de groupes, pour quelqu'un qui est connecté au site Web.

Je l'utilise pour tester mais ce n'est pas dynamique, donc c'est vraiment mauvais:

string strgroupids = "6";

Je veux l'utiliser maintenant. Mais la chaîne retournée est quelque chose comme1,2,3,4,5,

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

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

Je veux supprimer l' ,after, 5mais ça ne marche définitivement pas.

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

MSDN:

String.Remove (Int32):

Supprime tous les caractères de cette chaîne commençant à une position spécifiée et se poursuivant jusqu'à la dernière position

Qu'en est-il de cette façon

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

Un beaucoup plus propre.

Il ajoutera tous les éléments à l'intérieur groupIdsavec un ','entre chacun, mais il ne mettra pas un ','à la fin.

Les chaînes en c # sont immuables. Dans votre code, vous le faites strgroupids.TrimEnd(',');ou strgroupids.TrimEnd(new char[] { ',' });la strgroupidschaîne n'est pas modifiée .

Vous devez faire quelque chose comme à la strgroupids = strgroupids.TrimEnd(',');place.

Pour citer d' ici :

Les chaînes sont immuables - le contenu d'un objet chaîne ne peut pas être modifié après la création de l'objet, bien que la syntaxe le fasse apparaître comme si vous pouviez le faire. Par exemple, lorsque vous écrivez ce code, le compilateur crée en fait un nouvel objet chaîne pour contenir la nouvelle séquence de caractères et ce nouvel objet est affecté à b. La chaîne "h" est alors éligible pour la récupération de place.

Ajoutez une méthode d'extension.

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

puis utilisez:

yourString.RemoveLast(",");

Supprime toutes les virgules de fin:

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

C'est à l'envers cependant, vous avez écrit le code qui ajoute la virgule en premier lieu. Vous devez utiliser à la string.Join(",",g)place, en supposant que gest un string[]. Donnez-lui un meilleur nom que gtrop!

Au lieu d'ajouter une virgule pour chaque élément, vous pouvez simplement utiliser String.

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

Cela ajoutera le séparateur ("," dans ce cas) entre chaque élément du tableau.

string strgroupids = string.Empty;

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

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

Notez que l'utilisation d' ForEachici est normalement considérée comme "incorrecte" (lire par exemple http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Utilisation de LINQ:

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

Sans sous-chaîne d'extrémité:

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

En plus de la solution de sll: il est préférable de couper la chaîne au cas où il y aurait des blancs à la fin.

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

string.Joinc'est mieux, mais si vous voulez vraiment un LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Quelques notes:

• string.Join et foreach sont à la fois mieux que cela, approche beaucoup plus lente
• Pas besoin de retirer le dernier , car il n'est jamais ajouté
• L'opérateur d'incrémentation (+= ) est pratique pour l'ajout aux chaînes
• .ToString() est inutile car il est appelé automatiquement lors de la concaténation de non-chaînes
• Lors de la manipulation de grandes chaînes, StringBuilderdoit être pris en compte au lieu de concaténer des chaînes