Unity Hinge Jointのスクリプトについて

Question

前提・実現したいこと

UnityでUFOキャッチャーをつくっています。
右にいって奥に行ったらアームを開くようにしたいです。
アームの開き方はHinge Jointでしています。
Hinge JointのMotorのtargetVelocityをいじりたいですがうまくいきません。

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

特にありません

該当のソースコード

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

public class LeftArm : MonoBehaviour
{
    [Header("開いたかどうか？")]
    public bool ArmOpen = false;

    [Header("Script")]
    [SerializeField]
    private UFO ufo;
    [SerializeField]
    private BodyHit bodyHit;
    [SerializeField]
    private ArmHit armHit;
    [SerializeField]
    private ArmHit _armHit;

    void Update()
    {
        var hinge = GetComponent<HingeJoint>();
        var Motor = hinge.motor;
        hinge.motor = Motor;

        if (ufo.RightMove == true && ufo.BackMove == true)
        {
            ArmOpen = true;
        }

        if (armHit.CloseStart == true || bodyHit.CloseStart == true || _armHit.CloseStart == true)
        {
            ArmOpen = false;
        }

        if (ArmOpen == true)
        {
            Motor.targetVelocity = 100f;
        }
        else Motor.targetVelocity = -100;
    }
}

試したこと

Unityの設定画面でMotorの値をいじったらできました
Unityのスクリプトリファレンスも読んで作ってみましたがうまく動きませんでした。
HingeJoint.motor

Answer 1

ジョイントからモーターを取得した直後にジョイントへモーターを再設定してしまっているご様子ですが、そうではなくて...

    void Update()
    {
        var hinge = GetComponent<HingeJoint>();
        var Motor = hinge.motor;

        if (ufo.RightMove == true && ufo.BackMove == true)
        {
            ArmOpen = true;
        }

        if (armHit.CloseStart == true || bodyHit.CloseStart == true || _armHit.CloseStart == true)
        {
            ArmOpen = false;
        }

        if (ArmOpen == true)
        {
            Motor.targetVelocity = 100f;
        }
        else Motor.targetVelocity = -100;

        hinge.motor = Motor;
    }

という順序にしてみてはいかがでしょう。HingeJoint.motorはJointMotor型ですが、冒頭に...

struct in UnityEngine

と記されております。こういった構造体を返すプロパティの場合、「取得」→「書き換え」→「再代入」の手順を踏むケースが多いように思います。たとえばTransform.positionを書き換えてオブジェクトを移動させようとするときは、

var position = transform.position;
transform.position = position;
position.x = 10.0f;

ではなく、

var position = transform.position;
position.x = 10.0f;
transform.position = position;

とするでしょう。ParticleSystemの各種モジュールやNativeArrayのような特別な注意を要する例外はあるかもしれませんが、おおよそのstructはこのルールにならうかと思います。

Comments

[Sasaran]

ありがとうございます！無事に動きました！