地面をキャラクターが貫通してしまう

Question

unityで2Dのゲームを作っています。
下から上がってくる地面に乗っかって何秒耐えられるかのゲームなのですが、
肝心の地面に触れても、キャラが落ちていってしまいます。
スクリプトに間違いがあったらぜひ教えてください。
キャラクターのスクリプトです↓

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

public class PlayerIdou : MonoBehaviour
{
     public float SpeedX;    
　　　public static int SpeedY=-65;
     public float Muki;
     float screenLeft = -2;//左端の座標x
     float screenRight = 2;//右端の座標x
                                      
    void Start()
    {
        
    }

    // Update is called once per frame
    void Update()
    {
　　　　　SpeedX = SpeedX*0.7f;
        Muki = Muki*0.7f;
        SpeedY += 5;
        if (Input.mousePosition.x >= Screen.width / 2)
        {
                  if (Input.GetMouseButton(0)) {// 右側をタップしたら
                  SpeedX += 0.1f;
                  Muki -= 1;
                  }
        }
        else
        {
                  if (Input.GetMouseButton(0)) {// 左側をタップしたら
                  SpeedX -= 0.1f;
                  Muki += 1;
                  }
        }
     
        if(SpeedY > 0){
            void OnTriggerEnter(Collider other)
            {
        //接触したオブジェクトのタグが"Ground"のとき
        if(other.CompareTag("Ground"))
              {
            
            SpeedY = -65;
              }   
           }
    
        }

        transform.position += new Vector3 (SpeedX*0.5f, 0, 0);
         transform.rotation = Quaternion.Euler(0, 0, Muki*2);
        Vector3 pos = transform.position;
        pos.x = Mathf.Clamp(pos.x, screenLeft, screenRight);
        transform.position = pos;
        
    }

    internal static float getSpeedY()
    {
        throw new NotImplementedException();
    }

    public static int GetSpeedY(){
		return SpeedY;
	}

}

地面のスクリプトです↓

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

public class GameGround : MonoBehaviour
{

    public PlayerIdou playerIdou;

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

    // Update is called once per frame
    void Update()
    {
        float Jump;
        Jump = PlayerIdou.SpeedY;
        transform.position += new Vector3 (0, Jump*0.01f, 0);
    }
}

キャラクターにはPlayerのタグ、Rigidbody2D、Box Collider2Dをつけています。
地面にはGroundのタグと、Box Collider2Dをつけてあります。
このゲームは、キャラクターのy座標を変えたくないので、地面の位置を動かしています。

ネットで調べたことはほぼやったつもりですが、解決できなかったので質問させていただきました。
unityのバージョンは2020.3.14f1です。

Answer 1

まずOnTriggerEnter()がUpdate()の中に定義されている時点で間違っております。
関数の中に関数を定義した場合、その関数（今回の場合Update()）でしか呼び出すことはできません。
したがって、Unity本体からOnTriggerEnter()が呼び出せず、動作しません。

また、OnTriggerEnter()は3D用のものなので、2D用のもの（Rigidbody2Dなど）を使用する場合はOnTriggerEnter2D()でなければなりません。

また、OnTriggerEnter()を使っているということはIs Triggerがオンにしているとは思いますが、その場合ぶつかってもすり抜けてしまいます。
（スクリプトで制御しているのかもしれませんが、念のため）

---

追記：

変更箇所が多かったので、修正したコードを記載します。

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

public class PlayerIdou : MonoBehaviour
{
    public float SpeedX;
    public static int SpeedY = -65;
    public float Muki;
    float screenLeft = -2;//左端の座標x
    float screenRight = 2;//右端の座標x

    Rigidbody2D rigidbody2DComponent;
    bool tapLeft = false;
    bool tapRight = false;

    void Start()
    {
        rigidbody2DComponent = GetComponent<Rigidbody2D>();
    }

    // Update is called once per frame
    void Update()
    {
        if (Input.mousePosition.x >= Screen.width / 2)
        {
            if (Input.GetMouseButton(0))
            {// 右側をタップしたら
                tapRight = true;
            }
        }
        else
        {
            if (Input.GetMouseButton(0))
            {// 左側をタップしたら
                tapLeft = true;
            }
        }
    }

