Question

所以我必须从文本文件中的信息创建5个作业对象，看起来像这样

A 0 3

B 2 6

C 4 4

D 6 5

E 8 2

左列是它的名字，下一个是到达时间，最后一个是持续时间这就是我现在所拥有的

#include <iostream>
#include <fstream>
#include "Job.hpp"
using namespace std;

int main(int argc, const char * argv[]) {

  string fileinfo;
  string programname;


//gets Jobs.txt from commandline
programname.append(argv[1]);



Job jobs[5];
fstream afile;


//reads in from the file to create job objects
afile.open(programname);

if(afile.fail()){
    cout <<"file could not be opened " << programname <<endl;
}
while(!afile.eof())
{


    getline(afile,fileinfo);
    int i = 0;

        //cout <<fileinfo[0] <<endl;//gets next letter
        //cout <<fileinfo[2] <<endl;//gets next arrival time
        //cout << fileinfo[4] <<endl;//gets next duration time
    jobs[i].setletter(fileinfo[0]);
    jobs[i].setarrivaltime(fileinfo[2]);
    jobs[i].setduration(fileinfo[4]);
    i++;


}

afile.close();

cout << jobs[1].getletter() <<endl;
cout << jobs[2].getletter() <<endl;
cout << jobs[3].getduration() <<endl;

现在当我进入并打印出我的不同对象（如代码底部）中的值后，我关闭文件后，它们都包含来自文件第一行的信息。我不知道为什么因为技术上我在设置作业的所有值后增加'i'然后fileinfo获取文件的下一行，所以这对我来说似乎应该有效。但是，就像我从底部的那3个小丑得到的值一样，结果是

A

0

Job类

Job::Job(){}

Job::Job(char let, int arrive, int dura){
  letter = let;
  arrivaltime = arrive;
  duration = dura;
}

它还定义了所有的get和sets 所以你们可以帮助我能够正确读入文件并创建我的对象数组

Answer 1

int i = 0;

每次循环时，i都会被初始化为零。在将i初始化为0后，您的代码会立即执行此操作：

jobs[i].setletter(fileinfo[0]);
jobs[i].setarrivaltime(fileinfo[2]);
jobs[i].setduration(fileinfo[4]);

i在这里总是为零。这就是你所看到的。您的计算机将始终完全按照您的要求执行，而不是您希望它执行的操作。这是一个很好的经验法则，让你记住，继续前进。

i++;

这并不重要，因为在while循环的下一次迭代中，i将再次初始化为0。见上文。

 while(!afile.eof())

而且，this is going to be your second bug，你也会在修复你的第一个bug之后立即发现（在循环之前初始化i，而不是在循环之内）。

尝试从文本文件读取到对象数组

1 个答案: