变量未初始化？

Question

对于家庭作业问题，我必须定义一个可以在程序中用于表示分数的结构，以及一个可用于设置分数值的 set 函数，以及一个将打印的 print 函数分数很好。我们得到了 main 函数，根据问题输出应该是

(2/3) (1/5) (3/5)

这是我写的：

#include "library.h"
struct fraction
{
    int numerator;
    int denominator;
};

void set (fraction st, int n, int d)
{
    if (d>0)
    {
        n = st.numerator;
        d = st.denominator;
    }
}

void print(fraction st)
{
    int x = st.numerator;
    int y = st.denominator;
    print("(x/y)");
    print(" ");
}

void main()
{
    fraction a, b, c;
    set(a, 2, 3);
    set(b, 1, 5);
    set(c, 3, 5);
    print(a);
    print(b);
    print(c);
}

如果您想知道“library.h”是我的大学使用的大多数标准包含的快捷方式。

我不断收到变量“a”在未初始化的情况下被使用的错误。任何帮助将不胜感激。

Answer 1

如果set 函数预计会定义一个fraction，那么您应该为st.numerator 和st.denominator 成员变量分配n 和d 的值，如下所示：

void set (fraction st, int n, int d)
{
    if (d>0)
    {
        st.numerator = n;
        st.denominator = d;
    }
}

您还应该通过引用将fraction 变量传递给set 函数，即：void set(fraction& st, ...) 或指针void set(fraction* st, ...)，以便返回任何结果。

Answer 2

您将 a、b、c 按值传递给set()。当然，它们在 main()（和 print()）中是未初始化的。试试这个：

void set (fraction &st, int n, int d)
{
    if (d>0)
    {
        //n = st.numerator;
        //d = st.denominator;
        // I suppose this part should be:
        st.numerator = n;
        st.denominator = d;
    }

    /* Edit, thanks to @Tyler S comments:
       Not sure what author needs, but something like this
       should be here to really avoid uninitialized values. 

       Other options:
       Use unsigned int if you use only positive integers ( d>0 ).
       Use exceptions to handle negative inputs, zero denominator..
    */
    else
    {
        st.numerator = 1;
        st.denominator = 1;
    }
 }

在 main() 中：

set(a, 2, 3);
set(b, 1, 5);
set(c, 3, 5);

---

我还将print(..) 更改为void print(const fraction &st)。没有必要按值传递。想象一下更大的数据结构 - 复制只是为了打印它是浪费时间。

我建议检查pass by value/reference 和const correctness。