如何在 C 语言中返回一个指针

Question

我有一个返回指针的函数，我正在让它计算给定数字的阶乘。

这是我无法更改的骨架代码

#include<stdio.h>
#include<string.h>
//Read only region start

char* factorial (int input1)
{

    //Read only region end
    //Write code here


}

我写的代码是：

我也试试这个代码

#include<stdio.h>
#include<string.h>
//Read only region start

char* factorial (int input1)
{

    //Read only region end
    //Write code here
    char fact=1;
    while(input1!=0)
    {
        fact = fact * input1;
        input1--;
    }
    return fact;

}

左边是问题，右边是我正在写的代码。 它抛出的错误是“Segmentation fault”

该程序不需要main() 函数，因为mettl 在后端使用它来调用函数并评估结果。

Answer 1

我修改了你的代码，下面是它的正确版本：

unsigned int factorial (int input1)
{
  unsigned int res = 1;

  while(input1 > 1) {
    res = res * input1;
    input1--;
  }
  return res;
}

您的错误是返回指针 (char *) 而不是简单的字符（单字节值）。

我还将char 更改为unsigned int。为什么？

1. unsigned：因为您似乎希望您的结果是肯定的，否则您的 input1-- 循环将无法工作。因此，无符号允许您在达到数据类型限制之前获得更大的结果集（有符号字符的最大值为 127，无符号字符的最大值为 255）。

2. int：因为在幕后（在汇编代码中）传递到堆栈和 CPU 寄存器的数据大小是一个 int。 ASM 指令告诉只使用寄存器的一个字节（在 4 或 8 个字节中，取决于 x32 或 x64）。但是使用 byte 而不是 int 并没有任何性能提升。那么，如果它可以覆盖更大的结果集，为什么不使用 int 呢？ :-)

3. 我还在 > 1 条件下停止了循环，因为再运行一次迭代以进行 x1 乘法是没有用的。这是一个真正的坏蛋优化... 笑话 ;-))

---

更新：

根据您的 cmets，您无法更改函数定义，因此此代码将通过将结果作为字符串返回 (char *) 来工作：

char* factorial (int input1)
{
  unsigned int res = 1;
  char *str;

  while(input1 > 1) {
    res = res * input1;
    input1--;
  }
  str = malloc(32);
  sprintf(str, "%u", res);
  return str;
}

请注意，它分配了一个缓冲区来存储结果（即使在 64 位上，32 也应该没问题......）。您可能希望在使用后free() 缓冲区以避免内存泄漏。