出入力ｃ＋＋

Question

入出力のコードを書いていたのですが、エラーがでてしまいます。
問題点を指摘してくださるとうれしいです。

コード
/*
 * stream.cpp
 *
 *  Created on: 2015/10/29
 *      Author: 礼央
 */

#include<iostream>
#include<sstream>
#include<fstream>

int main() {
	using namespace std;
	ostringstream strbuf;

	int lucky = 7;
	float pi = 3.14;
	double e = 2.71;
//
	cout << "An in-memory stream" << endl;
	strbuf << "luckynumber:" << lucky << endl << "pi:" << pi << endl << "e:"
			<< e << endl;

	string strval = strbuf.str();
	cout << strval;

	ofstream outf;
	outf.open("mydata");
	outf << strval;
	outf.close();

	cout << "Read data from the file-watch for errors" << endl;
	string newstr;
	ifstream inf;
	inf.open("mydata");
	if (inf) {
		int lucky2;
		inf >> newstr >> lucky2;
		if (lucky != lucky2)
			cerr << "error! wrong" << newstr << lucky2 << endl;
		else
			cout << newstr << "OK" << endl;

		float pi2;
		inf >> newstr >> pi2;
		if (pi2 != pi)
			cerr << "Error! wrong" << newstr << pi2 << endl;
		else
			cout << newstr << "OK" << endl;
		double e2;
		inf >> newstr >> e2;
		if (e2 != e)
			cerr << "ERROR:Wrong" << newstr << e2 << endl;
		else
			cout << newstr << "OK" << endl;
		inf.close();
	}
}

Answer 1

istreamから>>でstring型変数へ読みだすと、次の区切り記号(whitespace)までを読み出します。
区切り記号はスペースやタブ文字や改行文字のようです。
これらの区切り記号を挟まずに数値を出力しているため、:の後の数字もnewstrに読まれてます。

フォーマットを変えてよいなら、:の後に１つスペースを挟むだけで解決します。

フォーマットを変えない場合は、getlineの区切り記号指定版を使うことも考えられます。
下記のように使います。
getline(inf, newstr, ':') >> lucky2;

ただ、':'を区切り記号にことで
①改行文字がnewstrに読まれてしまいます。
②':'はnewstrに読まれません。
①の影響が結構鬱陶しいし、対策は「汚く」なります。

少し面倒になりますが、行単位でstring型変数へ読み込んでfind_first_ofやfind_first_not_ofを使う定石的な方法を使った方がgetlineを使うより良いかも知れません。

Answer 2

std::ifstream operator>>( std::ifstream& ifs, std::string& str )
を確認しましょう．デフォルトでは数字も「：」も含めて全部読み取ります．
以下のようにオーバーライドするか，operator>>ではない独自関数を作って，そっちに任せましょう．

#include<iostream>
#include<sstream>
#include<fstream>
#include<cctype>

std::ifstream& operator>>( std::ifstream& ifs, std::string& str )
{
	char check = 0;
	if ( ifs >> check )
	{
		str += check;
		while ( isalpha( check ) && ifs.get( check ) )
		{
			str += check;
		}
	}
	else
	{
		ifs.setstate( std::ios_base::failbit );
	}
	
	return ifs;
}

int main() {
	using namespace std;
	ostringstream strbuf;

	int lucky = 7;
	float pi = 3.14;
	double e = 2.71;
	//
	cout << "An in-memory stream" << endl;
	strbuf << "luckynumber:" << lucky << endl << "pi:" << pi << endl << "e:"
		<< e << endl;

	string strval = strbuf.str();
	cout << strval;

	ofstream outf;
	outf.open("mydata.txt");
	outf << strval;
	outf.close();

	cout << "Read data from the file-watch for errors" << endl;
	string newstr;
	ifstream inf;
	inf.open("mydata.txt");
	if (inf) {
		int lucky2;
		inf >> newstr >> lucky2;
		if (lucky != lucky2)
			cerr << "error! wrong " << newstr << ", " << lucky2 << endl;
		else
			cout << newstr << "OK" << endl;

		float pi2;

		inf >> newstr >> pi2;
		if (pi2 != pi)
			cerr << "Error! wrong " << newstr << ", "<< pi2 << endl;
		else
			cout << newstr << "OK" << endl;

		double e2;
		inf >> newstr >> e2;
		if (e2 != e)
			cerr << "ERROR:Wrong " << newstr << ", " << e2 << endl;
		else
			cout << newstr << "OK" << endl;
		inf.close();
	}

Comments

[Chironian]

>以下のようにオーバーライドするか，operator>>ではない独自関数を作って，そっちに任せましょう．

標準ライブラリに既に存在している関数(http://www.cplusplus.com/reference/string/string/operator%3E%3E/)の動作を変えると混乱が生じやすいので、別関数を定義した方がより安全と思います。

[退会済みユーザー]

自分も別のアプローチしたほうが絶対いいよなぁこれと思いながら書いてましたｗ