Question

我正在尝试将逗号分隔的字符串转换为const char *的向量。使用以下代码，按预期输出

ABC_
DEF
HIJ

但我得到

HIJ
DEF
HIJ

我哪里错了？

代码：

#include <iostream>
#include <boost/tokenizer.hpp>
#include <vector>
#include <string>
using namespace std;

int main()
{
   string s("ABC_,DEF,HIJ");
   typedef boost::char_separator<char> char_separator;
   typedef boost::tokenizer<char_separator> tokenizer;

   char_separator comma(",");
   tokenizer token(s, comma);
   tokenizer::iterator it;

   vector<const char*> cStrings;

   for(it = token.begin(); it != token.end(); it++)
   {
      //cout << (*it).c_str() << endl;
      cStrings.push_back((*it).c_str());
   }

   std::vector<const char*>::iterator iv;
   for(iv = cStrings.begin(); iv != cStrings.end(); iv++)
   {
      cout << *iv << endl;
   }
   return 0;
}

http://ideone.com/3tvnUs

编辑：解决方案有以下答案的帮助：（PaulMcKenzie使用列表提供更简洁的解决方案。）

#include <iostream>
#include <boost/tokenizer.hpp>
#include <vector>
#include <string>
using namespace std;

char* createCopy(std::string s, std::size_t bufferSize)
{
   char* value = new char[bufferSize];
   memcpy(value, s.c_str(), (bufferSize - 1));
   value[bufferSize - 1] = 0;
   return value;
}

int main()
{
   string s("ABC_,DEF,HIJ");
   typedef boost::char_separator<char> char_separator;
   typedef boost::tokenizer<char_separator> tokenizer;

   char_separator comma(",");
   tokenizer token(s, comma);
   tokenizer::iterator it;

   vector<const char*> cStrings;

   for(it = token.begin(); it != token.end(); it++)
   {
      //cout << it->c_str() << endl;
      cStrings.push_back(createCopy(it->c_str(),
                                      (it->length() + 1)));
   }

   std::vector<const char*>::iterator iv;
   for(iv = cStrings.begin(); iv != cStrings.end(); iv++)
   {
      cout << *iv << endl;
   }

   //delete allocations by new
   //...
   return 0;
}

Answer 1

事情是这样的：boost::tokenizer::iterator不会返回您对字符串副本的所有权，而是指向内部副本。

例如，在运行代码后，我得到：

HIJ
HIJ
HIJ

解决方案是将cStrings.push_back((*it).c_str())替换为以下之一：

    char* c = new char[it->length() + 1];
    c[it->length()] = 0;
    cStrings.push_back(c);
    std::strncpy(c, it->c_str(), it->length());

看起来不漂亮，但你可能不会比这更快（至少如果你想使用boost::tokenizer。

其他选项是完全取代boost::tokenizer，例如strtok - 可以在此处找到一个示例：C split a char array into different variables

您也可以使用boost::algorithm::string::split，但之后您可能需要将string重新映射到const char*。

Answer 2

这是一种不需要动态分配的方法，同时为您提供了您正在寻找的std :: vector。诀窍是在＆＃34; permanent＆＃34;中创建你的参数。存储，然后只需设置指向该存储的指针向量。

以下代码使用std::list作为永久存储空间。原因是我们可以保证在将项添加到列表容器时，std::list中的项的迭代器不会失效。这是构建const char *的最终向量时的必要条件。

#include <iostream>
#include <boost/tokenizer.hpp>
#include <vector>
#include <string>
#include <list>

typedef std::vector<char> CharArray;
typedef std::list<CharArray> StringList;

using namespace std;

int main()
{
   StringList sList;

   string s("ABC_,DEF,HIJ");
   typedef boost::char_separator<char> char_separator;
   typedef boost::tokenizer<char_separator> tokenizer;

   char_separator comma(",");
   tokenizer token(s, comma);
   tokenizer::iterator it;

   vector<const char*> cStrings;

   for(it = token.begin(); it != token.end(); ++it)
   {
        // create an array of char and place on list
        sList.push_back(CharArray(it->begin(), it->end()));

        // null terminate this entry
        sList.back().push_back(0);

        // add the pointer to this entry to the vector of const char *.
        cStrings.push_back(&sList.back()[0]);
   }

   std::vector<const char*>::iterator iv;
   for(iv = cStrings.begin(); iv != cStrings.end(); iv++)
   {
      cout << *iv << endl;
   }
}

请注意，我们不必在此动态分配内存。您唯一需要确保的是StringList不会超出范围，因为这是您的参数所在。

矢量的逗号分隔标记到const char **

2 个答案: