インスタンスした継承先の子クラスを抽象クラス型の変数に入れた状態でメンバ変数にアクセスすると抽象クラスの初期値を参照してしまいます

Question

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Test : MonoBehaviour
{
    public StateDataClass currentState;
    public AttackState attackState;

    private void Start()
    {
        attackState = new AttackState();
        currentState = attackState;
        currentState.Method(this);
    }
}

public abstract class StateDataClass
{
    public string hoge = "抽象ステート";
    public bool piyo = false;
    public virtual void Method(Test test) { }
}

public class AttackState : StateDataClass
{
    public string hoge = "攻撃ステート";
    public bool piyo = true;

    public override void Method(Test test)
    {
        Debug.Log("hoge:" + hoge + "　　piyo:" + piyo);
        Debug.Log("test.attackState.hoge:" + test.attackState.hoge+ "　　test.attackState.piyo:" + test.attackState.piyo);
        Debug.Log("test.currentState.hoge:" + test.currentState.hoge + "   test.currentState.piyo:" + test.currentState.piyo);
    }
}

キャラクターのステートをクラスで管理するために、まず抽象クラスを作り、overrideしたメソッドと変数を持たせた継承先の子クラスAttackStateをTestキャラクター内でインスタンスしました。
TestキャラクターはcurrentStateという抽象クラス型の変数を持ち、この中にインスタンスした待機、攻撃、移動、といった子クラスを入れて状態遷移を行います。
しかしcurrentState経由でインスタンスした子クラスの変数にアクセスしたところ以下のように抽象クラスの変数が返ってきてしまいうまくいきません。

currentState経由で継承先の子クラスのメンバ変数にアクセスするにはどのようにすればよいでしょうか？

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
（追記）

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Test : MonoBehaviour
{
    public StateDataClass currentState;
    public AttackState attackState;

    private void Start()
    {
        attackState = new AttackState();
        attackState.Huga = 50;


        currentState = attackState;
        currentState.Method(this);

        if (currentState.piyo) { Debug.Log("現在のステートのpiyoがtrueの場合に処理する。piyoはステートによってtrueかflaseか差異がある（待機アニメはループ=trueだがジャンプアニメはfalse、など"); }

    }




}

public abstract class StateDataClass
{
    public string hoge = "抽象ステート";
    public bool piyo = false;

    public int Huga { get; set; } = 100;

    public virtual void Method(Test test) { }
}

public class AttackState : StateDataClass
{
    
    public string hoge = "攻撃ステート";
    public bool piyo = true;
    public int Huga { get; set; } = 50;


    public override void Method(Test test)
    {
     

        Debug.Log("hoge:" + hoge + "　　piyo:" + piyo);
        Debug.Log("test.attackState.hoge:" + test.attackState.hoge+ "　　test.attackState.piyo:" + test.attackState.piyo);
        Debug.Log("test.currentState.hoge:" + test.currentState.hoge + "   test.currentState.piyo:" + test.currentState.piyo);

        Debug.Log("huga:" + Huga);
        Debug.Log("test.currentState.Huga:" + test.currentState.Huga);

    }
}

プロパティを試してみたのですが、上手く行きませんでした。
currentStateの中にあるAttackStateのhogeにアクセスしたいのですが、currentState自身のhogeにアクセスしているように見えます。
実装の仕方が間違っているでしょうか？
実装したい運用としては、
・TestキャラクタのステートであるcurrentStateは常に変化する
・各継承先ステートのメンバ変数は各ステート毎に差異がある（待機ならアニメループ判定はtrueだが、ジャンプはfalse）
・currentState.Method()とするとcurrentStateに代入されている継承クラスのメソッドにアクセスできるように、currentStateの中にある継承先クラスのメンバにアクセスしたい

となります。

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
（追記）
TN8001さんの回答を元に記述したところ期待する結果が実装できました

public class Test : MonoBehaviour
{
    public StateDataClass currentState;
    public AttackState attackState;

    private void Start()
    {
        attackState = new AttackState();
 

        currentState = attackState;
        currentState.Method(this);

        if (currentState.piyo) { Debug.Log("現在のステートのpiyoがtrueの場合に処理する。piyoはステートによってtrueかflaseか差異がある（待機アニメはループ=trueだがジャンプアニメはfalse、など"); }

    }




}

public abstract class StateDataClass
{
    public string hoge = "抽象ステート";
    public bool piyo = false;

