Does anyone know why this simple player camera lock script is not working?

Unity Engine
1

Hello, I am making a simple 2D game and I am trying to lock the camera to the player using a C# script. The script works when I do not put the positions of the player and camera into Vector3 variables, but when I do the camera does not lock to the player and I do not see any errors appear in the console. Here is my code:

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

public class CameraLock : MonoBehaviour {

    public GameObject player;

    private Vector3 offset;
    private Vector3 cameraPos;
    private Vector3 playerPos;

    private void Awake()
    {
        cameraPos = transform.position;
        playerPos = player.transform.position;
    }

    private void Start()
    {
        offset = cameraPos - playerPos;
    }

    private void Update()
    {
        cameraPos = playerPos + offset;
    }
}

Can anyone tell me what I am doing wrong? Am I using the wrong variable types for storing the x,y, and z values of the positions? Thanks in advanced.

2

This should work (I don’t understand why you have the Awake and Start methods, when their content can easily go into the Start only, but it should work) Are you sure you’re placing the script onto the camera? Did you assign the player gameobject to the player variable?

3

Yes, I did all of that. I moved the assignments to Awake when I attempted to troubleshoot. This is the version that works:

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

public class CameraLock : MonoBehaviour {

    public GameObject player;

    private Vector3 offset;
    //private Vector3 cameraPos;
    //private Vector3 playerPos;

    private void Start()
    {
        // cameraPos = transform.position;
        // playerPos = player.transform.position;
        offset = transform.position - player.transform.position;
    }

    private void Update()
    {
        transform.position = player.transform.position + offset;
    }
}
4

Since cameraPos and playerPos values are assigned in start / awake, they arent updating. Simply assigning them again in the update function would (should) work, or you could do something like this :

    public GameObject player;

    private Vector3 offset;
    private Transform cameraTrans;
    private Transform playerTrans;


    private void Start()
    {
        offset = cameraTrans.position - playerTrans.postion;
    }

    private void Update()
    {
        cameraTrans.position = playerTrans.position + offset;
    }

I think.

5

In addition to what original-name stated you will probably want to use late update.

void LateUpdate()

as this will allow all change to the player object to take effect before moving the camera.