0

我希望有人可以修改我的代码，因为它太有问题了。有时它的工作，有时它不..

所以让我解释一下.. 文本文件数据如下

Line3D, [70, -120, -3], [-29, 1, 268]
Line3D, [25, -69, -33], [-2, -41, 58]

要阅读上面的行..我使用以下

char buffer[30];

cout << "Please enter filename: ";
cin.ignore();
getline(cin,filename);

readFile.open(filename.c_str());

//if successfully open
if(readFile.is_open())
{
//record counter set to 0
numberOfRecords = 0;

while(readFile.good())
{
//input stream get line by line
readFile.getline(buffer,20,',');


if(strstr(buffer,"Point3D"))
{
Point3D point3d_tmp;
readFile>>point3d_tmp;

// and so on...

然后我对 Line3d 的 ifstream 做了一个重载

ifstream& operator>>(ifstream &input,Line3D &line3d)
{
int x1,y1,z1,x2,y2,z2;

//get x1
input.ignore(2);
input>>x1;

//get y1
input.ignore();
input>>y1;

//get z1
input.ignore();
input>>z1;

//get x2
input.ignore(4);
input>>x2;

//get y2
input.ignore();
input>>y2;

//get z2
input.ignore();
input>>z2;
input.ignore(2);

Point3D pt1(x1,y1,z1);
Point3D pt2(x2,y2,z2);

line3d.setPt1(pt1);
line3d.setPt2(pt2);

line3d.setLength();
}

但问题是有时记录工作，有时不工作..我的意思是如果在这一点上

//i add a cout
cout << x1 << y1 << z1;
cout << x2 << y2 << z2;

//its works!
Point3D pt1(x1,y1,z1);
Point3D pt2(x2,y2,z2);

line3d.setPt1(pt1);
line3d.setPt2(pt2);

line3d.setLength();

但如果我拿走 cout 它就行不通了。我如何更改我的 cin.ignore() 以便可以正确处理数据，考虑数字范围是 -999 到 999

4

1 回答 1

0

我无法解释为什么这段代码会崩溃。这可能是因为您有未在此处发布的错误。但是，您会发现以这种方式编写 operator>> 更容易。

istream& operator>>(istream &input,Line3D &line3d)
{
    int x1,y1,z1,x2,y2,z2;
    char c1,c2,c3,c4,c5,c6,c7;

    input >> c1 >> x1 >> c2 >> y1 >> c3 >> z1 >> c4 >> c5 >> x2 >> c6 >> y2 >> c7 >> z2;

    Point3D pt1(x1,y1,z1);
    Point3D pt2(x2,y2,z2);

    line3d.setPt1(pt1);
    line3d.setPt2(pt2);
    line3d.setLength();
    return input;

}

使用 dummy char 变量（c1、c2 等）读取您不感兴趣的逗号和括号。此技术还将跳过您不感兴趣的空格。这是一种比使用忽略更可靠的方法。

您的代码中的其他错误operator>>应该使用istreamnot ifstream，并且应该return input;在最后。通过编写您operator>>的使用istream它，它将与任何类型的输入流（包括cin例如）一起使用，而不仅仅是文件流。

于 2012-11-15T16:57:57.860 回答