Question

该程序从文本文件“ penguins.txt”中读取行，并将数据复制到“ FeedingOutput.dat”中。在其他PC上运行正常，但是在笔记本电脑上运行时出现以下错误：

 Using Uninitialized Memory 'zFeeding'
 Using Uninitialized Memory 'zPercent'
 Using Uninitialized Memory 'dFeeding'
 Using Uninitialized Memory 'dPercent'
 Using Uninitialized Memory 'wFeeding'
 Using Uninitialized Memory 'wPercent'

文本文件“ penguins.txt”如下所示：

Zany A 5 4
Den B 4 8
Windi C 5 2.1

.txt和.dat文件都与.cpp文件位于同一目录中。

这是我的代码：


#include <iostream>
#include <fstream>
#include <string>
#include <cmath>
#include <iomanip>

using namespace std;


int main()
{

    double zFeeding; //Declaring variables
    double dFeeding;
    double wFeeding;
    double zPercent;
    double dPercent;
    double wPercent;
    double zFeedingNew;
    double dFeedingNew;
    double wFeedingNew; 

    char filename[50];
    string zName, dName, wName, zID, dID, wID;


    ifstream penguinInfo; //Allows input and output for the two different files
    ofstream dataOutput;


    cout << "Enter filename containing penguins information" << endl; //Asking for user to input file name, then opening that file
    cin.getline(filename, 50);
    penguinInfo.open(filename);
    dataOutput.open("FeedingOutput.dat");


    dataOutput << showpoint << fixed << setprecision(2); ////formating the output

    //this will set the information from penguins.txt to actual variables.
    penguinInfo >> zName, zID, zFeeding, zPercent, dName, dID, dFeeding, dPercent, wName, wID, wFeeding, wPercent;


    zFeedingNew = zFeeding + (zFeeding * (zPercent / 100)); //equations for new feeding amounts
    dFeedingNew = dFeeding + (dFeeding * (dPercent / 100));
    wFeedingNew = wFeeding + (wFeeding * (wPercent / 100));


    dataOutput << zName << " " << zID << " " << zFeedingNew << " lbs." << endl; //Outputs data to FeedingOutput.dat for Zany


    dataOutput << dName << " " << dID << " " << dFeedingNew << " lbs." << endl; //Outputs data to FeedingOutput.dat for Den


    dataOutput << wName << " " << wID << " " << wFeedingNew << " lbs." << endl; //Outputs data to FeedingOutput.dat for Windi


    penguinInfo.close(); //close files and requires approval to close the program
    dataOutput.close();
    system("pause");
    return 0;


}

我相信这可能是范围问题，但是我对c ++还是很陌生，所以我不确定什么地方出错了。

Answer 1

给出

penguinInfo >> zName, zID, zFeeding, zPercent, dName, dID, dFeeding, dPercent, wName, wID, wFeeding, wPercent;

根据operator precedence，operator >>的优先级高于operator,，并且与

相同

(penguinInfo >> zName), zID, zFeeding, zPercent, dName, dID, dFeeding, dPercent, wName, wID, wFeeding, wPercent;

即只有zName被设置为penguinInfo >> zName。

您可以将其更改为

penguinInfo >> zName >> zID >> zFeeding >> zPercent >> dName >> dID >> dFeeding >> dPercent >> wName >> wID >> wFeeding >> wPercent;

Answer 2

问题是comma operator并没有按照您的想象做。它只是丢弃左侧和右侧，然后继续右侧：

在逗号表达式E1，E2中，对表达式E1进行求值，将其结果丢弃（尽管如果具有类类型，则直到包含完整表达式的末尾才会销毁它），并且其副作用是在表达式E2的求值开始之前完成（请注意，用户定义的运算符，不能保证排序）（直到C ++ 17）。

最重要的是，它不会用任何东西填充逗号表达式中使用的所有变量。因此，您的变量保持未初始化状态（因为您没有在上面进行初始化）。

您实际上想要做的是像>>运算符那样链接<<运算符。看起来像这样：

penguinInfo >> zName >> zID >> zFeeding >> zPercent >> dName >> dID >> dFeeding >> dPercent >> wName >> wID >> wFeeding >> wPercent;

C ++“使用未初始化的内存..（变量名）”

2 个答案: