クラスの中にあるstring配列を初期化する方法

Question

下記のﾌﾟﾛｸﾞﾗﾑにて、1)は、ss1を経由して、string[]を宣言し初期化をしています。
2)は、経由なしで直接初期値を代入しています。
1)2)共にｴﾗｰもなく正常動作していますが、問題は無いのでしょうか？
2)には、static宣言が何処にもありません。

2)の部分に問題は無いのでしようか。
ご存知の方教えていただけないでしょうか？

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        for( int i=0 ; i < mem_com.Length ; ++i) {
            richTextBox1.Text += mem_com[i].aflag +"   " ;
            richTextBox1.Text += mem_com[i].aint +"   " ;
            richTextBox1.Text += mem_com[i].astr +"   " ;
            richTextBox1.Text += mem_com[i].astr_ar[1] +"\n" ;
        }
    }

    public mem_class[] mem_com = new mem_class[]
    {
        new mem_class(false,123,"a111",ss0),
        new mem_class(true, 456,"b222",ss1),                   // 1)
        new mem_class(true, 456,"b222",new string[] {"B000","B111","B222","B333"})//2)
    } ;

    static string[] ss0 = new string[] {"A000","A111","A222"} ;
    static string[] ss1 = new string[] {"B000","B111","B222","B333"} ;

}

public class mem_class
{
    public bool aflag { get ; set ; }
    public int aint { get ; set ; }
    public string astr { get ; set ; }
    public string[] astr_ar { get ; set ; }

    public mem_class(bool cl_aflag,int cl_aint,string cl_astr,string[] cl_astr_ar)
    {
        this.aflag = cl_aflag;
        this.aint = cl_aint;
        this.astr = cl_astr;
        this.astr_ar = cl_astr_ar ;
    }
}

動作環境
Windows7、Visual Studio 2015、C#

ご回答宜しくお願いいたします。

Answer 1

1にはやや問題がある気がします。
問題点はスコープが広く生存期間が長い静的変数ss1を通じてクラス内部の値を変更できてしまうことです。これにより一見するとmem_comと全く関係しないコードがmem_comの内容を書き換えてしまいます。

private void Form1_Load(object sender, EventArgs e)
{
    ss1[1] = "**UPDATE**";  // 一見するとmem_comとは全く関係なさそうなコードだが、実際は動作に影響する

    for( int i=0 ; i < mem_com.Length ; ++i) {
        richTextBox1.Text += mem_com[i].aflag +"   " ;
        richTextBox1.Text += mem_com[i].aint +"   " ;
        richTextBox1.Text += mem_com[i].astr +"   " ;
        richTextBox1.Text += mem_com[i].astr_ar[1] +"\n" ;
    }
}

上の例はわざと書いているので原因が判りますが、想定通り動かないという現象だけからこの原因を探し当てるのが困難であることは想像に難くありません。

Answer 2

問題ない。当たり前すぎて何が疑問なのかよくわかりません。
変数を共有したい、再利用したいなら、 static なり、フィールドで持つのもいい方法ですが、
その場で作ればいいのなら、そのときにnewすればいいというだけの話です。

Answer 3

こんにちは。

動作的には他の方がおっしゃっている通り、特に問題ありません。

もしかして、2)でnewしたものをどこにも保持していないかのように見えているのでしょうか？ 保持されていないと、GCが走れば回収されるので心配と？（違っていたらごめんなさい。）

2)でnewされたものは、仮引数のcl_astr_arに設定され、この仮引数が生きている限り（このコンストラクタからreturnするまで）解放されることはありませんのでご安心を。

Comments

[退会済みユーザー]

有難う御座いました。
開放が心配で･･･
保持されるのであれば、２）を使用します。

Answer 4

問題ないのでは？
というか「ss1を経由して」という方が逆に何の意味があるのかと疑問です。
エラーもなく正常動作が確認できているのに問題があるかも、とは何を懸念されているのでしょうか。