Roblox OnClick clothers giver script

Question

pnts = script.Pants 
shirt = script.Shirt 

function onClicked(playerWhoClicked) 

end 
function GiveClothes(character) 
if not character:findFirstChild("Shirt") then 
shirt:Clone().Parent = character 
else character:findFirstChild("Shirt"):Destroy() 
shirt:Clone().Parent = character 
end 

if not character:findFirstChild("Pants") then 
pnts:Clone().Parent = character 
else character:findFirstChild("Pants"):Destroy() 
pnts:Clone().Parent = character 
end 
end 

game.Players.PlayerAdded:connect(function(p) 
p.CharacterAdded:connect(function(char) 
wait(1.12) 
local plr = game.Players:findFirstChild(char.Name) 
print(char.Name) 

local groupid = 0 -- Id of your group 

local plr = game.Players:GetPlayerFromCharacter(part.Parent) 

if plr then 

if plr:IsInGroup(groupid) then 

if plr:GetRoleInGroup(groupId) >= 50 
then GiveClothes(char) 
end 
end 
end 

script.Parent.ClickDetector.MouseClick:connect(onClicked) 

Этот скрипт должен дать вам определенную одежду, если вы нажмете на кнопку (скрипт находится под кнопкой, и да есть также ClickDetector), но это дает вам одежду, если вы являются определенным рангами в Группе.

Но в настоящее время это не работает. Как это исправить?

Answer 1

Избегайте странные имена, как pnts, вы, вероятно, не умереть от добавления письма a

The ClickDetector already give you the player

И это по всему грязному

Попробуйте это:

local Pants = script.Pants 
local Shirt = script.Shirt 

local GroupID = 42 -- Group id here 

script.Parent.ClickDetector.MouseClick:connect(function(Player) 
    if not Player:IsInGroup(GroupID) then return end 
    if Player:GetRoleInGroup(GroupID) < 50 then return end 
    local Character = Player.Character 
    if Character == nil then return end 

    -- Get new shirt 
    local CharacterShirt = Character:findFirstChild("Shirt") 
    if CharacterShirt then CharacterShirt:Destroy() end 
    Shirt:Clone().Parent = Character 

    -- Get new pants 
    local CharacterPants = Character:findFirstChild("Pants") 
    if CharacterPants then CharacterPants:Destroy() end 
    Pants:Clone().Parent = Character 
end) 

Также не забудьте отправить errors, когда вы обратитесь за помощью