CSVデータの文字列行の分割関数について

質問

普段は、Delphi でコーディングしているものです。
CSVファイルの行を分割するのに関数を作成しました。(いままで使っていたDelphiのコードを移植しました。vectorとstringを使ってほぼ同じような処理を実現しています。）
これで一応動くのですが、vectorを関数内のstaticなオブジェクトにしたのが気になります。
このソースで問題はないでしょうか？
//
// To compile
// CL -GX splittest.cpp
//
#include <iostream.h>
#include <string>
#include<vector>
using namespace std;

vector<string> *SplitToVector(char dlm, string src)
{ static vector<string> vct;
int p;

vct.clear();
p = src.find(dlm);
while(p != src.npos){
vct.push_back(src.substr(0, p));
src = src.substr(p+1, src.length());
p = src.find(dlm);
}
if(src.length() > 0)
vct.push_back(src);
return &vct;
}

int main(void)
{ char sline[256];
vector<string> *vct;
int i,n;

strcpy(sline, "日本語,発音,備考");
vct = SplitToVector(',', sline);
cout << sline << endl;
n = vct->size();
for(i=0; i<n; i++)
cout << vct->at(i).c_str() << endl;
return 0;
}
以上、よろしくお願いします。

回答No.2

noname#168973 · ベストアンサー

No.1の方も仰っているように、利用側が十分に考慮すれば問題ないでしょうね。ただ、一般論から言えば利用者に制約を課す事はあまり良くないと思います。

## 利用者は多分、常にそんな事に気を配っていられないでしょうし、また Multi thread 等の 環境で(計らずとも)利用された場合、気を配る事すら難しいでしょうしね。

色々な解決方法があると思いますが、こんなのもあり??というのを一つ...

私は上記のような場合、最近なら

vector<string> SplitToVector(char dlm, string src)
{
　vector<string> vct;
　　: (vct に対する操作)
　return vct;
}

とします。## 笑わないで下さいね^^)

というのは、最近の賢い コンパイラーであれば上記を

void SplitToVector(vector<string>& vct, char dlm, string src)
{
　　: (vct に対する操作)
}

として自動でコンパイルしてくれます。

これの良い点は、利用者から見て

vector<string> vct = SplitToVector(...);

と記述できる事でしょうかね。

vector<string> vct;
SplitToVector(vct, ...);

だと、vct は値が戻されるのか、パラメータとして渡しているのか直感的に不明ですからね。

ですが、コンパイラーの実装依存なのはちょっとマイナスです;;)

お礼

貴重な回答ありがとうございます。

＞void SplitToVector(vector<string>& vct,・・・

なんと！参照になってしまうんですね。
まあ、今回はそのままコピーになっても問題ないレベルなので使ってみます。

呼び出し側で * や & が無くなると、ずいぶんすっきりしますね。

回答No.1

BLUEPIXY

>vectorを関数内のstaticなオブジェクトにしたのが気になります。
使える期間を意識していればいいんじゃないですかね。

mainで作ったvectorを引数として（参照で受けて）それを使う方が、個人的には好みですが。
それより、
int p;
は、
unsigned p;
の方がいいかも

お礼

すばやい回答ありがとうございます。

＞mainで作った・・・には好みですが。
Delphiのコードでは、Split関数のなかでCreateしたTStringListを呼び出し側で受けて、必要が無くなったらfreeしていました。　C++同じようにしようとすると、かなり面倒そうなのでやめたんです。

＞unsigned p;
さっそく修正します。C/C++ はこの辺に神経をつかいますね。