Wsrtjtyk

Hey I want to make my player invicible, when he takes damage and loose a health. ATM I cant make my player invicible, and also if he collides with several damage structures he takes more damage than intended I want him to lose 20% of his health and then be invincible for 2-3 sec.

This is attached to my damage structure

using System.Collections;

using System.Collections.Generic;

using UnityEngine;



    public class LoseHealth : MonoBehaviour {







       private void OnTriggerEnter2D(Collider2D collision)

        {

            if (collision.CompareTag("Player"))

            {

                HealthBarScript.health -= 20f;

            }



        }

    }

This is for my FullHearts on the canvas attached as a child to my "empty hearts"

  using System.Collections;

using System.Collections.Generic;

using UnityEngine;

using UnityEngine.UI;

using UnityEngine.SceneManagement;



public class HealthBarScript : MonoBehaviour {





    Image FullHearts;

    float maxHealth = 100f;

    public static float health;

    // Use this for initialization

    void Start () {

        FullHearts = GetComponent<Image>();

        health = maxHealth;

    }



    // Update is called once per frame

    void Update () {

        FullHearts.fillAmount = health / maxHealth;

        if (health == 0)

        {

            Application.LoadLevel(Application.loadedLevel);

        }

    }

}

ATM I have this on my playerscript which is not working all other scripts are working but just need it to only be 1 heart to lose no matter if the player hits multiple structures and then invicible to have time to recover and not hit the structure :

  if (!invincible)

        {

            if (col.gameObject.tag == "enemy")

            {

               // HealthBarScript.health -= 20f;

                // health -= 20; // subtract 1 form your total health

                invincible = true; // makes this whole function unusable since invincible is no longer false

                new WaitForSeconds(3);

                invincible = false; // makes this whole function reusable since invincible is false again

            }





        }

So I just made a simple collision 'game' using the method you provided and everything works fine. But now as I look further, I am seeing some redundancy in what you provided and your 'answer' is a bit confusing.

But your first script shows that every time the damage dealer collides with 'Player' it does damage.

private void OnTriggerEnter2D(Collider2D collision)

        {

            if (collision.CompareTag("Player"))

            {

                HealthBarScript.health -= 20f;

            }



        }

AND every time the 'Player' collides with an 'enemy' it does an invincible check which should be working. (Assuming the comments are removed). But as the comments stand there is no damage from the 'Player' hitting the damage structure but always (regardless of invincible) damage from the structure to the player.

if (!invincible)

    {

        if (col.gameObject.tag == "enemy")

        {

            // HealthBarScript.health -= 20f;

            // health -= 20; // subtract 1 form your total health

            invincible = true; // makes this whole function unusable since invincible is no longer false

            StartCoroutine(Invincible());                   // makes this whole function reusable since invincible is false again

        }





    }

}





IEnumerator Invincible()

{

    if (invincible == true)





    {

        yield return new WaitForSeconds(2);

        {





            invincible = false;

        }

    }

But when you hit again before the invincible check expires the script on the enemy is still dealing damage.

You have two scripts doing damage and only one does an invincible check.

Get rid of the damaging script on the structures, tag all the structures properly, comment the script on the player and you will be good to go. Or you can keep the script on the damage structures and just have each one check if the player is invincible by making that bool static.

Also you could have easily answered your own question with standard usage of Debug.Log("what the heck is happening"). Debug.Log should be your best friend.