Here's the code I have so far I think the problem is they share there Health.

mob/NPC//Defines a /mob/NPC
icon = 'Othersablar.dmi'
icon_state = "npc"
proc/move()//Now, this proc handles NPC movement
for(var/mob/M in oview())//loops over all mobs in view
if(get_step_away(src,M,3))//checks distance
if(!M.client)//Not a player
continue//continues through the loop
else//Player
walk_to(src,M,1,0)//Move to the person
else//4-or-more spaces away
continue
spawn(20) move()//loops the proc after 2 seconds
proc/attackplayer()//Handles attacking
for(var/mob/M in oview(1))//Within one space this time
src.Health-=10
spawn() DeathCheck()

New()//When created
attackplayer()//Calls the procs for it
move()