あるフィールドの値を引数として違うクラスに渡したい

Question

前提・実現したいこと

punch1クラスで定義したdamagePowerをpunchmanagerクラスに渡したい

あるクラスのフィールドで定義した整数を持つ変数を、
そのクラス内のメゾッドの引数として他のクラスのメゾッドに受け渡したいです

言葉が間違ってるかと思います、すみません

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

error CS0176: Member 'punch1.damagePower' cannot be accessed with an instance reference; qualify it with a type name instead
もしくは

該当のソースコード

public class punch1 : MonoBehaviour
{
   public static int damagePower= 1;　　//この数値
   public void OnTriggerEnter2D(Collider2D collision)
    {
        if (collision.tag == "Enemy")
        {
            var dp = new punch1();  //ここらへんが問題
            punchmanager.damagetrigger(dp.damagePower,collision);//punchmanagerクラスにdamagePowerを渡したい
        }
    }
}



public class punchmanager : MonoBehaviour
{
     public void damagetrigger(int damagePower, Collider2D collision) 
    {
        enemymanager enemy = collision.GetComponent<enemymanager>();
        enemy.OnDamage(damagePower);
    }
}

試したこと

staticで定義もしてみました

public static int damagePower=1;
if (collision.tag == "Enemy")
        {
            punchmanager.damagetrigger(punch1.damagePower,collision);//punchmanagerクラスにdamagePowerを渡したい
        }

Comments

[kotori_a]

「staticで定義したとき」のコードを実行するとどのようなエラーが出るのでしょうか？（上と全く同じエラー？それとも「An object reference is required for the nonstatic field, method, or property 'member'」というようなエラー？）
またはエラーが出ないのでしょうか？

[Q71]

C#はobject oriented languageです。orientedは「指向」と訳されますが、日本語の意味では「志向」が正解です。
object は「何か」「オブジェクト」と訳される場合が多いですが、「目的」という意味もあります。「目的志向言語」、目的を考えてそれに至る方法をかんがえる(プログラミング)言語であると考えてください。
ということで、変数やメソッドそれぞれの目的はなんでしょうか。damagePowerは何を表す変数で、その値はクラスに属するのか、インスタンスごとに異なるのか。そういう事を考えてください。そうすると、クラスメンバーの整理ができます。
まず、クラスメンバーの整理が必要だと思いました。

Answer 1

• damagePowerはstaticなので、クラス名でアクセスする必要があります。
• punchmanager.damagetrigger(クラス名.メソッド)という記述を行う場合、damagetriggerはstaticメソッドにする必要があります。

public class punch1 : MonoBehaviour
{
   public static int damagePower= 1;　　//この数値
   public void OnTriggerEnter2D(Collider2D collision)
    {
        if (collision.tag == "Enemy")
        {
            // damagePowerはstaticなので、クラス名(punch1)でアクセスする必要があります。
            punchmanager.damagetrigger(punch1.damagePower,collision);//punchmanagerクラスにdamagePowerを渡したい
        }
    }
}


public class punchmanager : MonoBehaviour
{
     // 上記のようにpunchmanager.damagetriggerという記述を行う場合、damagetriggerはstatic関数にする必要があります。
     public void static damagetrigger(int damagePower, Collider2D collision) 
    {
        enemymanager enemy = collision.GetComponent<enemymanager>();
        enemy.OnDamage(damagePower);
    }
}

Answer 2

現在提示されているコードだと、public static int damagePower= 1;とstatic付きで宣言されているので、これは静的フィールド(クラスフィールド)ですよね。静的フィールドをインスタンス名(ここではdp)に付けて使おうとすると、提示されているエラーメッセージが表示されます。

解決策は、クラス名に付けてフィールド名を書くpunch1.damamePowerなんですが、すでにご自分で書かれていますね。

もしかして、(質問文には書かれていませんが)本当はstaticにしたくないということなんでしょうか。その場合はそれがわかるように質問文を書かないと伝わりません。

Comments

[dedede914]

回答ありがとうございます
staticなしで書くとこのエラーが、
error CS0120: An object reference is required for the non-static field, method, or property 'punchmanager.damagetrigger(int, Collider2D)'
staticありでインスタンス化せずにpunch1.damamePower とかくと
error CS0120: An object reference is required for the non-static field, method, or property 'punch1.damagePower'
で同じようにエラーが出てしまいます

[Daregada]

やみくもにエラーをなくすことが目的になってませんか?
まず、「damagePower」はインスタンスごとに異なる値を取るのでしょうか、それともこのクラスで統一された値を取るのでしょうか。それによってstaticにするかどうかが決まります。

あと上のエラーは「punchmanager.damagetrigger」に関するエラーなので、そっちをstaticにするか、punchmanagerのインスタンスを作ってそれを代わりに指定するかする必要があります。