文字列の最後の文字を削除

データベースにリンクされたリストの多くの情報を取得しています。Webサイトに接続している人のために、一連のグループを作成します。

私はこれをテストに使用していますが、これは動的ではないため、本当に悪いです：

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の','間で追加しますが','、最後にa を置きません。

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は、文字列を連結する代わりに検討する必要があります