1

次のような文字列分割テンプレートを作成しようとすると、エラーが発生します。

struct splitX
{
  enum empties_t { empties_ok, no_empties };
};

template <typename Container>
Container& split(
  Container&                                 result,
  const typename Container::value_type&      s,
  typename Container::value_type::value_type delimiter,
  splitX::empties_t                           empties = splitX::empties_ok )
{
  result.clear();
  std::istringstream ss( s );
  while (!ss.eof())
  {
      typename Container::value_type field;
      getline( ss, field, delimiter );
      if ((empties == split::no_empties) && field.empty()) continue;
      result.push_back( field );
  }
  return result;
}

これは、以下のようなベクトルクラスで使用しようとすると、見出しからのエラーで失敗します。

std::getline(myfile,line);
std::vector<std::string> fields;
split(fields,line,' ');

// Test split function
std::cout << line << std::endl;
for(int i = 0; i < fields.size();i++)
  std::cout << fields[i];
4

2 に答える 2

4

わかりました、Microsoft Visual C++2008はあなたのを好きではありませんContainer::value_type::value_type。

あなたはこの巨大な構造を使うことができます：

template <typename T>
struct vte /*value_type_extractor*/
{
    typedef typename T::value_type type;
};

template <typename Container>
Container& split(
   Container&                                 result,
   const typename Container::value_type&      s,
   typename vte<typename vte<Container>::type>::type delimiter,
   splitX::empties_t                           empties = splitX::empties_ok )
{
  typedef typename Container::value_type string_type;
  typedef typename string_type::value_type elem_type;
  std::basic_istringstream<elem_type> ss( s );
  ...
}

または、STLスタイルで実行できます。

#inlcude <iterator>

template <typename OutputIterator, typename StringType>
OutputIterator split(
  OutputIterator                  out,
  const StringType&               s,
  typename StringType::value_type delimiter,
  splitX::empties_t               empties = splitX::empties_ok )
{
  typedef typename StringType::value_type elem_type;
  std::basic_istringstream<elem_type> ss( s );
  while (!ss.eof())
  {
      StringType field;
      std::getline( ss, field, delimiter );
      if ((empties == splitX::no_empties) && field.empty()) continue;
      *out++ = field;
  }
  return out;
}

//somewhere in code
{
    ...
    split(std::back_inserter(your_container), line, ' ');
}

または、テンプレートの神のふりをやめて、2つのオーバーロードされた関数を作成することもできます。

template <typename Container>
Container& split(
   Container&              result,
   const std::string&      s,
   char                    delimiter,
   splitX::empties_t       empties = splitX::empties_ok )
{
  std::istringstream ss(s);
  ...
}

template <typename Container>
Container& split(
   Container&              result,
   const std::wstring&     s,
   wchar_t                 delimiter,
   splitX::empties_t       empties = splitX::empties_ok )
{
  std::wistringstream ss(s);
  ...
}
于 2012-06-06T08:18:55.910 に答える
2

あなたのコードを視覚的に調べたところ、私は実際には何も悪いことを検出しませんでした。私はあなたのプログラムのカットアンドペーストを行い、mainあなたのテストコードを囲み、それをコンパイルしました（FC15のストックg++）。

splitコンパイラが関数にタイプミスを見つけました。split::no_emptiesである必要がありますsplitX::no_empties。また、警告を有効にすると、forループが符号付き型と符号なし型を比較していることが報告されました。

そのタイプミスとコンパイラの警告を修正した後、ルーチンはコンパイルされ、正常に実行されました。

于 2012-06-06T07:55:49.710 に答える