ofstreamオブジェクトをグローバル変数として扱いたい

Question

ofstreamのオブジェクトを複数の関数で共有して扱いたいのですが、どうすれば良いでしょうか。イメージとしては以下のとおりです。

void hoge(void){
  for (int i = 0; i < 5; i++){
    fstream ofs("test" + to_string(i) + ".txt");
    hoge1();
    hoge2();
    hoge3();
    ofs << endl;
  }
}

void hoge1(void){
  ofs << 1 << " ";
}
void hoge2(void){
  ofs << 2 << " ";
}
void hoge3(void){
  ofs << 3 << " ";
}

Answer 1

ofstreamオブジェクトをグローバル変数として扱いたい

というならば，単にグローバル変数にすれば良いのではないでしょうか？

//グローバル変数
std::ofstream ofs;

void hoge(void){
  for (int i = 0; i < 5; i++){
    ofs = ofstream("test" + to_string(i) + ".txt");
    hoge1();
    hoge2();
    hoge3();
    ofs << endl;
  }
}

ただし，目的に関して言えば，単純にこんなの↓で事足りる気がします．

void hoge1( std::ostream &OS ){
  OS << 1 << " ";
}

Comments

[mmtaro000]

ありがとうございます。助かりました。

Answer 2

グローバル変数を使うのではなく、引数や戻り値を利用しましょう。

「C言語 関数」とかで調べてください。

グローバル変数でも確かに実装自体はできます。ですがバグの温床になりやすいです。

自分ひとりでやるならいいですが、普通、会社レベルになるとチーム開発になります。
で、仮に簡単なプログラムとして、「入力値をテストの点数として仮定し、80点以上なら合格、それ未満なら不合格と表示するプログラム」を作るとします。

そして、グローバル変数 point (= 点数) を用意したとして、これにアクセスする場合、
グローバル変数はどこからでもアクセスが可能です。

よって、想定しない場所でデータの書き換えが行われる可能性があります。
確かに引数や戻り値も書き換え可能ですが、デバッグ時にはデータの状態や設定・取得している範囲を見ればいいだけです。

でも、グローバル変数だと、関係のない関数からでも触る事ができますから、「全ての範囲」を対象に調査しないといけません。

1人で作るなら勝手にすればいいのですが、1人で作る場合でも一か月前の自分が作ったものをマジで忘れているってこともザラです。(読めばわかるが、細かい部分までは...)

プログラミングは書いて終わり…ではありません。デバッグや保守も必要です。

後の事を考えると、やはりグローバル変数は使うのはやめておいた方がいいです。

ただし、Windows APIとかで複数のウィンドウプロシージャ(他の言語やライブラリだとイベントに相当するものを処理するやつ) で使い回す場合とか、競技プログラミングとかであえて使うってことはあります。

ですが、安易にグローバル変数を使うとバグの温床になり、修正困難に陥る可能性もありますから、使わない方が無難です。

なので、引数や戻り値によって対処しましょう。

書き方は調べてください。入門書とかにも載っていることです。

Comments

[mmtaro000]

「グローバル変数でも確かに実装自体はできます。ですがバグの温床になりやすいです。」
では実装方法を教えていただけますか？
仮にバグが起こったとしてもこちらの責任ですので、気兼ねなく教えていただければ幸いです。

[dodox86]

> グローバル変数でも確かに実装自体はできます。ですがバグの温床になりやすいです。

低評価をしたのは私ではありませんが、そこを押しての技術的な質問なのだと思います。例えば、std::coutのような使い方をしたいのかと。

[dodox86]

例えばログ出力のような、アプリケーション全般にわたる横断的な処理ではそのような使い方が決して無いこともないです。

[fana]

グローバル変数にする実装を私の回答の方に追記しました．
（単に，ローカルじゃない場所に定義すればいいだけっすよね？）

[mmtaro000]

すみません、解決しました。皆様解答ありがとうございました。