抱怨 FILE * 未初始化

Question

void openUpNow(FILE *x, FILE *y)
{
    x = fopen("xwhatever", "r");
    y = fopen("ywhatever", "r");
}

int _tmain(int argc,  _TCHAR* argv[ ])
{   
    FILE *x, *y;

    openUpNow(x, y);
}

警告 C4700：使用了未初始化的局部变量“x”

警告 C4700：使用了未初始化的局部变量“y”

补救措施？

Answer 1

无论如何，我认为这不是你想要做的。

假设您希望 openUpNow() 将文件打开到 x 和 y 中，您应该使用：

void openUpNow(FILE **x, FILE **y)
{
    *x = fopen("xwhatever", "r");
    *y = fopen("ywhatever", "r");
}

int _tmain(int argc,  _TCHAR* argv[ ])
{   
    FILE *x, *y;

    openUpNow(&x, &y);

    //  do stuff


}

换句话说，您需要传递将指针 x 和 y 放入函数中。

由于您的代码现在是这样，对 openUpNow() 的调用不会执行任何操作（并泄漏文件句柄），因为指针是按值传递的。

Answer 2

FILE *x = NULL;
FILE *y = NULL;

使用未初始化的变量通常会导致未定义的行为，因此编译器会发出警告，以便您可以了解此类变量。

此外，您需要通过引用而不是通过值传递指针，或者您在函数内部得到的是原始指针的副本。

void openUpNow(FILE **x, FILE **y)
               ^^^^^^^^^^^^^^^^^^ 

更正版本：

void openUpNow(FILE **x, FILE **y)
{
    *x = fopen("xwhatever", "r");
    *y = fopen("ywhatever", "r");
}

int _tmain(int argc,  _TCHAR* argv[ ])
{   
    FILE *x = NULL;
    FILE *y = NULL;

    openUpNow(&x, &y);

}

Answer 3

是的，该代码已损坏。在_tmain中，x和y未初始化并且具有垃圾值。然后将这些值传递给 openUpNow。幸运的是，openUpNow 忽略了它们。但随后您会丢弃 fopen 调用中的新值。你想要：

void openUpNow(FILE **x, FILE **y)
{
    *x = fopen("xwhatever", "r");
    *y = fopen("ywhatever", "r");
}

int _tmain(int argc,  _TCHAR* argv[ ])
{   
    FILE *x, *y;

    openUpNow(&x, &y);
}