複数種類の自機を作りたい場合の記述の方法をご教示ください

Question

前提・実現したいこと

2Dシューティングを作成したいと考えています。Unityに関しては初心者です。
ある目的から複数種類の自機が必要になるため、あらかじめPlayerStandardクラスに全種類の自機が共通して持つ変数と処理を記述し、それとは別に個別の自機が持つ変数と処理を各自のクラスが保持しているという状態を作りたいと考えました。

発生している問題・エラーメッセージ

PlayerStandardと個別の自機のクラス（ここではIchikaplayer）を以下のように設定し、
自機にIchikaplayer.csのみをアタッチしたところ、コンパイルエラーとなりました。

該当のソースコード

PlayerStandard.cs
using System.Collections;
using System.Collections.Generic;
using UnityEngine;

//すべての自機が共通して持つパラメータや動作機能の設定

public class PlayerStandard : MonoBehaviour
{
    // 移動スピード
    public float speed;

    //残機数
    public float rest;

    //エクステンドアイテム取得数
    public float lifechips;

    //エクステンドアイテム必要数初期値
    public float lifeinitial;

    //1回当たりのエクステンドアイテム必要数増加量
    public float lifegradiant;

    //エクステンドした回数
    public float extendedtimes;

    //エクステンドに必要なアイテム総数= lifeinitial + lifegradient * extendedtimes

    //自機のパワー
    public float playerpower;

    //シールド残量
    public float shieldtoughness;

    //シールド状態
    //0:ironclad 1:shieldbreak 2:shieldrecovering 3:absolute 4:wingmode
    //0の時以外は被弾すると死
    public int shieldvalidness;

    //MSキーが連打されているかどうか
    protected int MSbeat;

    //MSキーが何フレームの間連打されているか
    protected int MSbeattime;

    //MSキーが何フレームの間押下されているか
    protected int MSholdtime;

    //SWキーが連打されているかどうか
    protected int SWbeat;

    //SWキーが何フレームの間連打されているか
    protected int SWbeattime;

    //SWキーが何フレームの間押下されているか
    protected int SWholdtime;

    //DCキーが何フレームの間押下されているかどうか
    protected int DCholdtime;

    void Update()
    {
        //移動関連
        {
            // 右・左
            float x = Input.GetAxisRaw("Horizontal");

            // 上・下
            float y = Input.GetAxisRaw("Vertical");

            // 移動する向きを求める
            Vector2 direction = new Vector2(x, y).normalized;

            // 移動する向きとスピードを代入する
            GetComponent<Rigidbody2D>().velocity = direction * speed;
        }
        
        //MSキー入力関連＿初期状態ではZキー,入力補助として左Shift
        if (Input.GetKey(KeyCode.Z))　//MSキー押下状態の管理
        {
            MSholdtime++;   //押されている間ずっとカウントアップ
        }
        else
        {
            MSholdtime = 0; //押されていない場合はずっと0
        }

        if (Input.GetKeyUp(KeyCode.Z))  //MSキー連打状態の管理
        {
            MSbeat = 15;    //押された瞬間から15フレームの間連打状態
        }
        else if(MSholdtime == 0 && Input.GetKey(KeyCode.LeftShift))
        {
            MSbeat = 1;　　//Zが押されておらず、かつ左シフトの入力があればずっと連打状態
        }
        else
        {
            MSbeat--;       //押された瞬間でない状態(入力されていないor押しっぱなし)はフレームごとにカウントが1減少
        }
        if (MSbeat > 0)     //MSキー連打状態継続時間の管理
        {
            MSbeattime++;   //連打状態=カウントが正の状態と定義し、その継続時間をカウント
        }
        else
        {
            MSbeattime = 0; //0またはそれ以下になった場合連打継続時間をリセット
        }

        //SWキー入力関連＿初期状態ではXキー
        if (Input.GetKey(KeyCode.X)) //SWキー押下状態の管理
        {
            SWholdtime++;   //押されている間ずっとカウントアップ
        }
        else
        {
            SWholdtime = 0; //押されていない場合はずっと0
        }

        if (Input.GetKeyUp(KeyCode.X)) //SWキー連打状態の管理
        {
            SWbeat = 15;    //押された瞬間から15フレームの間連打状態
        }
        else
        {
            SWbeat--;       //押された瞬間でない状態(入力されていないor押しっぱなし)はフレームごとにカウントが1減少
        }
        if (SWbeat > 0)     //SWキー連打状態継続時間の管理
        {
            SWbeattime++;   //連打状態=カウントが正の状態と定義し、その継続時間をカウント
        }
        else
        {
            SWbeattime = 0; //0またはそれ以下になった場合連打継続時間をリセット
        }

        

        //被弾処理関連
    }
}

Ichikaplayer.cs
public class Ichikaplayer : PlayerStandard
{
        //ヒット数
        public float ichikahitcount;

        //ヒット継続時間
        public float ichikahittime;
}

    // Start is called before the first frame update
    void Start()
    {
        
    }

    // Update is called once per frame
    void Update()
    {
        
    }

試したこと

Ichikaplayer.csそのものを削除し、PlayerStandard.csのみをアタッチしたところ、
コンパイルエラーにならずに十字キーによってキャラを移動する動作はできました。
このことからクラスの継承の方法が間違っているのだろうと思っています。
身の程知らずの大きなものに手を出そうとした弊害と反省しております。
クラスの継承方法等、ご教示くださるとありがたいです。

補足情報（FW/ツールのバージョンなど）

Unity 2019.3.7f1を使用しています。

Answer 1

これって実際のコードですか？　だとしたら、Ichikaplayerクラスの定義が以下で終わってしまっているので、メソッドの定義がクラス定義の外に出てしまっているので、そのためにエラーが出ているのだと思います。

public class Ichikaplayer : PlayerStandard
{
        //ヒット数
        public float ichikahitcount;

        //ヒット継続時間
        public float ichikahittime;
}
// クラスの定義がここで終わっている

StartとUpdateメソッドをクラス定義の中に入れたらコンパイルエラーは解消されると思います。

Comments

[Melipofolia]

ありがとうございます
PlayerStandard.csのみをアタッチした時と同じように正常に動きました

[Melipofolia]

すみません見間違いでした。
動きませんでしたがこれはIchikaplayer側のUpdateに継承元側のUpdateが上書きされているということでよろしいでしょうか
可能であればどのようにすれば継承元継承先双方の内容が適応されるかもご教示願えないでしょうか

[edom18]

そうですね。Updateメソッドが上書きしてしまっています。なので、ベースクラスのUpdateをprotectedより上の公開レベルにした上で、virutal関数化する必要があります。

C#ではbaseクラスのメソッドを呼び出すにはbase.Hogeとする必要があるので、Ichikaplayer側ではbase.Updateを呼び出してやれば意図した動きになると思います。

ここからはよりよい実装という視点での意見になりますが、baseを呼び出さなければならない、というのはよくやるミスにつながるので、例えばベースクラス側でvirtualな差し込み用メソッドを定義しておき、派生クラスでは（必要であれば）それを呼び出す、というふうにするといいかと思います。

例えばこんな感じ↓

```
// PlayerStandard側

void Update()
{
// ... 前略

OnUpdate();
}

protected virtual void OnUpdate() { }

-------------------

// Ichikaplayer側

protected override void OnUpdate()
{
// Ichikaplayer側の差し込み処理
}
```

[Melipofolia]

ありがとうございます
そのあたりを勉強しながら組んでみます