リスト状のものを昇順並び替えするには？

Question

前提・実現したいこと

windows form applicationの
listBox1のデータを昇順並び替えしたいです。

1|サンプル1
3|サンプル3
2|サンプル2

簡単に書くとこんな感じのデータ（文字型）を
区切り文字「|」の前部分の値をキーにして
昇順に並び替えてlistbox1に戻したいです。

1|サンプル1
2|サンプル2
3|サンプル3

該当のソースコード

//リスト宣言
List<string> lt = new List<string>();

//昇順でリストに追加
for (int i = 0; i < listBox1.Items.Count; i++)
{
    int n_num = int.Parse(listBox1.Items[i].ToString().Split('|')[0]);

    if (lt.Count == 0)
    {
        lt.Add(listBox1.Items[i].ToString());
    }
    else
    {
        for (int q = lt.Count; q >0; q--)
        {
		int o_num = int.Parse(lt[q - 1].ToString().Split('|')[0]);
		if (n_num > o_num)
		{
		    lt.Insert(q, listBox1.Items[i].ToString());
		    break;
		}
		else
		{
		    if (q == 1)
		    {
		        lt.Insert(0, listBox1.Items[i].ToString());
		    }
		}
        }

    }
}

//リストボックスに戻す
listBox1.Items.Clear();
listBox1.Items.AddRange(lt.ToArray());

試したこと

一応動作はするのですが、
もっと良い方法を知りたいと思い
質問させていただきました。
よろしくお願いいたします。

Comments

[退会済みユーザー]

> もっと良い方法を知りたい
「もっと良い」が何を指すのかわからないので、具体的にお願いします。

[退会済みユーザー]

質問者さんの期待通り動作して問題がなければそれで良いと思いますけど・・・　どこに問題がある、もしくは気に入らないのでしょうか？　それを書いてくれないと適切な答えは得にくいと思いますけど。

[退会済みユーザー]

| の左側の数字は 0 ～ 9 の範囲なのか、10 以上もあるかとかで話が変わってきますが、そこのところはどうなのですか？

Answer 1

こんな感じでどうでしょう？

var sortedItems = listBox1.Items.AsEnumerable<string>().OrderBy(item => item.Split('|')[0]).ToArray();
listBox1.Items.Clear();
listBox1.Items.AddRange(sortedItems);

public static class CollectionsExtends
{
    public static IEnumerable<T> AsEnumerable<T>(this System.Collections.IEnumerable collection)
    {
        foreach (var e in collection) yield return (T)e;
    }
}

Answer 2

Linq使えばもっと簡潔に書けます。

var sortedItems = listBox1.Items
    .Cast<string>()
    .Select( x => new { Index = int.Parse( x.Split( '|' )[0] ), Text = x } )
    .OrderBy( x => x.Index )
    .Select( x => x.Text )
    .ToArray();

//リストボックスに戻す
listBox1.Items.Clear();
listBox1.Items.AddRange( sortedItems );

Answer 3

提示されたコードとほとんど同じだけど，ソート作業をListにやってもらう．

var list = new List< Tuple<int,string> >();
for( int i=0; i<listBox1.Items.Count; ++i )
{
    var ItemStr = listBox1.Items[i].ToString();
    list.Add( new Tuple<int,string>( int.Parse(ItemStr.Split('|')[0]), ItemStr ) );
}
list.Sort();
listBox1.Items.Clear();
foreach( var T in list ){   listBox1.Items.Add( T.Item2 );  }

Answer 4

中の値が数値や時間でしたらCompareでよろしいかと