親クラスと子クラスで共通のインスタンスを参照したい場合のベストプラクティスについて

Question

お世話になっております。

実装でちょっと困ったことがあり相談に乗っていただければと思います。

以下は現在書いているプログラムのダミーになりますが、今までstatic classで以下のように共通の変数を持たせていました。
(アクセサなどは適当です。動作確認していませんのであくまで例示コードだと思って補間していただけると助かります)

//Params.cs
static class Params{
  public int a{get;set;}=0;
  public double b{get;set;}=0;

}

//A.cs 親クラス
using static Params;
class A {
   B b;
   double aValue{get;set;}
    
   double someCalc{
       return a+aValue; 
   }

}

//B.cs 子クラス
using static Params;
class B {
   
   double bValue{get;set;}
    
   double someCalc{
    b = someOtherRewriteFunction(); //ｂの値を書き換えるメソッド
       return a+b+bValue; 
   }
}

しかし、この値は、serializeして保存することと最終的にAクラスのインスタンスごとに異なった値をとれた方が可用性がいいことに気づき、親クラスと子クラスから同じインスタンスを参照するように変更しようと思った所,staticでないクラスで書き直したところ以下のようになりました。

//Params.cs
[serializable]
class Params{
  public int a{get;set;}=0;
  public double b{get;set;}=0;
}

//A.cs
class A {
   Params params {get;set;}
   B b;
   
   public A(){
       this.params = new Params(); 
       this.b = new B(params); 
   }

   double aValue{get;set;}
    
   double someCalc{
       return params.a+aValue; 
   }

}

//B.cs
class B {
   Params  params {get;set;}
   public B(Params params){
      this.params = params;
   }
   double bValue{get;set;}
    
   double someCalc{
       params.b = someOtherRewriteFunction(); //ｂの値を書き換えるメソッド
       return params.a+params.b+bValue; 
   }
}

実際はParamsクラスの値を使用するメソッドが大量にあるので、引数を増やさないようにしたかったのでこうなりました(そもそも値もその時点で持っているものを計算で更新するので結局参照でもつ必要があるので引数不要だと思いました。)。この実装で問題点とかないでしょうか？もし他によりよい修正方法があれば教えていただきたいです。

Answer 1

親のポインタを持つ子クラスを作ることは度々ありますから、問題ないんではないでしょうかね。
※シリアライズするって要件が発生するとデフォルトでサポートされていないので色々悩まされますが。笑

しいて言えばそのデザインだとParam`をデシリアライズする際、AとB両方に再設定が必要ってことぐらいでしょうか。親子関係が明確であればネストしてしまっていいと思うんですが、先ほどいったようにネストするのはシリアライズ/デシリアライズを工夫する必要があって面倒です。

下記の形でparamsは後で代入っていうなら問題ないですけどね。

class A {
    private readonly B b;
    Params params { get; set; }

    public A(){
        this.params = new Params(); 
        this.b = new B(this); 
    }

    class B {
        private readonly A parent;
        public B(A parent){
            this.parent = parent;
        }
        double accessingToParentParam() {
            return parent.params.a + parent.params.b;
        }
    }
}

Comments

[退会済みユーザー]

確かに再代入の可用性考えるとインナークラス毎シリアライズした方が楽ですね。親のモデルの参照毎子に渡してしまうというやり方はrubyではよくやるのにC#だと盲点でした。
この実装いいですね。ありがとうございます！

[haru666]

java等でサポートされている親ポインタの参照っていうのがC#には無いので、このデザインで子クラス毎シリアライズするのは意外と面倒…だった気がします。たぶん。今なら良い解決策が出てるかもしれませんが。

[退会済みユーザー]

なるほど。試したことないので、検証してみますね。
最悪Paramsだけシリアライズして引数で引きずり回せばいいので抱えていた問題は解決しそうです。
ありがとうございました。