【C++】.csvファイルを配列に読み込み、行、列を取得したい

Question

前提・実現したいこと

私は現在、csv化したファイルを二次元配列に読み込み、指定した値と一致するデータの行数、列数(Excelで開いた時のような)を取得するプログラムを組みたいと考えています。そこでこちらの質問(https://teratail.com/questions/94162)を参考に、「csvファイルを読み込み、二次元配列に格納、それを配列から.csvに出力して格納を確認する」として以下のプログラムを書いたのですが、現在2つの問題が発生しています。

①下のファイルのeCsvの部分でエクスポートされたcsvファイルのデータの大部分が0に置き換えられてしまう(添付画像参照)
②①が解決し二次元配列への格納ができたとして、一致したデータの行数、列数をどう取得すればいいのかわからない

最終的にはこれを400×400のcsvファイルで利用したいと考えています。
まだまだ初学者ですので、お知恵を貸していただけると助かります。

###添付画像

###開発環境
Sublime Text 3
MinGW g++

該当のソースコード

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

#define MAP_HEIGHT 10
#define MAP_WIDTH 10

using namespace std;

int m_Map[10][10]; // csvデータ格納

int stoi(string str){ // stoi関数の定義
	int ret;
	stringstream ss;
	ss << str;
	ss >> ret;
	return ret;
}

void iCsv(){ // csvファイルのインポート
	
	ifstream ifs("Original/Test.csv");
	string line;
	int i;

    i = 0;

	while(getline(ifs, line)){
		
		string token = ",";
		istringstream stream(line);
		
		while(getline(stream, token)){
			m_Map[i/MAP_HEIGHT][i%MAP_WIDTH] = stoi(token.c_str());
			i++;
		}

	}

}

void eCsv(){ //正しく入力されたかどうか確認
	ofstream ofs("Converted/Test.csv");//ファイルの出力ストリーム
    	for (int i = 0; i < MAP_HEIGHT; i++){

	        for (int j = 0; j < MAP_WIDTH; j++){
        	    int num = i*j;
    	        ofs<<(int)m_Map[num / MAP_HEIGHT][num% MAP_WIDTH]<<"," ;
	        }

        ofs << endl;

    }

    ofs.close();

}

int main(){
	iCsv();
	eCsv();

	return 0;
}

Comments

[yumetodo]

そもそもstoi関数は自作しないでもC++11にあります。普通にstd::stoiとしてみてください。

[退会済みユーザー]

ご指摘ありがとうございます。私も最初はそのようにしていたのですが、エラーが発生してしまったため定義しています。私の環境のC++のバージョンが古いのかもしれません・・・。

[yumetodo]

お使いの環境を追記してください。gcc/clangなら-std=c++11をつければ利用できるはずです。

[退会済みユーザー]

環境追記しました。コンパイラはMinGW,g++です。

[yumetodo]

だいぶ前のmingw gccでも-std=c++11は使えるはずなのでどうぞ

[退会済みユーザー]

ありがとうございます。分かりました、やってみようと思います。

Answer 1

#include <iostream>
#include <string>
#include <fstream>

int main() {
  using namespace std;
  ifstream stream("test.csv");
  string line;
  int data[10][10];
  const string delim = ",";
  
  int row = 0;
  int col;
  while ( getline(stream, line) ) {
    col = 0;
    // delimを区切り文字として切り分け、intに変換してdata[][]に格納する
    for ( string::size_type spos, epos = 0;
        (spos = line.find_first_not_of(delim, epos)) != string::npos;) {
      string token = line.substr(spos,(epos = line.find_first_of(delim, spos))-spos);
      data[row][col++] = stoi(token);
    }
    ++row;
  }

  // よめたかな?
  for ( row = 0; row < 10; ++row ) {
    for ( col = 0; col < 10; ++col ) {
      cout << data[row][col] << " ";
    }
    cout << endl;
  }
}

コレ↑にtest.csv↓を食わせてみた

0,1,2,3,4,5,6,7,8,9
1,2,3,4,5,6,7,8,9,0
2,3,4,5,6,7,8,9,0,1
3,4,5,6,7,8,9,0,1,2
4,5,6,7,8,9,0,1,2,3
5,6,7,8,9,0,1,2,3,4
6,7,8,9,0,1,2,3,4,5
7,8,9,0,1,2,3,4,5,6
8,9,0,1,2,3,4,5,6,7
9,0,1,2,3,4,5,6,7,8

実行結果
0 1 2 3 4 5 6 7 8 9
1 2 3 4 5 6 7 8 9 0
2 3 4 5 6 7 8 9 0 1
3 4 5 6 7 8 9 0 1 2
4 5 6 7 8 9 0 1 2 3
5 6 7 8 9 0 1 2 3 4
6 7 8 9 0 1 2 3 4 5
7 8 9 0 1 2 3 4 5 6
8 9 0 1 2 3 4 5 6 7
9 0 1 2 3 4 5 6 7 8

Answer 2

こんばんは。

まだ初学者ということなので、まずはプログラムがうまく動かない場合の調べ方をお伝えします。

想定した挙動が得られない場合は、必ず「デバッグ」をしてみましょう。
一行ずつプログラムを動かしていく（ステップ実行といいます）と、どこでデータが0に書き換えられているのかがきっと見つかります。
どのような開発環境かは存じ上げませんが、大体はデバッグ実行中に変数の値を確認できます。
「ウォッチ」と呼ばれる機能です。
これをちゃんと使えば、どこに問題があるかがきっと見つかります。

このように短いソースコードなら、頑張れば必ずどこが悪いかがわかるはずです。

ちなみに、今回は関数のリファレンスと型変換がポイントになると思います。
関数のリファレンスとは、関数の引数と戻り値がどのようになっているか、という情報です。
プログラムをする上で、絶対に切り離せない概念です。
関数を使う際は、必ずその関数のリファレンスを確認してください。
C++において変数の型は非常に重要ですので、この機にきっちりと身につけましょう。
（なぜ0になるのだろう、という疑問があると思いますが、これはこの変数の型に起因しています。）
それと、よく使う関数は誰かが作ってくれていたり、標準であったりもしますよ。
プログラムは効率よく、楽できるところは楽をするようにしていきましょう。

追記）
②の解答ができていませんでした。
取得した結果をどのように出力するのかがわかりかねますが、一致している個所を探すのは簡単です。
行および列でループ処理を組んで、その中でif文で比較してやればよいだけです。
行数や列数は変数をループごとにカウントアップ（インクリメント）してあげれば、取得できますね。

Comments

[退会済みユーザー]

回答ありがとうございます。自分で調べたり、修正した結果、1行だけ正しく出力され、残りはみんな0になる、という状況になってしまいました。使用している環境はコードエディタにSublime Text、コンパイラにMinGWのg++、といった状況なのですが、使いこなせていない部分が多く、ステップ実行のやり方がよくわかりません・・・申し訳ないのですが、更に詳しいご指摘をいただきたいです。よろしくお願い致します。

追記)修正箇所について
eCsv内、jについてのfor文の部分を、

