這段代碼「長度指示器實現」有什麼問題？

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（）？

---

此外，這聽起來像是一個很好的時間拿出一個調試器並在您的代碼上運行它。如果它是一個無限循環，調試器會很快告訴你你卡在哪裏。