unity2dで自機が画面外に出ないようにしたい。

Question

前提・実現したいこと

unity,2Dでプレイヤーが操作する自機が
画面外に出ないようにしたいのですが、自機が動くだけで
画面外に出てしまいます。お知恵をお借りしたいです……。

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

UnityEngine.Vector2からfloatへ変換することができません。
などがでたり、あと、画面サイズを取得した方がいいとは思うんですが
取得する方法が分かりません…。
上記に描いた自機を画面外に出さない方法を知りたいです。。。

該当のソースコード

using System.Collections;

using System.Collections.Generic;

using UnityEngine;

public class player : MonoBehaviour

{

    //移動スピード

    public float speed = 5;

    // Use this for initialization

    void Start()

    {

    }

    // Update is called once per frame

    void Update()

    {

        var w = Screen.width;

        var h = Screen.height;

        // 右・左

        float x = Input.GetAxisRaw("Horizontal");

        // 上・下

        float y = Input.GetAxisRaw("Vertical");

        //移動する向き

        Vector2 dir = new Vector2( x, y);


        //移動する向きとスピード代入

        GetComponent<Rigidbody2D>().velocity = Mathf.Clamp(dir, x, y) * speed;


     }

}```

### 試したこと

Screen.widthを入れる箱をfloatにしたりしました。
dirにxとyを入れるときに、x,yのところをMathf.Clampを使ったりしました。

### 補足情報（FW/ツールのバージョンなど）
ツールはマイクロソフトヴィジュアルスタジオです。
一応、画面のサイズは2.4です。よろしくお願いいたします…。

Answer 1

移動先座標を直接指定するタイプの移動方法ならMathf.Clampによる範囲制限をやりやすいのですが、Rigidbody2D.velocityを操作するタイプの移動方法だと、ちょっと工夫を加える必要があるかと思います。

一案として、下記のような感じではいかがでしょうか?
説明もかねて冗長になっていますが、ご質問者さんのコードに合わせてご自由に改造してしまってください。

using UnityEngine;

public class player : MonoBehaviour
{
    // 移動スピード
    public float speed = 5;

    // 移動する向き
    private Vector2 dir;

    // 画面の左・右・上・下の境界の位置
    private float boundL;
    private float boundR;
    private float boundT;
    private float boundB;

    private new Rigidbody2D rigidbody2D;
    
    private void Start()
    {
        this.rigidbody2D = this.GetComponent<Rigidbody2D>();

        // あらかじめ、画面上下左右の縁がワールド空間上でどこに位置するか調べておく
        var mainCamera = Camera.main;
        var positionZ = this.transform.position.z;
        var topRight = mainCamera.ViewportToWorldPoint(new Vector3(1.0f, 1.0f, positionZ));
        var bottomLeft = mainCamera.ViewportToWorldPoint(new Vector3(0.0f, 0.0f, positionZ));
        this.boundL = bottomLeft.x;
        this.boundR = topRight.x;
        this.boundT = topRight.y;
        this.boundB = bottomLeft.y;
    }
    
    private void Update()
    {
        // 右・左
        var x = Input.GetAxisRaw("Horizontal");

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

        this.dir = new Vector2(x, y);
    }

    private void FixedUpdate()
    {
        // 現在の位置
        var position = this.rigidbody2D.position;

        // 範囲制限を加える前の速度
        var velocity = this.dir * this.speed;

        // 今回の物理状態更新での移動量
        var deltaPosition = velocity * Time.deltaTime;

        // 範囲制限を加えなかった場合の、予想される移動先
        var nextPosition = position + deltaPosition;

        // 予想移動先の座標をクランプする
        var clampedNextPosition = new Vector2(
            Mathf.Clamp(nextPosition.x, this.boundL, this.boundR),
            Mathf.Clamp(nextPosition.y, this.boundB, this.boundT));

        // クランプ後の移動先座標から移動量を求める
        var clampedDeltaPosition = clampedNextPosition - position;

        // クランプ後の移動量から速度を求める
        var clampedVelocity = clampedDeltaPosition / Time.deltaTime;

        // クランプ後の速度を代入
        this.rigidbody2D.velocity = clampedVelocity;
    }
}

Comments

[ropro2]

ありがとうございます！考えながらコードを打ち込んだところ、ちゃんと範囲制限されていました…！
最近、unityはじめたばかりで長いコードを見ても理解できないのですが、
さすが先人様たちのコードは違うなあと実感しました！
ありがとうございます！

Answer 2

Unity公式のSpace Shooter tutorialでも、対応例が説明されています。（3Dですが）
考え方はBongoさんの回答とほぼ同様かと思いますが、ご参考まで。

Moving the player - Unity
https://unity3d.com/jp/learn/tutorials/projects/space-shooter-tutorial/moving-player?playlist=17147

Comments

[ropro2]

ありがとうございます！見てみます！