Camera is Constantly Shaking Slowly after Updating its Position

Question

Please watch this video and tell me a fix. Thanks.

The boolean flag must become false when my camera reaches its final position. Currently, the flag becomes true and remains true. I need to find a way where when my camera moves to its first destination the flag becomes false and so on.

Here is the code:

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

public class camMOVE : MonoBehaviour {
    public Transform  handleview;
    public Transform pressureview;
    public Transform wallview;
    public Transform sechandleview;
    public Transform pressuretwoview;
    public Transform switchview;

    public GameObject handlebtn;
    public GameObject pressurebtn;
    public GameObject wallbtn;
    public GameObject handletwobtn;
    public GameObject pressuretwobtn;
    public GameObject switchbtn;

    public float transitionSPEED;
    Transform currentVIEW;
    public bool flag = false;
    public bool isReached = false;
    Vector3 currentangel;
    public List<GameObject> modelparts;

    private void Start(){
        handlebtn.SetActive (true);
        pressurebtn.SetActive (false);
        wallbtn.SetActive (false);
        handletwobtn.SetActive (false);
        pressuretwobtn.SetActive (false);
        switchbtn.SetActive (false);

        foreach (GameObject obj in modelparts) {

            obj.GetComponent<BoxCollider> ().enabled = false;
        }
    }

    private void Update(){
        if (flag ) {
            transform.position = Vector3.Lerp (transform.position, 
    currentVIEW.position, Time.deltaTime * transitionSPEED);


            //for camera rotation
            currentangel = new Vector3 (Mathf.LerpAngle 
    (transform.rotation.eulerAngles.x, 
    currentVIEW.transform.rotation.eulerAngles.x, Time.deltaTime * 
    transitionSPEED),
                Mathf.LerpAngle (transform.rotation.eulerAngles.y, 
    currentVIEW.transform.rotation.eulerAngles.y, Time.deltaTime * 
    transitionSPEED),
                Mathf.LerpAngle (transform.rotation.eulerAngles.z, 
    currentVIEW.transform.rotation.eulerAngles.z, Time.deltaTime * 
    transitionSPEED));

            transform.eulerAngles = currentangel;
        }
    }

    public void Handleview(){
        currentVIEW = handleview;
        handlebtn.SetActive (false);
        flag = true;
    }

    public void Pressureview() {
        currentVIEW = pressureview;
        pressurebtn.SetActive (false);
        flag = true;
    }

    public void Wallview() {
        currentVIEW = wallview;
        wallbtn.SetActive (false);
        flag = true;
    }

    public void Secondhandleview(){
        currentVIEW = sechandleview;
        handletwobtn.SetActive (false);
        flag = true;
    }

    public void Pressuretwoview(){
        currentVIEW = pressuretwoview;
        pressuretwobtn.SetActive (false);
        flag = true;
    }

    public void Switchview(){
        currentVIEW = switchview;
        switchbtn.SetActive (false);
        flag = true;

    }
}

Answer 1

1. you're not setting it to false anywhere

2. you really don't do this in Update - you will have to simply familiarize yourself with using a coroutine

It is then very easy.

The basic form is

In your code ..

StartCoroutine(MoveCamera());

and then

private IEnumerator MoveCamera() {
  
  while ( not at the new position ) {

    move the object a little
    yield return null;
  }
  set object to final exact position
}

You'll find 1000 examples online! As a rule, never use Update in Unity.

---

In practice, just use Tweeng ...

Once you're comfortable with this. You can just use Tweeng.

Tweeng:

https://stackoverflow.com/a/37228628/294884

in your example, you would simply say

StartCoroutine( 3f.Tweeng(
     (p)=>transform.position=p,
     fromPosition,
     toPosition) );

That would do it in 3 seconds (ie, 3f );

Enjoy

Answer 2

Basically, I agree with the above answer by Fattie. You could just use a Coroutine as shown below

IEnumerator PerformCameraMovement(Transform transformToMove,Vector3 fromPos,Vector3 toPos,float time)
    {
        float i = 0;
        float rate = 1 / time;
        while (i<1)
        {
            i += Time.deltaTime * rate;
            transformToMove.position = Vector3.Lerp(fromPos,toPos,i);
            yield return 0;
        }
    }

here in your case, transformToMove is the transform of your camera, fromPos is the position from where a camera should start transitioning, toPos is the position to where the camera should reach and finally time is the amount of time required to transition the camera fromPos to toPos

Coroutines work just like a thread.

Just call this Coroutine by using StartCoroutine(PerformCameraMovement(cameraTransform,cameraTransform.position,Vector3.zero,2.0f));

and placing it wherever required.

Now in the above case, the camera would move from its original position to Vector3.zero (origin) in exactly 2 seconds.

I would advise avoiding the use of Update for transitioning. It just won't work.

If you need a deeper understanding of coroutines just go through this blog below http://www.theappguruz.com/blog/how-to-use-coroutines-in-unity