Unity c# You are trying to create a MonoBehaviour using the 'new' keyword error

Hey guys so I got this annoying error, I don't get why in Unity we can't access other classes like this. I can't even inherit because you can't have 2 base classes. My other script has a variable which I want to reset the number after the battery is collected but it aint working.

Code (CSharp):

1. using UnityEngine;
2. using System.Collections;
3.  
4. public class BatteryPickup : MonoBehaviour {
5.  
6.    
7.  
8.     // Use this for initialization
9.     void Start ()
10.     {
11.        
12.     }
13.    
14.     // Update is called once per frame
15.     void Update ()
16.     {
17.         RaycastHit detectBatteries;
18.         if (Physics.Raycast(transform.position, transform.forward, out detectBatteries, 4))
19.         {
20.             if (detectBatteries.collider.tag == "Batteries" && Input.GetKeyDown(KeyCode.Q))
21.             {
22.                 Debug.Log("hi");
23.                 Destroy(detectBatteries.collider.gameObject);
24. //here is fails
25.                 flashlight ewa = new flashlight();
26.                 ewa.flashlightTimer = 55;
27.             }
28.         }
29.     }
30. }

 

use gameObject.AddComponent<flashlight>();

Or make the flashlight not inherit from MonoBehaviour.

You're not allowed to call new because Unity wants to be sure it has setup MonoBehaviours correctly using AddComponent so you don't get null gameObject variables etc..

 

What Antony said. Also, you can have an inheritance chain of MonoBehaviours just fine.

One more things: you are currently raycasting every frame, but are only doing something with it if the user is pressing Q - this is very wasteful. You should check if the user is pressing Q first, and only then do the raycast.

 

Oh I see thanks for feedback, I have been told optimization is for later, but I will do it thanks. Should I just copy and paste that code into start function?

 

I did that but how do I access the variable and reset it to 55?

 

help please

 

Code (csharp):

1.  
2. flashlight ewa;
3.  
4. void Start()
5. {
6.      ewa = gameObject.AddComponent<flashlight>();
7. }
8.  
9. void Update()
10. {
11.      // the rest of your code goes here
12.      {
13.           ewa.flashlightTimer = 55;
14.      }
15. }
16.  

I think this is wrong though, as you'd be adding a flashlight to a gameObject that is a battery pack. is this what you want to be doing? Wouldn't you already have a flashlight somewhere and you'd be adding the battery to that? In which case you'd want to get the flashlight from somewhere else rather than adding a new one.

 

Doesn't work, when I type AddComponent it doesn't pop up.

 

Sorry,

ewa = gameObject.AddComponent<flashlight>();

p.s. I edited my previous post to correct that.

 

Works! But when I start the game I get this error

MissingComponentException: There is no 'Light' attached to the "First Person Controller" game object, but a script is trying to access it.
You probably need to add a Light to the game object "First Person Controller". Or your script needs to check if the component is attached before using it.
flashlight.Update () (at Assets/scripts/flashlight.cs:33)

Do you want the code from my flashlight script?

 

Not really.

I think you need to read through the Unity manual and do some tutorials to get a better understanding of how Unity works.