对列表中的每个其他项目执行对列表中每个项目的操作,并使用排除项

Gar*_*ngs 5 c# list

我有一个没有重复的数据列表.对于这个例子,我会说我的清单是

List<string> list1 = new List<string>() { "A", "B", "C", "D" };
Run Code Online (Sandbox Code Playgroud)

我想对列表中的每个项目对列表中的每个其他项目执行我的操作,除非我已经对它们执行了操作(AB和BA)或者它们是相同的(AA).

例如.

A against B
A against C
A against D
B against C
B against D
C against D
Run Code Online (Sandbox Code Playgroud)

现在这很简单,但我的列表非常大,这个过程可能非常耗时.另外,对于我拥有的数据,我不需要对匹配数据运行操作,或者操作是否已经运行

例如.

A against A - Skip
A against B - Good
A against C - Good
A against D - Good
B against A - Skip (we already did A against B)
B against B - Skip
B against C - Good
B against D - Good
C against A - Skip
Run Code Online (Sandbox Code Playgroud)

等等.

我一直在寻找(我甚至不知道它是否存在)是一个简单的方法,我可以用来做这个,而不是启动两个循环并执行我的操作并保存结果以便以后进行比较.

循环通过列表O(n*n)但是因为我不需要比较超过一半的结果这是浪费时间,因为我知道我只需要检查O(n*(n/2))

我目前使用的代码如下

List<string> list1 = new List<string>() { "A", "B", "C", "D" };
List<string> list2 = new List<string>(list1);
List<string> listResult = new List<string>();

list2.Reverse();

int i = 0;
foreach (var a in list1)
{
    for (int j = 0; j < (list2.Count / 2); j++)
    {
        i++;
        Console.WriteLine("Looped {0} times", i);

        // Don't run against ourself
        if (a == list2[j])
            continue;

        if (listResult.Count(x => (x == a + list2[j]) || (x == list2[j] + a)) == 0)
        {
            listResult.Add(a + list2[j]);

            // Perform some operation here
            // operation(a, list2[j]);
        }
    }
}
Run Code Online (Sandbox Code Playgroud)

上面的代码工作正常(我需要调整list2.Count/2部分来计算奇数编号列表).

有一个更好的方法吗?我错过了一个LINQ扩展方法?我的问题是我真的不知道谷歌会做些什么.

我想知道是否有一个方法可以返回包含我想要的项目的列表,然后我将循环并执行我的操作.也许用的东西.SelectMany()

Chr*_*ris 10

对于列表中的每个条目,将其与列表中的所有条目进行匹配,因为之前的项目已经匹配.

List<string> list1 = new List<string>() { "A", "B", "C", "D" };

for( int i = 0; i < list1.Count - 1; i++ )
    for( int j = i + 1; j < list1.Count; j++ )
        Console.WriteLine( "{0} against {1}", list1[i], list1[j] );
Run Code Online (Sandbox Code Playgroud)

编辑:至于你的第二个问题,这样的事情怎么样:

public static class Extensions
{
    public static IEnumerable<U> Combinations<T, U>( this IEnumerable<T> list,
                                                     Func<T, T, U> combinator )
    {
        var temp = list.ToArray();
        for( int i = 0; i < temp.Length - 1; i++ )
            for( int j = i + 1; j < temp.Length; j++ )
                yield return combinator( temp[i], temp[j] );
    }
}
Run Code Online (Sandbox Code Playgroud)

然后可以像这样使用:

List<string> list1 = new List<string>() { "A", "B", "C", "D" };
var res = list1.Combinations( ( a, b ) => string.Format( "{0} against {1}", a, b ) );
Run Code Online (Sandbox Code Playgroud)

如果你可以支持IList而不是支持IEnumerable,你可以ToArray完全跳过这个电话.

  • 我喜欢这种方法,因为它是最干净,最快的.***AND***不使用额外的内存.这意味着它不会像其他人那样用*极大的`List`命中`Stack Overflow`异常. (3认同)

All*_*ice 5

只需针对列表中的所有元素运行每个元素.

for (int i = 0; i < list.Count; i++) {
    for (int j = i + 1; j < list.Count; j++) {
        //Run list[i] against list[j]
    }
}
Run Code Online (Sandbox Code Playgroud)

这样可以确保不会对自身或任何已经运行过的元素运行任何元素.