-2

我对 C++ 和一般编码相当陌生，我正在使用 Visual Studio 2013 Desktop 编写一个简单的程序来收集 .CSV 文件中的数据并对其执行一些操作。该程序似乎编译和运行良好，并要求我输入要打开的文件的名称。就像我打算的那样，如果我输入一个无效的文件名，程序将显示一条错误消息并终止，但是如果我输入正确的名称，我会收到一条消息，上面写着

“TestIO.exe 中 0x0F16A9E8 (msvcr120d.dll) 处的未处理异常：0xC0000005：访问冲突读取位置 0xCCCCCCC0。”

带有中断或继续选项。如果我按继续，它会再次显示相同的消息，并无限继续，直到我按“中断”并停止调试。我完全不知道这里发生了什么，有人可以对此有所了解吗？将不胜感激。

编辑：这是我的主要（）。希望这有助于更多，抱歉之前没有包括它。

int main()
{
    int numDays = 0, streams;
    string* date;
    string line, filename;
    DailyData* days;
    cout << "Enter file name: ";
    getline(cin, filename);
    ifstream infile;
    infile.open(filename); 
    if (infile.fail())
    {
        cout << "Error opening input file" << endl;
        return 0;
    }
    while (getline(infile, line))
        numDays++;
    date = new string[numDays];
    for (int i = 0; i < numDays; i++)
        getline(infile, date[i]);
    days = new DailyData[numDays];
    for (int i = 0; i < numDays; i++)
    {
        getData(date[i], streams);
        days[i] = DailyData(date[i], streams);
    }
    cout << "Max Streams: " << maxStreams(days, numDays) << endl;
    cout << "Min Streams: " << minStreams(days, numDays) << endl;
    cout << "Avg Streams: " << average(days, numDays) << endl;
    cout << "Tot Streams: " << total(days, numDays) << endl;
    delete[] days;
    delete[] date;
    infile.close();
    return 0;
}

编辑2：这是你们要求的一些东西

void getData(string& d, int& s)
{
    int start = 0, end = 0, i = 0;
    string p[14];
    while (start != string::npos)
    {
        end = d.find(",", start); 
        p[i] = d.substr(start, end - start);
        start = end + 1;
        i++;
    }
    d = p[0];
    s = atoi(p[5].c_str());
}

这是 DailyData 及其构造函数

class DailyData
{
    public:
    DailyData() :date("NULL"), streams(0){}
    DailyData(string d, int s) :date(d), streams(s){}
    string getDate(){ return date; }
    int getStreams(){ return streams; }
    friend ostream& operator << (ostream&, DailyData&);
    private:
        string date;
        int streams;
};

编辑 3：我将代码更改为使用向量而不是数组。除了更改 int main() 之外，我还确保更改所有函数定义/声明中的参数。我仍然收到与我最初遇到的相同的未处理异常错误。这是新的代码片段：

vector<string> date;
vector<DailyData> days;

//...

while (getline(infile, line))
{
    date.push_back(line);
    getData(date.back(), streams);
    days.push_back(DailyData(date.back(), streams));
}
numDays = days.size();
4

1 回答 1

2

在您已经阅读到文件末尾之后，您的代码似乎正在对输入文件调用 getline

while (getline(infile, line))
    numDays++;

// above reads every line in the file,
// then you call this, even though while(getline) has returned false:

for (int i = 0; i < numDays; i++)
    getline(infile, date[i]);

如果您是编程新手，请始终使用花括号来显示循环体，这样您就可以看到循环块执行开始和结束的位置。

此外，在实践中，最好学习使用调试器。这将是你作为程序员做过的最有价值的事情。

编辑：

以下是如何使用可动态调整大小的向量而不是固定大小的数组：

std::vector<string> dates;

//...

while( getline(infile, line) )
{
    dates.push_back(line);
}

如果您需要更多信息，请搜索 C++ 向量。

编辑2：

现在 getData 的代码已经发布，我认为问题就在那里。首先，这段代码：

while (start != string::npos)
{
    end = d.find(",", start); 
    p[i] = d.substr(start, end - start);
    start = end + 1;
    i++;
}

看起来很危险。

p[i] = d.substr(start, end - start);

如果 end == string::npos 会导致奇怪的行为，只要 d.find 找不到逗号就会发生。

同样 start = end + 1如果 end == string::npos不会做你所期望的

进一步的 p[i] = ... 仅适用于 i 小于 13 的值。我没有看到任何检查来保证这一点。

我的第一个建议是检查 end == string::npos 是否，如果不是，则仅执行其他操作。

希望对您有所帮助，此代码的行为在很大程度上取决于您的输入数据的外观，这就是为什么最好使用调试器的原因。

于 2014-05-26T01:43:10.000 回答