for (int j = 0; j < MAP_WIDTH; j++){
ofs << (int)m_Map[i][j] << ",";
}

としました。

[BellBlue]

お疲れ様です。
アウトプットは正しく動きそうになりましたね。

コンパイルするときのコマンドに、オプションとして -g を加えてください。
これによってできあがるexeには、「デバッグ情報」が付加されます。
このデバッグ情報を解析して表示してくれるのが「デバッガ」です。
MinGWをお使いとのことですので、「gdb」というデバッガを使用することになります。

次に、デバッガをどうやって起動するかという点です。
もちろんコマンドを叩いて動かせますが、色々と不便です。
そこで、お使いのエディタからデバッガを起動してやりましょう。

ただ、申し訳ないことに私はsublime textを使ったことがないので、具体的にご説明できません、、

お使いのエディタがgdbに標準対応しているかどうか調べて、対応していないならプラグインを入れて、デバッガを動かしてください。
これでステップ実行できるはずです。

最後に。
最新のソースは、一行目だけ正しく出力できているわけではないのでご注意ください。
偶然、そのように見えているだけです。
プログラミングでよく陥る罠です。。
「ここが悪いはず！」と思ってしまうと、他に注意が向かず、解決に時間を費やしてしまいます。
そして、どうしてもわからなかったら、インプットのCSVの値を不規則にしてみてください。

[退会済みユーザー]

返信ありがとうございます。gdbはプラグインがある、とのことなので、早速調べながらインストールしようと思います。

また、出力についてですが、本来1列目に並んでいる数字が1行目に並んでいるのではないか、と考えたのですが、合っているでしょうか。とすると、これは入力時に何かしらのエラーが起きている、と考えるのが妥当…そう考えると、iCsvのgetline関数あたりに原因があるのでしょうか？

[BellBlue]

もう解決されたかもしれませんが…
仰るとおりで、getline関数を調べていただければ原因は見つけられると思います。

[退会済みユーザー]

ありがとうございます。調べてみたところ、なんとかできそうです。丁寧なご教示ありがとうございました。