Usuń ostatni znak ciągu

Pobieram wiele informacji z listy połączonej z bazą danych i chcę utworzyć ciąg grup dla kogoś, kto jest podłączony do strony internetowej.

Używam tego do testowania, ale nie jest to dynamiczne, więc jest naprawdę złe:

string strgroupids = "6";

Chcę tego teraz użyć. Ale zwrócony ciąg znaków jest podobny1,2,3,4,5,

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

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

Chcę usunąć ,po, 5ale zdecydowanie nie działa.

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

MSDN:

String.Remove (Int32):

Usuwa wszystkie znaki z tego ciągu, zaczynając od określonej pozycji i kontynuując do ostatniej pozycji

Co powiesz na robienie tego w ten sposób?

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

Wiele czystsze.

Będzie dołączyć wszystkie elementy wewnątrz groupIdsz ','pomiędzy siebie, ale to nie będzie umieścić ','na końcu.

Ciągi w c # są niezmienne. Kiedy w kodzie zrobić strgroupids.TrimEnd(',');lub ciąg nie jest modyfikowany .strgroupids.TrimEnd(new char[] { ',' });strgroupids

Zamiast tego musisz zrobić coś takiego strgroupids = strgroupids.TrimEnd(',');.

Aby cytować stąd :

Ciągi są niezmienne - zawartość obiektu łańcuchowego nie może zostać zmieniona po utworzeniu obiektu, chociaż składnia sprawia, że ​​wygląda to tak, jakbyś mógł to zrobić. Na przykład, gdy piszesz ten kod, kompilator faktycznie tworzy nowy obiekt łańcuchowy do przechowywania nowej sekwencji znaków, a ten nowy obiekt jest przypisywany do b. Ciąg „h” kwalifikuje się wówczas do wyrzucania elementów bezużytecznych.

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 wszystkie przecinki końcowe:

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

Jest to jednak odwrotnie, napisałeś kod, który dodaje przecinek w pierwszej kolejności. Powinieneś użyć string.Join(",",g)zamiast tego, zakładając, że gjest string[]. Nadaj mu lepszą nazwę niż gteż!

Zamiast dodawania przecinka dla każdego elementu można po prostu użyć String.Join:

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

Spowoduje to dodanie separatora („,” w tym przypadku) między każdym elementem w tablicy.

string strgroupids = string.Empty;

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

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

Zauważ, że użycie ForEachtutaj 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 )

Korzystanie z LINQ:

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

Bez podciągów końcowych:

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

Dodatkowo do rozwiązania sll: Lepiej przyciąć łańcuch, jeśli na końcu są jakieś puste miejsca.

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

string.Joinjest lepszy, ale jeśli naprawdę chcesz LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Niektóre uwagi:

• string.Joini foreachoba są lepsze niż to, znacznie wolniejsze podejście
• Nie trzeba usuwać ostatniego, ,ponieważ nigdy nie jest dołączany
• Operator increment ( +=) jest przydatny do dołączania do łańcuchów
• .ToString() jest niepotrzebne, ponieważ jest wywoływane automatycznie podczas łączenia łańcuchów innych niż łańcuchy
• Podczas obsługi dużych ciągów StringBuildernależy rozważyć zamiast łączenia ciągów