C# ループの中で変数の初期化をする場合のメモリの負荷や処理速度について

Question

オープンソースのコードで遭遇したものなんですが、変数の初期化とメモリについて基本的なことも含めて確認したいことがあります。

whileループの中で変数の初期化と値のセットを行い、直後にもう一度値のセットをしているコードを見かけました。
この直後に行う値の再配置には何か意味があるのでしょうか？
メモリの負荷軽減や処理速度に関わるのではないかと考えました。

public Thread Tobj;

public void start()
{
  if (Tobj == null)
  {
    Tobj = new Thread(new ThreadStart(prc));
    Tobj.Start();
  }
}

private void prc()
{
  while (true)
  {
    int a = getRaw();
    a = getRaw(); //この部分は必要？

    if (a > 0)
    {
      Console.WriteLine(a);
    }
  }
}

また変数の初期化だけループの外で行った場合もメモリの負荷に違いなどはあるのかも疑問に思いました。

private void prc()
{
  int a;
  while (true)
  {
    a = getRaw();

    if (a > 0)
    {
      Console.WriteLine(a);
    }
  }
}

変数の初期化直後の値の再配置に意味はあるのか？
またループの外で変数を初期化してループ内では値の設定だけをする場合との違いはあるのか？
以上の2つが知りたいです。基本的な変数とメモリの負荷について理解できていない状態ですがご教示お願いします。

Answer 1

私も getRaw()が分かりません。名前からして、Rawデータ読み込みでしょうか?
最初のデータを読み捨てという意味でしょうか? それならば、アリ。

2つめの変数の初期化ですが、これは変数の宣言ですね。
初期の C(C#の元)は、関数の最初でしか、変数を宣言できなかったのでその名残かと思います。あるいは、ループの外で値を参照したいかのどちらかでしょう。

その辺しか、思いつきません。
また、この程度は、コンパイル時の最適化を考慮した場合、誤差としか思えません。
分かり易さが優先の範囲と考えます。

Comments

[gucha]

getRaw()では入力されている物理キーを確認して対応する値を返すというものでした。
常に監視させておくのでスレッドを分けているのだと思いました。
コンパイルで最適化されるということも知りませんでした。

Answer 2

getRaw() が何をやっているのかもわかりませんが、意味があるとは思えません。
新しいスレッドでコンソールに数値を出力するのも何の意味があるのかわかりません。

オープンソースということですが、オープンソースのソフトはだれでも作れますので品質が良いとは限りません。手本になるソースとは限らないでしょう。
メモリの負荷軽減や処理速度には寄与しないと思います。

###追記
ループの外での変数の宣言にもループの外で利用していない以上、意味がありません。利用していないスコープが変わるだけで IL レベルでは同じものになります。

Comments

[gucha]

やはり今回のケースでは特に差異はないのですね。
回答ありがとうございます。

Answer 3

初期化直後に代入しているのはちょっとソースみないと分かりませんね

後者はループの外で変数を宣言することでスコープが変わります。