    public int Huga { get; set; } = 100;

    public virtual void Method(Test test) { }
}

public class AttackState : StateDataClass
{
    
    //public string hoge = "攻撃ステート";
    //public bool piyo = true;
    //public int Huga { get; set; } = 50;

    public AttackState()
    {
        hoge = "攻撃ステート";
        piyo = true;
    }


    public override void Method(Test test)
    {
     

        Debug.Log("hoge:" + hoge + "　　piyo:" + piyo);
        Debug.Log("test.attackState.hoge:" + test.attackState.hoge+ "　　test.attackState.piyo:" + test.attackState.piyo);
        Debug.Log("test.currentState.hoge:" + test.currentState.hoge + "   test.currentState.piyo:" + test.currentState.piyo);

        Debug.Log("huga:" + Huga);
        Debug.Log("test.currentState.Huga:" + test.currentState.Huga);

    }
}

Answer 1

Visual Studioだとこういった警告が出るんですが、Unityのエディタでは出ないんですかね？

警告	CS0108	'AttackState.hoge' は継承されたメンバー 'StateDataClass.hoge' を非表示にします。非表示にする場合は、キーワード new を使用してください。

コンパイラの警告 (レベル 2) CS0108 | Microsoft Docs

やりたかったのはこういうことではありませんか？

public class AttackState : StateDataClass
{
    //public string hoge = "攻撃ステート";
    //public bool piyo = true;
    public AttackState()
    {
        hoge = "攻撃ステート";
        piyo = true;
    }
    // 以下省略
}

---

この警告はAttackState.hogeが宣言されたことによって継承元である抽象クラスStateDataClassのhogeをAttasckStateクラス内においては上書きしました、という認識で使っていたのですがこれは間違っていますでしょうか？

元のhoge（base.hogeとしてアクセスできます）は残ったまま、別のhogeをAttackStateに作っています。
意図してやることもありますが、そういう場合は「明示的に隠ぺいしてますよ」ということでnewと付けます。
new 修飾子 - C# リファレンス | Microsoft Docs

↑の回答コードでは値そのものを上書きしました。

プロパティを試してみたのですが、上手く行きませんでした。

プロパティならメソッドと同じように、virtualやabstractにできます。

namespace Qsg2zhvq2qgsipz
{
    public abstract class StateDataClass
    {
        public string hoge1 = "抽象ステート";
        public string hoge2 = "抽象ステート";

        public string hoge3 { get; set; } = "抽象ステート";
        public string hoge4 { get; set; } = "抽象ステート";

        public virtual string hoge5 { get; set; } = "抽象ステート";
        public abstract string hoge6 { get; set; }
    }

    public class AttackState : StateDataClass
    {
        public string hoge1 = "攻撃ステート"; // CS0108
        //public new string hoge1 = "攻撃ステート"; // CS0108が消える 意味は↑と同じ

        public string hoge3 { get; set; } = "攻撃ステート"; // CS0108
        //public new string hoge3 { get; set; } = "攻撃ステート"; // CS0108が消える 意味は↑と同じ

        public override string hoge5 { get; set; } = "攻撃ステート";
        public override string hoge6 { get; set; } = "攻撃ステート";

        public AttackState()
        {
            hoge2 = "攻撃ステート";
            hoge4 = "攻撃ステート";
            //Console.WriteLine(base.hoge1); // 抽象ステート
            //Console.WriteLine(base.hoge3); // 抽象ステート
            //Console.WriteLine(base.hoge5); // 抽象ステート
            //Console.WriteLine(base.hoge6); // エラー CS0205 抽象基本メンバーを呼び出すことはできません
        }
    }

    class Program
    {
        static void Main()
        {
            AttackState attackState = new AttackState();
            StateDataClass currentState = attackState;

            Console.WriteLine(attackState.hoge1); // 攻撃ステート
            Console.WriteLine(currentState.hoge1); // 抽象ステート

            Console.WriteLine(attackState.hoge2); // 攻撃ステート
            Console.WriteLine(currentState.hoge2); // 攻撃ステート

            Console.WriteLine(attackState.hoge3); // 攻撃ステート
            Console.WriteLine(currentState.hoge3); // 抽象ステート

            Console.WriteLine(attackState.hoge4); // 攻撃ステート
            Console.WriteLine(currentState.hoge4); // 攻撃ステート

            Console.WriteLine(attackState.hoge5); // 攻撃ステート
            Console.WriteLine(currentState.hoge5); // 攻撃ステート

            Console.WriteLine(attackState.hoge6); // 攻撃ステート
            Console.WriteLine(currentState.hoge6); // 攻撃ステート
        }
    }
}

