C++でcsvを2次元配列に格納

Question

#include <iostream>
#include <string> 
#include <stdint.h>
#include <vector>
#include <stdio.h>
#include <stdlib.h>
#include <cmath>
#include <iostream>
#include <math.h>
#include <fstream>
#include <sstream>
#include <algorithm> 
#include <chrono>
#include <utility> 
#include <time.h> 
#include <tuple> 
#include <cstdint> 
#include <cstdio>
#include <random>

int main()
{
	
	ifstream streamx("Bx.csv");
	string linex;
	double Br[364][184];
	const string delim = ",";
	int rowx = 0;
	int colx;
	while (getline(streamx, linex)) {
	colx = 0;
	for (string::size_type spos, epos = 0;
	(spos = linex.find_first_not_of(delim, epos)) != string::npos;) {
	string token = linex.substr(spos, (epos = linex.find_first_of(delim, spos)) - spos);
	Br[rowx][colx++] = stod(token);
	}
	++rowx;
	}
	cout << Br[4][57];//読み込めたかのチェック

	return 0;
}


```### 前提・実現したいこと

C++でのcsvファイル読み込みについて質問です．

読み込みcsvファイルの情報をdouble型の2次元配列に格納したいですが，ネットでいろいろ調べたり自分でコードを作ってみてもabort()has been calledなどいくつかエラーが表示されてしまい先に進めません．
Bx.csvという364*184のcsv形式ファイルをBr[364][184]という配列に入れたいと考えています．
質問前に散々調べましたがプログラミング初心者でして今現在の力では対応できませんでした．


また同規模のcsvを一度に20枚ほど読み込む必要があるのですがその際に変更すべき個所なども教えていただけますと幸いです．
何卒ご教示お願いします．

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


エラーメッセージ
debug error！
 abort() has been called





### 試したこと
以下の部分をmain関数の一部分に突っ込んでおり，これをコメントアウトすればプログラムは正常に動くので以上はこの部分だけだと思います．またCPUのメモリ容量の問題かと思いましたが，研究室の大きな計算機に放り込んでも同様のエラーを返されたので(そもそもそこまで大きなデータではないですし)それも無関係だと思います．


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

visual studio 2015/2019/g++は最新版にアップデートしてあります．

Comments

[BeatStar]

コードは『編集時の』、"<コード>" または "<code>" となっているものを押してからその中に書いた方がいいですよ。読みづらいですし。

[cateye]

コードは、コンパイル出来る状態て上げて下さい。

[Kinsho]

変更したしました．このサービスを使うのが今回初めてのため不慣れですみません．

[cateye]

ヘッダファイルにCのヘッダ（〜.h）が入っているのは何故でしょう？

[Kinsho]

csv読み込みでいろいろ検索していた時にいろいろincludeした残りですね．元のコードの他の部分を見てもおそらく使っていなさそうなので無視していただいて大丈夫です．

Answer 1

提示されたコードの問題点は、Bx.csvが想定している列数(184)や行数(364)を超えていることをチェックせずに、読み込んだデータを配列の要素に代入していることです。つまり、rowxやcolxの値が2次元配列のそれぞれの要素数以上の値になっても、Br[rowx][colx++] = stod(token);で値を設定してしまいます。

試しに、for文の継続条件を

rowx < 184 && colx < 364 &&
  (spos = linex.find_first_not_of(delim, epos)) != std::string::npos

に変更してみてください。

これでエラーが出ないのであれば、読み込んでいるCSVファイルの内容が「364*184のcsv形式」ではないということなので、CSVファイルの行数や列数をチェックしてください。

Comments

[Kinsho]

ありがとうございます．解決できました．

Answer 2

Br[rowx][colx++] = stod(token); の直前に
cout << "rowx = " << rowx << ", colx = " << colx << ", token = [" << token << "]\n";
をいれて、どこでおかしくなっているか見ましょう。

Comments

[Kinsho]

ありがとうございます！この通り実行したら
rowx = 0, colx = 0, token = [・ｿ0]
と出てきて普通のcsvではなくUTF-8(BOM付き)担っていたことに気づくことができました．

Answer 3

std::getline() のオーバーロード関数に区切り文字まで読み込めるバージョンがあるので、それを使ったほうがカンマ区切りの文字列を解析するのが楽なのではないでしょうか。

getline - cpprefjp C++日本語リファレンス

#include <fstream>
#include <iostream>
#include <sstream>
#include <string>
#include <vector>

using Mat = std::vector<std::vector<double>>;

/**
 * @brief 1行分パースする。
 * 
 * @param line 行
 * @param delim 区切り文字
 * @return std::vector<double> 値の一覧
 */
std::vector<double> parse_line(const std::string &line, char delim = ',')
{
    std::vector<double> values;

    std::istringstream ss(line);
    std::string token;
    while (std::getline(ss, token, delim))
        values.push_back(std::stod(token));

    return values;
}

/**
 * @brief CSV を解析する。
 * 
 * @param path CSV ファイルのパス
 * @return Mat 行列
 */
Mat parse_csv(const std::string &path)
{
    Mat mat;

    std::ifstream ifs(path);
    if (!ifs) {
        std::cout << "Failed to open csv path" << std::endl;
        return mat;
    }
    std::string line;

    while (std::getline(ifs, line)) {
        // 1行分解析する。
        std::vector<double> values = parse_line(line);
        mat.push_back(values);
    }

    return mat;
}

int main(int, char **)
{
    Mat mat;
    try {
        mat = parse_csv("sample.csv");
    } catch (std::exception &e) {
        std::cout << "Error: " << e.what() << std::endl;
        return 1;
    }

    // 出力する。
    if (!mat.empty()) {
        std::cout << "matrix size: " << mat.size() << "x" << mat.front().size() << std::endl;

        for (auto row : mat) {
            for (auto x : row)
                std::cout << x << " ";
            std::cout << std::endl;
        }
    }
}

Comments

[Kinsho]

こちらを使ってみても
debug error！
abort() has been called．
と表示されてしまいます．
問題はプログラムではなくcsvの方でしょうか？

[tiitoi]

例外キャッチするようにコードを修正したので、なにか表示されたら教えて下さい

[Kinsho]

一応コンパイルはできるようになりました！
ただ
実行環境の冒頭にError:invalid stod augumentと出てしまい，
さらに別枠に
Debug Assertion Failed!
Program: C:YWINDOWS ¥SYSTEM32¥MSVCP140D.dll File: CXprogram files (x86)¥microsoft visual studio 14.07vc¥include¥vector Line: 1234
Expression: vector subscript out of range
For information on how your program can cause an assertion failure, see the Visual $\mathrm{C}++$ documentation on asserts.
(Press Retry to debug the application)
というエラーメッセージが出てきますがこれはどういう意味でしょうか？

[tiitoi]

Error:invalid stod augument なので、数値としてパースできないものが CSV ファイルに含まれているのではないでしょうか。
CSV ファイルにヘッダーが入っていたりしませんか？CSV ファイルをどっかにアップロードするか、値の例とかを貼っていただけないと具体例なことはわかりません。

[Kinsho]

何とも不思議なもので，こちらの
Bx.csv
https://drive.google.com/file/d/1vcpLAAZIc6EL4QZ8YTfYthSwZu7UqGWR/view?usp=sharing
ではコンパイルが通りますが，
By.csv
https://drive.google.com/file/d/1hS6E8uwasTsvSsGB-saInZJOY9u2wujq/view?usp=sharing
では　invalid dtod augumentと出てきてしまいます．

これはどのような原因が考えられますか？

[tiitoi]

両方ともアクセス権がないと言われてファイルが閲覧できないです
共有リンクを取得して貼り付けてください
https://hep.eiz.jp/google-drive-share/#_8211

> これはどのような原因が考えられますか？

CSV ファイルに数値として変換できない値がある、CSV のフォーマットが間違っているなどファイルに原因があります。

[Kinsho]

申し訳ありません！
設定を変更致しましたが
Bx.csv
https://drive.google.com/file/d/1vcpLAAZIc6EL4QZ8YTfYthSwZu7UqGWR/view?usp=sharing
Bx.csv
https://drive.google.com/file/d/1hS6E8uwasTsvSsGB-saInZJOY9u2wujq/view?usp=sharing
でどうでしょうか？

[Kinsho]

調べていたところ普通のコンマ区切りのcsvとUTF-8(BOM付き)csvが混在していたようです．ご迷惑をおかけしました．