-1

可能重复：
如何为给定字符串键入集合

我正在尝试键入字符串的排列，例如字符串“123”应该给我 123 132 213 231 321 312 我需要帮助来修复我的代码

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace TestAAD
{
class Program
{
    static int len = 0;
    static int lenPerm = 0;
    private static void Permutations(string str, string perm , int i)
    {
        if (i == len) Console.WriteLine(perm);
        for (int k = 0; k < len; k++)
        {
             lenPerm = perm.Length;
            for (int j = 0; j < lenPerm; j++)
            {
                if ((perm[j] == str[(len - 1) - k]) && (len-1-k>0))
                    k++;
            }
            if((len-1-k) >=0)
            Permutations(str, perm + str[(len - 1) - k], i++);
        }
    }
    static void Main(string[] args)
    {
        string st = "123";
        len = st.Length;
        Permutations(st, "",0);
    }
}
}

如果有人可以帮助我，请谢谢大家

4

2 回答 2

1

PermutationsPermutations在循环len时间再次调用。这第二次Permutations调用Permutations在循环len时间内再次调用。这无休止地一次又一次地发生，或者至少直到你得到堆栈溢出为止。

当您使用递归调用时，您必须始终确保递归在某处停止。

If (job not done) {
    make recursive call
}

递归调用必须朝着完成工作迈出一步，否则它永远不会完成。

UPDATE 1（仅解决异常问题）

该方法难以阅读。len-1-k不要多次重复表达式，而是反转循环！for (int k = str.Length - 1; k >= 0; k--)并在内部循环中k减少。k--我也摆脱了len多余的变量。

在您的实施len-1-k中总是>= 0. 因此递归调用永远不会结束。因此我改变了减少的条件k。k即使已经是，它也会减少0。为了不让str[k]条件中的索引越界错误，k >= 0必须首先检查。

private static void Permutations(string str, string perm, int i)
{
    if (i == str.Length)
        Console.WriteLine(perm);
    for (int k = str.Length - 1; k >= 0; k--) {
        int lenPerm = perm.Length;
        for (int j = 0; j < lenPerm; j++) {
            if (k >= 0 && perm[j] == str[k])
                k--;
        }
        if (k >= 0)
            Permutations(str, perm + str[k], i++);
    }
}

public static void Start()
{
    string st = "123";
    Permutations(st, "", 0);
}

这不再产生无限递归，但结果仍然不正确。我让你弄清楚如何进一步改进代码。

更新 2（创建正确的排列）

最后这是我的工作实现

public static class PermutationBuilder
{
    private static char[] _characters;
    private static List<string> _list;

    public static IEnumerable<string> GetPermutations(string characters)
    {
        _characters = characters.ToCharArray();
        _list = new List<string>();
        AddPermutations("", 0);
        return _list;
    }

    private static void AddPermutations(string permutation, int level)
    {
        if (level >= _characters.Length) {
            _list.Add(permutation);
        } else {
            for (int i = 0; i < _characters.Length; i++) {
                char ch = _characters[i];
                if (ch != ' ') {
                    _characters[i] = ' ';
                    AddPermutations(permutation + ch, level + 1);
                    _characters[i] = ch;
                }
            }
        }
    }
}

请注意，我暂时用空格字符标记了已使用的字符。

你这样称呼它

foreach (string permutation in PermutationBuilder.GetPermutations("123")) {
    Console.WriteLine(permutation);
}
于 2012-12-08T15:59:47.053 回答
1

排列很容易做到。

public string[] FindPermutations(string word)
{
    if (word.Length == 2)
    {
        char[] c = word.ToCharArray();
        string s = new string(new[] { c[1], c[0] });
        return new[]
            {
                word,
                s
            };
    }

    List<string> result = new List<string>();

    string[] subsetPermutations = FindPermutations(word.Substring(1));
    char firstChar = word[0];
    foreach (string temp in subsetPermutations
                    .Select(s => firstChar.ToString(CultureInfo.InvariantCulture) + s))
    {
        result.Add(temp);
        char[] chars = temp.ToCharArray();
        for (int i = 0; i < temp.Length - 1; i++)
        {
            char t = chars[i];
            chars[i] = chars[i + 1];
            chars[i + 1] = t;
            string s2 = new string(chars);
            result.Add(s2);
        }
    }
    return result.ToArray();
}
于 2012-12-08T16:02:29.847 回答