Feedback 2D Enemy Script

Hello everyone! I'm helping create a game for a college final and am not very well versed in C# or computer programming as a whole. I created an enemy script that is supposed to trigger an enemy attack when the player is close. For reference, I used an enemy tutorial from inScope Studios on Youtube and modified it to use OnTriggerEnter rather than OnCollider.

Code (CSharp):

1. using System.Collections;
2. using System.Collections.Generic;
3. using UnityEngine;
4.  
5. public class EnemyScript : MonoBehaviour
6. {
7.     [SerializeField]
8.     private Animator animator;
9.  
10.     private Transform enemy;
11.  
12.     private GameObject Enemy;
13.  
14.     // Start is called before the first frame update
15.     void Start()
16.     {
17.  
18.     }
19.  
20.     // Update is called once per frame
21.     void Update()
22.     {
23.  
24.     }
25.  
26.     public void OnTriggernEnter(string enemytag, GameObject other)
27.     {
28.         if (enemytag == "Enemy" && other.tag == "Player")
29.         {
30.             other.GetComponent<PlayerMoveScript>().TakeHit();
31.         }
32.     }
33.  
34.     public void StopAttack()
35.     {
36.         animator.SetBool("Attack", false);
37.     }
38.     public void Attack()
39.     {
40.         GameObject go = Instantiate(Enemy, enemy.position, Quaternion.identity);
41.         Vector3 direction = new Vector3(transform.localScale.x, 0);  
42.  
43.     }
44.  
45. }

The script unfortunately does not work and I am not sure how to fix it.

 

public void OnTriggernEnter(string enemytag, GameObject other)

Your code does not match the required method signature for OnTriggerEnter. Please see the documentation and example code here. It must match this pattern or it will not work:
https://docs.unity3d.com/ScriptReference/MonoBehaviour.OnTriggerEnter.html

 

Thank you for letting me know! I fixed it to match and added a line of code that calls for a comparison of the game object tags to trigger an enemy hit but I don't think it's doing anything.

Code (CSharp):

1.  public void OnTriggerEnter(Collider other)
2.     {
3.         if (gameObject.tag == "Enemy" && other.tag == "Player")
4.         {
5.             other.GetComponent<PlayerMoveScript>().TakeHit();
6.         }
7.     }

 

That was actually my mistake! I didn't realize I had added an extra n.
And here is the link:

I tried doing it his way, but the addition of an interface seemed too complicated for what I'm trying to create since non of my enemies fire projectiles and have very simple attacks.