如果出现错误，我该如何返回 - C

Question

基本上我有这个数据结构：

typedef struct Data {
    char *str;
    int n;
} TData, *AData;

我想使用 malloc 为 str 分配空间（在该变量中，我放置了一个从文件中读取的字符串，老实说，考虑到我的问题的主题，这并不重要）。因此，我创建了一个函数TData InitData(const char *file_name)：

TData InitData(const char *file_name) {
    TData data;

    data.str = malloc(1024);
    if (!data.str) {
        printf("Allocation error!\n");
        // I have no idea what to return here in order to exit the function.
        return (...);
    }

    return data;
}

现在，在我的“main()”函数中，我会这样调用：

int main(int argc, const char * argv[]) {
    TData data;
    if(!(data = ReadExpression(argv[1]))) {
        return 1;
    }
    // I do something with my data and the program ends.
    return 0;
}

如何使InitData函数中的return语句有效？

我知道另一种方式，可能推荐的方式是将TData data从main发送到InitData作为AData类型的参数并将函数的类型更改为int（会导致类似：int InitData(const char *file_name, AData data)) 或简单地使用AData。

Answer 1

• 根据经验，始终将结构作为指针参数而不是按值传递。
• 根据经验，编写复杂函数时，请保留返回值以防出错。

所以在你的情况下，将函数重写为：

bool InitData (TData* data, const char *file_name);

bool 可以用枚举替换以获得更详细的错误代码。现在函数结果只有“ok”或“error”。

Answer 2

C 没有内置的多值返回，所以通常你要做的是

int InitData(const char *file_name, TData *returnedData) { ... }

InitData 的返回值是错误/成功标志，TData 结构通过指针返回。

Answer 3

我在C做的不多，但是很多功能都是这样的：

int foo(intargs, outargs);

这背后的事情是您分配对象并将其作为引用传递给函数内部。

函数的返回值将是enum 或类似的值，用于比较以确定发生了什么。

Answer 4

我喜欢 MicroVirus 建议的方式。接受数据作为参数也可用于多线程代码安全。此外，您可以返回操作结果以检查调用代码。您的 TData 不是指针，因此您不能将其设置为 NULL，我认为这是一种告诉无效案例的方法。但是，这里还有另一个问题！ InitData 中的 TData 数据是一个局部变量，应在函数返回时复制。我认为这不是性能问题的好习惯。我建议，接受对象作为参数（例如作为指针）将函数的结果返回给调用代码并检查数据的有效性。您可以将数据设置为 NULL 或不设置，但是当您返回结果值时，这会使一切都变得清晰，而无需检查数据对象本身。

Answer 5

正常返回data，但将main()中的代码更改为：

data = ReadExpression(argv[1]);
if ( !data.str ) {
    return 1;
}

您不能返回标量值来代替struct，并且您的问题表明您已经知道可以使用指针但选择不使用，因此这是最简单的替代方法。

Answer 6

根据if (!data.str) 结果，您可以在数据结构集中设置一个标志，并始终返回data。 然后在你想知道发生了什么的时候检查标志。

只是为了澄清： 你可以写这样的东西，或者类似的东西：

`typedef struct Data 
{
    char *str;
    int n;
    char myflag;
    const char *file_name;
} TData, *AData;`

`void InitData(TData *data) 
{
    data.str = malloc(1024);
    if (!data.str) 
    {
        printf("Allocation error!\n");
        myflag=0;
    }
    else
    {
        myflag=1;
    }
    return data;
}`

但也见 cmets。只是想提供另一种观点。