Видалити останній символ рядка

Я отримую багато інформації у списку, пов’язаному з базою даних і хочу створити рядок груп для тих, хто підключений до веб-сайту.

Я використовую це для тестування, але це не динамічно, тому це дуже погано:

string strgroupids = "6";

Я хочу зараз цим скористатися. Але повернутий рядок - це щось на кшталт1,2,3,4,5,

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

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

Я хочу видалити ,після, 5але це точно не працює.

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

MSDN:

String.Remove (Int32):

Видаляє всі символи з цього рядка, починаючи з заданої позиції та продовжуючи останню позицію

Що робити з цим способом

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

Багато чистіше.

Він буде додавати всі елементи всередині groupIdsз ','між собою, але це не буде поставити ','в кінці.

Рядки в c # незмінні. Коли у вашому коді ви strgroupids.TrimEnd(',');або рядок не змінюється .strgroupids.TrimEnd(new char[] { ',' });strgroupids

Вам потрібно зробити щось подібне strgroupids = strgroupids.TrimEnd(',');замість цього.

Цитувати звідси :

Рядки незмінні - вміст рядкового об'єкта не може бути змінено після створення об’єкта, хоча синтаксис видає, що ви можете це зробити. Наприклад, коли ви пишете цей код, компілятор фактично створює новий рядковий об'єкт для утримання нової послідовності символів, і цей новий об'єкт присвоюється b. Потім рядок "h" є придатним для збору сміття.

Додати метод розширення.

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

тоді використовуйте:

yourString.RemoveLast(",");

Видаляє будь-які затримки:

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

Це все назад, але ви написали код, який додає коску в першу чергу. Ви повинні використовувати string.Join(",",g)замість цього, припускаючи, що gце string[]. Назвіть це краще ім'я, ніж gтеж!

В якості альтернативи доданню коми для кожного елемента ви можете просто використовувати String.Join:

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

Це додасть сеператор ("," у цьому випадку) між кожним елементом масиву.

string strgroupids = string.Empty;

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

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

Зауважте, що використання ForEachтут зазвичай вважається "неправильним" (читайте, наприклад, http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Використання деяких LINQ:

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

Без кінцевої підрядки:

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

Додатково до рішення sll: Краще обрізати рядок у випадку, якщо в кінці є деякі порожні (і).

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

string.Joinкраще, але якщо ви дійсно хочете LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Деякі примітки:

• string.Joinі foreachобидва кращі, ніж цей, набагато повільніше, підхід
• Не потрібно видаляти останню, ,оскільки вона ніколи не додається
• Оператор приросту ( +=) зручний для додавання до рядків
• .ToString() є непотрібним, оскільки його називають автоматично при об'єднанні не рядків
• При обробці великих струн StringBuilderслід враховувати замість об'єднувальних рядків