Hearth based health system help

You've attached a '.unitypackage' file, you should instead attach your C# file.

I would have checked the script out but sorry I'm not gonna open unitypackages.

Welcome to Unity! Most new developers share the same mistake that you did. Writing the script. On answers.unity3d.com, there is a feature where you can copy/paste your script to make it readable. Here is a tutorial:

This bar on the top has a feature which is the button. Click that, then a text box will come up and then you can paste your script there. Hope this helps!

The script in general seems very well made. There's one or two small logic errors, and areas which could probably be improved performance wise, but apart from that it's very good for your first self-made script.

Good practice would be to mark all of the serialized variables as private.

A more efficient way of getting the current children would be using a list (https://msdn.microsoft.com/en-us/library/6sh2ey19(v=vs.110).aspx), so instead of getting 'transform.getchild' you can get the child which would be stored in the list (this assumes you only add children in this script). Transform.child can be quite heavy so you should figure out a way to add and remove the correct amount of children from the list using the logic you already have.

Also the if statement around the first for loop is redundant, the for loop wont fire if childcount is greater than maxvalue because the 'i' would be greater than maxvalue-childcount.

The final thing I see is that you are not casting 'clone' to a gameobject, it would create an error if you tried to use it, so do this instead:

 GameObject clone = (GameObject)Instantiate(childPrefab, transform);