Unityでキャラに物を持たせたい

Question

前提・実現したいこと

タイトル通り、物を持たせたいです
Unityの２Dです
イメージとしてはマリオが甲羅を持っている感じです

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

発生している問題は
物をキャラクタの手（腕）辺りの上に移動させようとすると、なぜか足部分に移動しくっついてしまうことがあります。うまくいくときもありますが、3割くらいの確率で足に移動してしまいます。

該当のソースコード

if (collision.gameObject.tag == "Mario") {
//rb2d.isKinematic = true;
if (Input.GetKey (KeyCode.RightArrow)) {
if (facingRight == true) {
w = Mario.transform.position;
w.x += 1.3f;
w.y += 0.15f;
transform.position = w;
Debug.Log (w);
}
} else if (Input.GetKey (KeyCode.LeftArrow)) {
if (facingRight == false) {
w = Mario.transform.position;
w.x -= 1.3f;
w.y += 0.15f;
transform.position = w;
}
}
transform.parent = GameObject.Find ("マリオ").transform;

試したこと

キャラの腕の位置のほんの少し上に物を移動させ、とても短い距離を落下させてキャラと物を接触させています
物を移動させる位置を修正したりしましたが解決できません

補足情報（FW/ツールのバージョンなど）

キャラと物のリジッドボディはオンのままです
物のリジッドボディをオフにすると移動してくっつきはしますが、何も操作していないと、キャラが物に押されているような現象が起き、その場にとどまることができません

他にしていることは、キャラと物に摩擦を加えています。これはキャラが止まったとき、物が腕からストーーーんと飛んで行ってしまうからです。接触時は親子関係にもしていますが、それでもうまく行かなかったので摩擦を加え、飛んでいくことを減らしました

ソースコードのfacingRightは、キャラの向きによって腕の位置も変わるので、向きを判定してそれにあわせた位置に物を移動させています

些細なことでもコメントよろしくお願いします

Comments

[stdio]

該当のソースコードでしっかり枠に収まっていませんよ。

[pokapoka2804]

編集しました、ご指摘ありがとうございます！！

Answer 1

このスクリプトだと、
「衝突時に左右キーのどちらも押されていない場合」と
「衝突時に右キーが押されていてfacingRightがfalseの場合」と
「衝突時に左キーが押されていてfacingRightがtrueの場合」には
オブジェクトの移動が実行されません。
その結果衝突時の場所に留まる為、その位置が丁度足の位置なのではないでしょうか。

持っている間、物とキャラの衝突を維持させておく必要があるのかは疑問ですが、
直接的な原因は上記かと思いますので確認してみてください。

Comments

[pokapoka2804]

if（左右キー）の中のif (facingRight == true) {、if (facingRight == false) {
を消してみたら足の位置でくっつくことがなくなりました！
また、
物が上から降ってきたとき、ジャンプして左右キーを放し上から物に着地したときが「衝突時に左右キーのどちらも押されていない場合」だと思うので、
コードは物のスクリプトに書いていたのですが、キャラのスクリプトにカット＆ペーストし書き換えて、キャラの向きによって物を移動させる位置を指定することにしました（物の向きで移動させるとキャラの反対側に移動することがあるため）。

めちゃくちゃ悩んでいたのでとても嬉しいです、ありがとうございました！

if (collision.gameObject.tag == "Koura") {
if (Input.GetKey (KeyCode.RightArrow)) {
m = transform.position;
m.x += 1.3f;
m.y += 0.15f;
collision.transform.position = m;
} else if (Input.GetKey (KeyCode.LeftArrow)) {
m = transform.position;
m.x -= 1.3f;
m.y += 0.15f;
collision.transform.position = m;
} else if (facingRight == true) {
m = transform.position;
m.x += 1.3f;
m.y += 0.15f;
collision.transform.position = m;
} else if (facingRight == false) {
m = transform.position;
m.x -= 1.3f;
m.y += 0.15f;
collision.transform.position = m;
}
}

[sakura_hana]

解決して何よりです。
余談ですが、この内容だと以下のように短く出来ます。

if (collision.gameObject.tag == "Koura") {
m = transform.position;
m.x += (Input.GetKey (KeyCode.RightArrow) || facingRight) ? 1.3f : -1.3f;
m.y += 0.15f;
collision.transform.position = m;
}

「||」→「または」の意味になります。（いずれかがtrueならtrue、全部falseならfalse）
「(〜〜) ? 値A : 値B」→三項演算子と言います。?の前がtrueなら値A、falseなら値Bが使用されます。
「facingRight」→bool型変数は==を省略すると「trueの場合」を示します。
今回は使いませんでしたが「!facingRight」とすると更に反転して「falseの場合」になります。

if文を連ねるのも流れが分かりやすいのですが、同じコードは何度も書かないようにするとデバッグや修正がしやすくなるのでオススメです。
プログラムではこういう「コードの見直し（良い書き方に直す）」ことを「リファクタリング」というので、余裕が出来たら調べてみるといいと思います。

[pokapoka2804]

三項演算子、本では見たことありましたがこういう時に使うんですね！
リファクタリング調べてみました。他の部分もリファクタリングできないか考えてみます！
今日はsakuraさんの丁寧な説明のおかげで目から１００枚くらい鱗が落ちました。
もっともっと経験を積んでかっこいい人になります！
そして、良いものを作ったり、悩んでいる人をお助けします
本当にありがとうございます！！