请解释为什么这个 C 代码给我一个分段错误？

Question

我是一名尝试学习 C 的初学者。以下代码在我运行时出现分段错误。谁能解释原因，或者告诉我我的错误？

struct frac sum(struct frac f1, struct frac f2);
struct frac {
int *numer;
int *denom;
};
struct frac sum(struct frac f1, struct frac f2) {
struct frac rv;
*rv.numer = (*f1.numer)*(*f2.denom) + (*f2.numer)*(*f1.denom);
*rv.denom = (*f1.denom)*(*f2.denom);
return rv;
}
int main() {
int n = 5;
int d = 10;
struct frac myFrac1 = {&n, &d};
struct frac myFrac2 = {&n, &d};
struct frac myFrac3 = sum(myFrac1, myFrac2);
return 0;
}

Answer 1

程序会在第二行崩溃

struct frac rv;
*rv.numer = (*f1.numer)*(*f2.denom) + (*f2.numer)*(*f1.denom);

因为，您在堆栈中定义了 rv。 rv.number 可以是任何东西。

在大多数情况下，*(rv.numer) 是你不能写的地方。

Answer 2

"...当我运行它时，代码给了我一个分段错误。谁能解释原因，或者告诉我我的错误"

首先，如果在调试模式下编译，可能会出现一些问题，如果您能够看到它们并通过解决每个问题做出响应，可能会阻止您遇到段错误：

当我编译时出现正常警告：

不初始化变量是程序员（新老）最常被忽略但代价高昂的错误之一。始终初始化。

为什么会出现段错误 - 在您的原始代码中，我在 运行 时看到了这个：

尝试取消引用未初始化的指针几乎总是会导致分段错误。

只是一些可以解决上述错误的建议：
为了在整个示例改编中保持可读性，您的代码使用typedef 创建frac_s。以原始帖子的方式使用指针需要在地方进行动态内存分配，非指针也可以很好地服务，即struct 成员。所以，注意使用非指针成员。下面的指针使用仅限于函数参数和sum(,,) 的返回类型，从而大大简化了代码，但需要动态内存分配和free()。将以下内容与您的代码进行比较以查看更改的程度...

typedef struct  {
    int numer;
    int denom;
}frac_s;


frac_s * sum(frac_s *f1, frac_s *f2);

frac_s * sum(frac_s *f1, frac_s *f2) {
    frac_s * rv = calloc(1, sizeof(*rv)) ;
    if(rv)
    {
        rv->numer = (f1->numer)*(f2->denom) + (f2->numer)*(f1->denom);
        rv->denom = (f1->denom)*(f2->denom);
    }
    return rv;
}

int main(void) {
    int n = 5;
    int d = 10;

    frac_s myFrac1 = {n, d};
    frac_s myFrac2 = {n, d};
    frac_s *myFrac3 = sum(&myFrac1, &myFrac2);
    if(myFrac3)
    {
        //use myFracs;
        free(myFracs);
    }

    return 0;
}

Answer 3

试试这个

struct frac sum(struct frac f1, struct frac f2);

struct frac {
    int *numer;
    int *denom;
};

struct frac sum(struct frac f1, struct frac f2) {
    struct frac rv;
    rv.numer = (int*)malloc(sizeof(int));
    rv.denom = (int*)malloc(sizeof(int));
    *rv.numer = (*f1.numer)*(*f2.denom) + (*f2.numer)*(*f1.denom);
    *rv.denom = (*f1.denom)*(*f2.denom);
    return rv;
}

int main() {
    int n = 5;
    int d = 10;

    struct frac myFrac1 = {&n, &d};
    struct frac myFrac2 = {&n, &d};
    struct frac myFrac3 = sum(myFrac1, myFrac2);

    return 0;
}

结果是rv.numer和rv.denom最后都指向100。