C# クラス配列の宣言方法

Question

###前提・実現したいこと
CからC#へ置き換えをおこなっています。
構造体をクラスに置き換えていますが
入れ子になっている場合、
下記宣言方法が問題ないのかアドバイスをいただけないでしょうか。

特にエラー等発生なく使用できましたが、間違った考え方だといけないので
質問させていただいています。

この構造体の用途は
グローバル変数として宣言し、ファイルから読込んだ値を流しこみ、
アプリ起動中常時参照します。

###該当のソースコード

typedef struct{
   int  A;
   int  B[10];
   int  C[10];
}STR_A;

typedef struct{
   char   D[256];
   STR_A  E[100];
}STR_B;

typedef struct{
   STR_B  F[200];
   STR_B  G[200];
}STR_C;

public class CLS_A
{
   public int A;
   public int[] B = new int[10];
   public int[] C = new int[10];
}

public class CLS_B
{
   public string D;
   public CLS_A[] E = new CLS_A[100];

   public CLS_B()
   {
      int i;
      for (i = 0; i < 100; i++)
      {
         E[i] = new CLS_A();
      }
   }
}

public class CLS_C
{
   public CLS_B[] F = new CLS_B[200];
   public CLS_B[] G = new CLS_B[200];

   public CLS_C()
   {
      int i;
      for (i = 0; i < 200; i++)
      {
         F[i] = new CLS_B();
         G[i] = new CLS_B();
      }
   }
}

Answer 1

int の宣言は、forの中に入れられる。forの中だけでしか使わない変数なので、こっちのほうが絶対正しい。

      for (int i = 0; i < 200; i++)
      {
         F[i] = new CLS_B();
         G[i] = new CLS_B();
      }

あと、初期化はこんな感じにかくのもありでは？
とか思いますけど。

F = Enumerable.Range(1,200).Select(_=>new CLS_B()).ToArray()

あと、パブリックのものがただのフィールドで、プロパティではないですね。
これもあまり良くありません。
クラスの外から見えるものは、プロパティを使う、というのが一応作法です。直接触れるようにはしない。

そのため、

public class CLS_C
{
   public CLS_B[] F {get; private set;} = Enumerable.Range(1,200).Select(_=>new CLS_B()).ToArray();
   public CLS_B[] G {get; private set;} = Enumerable.Range(1,200).Select(_=>new CLS_B()).ToArray();
}

こんな感じ書くのがいいのですかね。
外から、Setすることがないので、たぶん、privateをつけるべきかな。
元のコードだと、ノーガードで外から変更できるので、ちょいとやばいと思いますね。
予め防げるものは防ぐが原則なので、外からの変更は、メソッドを介すという原則がいいと思います。

追記
Enumerable.Repeatの仕様を勘違いしていたので修正

Comments

[MMashiro]

実際にあまり使ったことがないので間違えていたらすいません

F = Enumerable.Repeat(new CLS_B(), 200).ToArray();
こちらの書き方だと実体は一つだけで配列の中には第１引数のインスタンスへの参照が200個生成されるとかだった気がします
CLS_B型が参照型の為に発生する問題なのでCLS_B型をstruct(値型)に変更して問題ないのであればstructにすることで回避可能です

[ozwk]

どうしてもLinqしたいなら
Enumerable.Range(1,200).Select(_=>new CLS_B()).ToArray()
とかどうでしょう

[kiichi54321]

お二人方、正しいですね。私は、Repeatの仕組みを勘違いしていました。

[rkw4a]

回答ありがとうございました。

Answer 2

自分的にですが（あくまで自分的にです）、.NET Framework の C# にパブリックフィールドや配列を使うのはピンときません。

可能であれば、代わりにパブリックプロパティ、List<T> 型を使うということも考えてはいかがでしょう？

具体的には以下の通りです。質問者さんの目的には合わないかもしれませんが、ご参考まで。

public class CLS_A
{
   public int A { get; set; }
   public List<int> B { get; set; }
   public List<int> C { get; set; }

   public CLS_A()
   {
      this.B = new List<int>();
      this.C = new List<int>();
   }
}

public class CLS_B
{
   public string D { get; set; }
   public List<CLS_A> E { get; set; }

   public CLS_B()
   {
      this.E = new List<CLS_A>();
   }
}

public class CLS_C
{
   public List<CLS_B> F { get; set; }
   public List<CLS_B> G { get; set; }

   public CLS_C()
   {
      this.F = new List<CLS_B>();
      this.G = new List<CLS_B>();
   }
}

Comments

[rkw4a]

配列よりlistのほうが利便性が良いので、通常(処理速度やメモリを気にしない)であればlistを優先して使用するという考えでよいのでしょうか？

[退会済みユーザー]

使いやすさ優先で考えれば良いと思います。実際、処理速度やメモリが問題になるということはまずなさそうですし。

[rkw4a]

回答ありがとうございました。

Answer 3

クラスの基本的な宣言としては問題ありませんが、アプリから常時参照する（唯一の値）のであればstaticクラスにすることをお勧めします。
C#でのポイントは、常時唯一の値を保持したい場合にはstaticを使用することまた名前空間も重要ですので一度調べては如何でしょうか？
＜追記＞
初めてなもので、ソースが下にまだ隠れていたのを見逃していました。
補足ポイント
①上のCでは、構造体の宣言のみですがC#では、「new CLS_A()」で値を放り込んでいますね。実際に使うときに値を入れるべきではないでしょうか？
②C#もCの構文と同じようにfor文の中で（int i = 0; i < 100; i++)とできますよ。

Comments

[rkw4a]

staticクラス参考になりました。

Answer 4

まったく問題ありません。