How do I fix my (killer) choosing script?
Hi reader, I was wondering how I can fix up this script, It's using math.random to choose a random player out of the players. It's coming up with errors saying 'Attempt to index local killer a number value' but I don't want it to be a number, I want it to be the chosen players name. I was wondering if anyone could help fix my script.
1 | for i, player in pairs (game.Players:GetPlayers()) do |
2 | local Killer = math.random(1,1) |
3 | workspace[Killer.Name]:MoveTo(game.Workspace.spawntest) |
4 | end |
2 answers
math.random returns numbers, not players. With this, you can use select a random player from a list of players.
1 | local Players = game.Players:GetPlayers() -- This list won't automatically update, so be sure to run this again after any time has passed. |
2 | local Killer = Players[ math.random(#Players) ] --This generates a random number between 1 and the number of players, and uses that as an index to select the player from the list. |
3 | Killer.Character:MoveTo(game.Workspace. |
You should also put math.randomseed(tick()) before you generate random numbers to make them seem more random. Random numbers generated by computers aren't truly random, but using a unique number as a seed will make the output appear random.
Edit: The # operator only works on tables. In this case, the table it applies to is Players - a list created by calling game.Players:GetPlayers(). You don't need a loop to do any of this.
0
0
0
0
0
Ok then.
It looks like you (for whatever reason) expected math.random() to give you a userdata rather than an integer.
So for starters, lets create a valid example of how to use math.random(). Say I want to print a random number from 1 through 10
1 | num = math.random(1,10) |
2 | print(num) |
or
1 | print(math.random(1,10)) |
What about choosing a random userdata!??!
We're getting there, first I want to do a little briefing on tables:
1 | tab = {} -- tab is our table |
2 | table.insert(tab,'oof') -- this inserts the string "oof" into our table |
3 | print(#tab) -- this will print the length of the table |
4 | print(tab[1]) -- this will print the first value of the table, which is "oof" |
5 | table.remove(tab, 1) --this removes the first value of the table |
Now we're there!
Now, if this is a script in game.ServerScriptService:
01 | plrs = {} |
02 | game.Players.PlayerAdded:Connect(function(plr) |
03 | table.insert(plrs, plr.Name) |
04 | end) |
05 |
06 | game.Players.PlayerRemoving:Connect(function(plr) |
07 | for i,v in pairs (plrs) do |
08 | if v == plr.Name then table.remove(plrs,i) |
09 | end |
10 | end |
11 | end) |
12 |
13 | function pickrandomplayer() |
14 | local num = math.random(1,#plrs) |
15 | workspace[plrs[num]]:MoveTo(game.Workspace.spawntest) |
16 | end |
Then you run pickrandomplayer() whenever you need to pick a player.
Note: You should probably research the roblox wiki before you ask questions here.
0