web-dev-qa-db-ja.com

List <T>の2つのアイテムを交換します

list<T>内の2つのアイテムの位置を入れ替えるLINQの方法はありますか?

63
Tony The Lion

C#:Swapメソッドの適切な実装 のMarcからの回答を確認してください。

public static void Swap<T>(IList<T> list, int indexA, int indexB)
{
    T tmp = list[indexA];
    list[indexA] = list[indexB];
    list[indexB] = tmp;
}

linq-i-fiedのようにできます

public static IList<T> Swap<T>(this IList<T> list, int indexA, int indexB)
{
    T tmp = list[indexA];
    list[indexA] = list[indexB];
    list[indexB] = tmp;
    return list;
}

var lst = new List<int>() { 8, 3, 2, 4 };
lst = lst.Swap(1, 2);
100
Jan Jongboom

誰かがこれを行う賢い方法を考えるかもしれませんが、そうすべきではありません。リスト内の2つのアイテムを交換することは本質的に副作用がありますが、LINQ操作には副作用がないはずです。したがって、単純な拡張メソッドを使用するだけです。

static class IListExtensions {
    public static void Swap<T>(
        this IList<T> list,
        int firstIndex,
        int secondIndex
    ) {
        Contract.Requires(list != null);
        Contract.Requires(firstIndex >= 0 && firstIndex < list.Count);
        Contract.Requires(secondIndex >= 0 && secondIndex < list.Count);
        if (firstIndex == secondIndex) {
            return;
        }
        T temp = list[firstIndex];
        list[firstIndex] = list[secondIndex];
        list[secondIndex] = temp;
    }
}
30
jason

既存のSwapメソッドはないため、自分で作成する必要があります。もちろん、それをlinqifyできますが、それは1つの(書かれていない?)ルールを念頭に置いて行う必要があります:LINQ操作は入力パラメーターを変更しません!

他の「linqify」回答では、(入力)リストが変更されて返されますが、このアクションはそのルールを妨げます。並べ替えられていないアイテムのリストがある場合に奇妙な場合は、LINQの「OrderBy」操作を実行し、入力リストも(結果のように)並べ替えられることを発見します。これは許可されていません!

だから...これをどうやってやるの?

私が最初に考えたのは、コレクションの繰り返しが完了した後にコレクションを復元することでした。しかし、これはdirtyソリューションなので、使用しないでください:

static public IEnumerable<T> Swap1<T>(this IList<T> source, int index1, int index2)
{
    // Parameter checking is skipped in this example.

    // Swap the items.
    T temp = source[index1];
    source[index1] = source[index2];
    source[index2] = temp;

    // Return the items in the new order.
    foreach (T item in source)
        yield return item;

    // Restore the collection.
    source[index2] = source[index1];
    source[index1] = temp;
}

このソリューションは、入力リストを元の状態に復元してもdoesを変更するため、汚れています。これにより、いくつかの問題が発生する可能性があります。

  1. リストは読み取り専用であり、例外をスローします。
  2. リストが複数のスレッドで共有されている場合、この関数の実行中に他のスレッドのリストが変更されます。
  3. 反復中に例外が発生した場合、リストは復元されません。 (これは、Swap関数内にtry-finallyを記述し、restore-codeをfinallyブロック内に配置することで解決できます)。

より良い(そしてより短い)解決策があります:元のリストのコピーを作成するだけです。 (これにより、IListの代わりにIEnumerableをパラメーターとして使用することも可能になります):

static public IEnumerable<T> Swap2<T>(this IList<T> source, int index1, int index2)
{
    // Parameter checking is skipped in this example.

    // If nothing needs to be swapped, just return the original collection.
    if (index1 == index2)
        return source;

    // Make a copy.
    List<T> copy = source.ToList();

    // Swap the items.
    T temp = copy[index1];
    copy[index1] = copy[index2];
    copy[index2] = temp;

    // Return the copy with the swapped items.
    return copy;
}

このソリューションの欠点の1つは、メモリ全体を消費するリスト全体をコピーするため、ソリューションがかなり遅くなることです。

次の解決策を検討してください。

static public IEnumerable<T> Swap3<T>(this IList<T> source, int index1, int index2)
{
    // Parameter checking is skipped in this example.
    // It is assumed that index1 < index2. Otherwise a check should be build in and both indexes should be swapped.

    using (IEnumerator<T> e = source.GetEnumerator())
    {
        // Iterate to the first index.
        for (int i = 0; i < index1; i++)
            yield return source[i];

        // Return the item at the second index.
        yield return source[index2];

        if (index1 != index2)
        {
            // Return the items between the first and second index.
            for (int i = index1 + 1; i < index2; i++)
                yield return source[i];

            // Return the item at the first index.
            yield return source[index1];
        }

        // Return the remaining items.
        for (int i = index2 + 1; i < source.Count; i++)
            yield return source[i];
    }
}

そして、パラメーターをIEnumerableに入力する場合:

static public IEnumerable<T> Swap4<T>(this IEnumerable<T> source, int index1, int index2)
{
    // Parameter checking is skipped in this example.
    // It is assumed that index1 < index2. Otherwise a check should be build in and both indexes should be swapped.

    using(IEnumerator<T> e = source.GetEnumerator())
    {
        // Iterate to the first index.
        for(int i = 0; i < index1; i++) 
        {
            if (!e.MoveNext())
                yield break;
            yield return e.Current;
        }

        if (index1 != index2)
        {
            // Remember the item at the first position.
            if (!e.MoveNext())
                yield break;
            T rememberedItem = e.Current;

            // Store the items between the first and second index in a temporary list. 
            List<T> subset = new List<T>(index2 - index1 - 1);
            for (int i = index1 + 1; i < index2; i++)
            {
                if (!e.MoveNext())
                    break;
                subset.Add(e.Current);
            }

            // Return the item at the second index.
            if (e.MoveNext())
                yield return e.Current;

            // Return the items in the subset.
            foreach (T item in subset)
                yield return item;

            // Return the first (remembered) item.
            yield return rememberedItem;
        }

        // Return the remaining items in the list.
        while (e.MoveNext())
            yield return e.Current;
    }
}

Swap4は、ソース(のサブセット)のコピーも作成します。最悪のシナリオでは、Swap2関数と同じくらい遅く、メモリを消費します。

10
Martin Mulder

リストにはReverseメソッドがあります。

your_list.Reverse(i, 2) // will swap elements with indexs i, i + 1. 

ソース: https://msdn.Microsoft.com/en-us/library/hf2ay11y(v = vs.110).aspx

7
user1920925

順序が重要な場合は、リスト内のシーケンスを示す「T」オブジェクトのプロパティを保持する必要があります。それらを交換するには、そのプロパティの値を交換し、それを.Sort(sequenceプロパティとの比較)で使用します

0
CaffGeek