[HELP,C#] Update() Function not running at all!

Hello Everyone!

I have coded this to change weapons in my fps game. The weapon change, start() and SelectWeapon() functions work properly but the update() function doesn't work at all.

Code (CSharp):

1. using UnityEngine;
2.  
3. public class WeaponControllerSwitching : MonoBehaviour
4. {
5.     public int selectedWeapon = 0;
6.  
7.     // Start is called before the first frame update
8.     void Start()
9.     {
10.         SelectWeapon();  
11.     }
12.  
13.     // Update is called once per frame
14.     void Update()
15.     {
16.         int previousSelectedWeapon = selectedWeapon;
17.         Debug.Log("Updating");
18.         if (Input.GetAxis("Mouse ScrollWheel") > 0f)
19.         {
20.             if (selectedWeapon >= transform.childCount - 1)
21.                 selectedWeapon = 0;
22.             else
23.                 selectedWeapon++;
24.  
25.         }
26.         if (Input.GetAxis("Mouse ScrollWheel") > 0f)
27.         {
28.             if (selectedWeapon <= transform.childCount - 1)
29.                 selectedWeapon = 0;
30.             else
31.                 selectedWeapon--;
32.  
33.         }
34.  
35.         if (previousSelectedWeapon != selectedWeapon)
36.         {
37.             SelectWeapon();
38.         }
39.     }
40.  
41.     void SelectWeapon()
42.     {
43.         int i = 0;
44.         foreach (Transform weapon in transform)
45.         {
46.             if (i == selectedWeapon)
47.                 weapon.gameObject.SetActive(true);
48.             else
49.                 weapon.gameObject.SetActive(false);
50.             i++;
51.         }
52.     }
53. }
54.  

 

Hi,

What do you mean with "doesn't work at all." Please elaborate and explain. Do you mean that your Update function does not run at all? Or do you mean that the code in your Update does not do what you imagine it should do? Or something else, like you get some errors you don't know how to deal with?

 

Yes , sorry for not being clear.
I believe the update() function is supposed to run every frame but it doesn't even run once!

 

If you put to your Update something like:

Code (CSharp):

1. Debug.Log("Update running.");

Do you see anything in your console window? Always first check the basics if something seems to be failing.

EDIT: just out of curiosity I checked your code, it does run just fine here, no errors when compiling. So there must be some logic error how you expect it to function. Or there's that unlikely situation that the Update really doesn't run. But try that Debug print first to see if it's running there.

 

I assume that the Update function is called, but doesn't result in anything happening.

Reason for this is that you have 2 if-clauses that compare the same thing "if (Input.GetAxis("Mouse ScrollWheel") > 0f)". The second one probably needs to be "< 0f".

What currently happens is that the first if-clause does selectedWeapon++ and then the second if-clause does selectedWeapon-- .. essentially putting you on the weapon you started with again. So no weapon is switched.

EDIT: spelling + layout

 

Open Visual Studio, set some breakpoints, click "Attach to Unity" and play the game in Unity. There you can see what's wrong line by line.

 

The OP might just have the object disabled or have not added script to any gameobject in active scene. Update will never run in both cases.

 

@palex-nx yes of course. That's why I asked to test if the script produces any debug prints... It's a good place to start.

 

Yes, the code compiles , no errors after compiling or while playing game.

Yes , I put the Debug.Log("Updating") in the code and it doesn't output in the console!

 

Yes but if the function is callimg amd running,then it should have outputted "Updating" string to the console which it doesn't.

 

The weapons ( theres like 3 weapons object ) are children of a empty game object Melee. The melee is child of fps_hands object.

I have attached the script to the Melee empty game object.
The Melee empty game object are enabled always (i have checked it in inspecter)

 

Almost certainly, the object is getting deactivated. Just just need to confirm that and find out why, and this code will help you do that:

Code (csharp):

1. void OnDisable() {
2. Debug.Log("We got disabled!");
3. }

I'm fairly certain that at some point this object is getting disabled or destroyed, and this will help you find out where. If you see this line, click on it in the console, and look at the lines below it in the console. That's called the stacktrace, and it's basically the path the code took to get to that function. If you check out where it came from, you should be able to see why it got disabled.

 

Okay, I will test the code out.

 

I put the debug.log inside the update() function. Codes in the start() work as expected.