为什么 C++ strtok() 为我工作吗？

Question

该程序应该通过 cin 接收输入，对其进行标记，然后输出每个输入以向我表明它工作正常。但事实并非如此。

该程序编译时没有错误，并接受输入，但无法输出任何内容。

我做错了什么？

int main(int argc, char* argv[])
{
  string input_line;

  while(std::cin >> input_line){
    char* pch = (char*)malloc( sizeof( char ) *(input_line.length() +1) );

    char *p = strtok(pch, " ");
    while (p != NULL) {
      printf ("Token: %s\n", p);
      p = strtok(NULL, " ");
    }
  }
  return 0;
}

我按照这里的代码示例： http://www.cplusplus.com/reference/ clibrary/cstring/strtok/

谢谢。

Answer 1

看起来您忘记将 input_line 的内容复制到 pch：

strcpy(pch, input_line.c_str());

但我不确定您为什么要进行字符串标记化。正在做cin>> input_line 不会读取一行，而是读取一个令牌..所以你无论如何都会得到令牌？

Answer 2

这更像是一个正确性的帖子，汉斯有你的问题。

获取一行输入的正确方法是使用 getline：

std::string s;
std::getline(std::cin, s);

std::cin 无论如何都会在空格处中断，因此如果您输入 asd 123 并运行您的代码，input_line 首先会是“asd”，然后在循环中第二次是“123”（无需等待输入）。

也就是说，获取结果的一种简单方法是使用 stringstream。每当您显式分配内存时，尤其是使用 malloc 时，您可能会遇到困难。这是标记字符串的一种可能的解决方案：

#include <sstream>
#include <string>
#include <iostream>

int main(void)
{
    std::string input;
    std::getline(std::cin, input);

    std::stringstream ss(input);
    std::string token;
    while(std::getline(ss, token, ' '))
    {
        std::cout << token << "...";
    }

    std::cout << std::endl;
}

如果您确实想使用 strtok，您可能会这样做：

#include <cstring>
#include <string>
#include <iostream>
#include <vector>

int main(void)
{
    std::string input;
    std::getline(std::cin, input);

    std::vector<char> buffer(input.begin(), input.end());
    buffer.push_back('\0');

    char* token = strtok(&buffer[0], " ");
    for (; token; token = strtok(0, " "))
    {
        std::cout << token << "...";
    }

    std::cout << std::endl;
}

记住，手动内存管理是不好的。对数组使用向量，可以避免泄漏。 （你的代码有！）

Answer 3

您没有初始化您的字符串。 插入。

strcpy(pch, input_line.c_str());

在 malloc 行之后

Answer 4

GMan 的答案可能更好，更纯粹的 c++。这更像是专门使用 strtok() 的混合，因为我认为这是您的目标。

我使用了 strdup()/free() 因为这是复制字符串的最简单方法。在这个问题中，您正在泄漏内存，因为您的 malloc() 没有匹配的 free() 。

也算符>>字符串会在空格处中断，因此不适合获取行。请改用getline()。

token.cpp

#include <iostream>
#include <string>
#include <cstring> /* for strtok() and strdup() */
#include <cstdlib> /* for free() */

int main(int argc, char * argv[]){
    std::string line;

    while(getline(std::cin, line)){
        char *pch = strdup(line.c_str());

        char *p = strtok(pch, " ");

        while(p){
            std::cout<<"Token: "<<p<<std::endl;
            p = strtok(NULL, " ");
        }

        std::cout <<"End of line"<<std::endl;
        free(pch);
    }
    return 0;
}

当您运行此程序时，您会得到看似正确的结果/

$ printf '嗨，我喜欢标记\n在新行上也\n\n空白很好'|./token
代币：嗨
令牌：有，
代币：I
代币：喜欢
代币：代币
行尾
令牌：开启
代币：新
令牌：行
代币：太
行尾
行尾
令牌：空白
令牌：是
令牌：很好
行尾

Answer 5

或者使用这个：

pch = strdup(input_line.c_str());