    void FixedUpdate()
    {
        SpeedX = SpeedX * 0.7f;
        Muki = Muki * 0.7f;
        SpeedY += 5;

        if (tapRight)
        {// 右側をタップしたら
            SpeedX += 0.1f;
            Muki -= 1;
        }
        tapRight = false;

        if (tapLeft)
        {// 左側をタップしたら
            SpeedX -= 0.1f;
            Muki += 1;
        }
        tapLeft = false;

        Vector3 pos = transform.position + new Vector3(SpeedX * 0.5f, 0, 0);
        rigidbody2DComponent.MoveRotation(Quaternion.Euler(0, 0, Muki * 2));
        pos.x = Mathf.Clamp(pos.x, screenLeft, screenRight);
        rigidbody2DComponent.MovePosition(pos);
    }

    internal static float getSpeedY()
    {
        throw new NotImplementedException();
    }

    public static int GetSpeedY()
    {
        return SpeedY;
    }

    void OnCollisionEnter2D(Collision2D other)
    {
        //接触したオブジェクトのタグが"Ground"のとき
        if (other.gameObject.CompareTag("Ground"))
        {
            SpeedY = -65;
        }
    }
}

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

public class GameGround : MonoBehaviour
{

    public PlayerIdou playerIdou;
    Rigidbody2D rigidbody2DComponent;

    // Start is called before the first frame update
    void Start()
    {
        rigidbody2DComponent = GetComponent<Rigidbody2D>();
    }

    void FixedUpdate()
    {
        float Jump;
        Jump = PlayerIdou.SpeedY;
        rigidbody2DComponent.MovePosition(transform.position + new Vector3(0, Jump * 0.01f, 0));
    }
}

また、Box Collider2DとRigidbody2Dを両方に付けて、Rigidbody2Dを以下のように設定しました。

• Body Type: Kinematic
• Use Full Kinematic Contacts: ON
• Collision Detection: Continuous（上記2つで上手くいかない場合）

以下、変更した箇所についての説明です。

• OnTriggerEnter()からOnCollisionEnter2D()に変更しております。
  Is Triggerを解除した場合は、OnTriggerEnter2D()では取得できないため、OnCollisionEnter2D()にしています。
• 物理判定や移動を行うものについては、Update()ではなくFixedUpdate()を使います。
  ただし、操作入力の取得は逆にUpdate()でないといけないので、tapLeftやtapRightのように変数を仲介させるようにしましょう。
• 移動させる処理もtransformだと貫通することがあるので、Rigidbody2Dを使って移動させます。
  移動方法は色々ありますが、今回はMovePosition()やMoveRotation()を使っています。
  ただし、これらはRigidbody2DがKinematicでないと正しく動作しないため、そのように設定しております。
  また、オブジェクトの大きさや速度次第では、これでも貫通する可能性があるので、必要に応じてContinuousを設定する必要があります。

あと、変更はしていませんが、気になった点として以下があります。

• SpeedYがstaticですが、そうした意味が理解できませんでした。
  むしろ、staticだと「Playerを2つ以上用意できない」「Playerを破棄した後もSpeedYの値が残ってしまう」という問題があります。
  GameGroundにplayerIdouを用意してあるのだから、それにPlayerIdouをインスペクターで指定させればいい話だと思いますが、どうでしょう。

Comments

[youngdoonatsu]

ご回答ありがとうございます。OnTriggerEnter2D()を使用すると、"Script error: OnTriggerEnter2D
This message parameter has to be of type: Collider2D　The message will be ignored."というエラーが出てしまいます。

[youngdoonatsu]

本当にごめんなさい！上のは嘘ですすみません。しっかりプログラムを設定してなくて起きてしまいました。
エラーは無くなりましたが、まだ地面を貫通してしまいます…Box ColliderのIs triggerはどちらも解除済みなのですが…

[fiveHundred]

少し前に追記しました。

Answer 2

僕も同じことが昔あったのですが…
コードの問題ではなく
キャラクターと地面の方に
Colliderがついているかかくにんしてみてください。

Comments

[youngdoonatsu]

Colliderついてました