unity2Dシューティングの自機を画面から出なくしたいです

###前提・実現したいこと
現在unityを用いて2Ｄシューティングの製作に挑戦しています。
その中で自機の製作を現在しているのですがそこで不明なことがあり質問させていただきました。
自分はまだまだunityでの製作経験はほとんどなく、本やさまざまなサイトを参考させていただきながらプログラムを書いているようなレベルです。
今回お聞きしたいのは移動制限のかけ方です。
現在移動は横にのみ可能にしています。

###発生している問題・エラーメッセージ
ＩＦ文を使ったのですがエラーは吐かれず、起動もしっかりとしてくれます。
ですが横に移動をしていても止まってくれずにそのまま画面外に行ってしまいます。
エラーが吐かれていないせいで見当もつかない状態です。ＩＦ文を使うこと自体が間違っているのでしょうか？

###該当のソースコード

c#
1using UnityEngine;
2using System.Collections;
3
4public class player : MonoBehaviour
5{
6
7    public float speed = 5;
8    void Update()
9    {
10        float x = Input.GetAxisRaw("Horizontal");
11        float y = 0;
12        Vector2 direction = new Vector2(x, y).normalized;
13
14        GetComponent<Rigidbody2D>().velocity = direction * speed;
15
16        if (x >= 5)
17        {
18            x = 5;
19        }
20        else if(x <= -5)
21        {
22            x = -5;
23        }
24    }
25
26
27}

###試したこと
座標の数値自体が間違っているのかといろいろ変えてみたがダメでした。

###補足情報(言語/FW/ツール等のバージョンなど)
言語はC#
unityのverは5.3.5f1です。

行動規範の内容に同意します

回答1件

ベストアンサー

Inputはコントローラを扱う機能です。
float x = Input.GetAxisRaw("Horizontal");
とありますが、これはスティックをどれだけ左右に傾けたかが取れるだけでplayerの位置とは何の関係もありません。

playerの位置を一定の枠内に制限したいなら、playerの位置情報を操作しましょう。
(ワールド座標系なので、５程度だとほとんど身動き取れないかもしれませんが・・・）

C#
1Vector3 pos = transform.position;
2
3if (pos.x >= 5)
4{
5    pos.x = 5;
6}
7else if(pos.x <= -5)
8{
9    pos.x = -5;
10}
11transform.position = pos;