Xóa char cuối cùng của chuỗi

Tôi đang lấy rất nhiều thông tin trong một danh sách, được liên kết với cơ sở dữ liệu và tôi muốn tạo một chuỗi các nhóm, cho một người nào đó được kết nối với trang web.

Tôi sử dụng điều này để kiểm tra nhưng điều này không năng động, vì vậy nó thực sự tồi tệ:

string strgroupids = "6";

Tôi muốn sử dụng nó bây giờ. Nhưng chuỗi trả về là một cái gì đó như1,2,3,4,5,

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

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

Tôi muốn xóa ,sau 5nhưng nó chắc chắn không hoạt động.

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

MSDN:

Chuỗi.Remove (Int32):

Xóa tất cả các ký tự khỏi chuỗi này bắt đầu tại một vị trí đã chỉ định và tiếp tục qua vị trí cuối cùng

Làm gì theo cách này

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

Một rất nhiều bụi.

Nó sẽ nối tất cả các yếu tố bên trong groupIdsvới một ','giữa, nhưng nó sẽ không đặt ','cuối cùng.

Chuỗi trong c # là bất biến. Khi trong mã của bạn, bạn làm strgroupids.TrimEnd(',');hoặc strgroupids.TrimEnd(new char[] { ',' });các strgroupidschuỗi không được sửa đổi .

Bạn cần phải làm một cái gì đó như strgroupids = strgroupids.TrimEnd(',');thay vào đó.

Để trích dẫn từ đây :

Chuỗi là bất biến - nội dung của một đối tượng chuỗi không thể thay đổi sau khi đối tượng được tạo, mặc dù cú pháp làm cho nó xuất hiện như thể bạn có thể làm điều này. Ví dụ, khi bạn viết mã này, trình biên dịch thực sự tạo ra một đối tượng chuỗi mới để giữ chuỗi ký tự mới và đối tượng mới đó được gán cho b. Chuỗi "h" sau đó đủ điều kiện để thu gom rác.

Thêm một phương thức mở rộng.

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

sau đó sử dụng:

yourString.RemoveLast(",");

Xóa mọi dấu phẩy:

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

Điều này là ngược, mặc dù, bạn đã viết mã thêm dấu phẩy ở vị trí đầu tiên. Bạn nên sử dụng string.Join(",",g)thay thế, giả sử glà a string[]. Đặt cho nó một cái tên tốt hơn gquá!

Thay thế cho việc thêm dấu phẩy cho mỗi mục, bạn chỉ có thể sử dụng String.Join:

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

Điều này sẽ thêm seperator ("," trong trường hợp này) giữa mỗi phần tử trong mảng.

string strgroupids = string.Empty;

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

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

Lưu ý rằng việc sử dụng ForEachở đây thường được coi là "sai" (đọc ví dụ http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Sử dụng một số LINQ:

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

Không có lớp nền cuối:

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

Bổ sung cho giải pháp của sll: Tốt hơn là cắt chuỗi trong trường hợp có một số trống ở cuối.

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

string.Joinlà tốt hơn, nhưng nếu bạn thực sự muốn có một LINQ ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Một số lưu ý:

• string.Joinvà foreachcả hai đều tốt hơn thế này, cách tiếp cận chậm hơn rất nhiều
• Không cần phải xóa cái cuối cùng ,vì nó không bao giờ được thêm vào
• Toán tử gia tăng ( +=) thuận tiện cho việc nối vào chuỗi
• .ToString() là không cần thiết vì nó được gọi tự động khi nối các chuỗi không
• Khi xử lý các chuỗi lớn, StringBuildernên xem xét thay vì nối các chuỗi