C# と VB.NET の質問掲示板

開発環境:cint-5.16.19-win32
使用言語:C

C言語で、グローバル変数をなくして関数化したいです。
下のように考えましたが、read と write で変数を
共有化する方法が思いつきません。
何とかグローバル変数を使わずに、できないものでしょうか。
enum
global_variable
{
  VAR1 = 0,
  VAR2
};
int global_read(int name)
{
  static int var1 = 1;
  static int var2 = 2;
  switch(name)
    {
    case VAR1:return var1;
    case VAR2:return var2;
    default: return 0;
    }
}
void global_write(int name, int value)
{
  static int var1 = 1;
  static int var2 = 2;
  switch(name)
    {
    case VAR1:var1 = value;break;
    case VAR2:var2 = value;break;
    default: return;
    }
}
int main(void)
{
  int a = 0;
  int b = 0;
  a = global_read(VAR1);
  b = global_read(VAR2);
  global_write(VAR1, 3);
  global_write(VAR2, 4);
  return 0;
}

うーん...どぉしよっかなー...

typedef struct {
  int iVal;
  long lVal;
} global_t;

global_t* get_global() {
  static global_t* instance = NULL;
  if ( instance == NULL ) instance = malloc(sizeof(global_t));
  return instance;
}

// 最後に一回、必ず呼ぶべしっ!
void destroy_global() {
  free(get_gloval());
}

// つかいかた
int main() {
  long n = get_global()->lVal; // read;
  get_global()->iVal = (int)n; // write;
  destroy_global(); // 忘れんなやｺﾞﾙｧ!
  return 0;
}

コレ↓でも構わんですけどね。

typedef struct {
  int iVal;
  long lVal;
} global_t;

global_t* get_global() {
  static global_t instance;
  return &instance;
}

// つかいかた
int main() {
  long n = get_global()->lVal; // read;
  get_global()->iVal = (int)n; // write;
  return 0;
}

> あ、malloc を使う必要はないと。
> こっちの方がすっきりしていていいですね。

そう断じるのは早計かと。case-by-caseでしょう。
初期値を与えたいなら malloc の方が楽だったりします。

global_t* get_global() {
  static global_t* instance = NULL;
  if ( instance == NULL ) {
    instance = malloc(sizeof(global_t));
    instance->iVal = 123;
    instance->lVal =456;
  }
  return instance;
}

全体的になにをやりたいのかよくわからんけど、俺なら素直にグローバル変数にするだろう、と思う・・・
カプセル化もなにもないので、誤使用に対する安全性が増しているとは思えないし
性能的にただ無駄なことやっているだけと思われるし
全体的に見通しがよくなったか？と問われるとそうは思えないし

C++ でいうところの悪例
class getset_bad_example {
  int value;
public:
  void set(int v) { value=v; }
  int get() const { return value; }
};
と何一つ変わらんわけで・・・

C で終了時処理が必要なら atexit が使えるよん。

C++なら名前空間 global で囲むだけかも。
namespace global {
  int iVal;
  long lVal;
}
※ ただし糸色文寸に using namespace global 書いちゃダメ!

その前にグローバル変数をできる限り減らすよう努力しますが。