Is this the right way to use a function, or is the whole script wrong?

ZombieA is in lighting and gets cloned and gets put in workspace. This script is for when the zombie dies and the player gets money, which is called RealMoney and is in the player and not in the leaderboard.

local ZombieA = game.Workspace:WaitForChild("ZombieA")

game:GetService('Players').PlayerAdded:Connect(function(player)
        player.CharacterAdded:Connect(function(character)
            ZombieA:WaitForChild("Zombie").Died:Connect(function(H)
	while H == true
	do
		player.RealMoney.Value = player.RealMoney.Value + 1000
		
	end
            
				
			
				
			
            end)
        end)
    end)