Why does this close so many of parts instead of just 1 of each??

local player = game.Players.LocalPlayer
local mageval = script.Parent.Parent
script.Parent.MouseButton1Click:connect(function(changecostume)
	local characterpart = player.Charachter:GetChildren()
	for i,v in pairs(characterpart()) do
		if v.ClassName == "CharacterMesh" then 
			v:remove()
		else
			print("NotAPart")
		end
	end
	wait(2)
	local costumechose = player.Costumes.Skeleton:GetChildren()
	for i,v in pairs(costumechose()) do
		if v.ClassName == "CharacterMesh" then 
			local clone = v:clone()
			clone.Parent = player.Character
		else
			print("NotAPart")
		end
	end
end)

why does this clone so many of the same parts? Do I need to add a debounce?