Shots

Hello guys

I try to create script for shots, but it is not working with any key, eventhough the unity is not displaying any errors in control panel. Could you take a look?

[System.Serializable]
public class Boundary
{
public float xMin, xMax, yMin, yMax;
}

public class PlayerController : MonoBehaviour
{
public float speed;
public float tilt;
public Boundary boundary;

public GameObject shot;
public Transform shotSpawn;
public float fireRate;

private float nextFire;

void update()
{
if (Input.GetButton("Fire1") && Time.time > nextFire)
{
nextFire = Time.time + fireRate;
Instantiate(shot, shotSpawn.position, shotSpawn.rotation);
}
}

void FixedUpdate ()

{
float moveHorizontal = Input.GetAxis("Horizontal");
float moveVertical = Input.GetAxis ("Vertical");

Vector3 movement = new Vector3(moveHorizontal, moveVertical, 0.0f);
GetComponent<Rigidbody>().velocity = movement * speed;


Thank you!

GetComponent<Rigidbody>().position = new Vector3
(

Mathf.Clamp(GetComponent<Rigidbody>().position.x, boundary.xMin, boundary.xMax),
Mathf.Clamp(GetComponent<Rigidbody>().position.y, boundary.yMin, boundary.yMax)
);

GetComponent<Rigidbody>().rotation = Quaternion.Euler(0.0f, 0.0f, GetComponent<Rigidbody>().velocity.x * -tilt);
}
}

 

Debug.Log it!

 

Unity special methods always start with a capital letter. so update should be Update

And use code tags the next time you post code.

 

Not wrong, but technically all methods in C# should start with a capital letter. The same goes for class and property names.

 

Well, stylistically yes, but the Unity ones are the ones that will make code actually fail to function, as opposed to just hard to read.

 

Sorry, I'm new, will use tags for the future

The debug showed " PlayerController.cs(29,23): error CS1002: ; expected

 

Ok the upper case for :Update" solved the problem, thanks guys!