Usuń ostatni znak łańcucha

A co z robieniem tego w ten sposób

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

Alot cleaner.

Doda wszystkie elementy wewnątrz groupIds z ',' pomiędzy każdym, ale nie umieści ',' na końcu.

Ciągi w c# są niezmienne. Gdy w Twoim kodzie robisz strgroupids.TrimEnd(','); LUB strgroupids.TrimEnd(new char[] { ',' }); strgroupids ciąg nie jest modyfikowany .

Cytuję z tutaj :

Łańcuchy są niezmienne--zawartość obiektu string nie może być zmieniany po wytworzeniu obiektu, choć składnia czyni go wygląda na to, że możesz to zrobić. Na przykład, kiedy piszesz ten kod, kompilator faktycznie tworzy nowy obiekt string do hold the new sekwencji znaków, a ten nowy obiekt jest przypisany do b. W ciąg " h " kwalifikuje się wtedy do usuwania śmieci.

Dodaj metodę rozszerzenia.

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

Następnie użyj:

yourString.RemoveLast(",");

Usuwa przecinki końcowe:

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

To jest jednak odwrotnie, napisałeś kod, który dodaje przecinek na pierwszym miejscu. Zamiast tego należy użyć string.Join(",",g), zakładając, że g jest string[]. Nadaj jej też lepszą nazwę niż g!

W C# 8 wprowadzono zakresy i indeksy , dając nam nowe, bardziej zwięzłe rozwiązanie:

strgroupids = strgroupids[..^1];

Jako alternatywę do dodawania przecinka dla każdego elementu można po prostu użyć String.Join:

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

Spowoduje dodanie separatora ("," w tym przypadku) pomiędzy każdym elementem tablicy.

string strgroupids = string.Empty;

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

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

Zauważ, że użycie ForEach tutaj jest zwykle uważane za " złe " (czytaj na przykład http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Używając LINQ:

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

Bez podciągu końcowego:

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

Dodatkowe rozwiązanie sll: lepiej przyciąć łańcuch w przypadku, gdy są jakieś puste (s) na końcu.

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

string.Join jest lepsze, ale jeśli naprawdę chcesz LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Kilka uwag:

• string.Join i foreach są lepsze od tego, znacznie wolniejsze, podejście
• nie ma potrzeby usuwania ostatniego ,, ponieważ nigdy nie jest dołączany
• operator increment (+=) jest przydatny do dodawania do łańcuchów
• .ToString() jest niepotrzebny, ponieważ jest wywoływany automatycznie podczas łączenia nie-ciągów
• podczas obsługi dużych łańcuchów, StringBuilder powinny być rozważane zamiast konkatenacji strun

Nie ma na to żadnego "szybkiego i brudnego" sposobu. Zwykle tak:

mystring= string.Concat(mystring.Take(mystring.Length-1));