Чтение из файла после достижения конца файла

Question

У меня есть функция с именем readNextString(ifstream &file , char* &pBuffer), которая извлекает следующую строку из файла, до достижения ',' или '\n', удаляет пробелы в начале и в конце строки, сохраняет остальное в pBuffer и возвращает true, если все работает нормально - false в противном случае. Все работает до тех пор, пока не будет достигнут конец файла. Когда установлен флаг eof, я не могу перенести указатель get. Я попытался это:

if(file.eof()) 
{ 
    file.clear(); 
    file.seekg(0 , ios::end) 
} 

..., а затем удалить пробельные в конце строки. Это почти помогло. Функция извлекает строку без пробелов, но я получаю бесконечный цикл.

Мой фактический вопрос: как я могу проверить, является ли следующий символ EOF, и если я не могу - есть ли альтернативный способ сделать это?

Вот моя фактическая функция:

bool readNextString(ifstream &file , char* &pBuffer) 
{ 
    if(file.eof()){ 
     return false; 
    } 
    for(; file.good() && isWhitespace(file.peek()) && !file.eof() ; file.seekg(1 , ios::cur)) 
     ; 
    if(file.eof()){ 
     cout << "The file is empty.\n"; 
     return false; 
    }else{ 
     streamoff startPos = file.tellg(); 
     cout << "startPos : " << startPos << endl; 
     for(;file.good() && file.peek()!='\n' && file.peek()!=',' && file.peek()!= EOF; file.seekg(1 , ios::cur)) 
      ; 
     streamoff A = file.tellg(); 
     cout << "A : " << A << endl; 
     file.seekg(-1 , ios::cur); 
     for(;file.good() && isWhitespace(file.peek()) ; file.seekg(-1 , ios::cur)) 
      ; 
     file.seekg(2 , ios::cur); 
     streamoff endPos = file.tellg(); 
     cout << "endPos : " << endPos << endl; 
     pBuffer = new char[endPos-startPos]; 
     if(pBuffer) 
     { 
      file.seekg(startPos , ios::beg); 
      file.get(pBuffer , endPos-startPos , ',' || '\n'); 
      for(;file.good() && file.peek()!='\n' && file.peek()!=',' && file.peek()!= EOF; file.seekg(1 , ios::cur)) 
       ; 
      file.seekg(2 , ios::cur); 
      streamoff temp = file.tellg(); 
      cout << "temp : " << temp << endl; 
      return true; 
     }else{ 
      cout << "Error! Not enough memory to complete the task.\nPlease close some applications and try again.\n"; 
      return false; 
     } 
    } 
} 

И это единственное место, где я это называю:

void printCities() 
{ 
    ifstream city ; 
    city.open("cities.txt", fstream::in); 
    if(city.is_open()) 
    { 
     char *currCity; 
     int counter = 1; 
     while(readNextString(city , currCity)) 
     { 
      cout << counter++ << ". " << currCity << endl; 
      delete[] currCity; 
      currCity = NULL; 
     } 
     if(city.eof()) 
      cout << "There are no cities added.\n"; 
     city.close(); 
    }else 
     cout << "Error by opening 'cities.txt'.Make sure that the file exist and try again.\n"; 
} 

Надеется, что я достаточно ясно. Если вы обнаружите некоторые другие ошибки или возможные ошибки, я буду рад услышать их и узнать из него.

Answer 1

Ok.I сделал это иначе! getline() для WIN! : D Так что это мой код (на этот раз гораздо более удобным для чтения):

bool readNextString(ifstream &file , char pBuffer[]) 
{ 
    while(isWhitespace(file.peek()) && !file.eof()) 
     file.ignore(1); 
    if(!file.eof()) 
    { 
     streamoff start = file.tellg(); 
     stringstream toComma; 
     if(file.getline(pBuffer , 200 , ',')) 
     { 
      toComma << pBuffer; 
      toComma.getline(pBuffer ,200, '\n'); 
      int i=strlen(pBuffer)-1; 
      for(; isWhitespace(pBuffer[i]) ;i--) 
       ; 
      pBuffer[i+1] = '\0'; 
      file.clear(); 
      file.seekg(start + strlen(pBuffer) , file.beg); 
      return true; 
     }else return false; 
    } 
    return false; 
} 

Я сделал некоторые изменения в моей другой функции тоже:

void printCities() 
{ 
    ifstream city ; 
    city.open("cities.txt", fstream::in); 
    if(city.is_open()) 
    { 
     if(!isEmpty(city)) 
     { 
      char currCity[200]; 
      int counter = 1; 
      while(readNextString(city , currCity) && counter < 10) 
       cout << counter++ << ". " << currCity << endl; 
     }else 
      cout << "There are no cities added.\n"; 
     city.close(); 
    }else 
     cout << "Error by opening 'cities.txt'.Make sure that the file exist and try again.\n"; 
} 

Функциональный isEmpty(ifstream &file) возвращает истину, если файл пуст и false в противном случае.

Спасибо всем за помощь! С уважением!

Answer 2

Как проверить, является ли следующий символ EOF?

Как это

if (file.peek() == EOF) 
{ 
    // next char is EOF 
    ... 
} 

Answer 3

Во-первых, не использование стремятся пропустить пробелы. Просто получите символ и сделайте это.

Во-вторых, вы, кажется, неправильно понимаете значение istream::good() и istream::eof(). Существует никогда в любое время, где istream::good() уместно, и istream::eof() не обычно неподходящими после ввода имеет не удалось. Что касается вашего цикла прыгая белое пространство, обычное решение будет:

while (isspace(file.peek()) && file.peek() != EOF) { 
    file.get();  // ignore read character... 
} 

Аналогичные комментарии справедливы и для других циклов, а также, за исключением того, что вы не хотите, чтобы игнорировать символы чтения. Для того, чтобы собрать символов до следующего ',', например:

std::string field; 
while (file.peek() != ',' && file.peek() != EOF) { 
    field.push_back(file.get()); 
} 

(и ваш file.get(pBuffer, endPos - startPos, ',' || '\n') , конечно, не делать то, что вы ожидаете, что, выражение ',' || '\n' всегда будет вычисляться true, который, когда преобразуется к char, является '\01.)

Наконец, в то время как выше стратегия будет работать, это гораздо предпочтительнее для ввода больших блоков текста в std::stream и разобрать это.Если текст ориентирован на линию, используйте что-то вроде:

std::string line; 
while (std::getline(file, line)) { 
    // Parse line, using std::istringstream if appropriate, 
    // although this doesn't seem to be the case for your code. 
} 

Это на порядок проще, чем то, что вы делаете.