C ++中的无限循环为什么？

Question

我很抱歉发布超长代码，但是当我运行这段代码时，我看到的只是—— 堆大小：1638652 获取int： 获取int： 获取int： 获取int： 获取int： 堆大小：1638653 它不断循环，堆大小加一。

#include <iostream>
#include <fstream>
#include <algorithm>
#include <vector>
#include <exception>

#ifndef WX_REPORT_H
#define WX_REPORT_H


#include <string>
#include <sstream>
using std::string;
using std::stringstream;

typedef struct WX_REPORT

{
    string unitType;
    string stationName;
    string time;
    string gpsLoc;
    int pressure;
    int windSpeed;
    int temperature;
    int humidity;
    int windDirection;

    string toString()
    {
        stringstream str;
        str << stationName << ": " << time << "\t" << gpsLoc << "\n";
        str << pressure << "\n" << windSpeed << "\n" << temperature << "\n";
        str << humidity << "\n" << windDirection;
        return str.str();
    }
}
WXReport;

#endif
/*
 * Reports must be in the following format:
 * M or I // Metric or imperial units
 */
using namespace std;

vector<WXReport*> heap;

bool compTime(const WXReport* a, const WXReport* b) {
    if(a->time < b->time) { // timing
        return false;
    } else {
        return true; // commands to return true
    }
}

void heapAdd(WXReport* wx) {
    heap.push_back(wx);
    push_heap(heap.begin(), heap.end());
}

WXReport* heapPop() { // header popup
    pop_heap(heap.begin(), heap.end());
    WXReport* rep = heap.back();
    heap.pop_back();
    return rep;
}

void getInt(istream &input, int &i) {
    string temp;
    input>>temp;
    cout<<"Getting int: "<<temp<<endl;
    i = atoi(temp.c_str());
}

void readInFile(string filename) {
    ifstream input(filename);
    WXReport *report;
    while(!input.eof()) {
        report = new WXReport();
        getline(input, report->unitType);
        getline(input, report->stationName);
        getline(input, report->time);
        getline(input, report->gpsLoc);
        getInt(input, report->pressure);
        getInt(input, report->windSpeed);
        getInt(input, report->temperature);
        getInt(input, report->humidity);
        getInt(input, report->windDirection);
        heapAdd(report);
        cout<<"Heap size: "<<heap.size()<<endl;
    }
}

int menu() {
    cout<<"\n\nPlease select one: "<<endl;
    cout<<"1) Read in another file"<<endl;
    cout<<"2) Display the fastest wind speed"<<endl;
    cout<<"3) Display weather stations by name"<<endl;
    cout<<"4) Display all weather reports"<<endl;
    cout<<"5) Remove a weather report"<<endl;
    cout<<"6) Write weather reports to file"<<endl;
    cout<<"0) Exit"<<endl;
    int choice;
    cin>>choice;
    return choice;
}

void printAllReports() {
    cout<<"Printing all reports"<<endl;
    for(WXReport* rep: heap) {
        cout<<rep->toString()<<endl;
    }
    cout<<"Done printing reports"<<endl;
}

int main(int argc, char* argv[]) {
    string filename = "report.txt";
    readInFile(filename);

    int choice = menu();
    while(choice != 0) {
        switch(choice) {
            case 1:
                cout<<"What file would you like to read in?"<<endl;
                cin>>filename;
                readInFile(filename);
                break;
            case 2:
                cout<<"Has not been implemented"<<endl;
                break;
            case 3:
                cout<<"Has not been implemented"<<endl;
                break;
            case 4:
                printAllReports();
                break;
            case 5:
                cout<<"Has not been implemented"<<endl;
                break;
            case 6:
                cout<<"Has not been implemented"<<endl;
                break;
            default:
                cout<<"Invalid choice, please try again."<<endl;
        }
        choice = menu();
    }
    cout<<"Thank you!"<<endl;

    return 0;
}

Answer 1

重要的部分。如果您没有阅读其他内容，请阅读以下内容：始终检查错误代码和返回值。

在ifstream input(filename); 之后，您不知道文件是否已打开。使用input.is_open() 进行测试可以解决这个问题。

如果文件未打开，所有对getline 的调用都会失败，eof() 也会失败。文件未打开，无法读取文件结尾，也无法退出循环。即使文件打开了，如果你不检查getline的输出，你怎么知道你读到了一行？

流的一个有趣部分是，如果您测试流，它会告诉您它是否处于错误状态，因此您可以编写看起来像

的代码

if (getline(...) && getline(...) && ...)

因此，您不必制作大量 if-else-if 或大量嵌套 if。第一次读错了，你就出局了。

if eof() 的问题包含在问题的 cmets 中。基本是在开始阅读之前您不知道文件是否已结束。另外，如果你在一堆读取的中间碰到文件末尾会发生什么？

所以读一行。如果它很好，请阅读下一行等...直到完成。

getInt 不是必需的。

int val;
input >> val;

如果流可以解析为int，则将整数加载到val。如果不能，则输入标记为错误，您可以检查原因。可能无法解析。可能是文件结尾。

int val;
if (input >> val)
{
    //do stuff
} 
else
{
    //no int here. Do other stuff
}

就像上面一样，你可以链接指令并获取

if (input >> val >> anotherval >> ...)