Dizenin son karakterini sil

Bir veritabanına bağlı bir listede çok fazla bilgi alıyorum ve web sitesine bağlı biri için gruplar dizesi oluşturmak istiyorum.

Bunu test etmek için kullanıyorum ama bu dinamik değil, bu yüzden gerçekten kötü:

string strgroupids = "6";

Bunu şimdi kullanmak istiyorum. Ama döndürülen dize şuna benzer1,2,3,4,5,

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

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

Ben Silmek istediğiniz ,sonra 5ama kesinlikle çalışmıyor.

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

MSDN:

String.Remove (Int32):

Belirtilen bir konumdan başlayıp son konumdan devam ederek bu dizeden tüm karakterleri siler

Bu şekilde yapmaya ne dersin?

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

bir sürü temizleyici.

İçerideki tüm öğeleri her birine groupIdsbir arada ekler ',', ancak ','sonuna bir a koymaz.

C # dizeleri değişmez. Kodunuzdaki Bunu yaptığınızda strgroupids.TrimEnd(',');veya dize değiştirilmezstrgroupids.TrimEnd(new char[] { ',' });strgroupids .

Gibi bir şey yapmalısın strgroupids = strgroupids.TrimEnd(','); yerine .

Buradan alıntı yapmak için :

Dizeler değiştirilemez - sözdizimi bunu yapabiliyormuş gibi görünmesine rağmen, bir dize nesnesinin içeriği nesne oluşturulduktan sonra değiştirilemez. Örneğin, bu kodu yazdığınızda, derleyici aslında yeni karakter dizisini tutmak için yeni bir dize nesnesi oluşturur ve bu yeni nesne b'ye atanır. Daha sonra "h" dizesi çöp toplama için uygundur.

Bir uzantı yöntemi ekleyin.

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

sonra kullan:

yourString.RemoveLast(",");

Sondaki virgülleri kaldırır:

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

Bu geriye doğru olsa da, ilk etapta virgül ekleyen kodu yazdınız. Bunun string.Join(",",g)yerine, gbir string[]. gÇok daha iyi bir isim verin !

Her öğe için virgül eklemenin alternatifi olarak String.Join'i kullanabilirsiniz:

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

Bu , dizideki her öğe arasına ayırıcı (",") ekler .

string strgroupids = string.Empty;

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

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

Kullanımı geldiğini hatırlatırız ForEachburada normalde "yanlış" olarak kabul edilir (örneğin okuma http://blogs.msdn.com/b/ericlippert/archive/2009/05/18/foreach-vs-foreach.aspx )

Bazı LINQ kullanma:

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

Uç alt dizeleme olmadan:

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

Sll'nin çözümüne ek olarak: Sonunda boş (lar) olması durumunda dizeyi kırpmak daha iyidir.

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

string.Joindaha iyidir, ancak gerçekten bir LINQ istiyorsanız ForEach:

var strgroupids = string.Empty;

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

    strgroupids += g;
});

Bazı notlar:

• string.Joinve foreachikisi de bundan daha iyi, çok daha yavaş bir yaklaşım
• Sonuncuyu kaldırmaya gerek yok ,Asla eklenmediğinden
• Artış operatörü (+= ) dizelere eklemek için kullanışlıdır
• .ToString() dize olmayanları birleştirirken otomatik olarak çağrıldığından gereksizdir
• Büyük dizeleri StringBuilderkullanırken dizeleri birleştirmek yerine dikkate alınmalıdır