这段代码“长度指示器实现”有什么问题？

Question

这是长度指示符字段的实现 但它挂起了，我认为陷入了循环并且没有显示任何东西。

// readx22.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include "iostream"
#include "fstream"
#include "stdio.h"
using namespace std;


class Student
{
public:
 string id;
 size_t id_len;
 string first_name;
 size_t first_len;
 string last_name;
 size_t last_len;
 string phone;
 size_t phone_len;
 string grade;
 size_t grade_len;
 void read(fstream &ven);
 void print();
};
void Student::read(fstream &ven)
{
 size_t cnt;
 ven >> cnt;
 id_len=cnt;
    id.reserve( cnt );
    while ( -- cnt ) {
        id.push_back( ven.get() );
    }

 ven >> cnt;
 first_len=cnt;
    first_name.reserve( cnt );
    while ( -- cnt ) {
        first_name.push_back( ven.get() );
    }

 ven >> cnt;
 last_len=cnt;
    last_name.reserve( cnt );
    while ( -- cnt ) {
        last_name.push_back( ven.get() );
    }

 ven >> cnt;
 phone_len=cnt;
    phone.reserve( cnt );
    while ( -- cnt ) {
        phone.push_back( ven.get() );
    }

 ven >> cnt;
 grade_len=cnt;
    grade.reserve( cnt );
    while ( -- cnt ) {
        grade.push_back( ven.get() );
    }

}
void Student::print()
{
// string::iterator it;
 for ( int i=0 ; i<id_len; i++)
  cout << id[i];

}
int main()
{
 fstream in;
 in.open ("fee.txt", fstream::in);
 Student x;
 x.read(in);
 x.print();
 return 0;
}

谢谢

Answer 1

您可能应该在任何地方都使用 cnt-- 而不是 -- cnt。第一个零字节字符串将触发一个极大的循环，最终耗尽所有内存（64 位操作系统上可能除外）。实际上，甚至不用担心这个修复。循环get()效率极低，只需调用read()即可。

Answer 2

如果我正确理解你的问题，那么当你运行这个时，你会看到无限循环吗？

我很想知道每个循环之前的 cnt 是什么。

另外，您的代码实际上是否达到了x.print()？

---

另外，这听起来像是取出调试器并在代码上运行它的好时机。如果是无限循环，调试器会很快告诉您卡在哪里。