Character accessories wont remove?

Grammarly_exe · October 26, 2024, 8:09pm

Im trying to remove the players accessories but not ones such as face, neck, hair, and hat. But when i tried running it nothing happened, no errors.

My script:

wait(0.5)


wait(0.5)

game.Players.PlayerAdded:Connect(function(player)
	player.CharacterAppearanceLoaded:Connect(function(char)
		for i, v in pairs(char:GetDescendants()) do
			if v:IsA("Accessory") then
				if v.AccessoryType == Enum.AccessoryType.Hat or Enum.AccessoryType.Face or Enum.AccessoryType.Hair or Enum.AccessoryType.Neck then
					return
				else
					v:Destroy()
				end
			end
		end
	end)
end)

Ive also looked at other posts but literally nothing is working

Iovedistrict · October 27, 2024, 12:25am

Have you verified that you’re using the correct operator?

It looks like your current condition checks if the player’s accessory type matches any of the listed options, which should return the accessory if it does.

Bones:

for i, v in pairs(char:GetDescendants()) do
    -- Consider changing this to GetChildren() for better performance
    if v:IsA("Accessory") then
        -- Check if the accessory type is a hat or any of the following
        if v.AccessoryType == Enum.AccessoryType.Hat or Enum.AccessoryType.Face or Enum.AccessoryType.Hair or Enum.AccessoryType.Neck then
            return -- Consider returning instead of destroying
        else
            v:Destroy()
        end
    end
end

Additionally, note that using or in this manner won’t work correctly. You must specify the comparison for each condition. For example, this line:

if v.AccessoryType == Enum.AccessoryType.Hat or Enum.AccessoryType.Face or ...

is incorrect. Instead, it should be:

if v.AccessoryType == Enum.AccessoryType.Hat or v.AccessoryType == Enum.AccessoryType.Face or ...

This ensures that each condition is properly evaluated.

Solution:

Change the operators or switch return and v:Destroy()