2016-05-29 33 views
2

递归时,递归中的收益率返回值也不会返回值。当我调用代次排列时,第2个GeneratePermutations(list, startCount, permutationCount)确实返回了6次(yield return permutationList;)。但由于某种原因,第一个GetPermutations中的result不包含任何内容,即使我在结果上调用了.ToList()即使在“.ToList()”调用

这是否有什么做递归?

Test.cs

IEnumerable<int[]> actual = _sut.GetPermutations(3).ToList(); 

Perm.cs

public class Perm 
{ 
    public IEnumerable<int[]> GetPermutations(int permutationCount) 
    { 
     int[] permutationList = Enumerable.Range(1, permutationCount).ToArray(); 
     IEnumerable<int[]> result = GeneratePermutations(permutationList, 0, permutationCount - 1).ToList(); 

     // Doesn't contain any value! 
     return result; 
    } 

    // http://stackoverflow.com/a/756083/4035 
    private IEnumerable<int[]> GeneratePermutations(int[] permutationList, int startCount, int permutationCount) 
    { 
     if (startCount == permutationCount) 
     { 
      // Does return 6 times here. 
      yield return permutationList; 
     } 
     else 
     { 
      for (int i = startCount; i <= permutationCount; i++) 
      { 
       Swap(ref permutationList, startCount, i); 
       GeneratePermutations(permutationList, startCount + 1, permutationCount).ToList(); 
       Swap(ref permutationList, startCount, i); 
      } 

     } 
    } 

    // http://stackoverflow.com/a/2094316/4035 
    public static void Swap(ref int[] list, int index1, int index2) 
    { 
     int tmp = list[index1]; 
     list[index1] = list[index2]; 
     list[index2] = tmp; 
    } 
} 
+0

与您的问题无关:在GetPermutations()中获取您的数据类型quirkiness。你不需要'List ':扔掉铸件。 –

+0

@QualityCatalyst:我只是在源代码和代码中都改变了建议的类型。 – Sung

+1

请注意''Swap'方法中不需要'ref'。 –

回答

3

你的问题是,你永远不回你的递归调用的结果和你有相同的阵列工作的所有时间。这将是你的代码最快的修复(不是最好的,但我敢肯定你会从这里看着办吧):BTW

IEnumerable<int[]> perms = GeneratePermutations(
    new List<int>(permutationList).ToArray(), 
    startCount + 1, permutationCount); 

foreach(int[] perm in perms) 
    yield return perm; 

:可以使整个类的静态和通用

Proof on dotNetFiddle

+0

是的,的确,问题在于我正在研究相同的'permutationList'并每次修复问题时创建一个副本。谢谢 – Sung

相关问题