C#でクラスのメソッドから親？クラスのフィールドに値を格納したい

Question

ClassAのフィールド値ValAをClassBのメソッドから値をsetできるでしょうか？

class ClassA {
 public ClassB ClB {get; set;}
 public int ValA {get; set;}  // ★ClassBのSetClassAvalueメソッドから値をsetしたい
}

class ClassB {
 public void SetClassAvalue(int value){
  // ★ここからvalueの値をsetしたい
 }
}

ちょっと意味不明かもしれませんが、

ClassA clA = new ClassA();
clA.ClB.SetClaassAvalue(3);

を実行するとclA.ValAに3が格納されるようにしたいのです。

どうぞよろしくお願い致します。

開発環境：Windows10 VS2015 C# .NetFramework4.5

Answer 1

その設計がおかしいです。
ClassAにClassBを継承させてパラメータを保持するか、
ClassAを経由してClassBにアクセスするのが良いです。

どうしてもしたければSetClaassAvalueにClassAのオブジェクトを渡すしかないのでは？
リフレクションする意味も分かりませんし。。。

Comments

[退会済みユーザー]

ありがとうございます！
たしかになんとなく設計がおかしい気がしていました・・・。
こんなふうな使い方もできるのかな？という疑問から質問させていただきましたがやはりおかしかったのですね。

[himakuma]

プロパティにしている意味がなくなりますし。。。

[himakuma]

やりたい内容しだいですが、ClassA、ClassBを持つClassCを作ってもいいかもですね

[退会済みユーザー]

ありがとうございます！
そうですね、ちょっと設計を見直そうかと思います。
ClassCを作ると全般的に変えないといけなくなるので・・・。

[himakuma]

ishi9さんにコメントした内容を見ると平均値を算出する共通の継承クラスを作った方が良い気がしますが？

[himakuma]

拡張メソッドを使っても良いかもです。（速度を求めるならやめた方が良いです）
もしかしてUnityですか？？

[退会済みユーザー]

再度ありがとうございます！

>平均値を算出する共通の継承クラスを作った方が良い気がしますが？

うう・・すみません、そもそも設計を変える前提なのですよね。
やはり最初にご回答いただいた「ClassAを経由してClassBにアクセスするのが良いです。」がシンプルで良いような気がしています。
あとUnityではないです。

[himakuma]

ClassAをstaticで保持は駄目ですよね？

[退会済みユーザー]

今のところそうですね・・・。

Answer 2

こんにちは。

他の方たちがおっしゃっているようにやりたいことをもう少し整理されたほうが良いとは思います。
とりあえず実現には子から親へ変更イベントを起こすのが簡単だと思います。

class ClassA
{
    public ClassB ClB { get; set; }
    public int ValA { get; set; }  // ★ClassBのSetClassAvalueメソッドから値をsetしたい

    public ClassA()
    {
        ClB = new ClassB();
        ClB.SetAvalue += (value) =>
        {
            ValA = value;
        };
    }
}

class ClassB
{
    public event Action<int> SetAvalue;
    public void SetClassAvalue(int value)
    {
        // ★ここからvalueの値をsetしたい
        this.SetAvalue?.Invoke(value);
    }
}

Comments

[退会済みユーザー]

ありがとうございます！
とりあえず質問内容からいえばイベント発生が簡単なんですね！
まずは設計を見直してみて勉強としてイベントもやってみようと思います。

Answer 3

その例だけだと、下記のように書けばいいのでは？で話が終わってしまいます。

clA.ValA = 3;

「こう書けないのか？」の前に「こういう事がしたい」というのがあるはずなので、
それをちゃんと質問に書いていただけると、「ならこういうテクニックがあるよ」と回答しやすいのですが。

Comments

[退会済みユーザー]

ありがとうございます！
やりたいことがあるのですが、その前にそもそも質問の内容みたいなことって普通はやってもいいのかやらないのかの疑問があったのでお聞きしました。

例えばClassBの中にいろんな値をsetするのだけど、そのsetした値の平均値とかをClassAにも格納しておきたいなあと思ったのです。
それだとClassBに平均値のフィールドを作ればって話になりますが、ClassC、ClassDというのもあってClassCに値が入ったら平均値を作り、ClassDにも値が入ったら平均値を作り・・・などちょっと複雑でして、どうしてもClassAに値を保持しておきたかったのです。

[ishi9]

まだちょっと要領を得ない感じですね。

まずはプログラムの構造やら設計の話は置いておいて、
やりたいことを紙に書いて整理するといいかもしれませんね。

設計どうこうの話を軽くいいますと、クラス同士はできるだけ依存関係がないことが理想で
クラスA　｜　クラスB　無関係
クラスA　→　クラスB　一方通行
クラスA　←→　クラスB　互いに依存しあう

設計の内容によりますが、「互いに依存しあう」まで行くと、悪い設計になっている可能性が高いです。できるだけ一方通行の依存性になるよう設計すべきです。

まぁその辺は一旦置いておいてやりたいことを紙に整理するの試してみてください。

[ishi9]

補足ですが、「依存する」というのがどういう状態かというと
（一つの目安ですが）そのクラスを消してコンパイルが通るかどうかです。

例えば一般的な親子関係は子を消してもコンパイルは通りますが、
親を消して子だけ残すとコンパイルは通りません。これが一方通行です。

今のその方向で進むと、恐らくどのクラスを消してもコンパイルが通りませんし、
いざ、いらなくなったので削除しようとしたり一部のものだけ挙動を変えようとするとかなり大変そうです。

[退会済みユーザー]

ありがとうございます！
設計が悪いというのがわかってきました！
具体的に教えてくださって感謝します
ちょっと設計を見直そうと思います！