Collision detection is not working

Hello, i have a little problem, my script wont detect collision with the ennemies, i tried with other gameobject but its not working!

Code (CSharp):

1. void OnCollisionEnter(Collision col)
2.     {
3.         if (col.gameObject.name == "Monster!(Clone)")
4.         {
5.             Debug.Log("has collide");
6.             currentHealth -= 5;
7.             healthBar.value = CalculateHealth();
8.         }
9.     }

 

Just some basic troubleshooting: if you place a Debug.Log call above your condition, does that execute? This tells whether the problem lies in the collision or in the subsequent name comparison.

 

no its not executing

 

Did you see this part of the manual?
"Note: Collision events are only sent if one of the colliders also has a non-kinematic rigidbody attached. Collision events will be sent to disabled MonoBehaviours, to allow enabling Behaviours in response to collisions."
https://docs.unity3d.com/ScriptReference/MonoBehaviour.OnCollisionEnter.html

Also, checking with a name like that is no good, super prone to error and very limited.
My suggestion to you is to use tags.

 

Or checking against the script directly can avoid nasty tags as well.
E.g.

Code (CSharp):

1. Monster mon = col.GetComponent<Monster>();
2. if (mon != null) // Do something

 

Why do its not working with a rigidbody?

 

Do i need to activate "IS trigger"?

 

https://docs.unity3d.com/Manual/CollidersOverview.html

Also it depends which object you put this script on.

 

the script is on my player

 

You are using OnCollisionEnter instead of OnTriggerEnter. I think it's the problem, and you are also comparing with game object name, and you should use tags instead. Try something like this:

Code (CSharp):

1. void OnTriggerEnter(Collider other)
2.     {
3.         if (other.tag == "Ennemies") // I have used the same tag that you attached to your enemy character
4.         {
5.             Debug.Log("has collide");
6.             currentHealth -= 5;
7.             healthBar.value = CalculateHealth();
8.         }
9.     }

Hope it helps you.
Remember to set one of the colliders as IsTrigger

 

Finally !!! thanks you so much @diasrodrigo !

 

thx xd