I am in need of help, my script when a player touches a part, they get money is not working, why?

:WaitForChild() is a method, which means it uses a colon to call it. .Touched() is an event, the T is capitalized.

You also need a way to prevent spam, we'll use something called a debounce. It's a way for the script to check if we can touch the brick again, it's just a bool.

local Goal = script.Parent
local debounce = true

local function onModelTouched(hit)
	if hit and hit.Parent and debounce then 
		debounce = false
		local player = game.Players:GetPlayerFromCharacter(hit.Parent)
		local character = player and player.Character
		local humanoid = character and character:FindFirstChild("Humanoid")
		-- if player isn't dead
		if humanoid and humanoid.Health > 0 then 
			local leaderstats = player:FindFirstChild("leaderstats")
			local Money = leaderstats and leaderstats:FindFirstChild("Money")
			if Money then
				-- Edit the value, not the object.
				Money.Value = Money.Value + 1
			else
				print (" can't find stats ", leaderstats, Money)
			end
		end
		-- prevent spam 
		wait(1)
		debounce = true
	end
end

Goal.Touched:Connect(onModelTouched)