Проблема с использованием getline и strtok вместе в программе - PullRequest
Новая
       43

Проблема с использованием getline и strtok вместе в программе

5 голосов
/ 21 июня 2010

В приведенной ниже программе я намерен прочитать каждую строку в файле в строку, разбить строку и отобразить отдельные слова. Проблема, с которой я сталкиваюсь, заключается в том, что теперь программа выводит только первую строку в файле. Я не понимаю, почему это происходит? 

#include<iostream>
#include<string>
#include<fstream>
#include<cstdio>
using namespace std;

int main()
{
    ifstream InputFile("hello.txt") ;
    string store ;
    char * token;

    while(getline(InputFile,store))
    {
        cout<<as<<endl;
        token = strtok(&store[0]," ");
        cout<<token;
        while(token!=NULL)
        {
        token = strtok(NULL," ");
        cout<<token<<" ";
        }

    }

}

Ответы [ 3 ]

3 голосов
/ 21 июня 2010

Я новичок в C ++, но я думаю, что альтернативный подход может быть: 

while(getline(InputFile, store))
{
    stringstream line(store); // include <sstream>
    string token;        

    while (line >> token)
    {
        cout << "Token: " << token << endl;
    }
}

Это будет анализировать ваш файл построчно и токенизировать каждую строку на основе разделения пробелов (так что это включает в себябольше, чем просто пробелы, такие как табуляции и новые строки).

2 голосов
/ 21 июня 2010

Ну, здесь есть проблема. strtok() принимает строку с нулевым символом в конце, а содержимое std::string не обязательно должно заканчиваться нулем.

Вы можете получить строку с нулевым символом в конце std::string, вызвав для нее c_str(), но это вернет const char* (т.е. строка не может быть изменена). strtok() принимает char* и изменяет строку при ее вызове.

Если вы действительно хотите использовать strtok(), то, на мой взгляд, самым чистым вариантом будет скопировать символы из std::string в std::vector и завершить нулем вектор: 

std::string s("hello, world");
std::vector<char> v(s.begin(), s.end());
v.push_back('\0');

Теперь вы можете использовать содержимое вектора в виде строки с нулевым символом в конце (используя &v[0]) и передать ее в strtok().

Если вы можете использовать Boost, я бы рекомендовал использовать Boost Tokenizer . Он обеспечивает очень чистый интерфейс для токенизации строки.

0 голосов
/ 21 июня 2010

То, что говорит Джеймс МакНеллис, является правильным.

Для быстрого решения (хотя и не лучшего) вместо 

string store

используйте 

const int MAX_SIZE_LINE = 1024; //or whatever value you consider safest in your context.
char store[MAX_SIZE_LINE];
...