No, thats local variable! Everytime you run the command, a new oldHealth is declared.
Floats have a default value of 0.0 = death
Isn't the [MAX_PLAYERS] wasting too many cells? [playerid] would do the same work won't it? Yes I know that it's declared as local but the cell size is global.