Удалить последний символ строки

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

Я использую это для тестирования, но это не динамично, так что это действительно плохо:

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следует рассматривать вместо объединения строк