我的堆算法代码有什么问题？

Question

我的作业要求我编写一个程序，该程序从终端（argc 和 argv）获取一个字符串并打印所有可能的排列。 我曾尝试使用堆算法，但似乎并没有奏效。 下面是我的功能。

char **getPermutation(char * in)
{
//n is the size of the input string.
 int n = strlen(in);
 int count[n];
 int counter= 0;
 char copy[n];
 char **permutations = malloc(sizeof(char*)*(factorial(n)));
 permutations[0] = in;
 strcpy(in, copy);
 counter++;
 for( int i = 1; i < n;)
 {

  if (count[i] < i){
   if (i%2==0){
    swap(&in[0],&in[i]);
   }
   else
   {
    swap(&in[count[i]],&in[i]);
   }
    permutations[counter] = in;
    strcpy(in, copy);
    counter++;
    count[i]++;
    i = 1;
  }
  else
  {
   count[i] = 0;
   i++;
   }
  }
 return permutations;
 }

该函数必须返回指向指令指定的字符指针的指针。 这也是为什么有这么多变量的原因（尽管我不确定如何处理字符串的副本。我很确定我需要它）。 测试表明程序会循环，经常循环太多并最终遇到段错误。 似乎交换的字符串并没有使其成为最重要的返回数组。

Answer 1

下面是清理内存分配的代码返工，它解决了上述注释中提到的一些问题。 此外，您的算法中有一个错误，这条语句strcpy(in, copy); 使您无法获得所有排列（导致重复。）此代码有效但尚未完成，它可以使用更多错误检查和其他收尾工作：

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

unsigned int factorial(unsigned int n)
{
    /* ... */
}

void swap(char *a, char *b)
{
    /* ... */
}

char **getPermutations(const char *input)
{
    char *string = strdup(input);

    size_t length = strlen(string);

    char **permutations = calloc(factorial(length), sizeof(char *));

    int *counts = calloc(length, sizeof(int)); // point to array of ints all initialized to 0

    int counter = 0;

    permutations[counter++] = strdup(string);

    for (size_t i = 1; i < length;)
    {
        if (counts[i] < i)
        {
            if (i % 2 == 0)
            {
                swap(&string[0], &string[i]);
            }
            else
            {
                swap(&string[counts[i]], &string[i]);
            }
            permutations[counter++] = strdup(string);
            counts[i]++;
            i = 1;
        }
        else
        {
            counts[i++] = 0;
        }
    }

    free(counts);
    free(string);

    return permutations;
 }

int main(int argc, char *argv[])
{

    char *string = argv[1];

    char **permutations = getPermutations(string);

    unsigned int total = factorial(strlen(string));

    for (unsigned int i = 0; i < total; i++)
    {
        printf("%s\n", permutations[i]);
    }

    free(permutations);

    return 0;
}

输出

> ./a.out abc
abc
bac
cab
acb
bca
cba
>