C++において構造体にcsvファイルから読み込んだ値を定義したい

Question

tCarCtrl構造体にcsvファイルから読み込んだ値（ary[0]，ary[1]）を定義したいと考えています。
以下の様にコードを考えたのですが,　エラーが発生してしまっています。
対策についてどなたか教えていただけないでしょうか。
宜しくお願い致します。

開発環境
Windows8.1 Pro (64bit)
Excel 2016 (64bit)
Visual Studio 2017

対象とするコード

typedef struct {
	FILE *file1;
	char yy[256];
	char *ary[2];
	file1 = fopen("output.csv", "r")
	/*ファイルの終わりまで繰り返し読み込む*/
	while (fgets(yy, 256, file1) != NULL)
	{
		printf("%s\n", yy);
		ary[0] = strtok(yy, ",");
		ary[1] = strtok(NULL, ",");
	}
	fclose(file1);

} tCarCtrl;

発生しているエラーの例

while (fgets(yy, 256, file1) != NULL)
　⇒エラーC2059	構文エラー: '='	

{
　⇒エラーC2334	'{' の前に予期しないトークンがありました。関数の本体は無視されます

fclose(file1);
　　⇒エラーC2061　構文エラー: 識別子 'file1'	
　　　エラーC4430　型指定子がありません - int と仮定しました。メモ: C++ は int を既定値としてサポートしていません	

Comments

[cateye]

fopen()のエラーチェックは？・・・そもそもメソッドでもないのに、実行文が有るのはなぜでしょう？

また、構造体ではなくエラーになるソースを貼り付けて下さい。
エラーの出る行が間違っている可能性もありますが、そこまでのソースに誤りが有る場合もあります。

[muton]

・fopenの行ではエラーが発生していません。
・typedef structで実行分を行うこと自体が間違いなのでしょうか？tCarCtrl構造体にcsvファイルから読み込んだ値へのポインタを追加する必要があるのですが、その方法が全く分からなくて、上記のコードを書きました。

素人質問で申し訳ございません。ご教授頂けると非常に嬉しいです。
エラーコードに関しては追加します。

[cateye]

>fopenの行ではエラーが発生していません。・・・コンパイルエラーの事ですか？
まぁ、コンストラクタはおいておいてｗ　メソッドも何も定義しないでいきなりfopen()出来ると思いますか？そこに制御が渡ることはないですよ。

Answer 1

C++らしく書いてみた。

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

コレ↑(output.csv) を コレ↓に食わすと

#include <iostream> // cout, endl
#include <fstream>  // ifstream
#include <array>    // array
#include <string>   // string
#include <cstring>  // strtok()
#include <vector>   // vector

struct tCarCtrl {
  std::array<std::string,2> ary;

  tCarCtrl(const std::string& line) {
    static char yy[256];
    strcpy(yy, line.c_str());
    ary[0] = strtok(yy, ",");
    ary[1] = strtok(nullptr, ",");
  }
};

std::ostream& operator<<(std::ostream& stream, const tCarCtrl& car) {
  return stream << '(' << car.ary[0] << ',' << car.ary[1] << ')';
}

int main() {
  using namespace std;

  vector<tCarCtrl> cars;
  {
    ifstream file("output.csv");
    if ( !file.is_open() ) return 1;

    string line;
    while ( getline(file, line) ) {
      cars.emplace_back(line);
    }
  }

  for ( const auto& car : cars ) {
    cout << car << endl;
  }
}

コレが出てくる。

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

Comments

[muton]

回答ありがとうございます。
参考にさせて頂き、コードを組んだ結果、csvファイルから読み込んだ値を定義できたため、ベストアンサーとさせて頂きます。

Answer 2

char *ary[1];

と定義してるなら、

    ary[1] = strtok(NULL, ",");

範囲オーバしてますぜ

んで、そもそもtypedef struct でするような内容ではないですな。
そこんとこどーにかしましょう

Comments

[muton]

回答ありがとうございます。
typedef struct以外でこのコードを書いた場合、エラーも起きず、
csvファイルの一行目をary[0]と二行目をary[1]と定義できていた(printfで確認済)のですが、範囲オーバーしてしまっているのでしょうか。

別件（読み込んだ値をシュミレーション画面上に描画するため）でtCarCtrl構造体にcsvファイルから読み込んだ値を定義する必要があるのですが、typedef struct外で読み込んでtypedef struct内で定義すれば良いのでしょうか？

素人質問で申し訳ございません。ご教授頂けると非常に嬉しいです。

[y_waiwai]

範囲オーバはコンパイル時にはエラー出ません
ただ、まが悪ければ暴走するだけです

typedef というのは型の再定義を行う命令です
そーゆー使い方はできません

[muton]

回答ありがとうございます。
tCarCtrl構造体にcsvファイルから読み込んだ値へのポインタを追加する必要があるのですが、それは可能なことなのでしょうか？

何度も申し訳ございません。宜しくお願い致します。

[y_waiwai]

どんなポインタでも追加するのは可能です

＃それがなにを指しているのか、というのをしっかり把握しておきましょう

Answer 3

こんにちは。

C++は構造体の中に関数を定義することはできますが、関数定義でない実行文を直接書くことはできないですよ。きちんと関数を定義しましょう。（定義方法は基礎ですので一度ご自身で調べてみて下さい。）

また、char *ary[1];は要素数が 1個 の配列を定義しています。2個必要なら、char *ary[2];と定義しましょう。

それと蛇足かもしれませんが、typedef struct {・・・} XXXX; は古い構文です。C++でこの形式の記述はあまり見かけません。struct XXXX {・・・};形式で記述することがほとんどです。

Comments

[muton]

回答ありがとうございます．
・不勉強で申し訳ございません。定義方法について調べてみます。
・char *ary[1]→char *ary[2]の様に変更します。
・研究で扱うオープンソースのカーシュミレーションのコードをいじっているという現状なのですが、古い構文が混じってしまっているのかも知れません。

[muton]

別件（読み込んだ値をカーシュミレーション画面上に描画するため）でtCarCtrl構造体にcsvファイルから読み込んだ値へのポインタを追加する必要があるのですが、それは可能なことなのでしょうか？

素人質問で申し訳ございません。宜しくお願い致します。

[Chironian]

> それは可能なことなのでしょうか？
問題提議があいまいなので良く分からないですが、C++はC言語としても使えますので、恐らく可能です。

そのデータやポインタは可変個存在すると思いますが、それぞれの記憶領域をどのようにして確保するのか？などいくつかの課題があります。（std::vectorやstd::list等が検討対象になると思います。）