Comments

[ko_yu]

できました！ありがとうございます！でも原理がわからないので教えてください。

なぜAttackStateという継承先のクラス名と同じクラスを書き、そこにメンバ変数を代入すると期待した結果になるのでしょうか？
また、私の環境でも警告が出ているのですが、この警告はAttackState.hogeが宣言されたことによって継承元である抽象クラスStateDataClassのhogeをAttasckStateクラス内においては上書きしました、という認識で使っていたのですがこれは間違っていますでしょうか？
currentState.hogeはcurrentStateに代入されているAttckStateのhogeにアクセスする、という認識で使っていたのですが、 AttackStateクラス内にもう一度 AttackStateクラスを書くことでどうして差が出るのでしょうか？

[TN8001]

> できました！ありがとうございます！でも原理がわからないので教えてください。

追記しました。
ちょっとややこしい（回りくどい？）コードのため混乱されているのかもしれませんが、もっと単純な話です。
CS0108が出ているということは、別のメンバーができてしまっているということです。

[ko_yu]

ケースごとの例のコードありがとうございます！
なるほど、継承先クラスのフィールドで同名変数を宣言する、もしくはoverrideしないプロパティは、上書きではなく継承先と抽象元にhogeが１つずつある状態になっていたということでしょうか。
コンストラクタを使う、overrideしたプロパティは抽象クラスの変数をきちんと引き継いで派生クラスにだけhogeを用意したということでしょうか。
そして同名変数宣言 or overrideしない場合は、currentState経由だとbase.hogeにアクセスしてしまう、ということなのだと理解しました。

詳しく教えてくださってありがとうございます！
このCS0108は何度か見ていたのですが、いまいち理解できないままコードを打っていたので今日から認識を改めようと思います。

[TN8001]

ちょっと例が足りず勘違いさせたかもしれません（更新しました）

> 継承先クラスのフィールドで同名変数を宣言する、もしくはoverrideしないプロパティは、上書きではなく継承先と抽象元にhogeが１つずつある状態になっていたということでしょうか。

hoge1・hoge3のことですね。そうなります。
overrideは無関係です。hoge5も同様です。

> コンストラクタを使う、overrideしたプロパティは抽象クラスの変数をきちんと引き継いで派生クラスにだけhogeを用意したということでしょうか。

hoge2・hoge4に関しては、単にStateDataClassのメンバーを継承しているだけですので実体は１個です。
hoge5は上の通り２つということになります。

> そして同名変数宣言 or overrideしない場合は、currentState経由だとbase.hogeにアクセスしてしまう、ということなのだと理解しました。

hoge1・hoge3のような状態になるのは、あまりうれしくないので普通は使いません。
そうせざるを得ない時（親クラスに手を入れられない場合等）に、仕方がなくやる感じです。

やりたいことがあんまりわかっていないのですが、hoge2・hoge4で困らないのであればそれでいいと思います。
hoge2・hoge4でやりたいことができないのであれば、hoge5・hoge6ということになりますね（あるいはinterface）

[ko_yu]

捕捉ありがとうございます！
なるほど、overrideではなくvirtualかabstarctの違いということですね。
ステート毎のアニメのループ判定用にbool変数を各ステートに持たせたかったので当面は2,4で対処できそうです。

詳しく解説してくれてすごく助かりました！

Answer 2

変数はoverrideできないので、そういう場合はプロパティとしましょう

Comments

[ko_yu]

回答ありがとうございます。プロパティを使ってみたのですが、うまくいきませんでした。
継承先クラスのメンバ変数自体は変わっているようなのですが、currentState.Hugaと書くと、どうもcurrentStateを経由して中のメンバ変数にアクセスしているというより、currentState自体の変数を参照している気がします。
Testキャラクタのステートは常に変化してcurrentStateに格納される、継承先ステート毎に異なるメンバ変数にcurrentStateを経由してアクセスするにはどのようにすべきでしょうか？

[y_waiwai]

プロパティにして、get、setに関数を割り当て、そいつをoverrideしましょう

[ko_yu]

overrideしたプロパティでできました！ありがとうございます。