C# 継承の使い方...

Question

UnityでBaseMagicを継承したFireクラスを作成したところエラーが４つでてしまったのですが原因が分かりません。どこがおかしいのか教えていただきたいです。よろしくお願いします。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using BattleScene;
using BattleScene.Chara;

namespace BattleScene.Magic{

public class BaseMagic : IMagic
{
    public int DownMP { get; set; }
    TextManager textmanager = GameObject.Find("battletext").GetComponent<TextManager>();

    public void Use(Player attacker,Player defender);

    public bool canUse(Player player){
       if(player.MP >= DownMP){
           return true;
       }
       return false;
    }
}

}

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using BattleScene;
using BattleScene.Chara;

namespace BattleScene.Magic{

public class Fire : BaseMagic
{
    public Fire(){
        public int maxDamage { get; set; } = 30;
        public int minDamage { get; set; } = 10
        base.DownMP = 20;
    }
    
    public override void Use(Player attacker,Player defender){
        int calcedDamage = UnityEngine.Random.Range(this.minDamage,this.maxDamage+1);
        base.textmanager.battleLog($"{attacker.PlayerName}のファイア！   {defender.PlayerName}に{calcedDamage}のダメージ");
        attacker.MP = attacker.MP-base.DownMP;          //mp消費-20
        defender.damage(calcedDamage);
        attacker.AttackFinished = true;
	}
}

}

Assets\MainBattle\BattleScene\Magic\Fire.cs(11,19): error CS1513: } expected
Assets\MainBattle\BattleScene\Magic\Fire.cs(14,9): error CS1519: Invalid token 'base' in class, struct, or interface member declaration
Assets\MainBattle\BattleScene\Magic\Fire.cs(14,21): error CS1519: Invalid token '=' in class, struct, or interface member declaration
Assets\MainBattle\BattleScene\Magic\Fire.cs(26,1): error CS1022: Type or namespace definition, or end-of-file expected

Comments

[退会済みユーザー]

まずエラーコードとエラーメッセージでググってください。
単純なコンパイルエラーはエラー内容をきちんと読んで調べれば自力で解決出来る筈です。
翻訳も検索も面倒だから調べてという話ならお断りします。

[退会済みユーザー]

継承の方法とかの以前の問題で、単純に } が足りないとかの文法ミスのようですが？

Answer 1

minDamageの行、セミコロンが抜けてます。
あとコンストラクタの中でpublic使ってるがおかしいと思います。
そのpublicと書いている行であなたは何をやりたいのか、そのために使うべき正しい書き方はどのようなものかを確認してください。

Answer 2

    public Fire(){
        public int maxDamage { get; set; } = 30;
        public int minDamage { get; set; } = 10
        base.DownMP = 20;
    }

「関数内で定義したものは関数でしか使えない」ので、maxDamageとminDamageはFire()内でしか使えませんし、その上で「どこでもアクセス出来るようにする」という意味のpublicの付けているのですから、コンパイラから見ればもはや意味不明でしょう。
もしmaxDamageとminDamageを他のクラスから参照したいのであれば、Fire()外で定義すべきです。

Comments

[Tomo0225]

コンストラクタ内でmaxDamageとminDamageを定義しているのがいけなかったんですね。言われておかしいことだと気づきました。
助かりました。ありがとうございます。