ファイルを管理するクラスの内容をどうすべきかわからない

Question

前提・実現したいこと

c++でゲームを作っています。
テキストファイルのデータを読み込んで変数に格納する必要がでてきたので、新しくファイルに関する仕事をするクラスを用意しようと思ったのですが、どこまで仕事を任せれば良いかわかりません。

例えば、データを格納したい変数を参照渡ししてそのクラスの中でファイルオープン、データの読み込み、格納を行うか、ファイル名を渡して、そのクラスの中でファイルオープンだけし、ifsteam型の変数を戻り値として受け取り、呼び出し元のクラスでデータの読み込み格納をするかと言った感じです。

普通はどのようにするのでしょうか。

また、今後データの保存も行う予定なのですが、この機能はファイルを読み込むクラスに追加すれば良いのでしょうか。

Answer 1

• データを管理するクラス(仮にgame_dataクラスとする)
• データを管理するクラスをファイルから作るフリー関数(std::basic_istream&を受け取ってgame_dataを返す)
• データを管理するクラスからファイルを作るフリー関数(std::basic_ostream&とconst game_data&を受け取る)

を作ればいいんではないかと

Comments

[inkmg]

回答ありがとうございます。フリー関数を使うということは考えていませんでした。

Answer 2

こんにちは。

ある程度決まった処理があるなら、そのカスタムなファイル管理クラスにその「決まった」処理を任せてしまえば、そのファイル管理クラスを使う人は「決まった処理」に煩わされずに使えるので楽ですし、バグも減らせます。

しかし、その決まった処理がほとんどないなら、fstreamをそのまま使えばよいです。後者の例は正にこちらに該当します。
fstreamのコンストラクタへパラメータをパススルーするだけの関数を別途作る意味はほぼありません。auto ofs = std::ofstream("foo.txt");は多くの人が意味を把握できます。auto ofs = create_file("foo.txt");と書かれていた場合、create_file()関数の定義を見に行く必要があります。そこに、return std::ofstream(param);なんて書かれていたらがっかりです。怒りさえ覚える人も居ると思います。

Comments

[yumetodo]

怒りさえ覚えるｗ
まあわかる。

[inkmg]

回答ありがとうございます。決まった処理はないので、今回はfstreamを使おうと思います。また今後無駄な関数を作らないよう気を付けたいと思います。

Answer 3

どこまで仕事を任せれば良いかわかりません。

設計次第。要件次第です。
任せるのは作る人の仕事です。
まず全ての仕事を洗い出し、バランスや再利用性などを考えて
自身で与えて行くと良いです。
「これが答え」というのが存在しないので、実際の要件と照らし合わせ、
調整していくしかないです。
どんなに熟練者であってもいきなり100%のものはできません。
1つ1つ調整していくものです。

Comments

[inkmg]

回答ありがとうございます。柔軟に思考しなければならないのですね。プログラミング初心者なので、これからそのような力をつけられるよう頑張ります。

Answer 4

c++でゲームを作っています。

シリアライズをどこまでこだわるかによると思いますが。
個人的にはjson11を押しておきます。