Random loot script help

Help and Feedback Scripting Support
#21

Ok, I changed it to a script but still nothing.
I am sorry to put you through all this but I would not have done all this by myself

#22 
math.randomseed(tick())

script.Parent.Touched:connect(function(hit)
    print(“A”)
    if hit.ParentFindFirstChild(“Humanoid”) then
        print(“B”)
        script.Disabled = false
        local player = game.Players:FindFirstChild(hit.Parent.Name)
        local random = math.random(1,100)

        if random > 0 and random < 50 then
            print(“0-50”)
            local sword1 = game.ReplicatedStorage.sword1:Clone() 
            sword1.Parent = player.BackPack
        elseif random > 50 and random < 100 then
            print(“50-100”)
            local pistol1 = game.ReplicatedStorage.Pistol1:Clone()	
        	pistol1.Parent = player.Backpack
        end

        script.Parent:Destroy()
    end
end)
#23

Add print and show me the output

#24

Do you want me to test then take a screenshot with the output

#25

ok here we go:

screenshot num 4
screenshot num 41456×852 187 KB

#26

The error is in the print maybe your “ are wrong or something like that
Create an script and copy the:

 print(“hello world”)

Just modify the hello world into A or B, etc

#27

ok well I found this:

Screenshot num 5
Screenshot num 51455×847 162 KB

#28

Oh, change the line 5 into this

hit.Parent:FindFirstChild(“Humanoid”)
#29

well… found this too:

Screenshot num 6
Screenshot num 61453×848 169 KB

#30

Oh sorry :neutral_face:, use this:

if hit.Parent:FindFirstChild(“Humanoid”) then
#31

well… again:

Screenshot num 7
Screenshot num 71455×847 169 KB

#32

NO, change in the line 5 :slight_smile:
The third was correct

#33

well… sorry about that one thats all me

1 Like
#34

ok so now there is no errors and when I touched the “crate” it disappeared but gave me nothing.

#35

@baseballcal15 Can you show me the output?

#36

ok so here’s this:

Screenshot num 8
Screenshot num 81917×670 276 KB
(The rest of the output is just my tycoon stuff, and also there is stuff in my tycoons that need touch up so ignore that)

#37

Generally you can define the player and test to see if it is a player, which removes the need for a humanoid

script.Parent.Touched:Connect(function(hit)
   local player = game.Players:GetPlayerFromCharacter(hit.Parent)
   if player then
      local randomNumber = math.random(1,100)
      if randomNumber  <= 50 then
         game.ReplicatedStorage.sword1:Clone().Parent = player.Backpack
      else
         game.ReplicatedStorage.pistol1:Clone().Parent = player.Backpack
      end
      script.Parent:Destroy()
   end
end)
#38

wow. kingerman yours worked thank you
Also thank you to gogols who helped out through everything.

#39

No problem, if you don’t mind marking it as the solution so people know that this has been solved :slight_smile:

#40

just did that but thank you to you both again. :grin:

2 Likes