C++ vector参照渡しで要素追加をしたい

Question

実現したいこと

C++でvectorを参照渡しして、要素を追加するユーザ関数を作ろうとしています。
空白を含む文字列を空白で区切ってvectorに格納する機能を持った関数です。

前提

vectorを渡しても要素を追加することができない状態です。

発生している問題・エラーメッセージ

なし

該当のソースコード

#include <iostream>
#include <string>
#include <vector>
#include <algorithm>
#include <iterator>

using namespace std;

void vectorSplit(vector<string> &v,string str);

int main()
{
    string s;
    while(1) {
        getline(cin,s);
        cout << s << endl;
        if(cin.fail()) {
            cout << "unti" << endl;
            break;
        }
        vector<string> info;
        vectorSplit(info,s);
        cout << info.size() << endl;
    }
}

void vectorSplit(vector<string> &v,string str)
{
    string tmp = "";
    for(int i=0;i<str.length()-1;i++) {
        if(str[i] == ' ') {
            cout << tmp << endl;
            v.push_back(tmp);
            tmp = "";
            continue;
        }
        tmp += str[i];
    }
}

試したこと

vectorSplitにvectorを渡してもサイズが0のままです。

補足情報（FW/ツールのバージョンなど）

Comments

[episteme]

再現しません。vector::size() は変化します。

[int32_t]

実行時にどんな入力をして動作確認していますか? 「cout << tmp << endl;」は実行されましたか?

[episteme]

ふたつ以上の空白が続いたときに誤作動するけど、いいのかな?

Answer 1

[ご参考] 僕ならこう書く(vectorの参照渡しではなく、イテレータ渡し):

#include <string>

template <typename OutputIterator>
OutputIterator split(OutputIterator out, const std::string& str, 
                     const std::string& delim =" \t\n") {
  std::string token;
  bool in_token = false;
  for ( char ch : str ) {
    if ( delim.find(ch) != std::string::npos ) {
      if ( in_token ) { *out++ = token; in_token = false; }
    } else {
      if ( in_token ) { token += ch; } 
      else { token = ch; in_token = true; }
    }
  }
  if ( in_token ) { *out++ = token; }
  return out;
}

// ----- おためし

#include <vector>
#include <iostream>
#include <iterator>

int main() {
  using namespace std;
  while ( true ) {
    string s;
    getline(cin,s);
    if(cin.fail()) {
      cout << "unti" << endl;
      break;
    }
    vector<string> info;
    split(back_inserter(info), s);
    for ( const auto& item : info ) {
      cout << '[' << item << "] ";
    }
    cout << endl;
  }
}

Answer 2

質問文の vectorSplit() は、

• 空白が含まれていない入力文字列だと何も起きない
• 入力文字列の最後の１文字が無視される

という問題があるように見えます。

    for (int i = 0; i < str.length(); ++i) { // length() から1引く必要はない
        ...
    }
    v.push_back(tmp);  // forループ後に最後の項目を追加する必要がある