データが正しく出力されません…

Question

前提・実現したいこと

C++を用いて自動採点のシステムを開発しているのですが、
out_of_rangeの例外がどこから発生して、どう処理したらいいのかが分かりません。
今度はデータが正しく出力されない問題が発生しました…

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

~~terminate called after throwing an instance of 'std::out_of_range'what(): basic_string::substr: __pos (which is 4) > this->size() (which is 0)This application has requested the Runtime to terminate it in an unusual way.
Please contact the application's support team for more information.
~~

該当のソースコード

#include <iostream>
#include <fstream>
#include <string>
#include <vector>
#include <utility>
#include "AnswerCheck.h"

typedef vector<string> Type;

Type answer;

Type putdata(string s){
	int cnt = 0, pnt = 0, cha = 0;
	Type buf;
	
	while(1){
		if(buf.size() == 13)	break;
		if(s[cha] != ','){
			cnt++;
		}
		else{
		**  try{
				buf.push_back(s.substr(pnt + 1, cnt - 2));
			}
			catch(...){
				buf.push_back("");
			}
			pnt += ++cnt;
			cnt = 0;
		}
		cha++;
	}
	
	return buf;
}

int main(void){
	ifstream ifs("data.csv");
	ofstream log("log.csv", ios::trunc);
	int cnt = 0;
	
	log << "Class,Number,A1,A2,A3,A4,A5,B1,B2,B3,B4,B5,B6,A Result,Sum Result" << endl;
	
	string s;
	while(getline(ifs, s)){
		int a_result = 0, result = 0;
		if(cnt != 0){
			Type buf = putdata(s);
			string id = buf[1];
			pair<string, string> user;
				user.first  = id.substr(4, 1);
				user.second = id.substr(5, 2);
			cout << "Class  : " << user.first  << endl
					 << "Number : " << user.second << endl;
			log << user.first << ","
					<< user.second << ",";
			for(int a = 2; a < buf.size(); a++){
				int score = check(buf[a], a - 2);
				result += score;
				if(a < 7)	a_result += score;
				log << score << ",";
			}
			log << a_result << "," << result << flush << endl;
		}
		
		cnt++;
	}
}

試したこと

**の場所で例外が発生しているのかと思い、try,catchを入れたのですが、変わりませんでした。

補足情報

「#include "AnswerCheck.h"」は問題採点用のヘッダで、こちらにエラーが無い事は確認済みです。
また、読み込みファイルの「data.csv」は読み込むデータが1つのみ(Googleフォームの結果を落としているので2行)の場合は問題なく動きます。

追加

「data.csv」のサンプルです。
data.csv

開発環境はMinGW6.3.0です。
エディタはTeraPadを使用しています。

回答者のご指摘からmain関数にtry,catchを入れたところ、ひとまずエラーが解消されました。
ですが、入力したデータは4つ連続しているのですが、出力した「log.csv」には以下のように無駄に空白が入ってしまいました…
良ければ原因は何処なのか教えてほしいです…

Comments

[tiitoi]

data.csv のサンプルを追記いただくことはできますか？

[yohhoy]

デバッガ(gdbやlldb)は使えますか？開発環境も追記された方が良い気がします。

Answer 1

問題点

提示された CSV ファイル

"タイムスタンプ","ユーザー名(例：sth1F45)","A1 回答欄","A2 回答欄","A3 回答欄","A4 回答欄","A5 回答欄","B1 回答欄","B2 回答欄","B3 回答欄","B4 回答欄","B5 回答欄","B6 回答欄","C1 回答欄","C2 回答欄","C3 回答欄","C4 回答欄"
"2018/09/10 9:01:29 午後 GMT+9","sth1F45","ABC","AEG","ABCD","ABCE","ADHJ","ABDF","AGLK","AFJK","AEIC","AOFH","AKGJ","","","",""
"2018/09/10 10:03:37 午後 GMT+9","sth1A39","ADE","ABG","ACEH","ABDHKI","AGCH","ABDFHJIJHJI","ABCIMFJMHKML","ABEGLMN","ADGIJK","DGILJLNMN","ADFKNOMPLP","a
i
u
e
o"

CSV のパーサーが不十分ではないですか。

1. getline(ifs, s) で1行取ってくる。
2. putdata() に渡して、カンマで分割し、リストで返す。

となっていますが、CSV の値に改行を含むものがあるので、4行目以降のパースは意図どおり動きません。
例えば、getline(ifs, s) の3回目の呼び出しで渡される引数は s="i" となっていました。

解決策

getline() を使わず、ダブルクオーテーション中に改行があることを考慮したパーサーを作成する必要があります。
自作してもよいかと思いますが、CSV を扱える C++ のライブラリを探してみてはどうでしょうか。

Answer 2

こんにちは。

basic_string::substr: __pos (which is 4) > this->size() (which is 0)

まず、basic_stringはstringのことです。basic_string<char>のtypedefがstringですので。
つまり、string.substrで posが4で長さゼロの文字列の時に例外が発生しているようなエラーメッセージです。該当し得る箇所にもう一つ以下の文がありますね。

user.first = id.substr(4, 1);

恐らくbuf[1]の長さ0の文字列が返ってくるようなデータを渡されているのだと思います。
putdata()関数に渡した s とputdata()関数の戻り値を出力して確認することがおすすめです。

Answer 3

データを見ないとなんとも言えませんが、

user.first = id.substr(4, 1);
user.second = id.substr(5, 2);

この辺は問題無いですか? エラーメッセージから、怪しくあります。
また、try {} の範囲ですが、** の箇所だけでなく、他の箇所はどうでしたか?
main() の中を適当に入れてみてはどうでしょうか?

Comments

[Removed_Past]

ご指摘された箇所にtry catchを入れたところ、エラーは解消されました！
ただ、質問を編集しましたが、今度はデータが正しく出力されないという問題が今度は発生してしまいました…
よろしければこちらの方にも御力をお貸しいただけないでしょうか？

[pepperleaf]

データ内に改行コードが入っています。
getline()は改行までを一行と見なすためです。
csvデータとして読む工夫が必要です。C++に csvデータを読み込むライブラリがあると思いますが、直ぐには分かりません。
或いは、単になる文字列として読込み、"" を一塊のデータとするコードが必要となります。