Scripting Helpers is winding down operations and is now read-only. More info→

How to select a NEW random number?

Asked by

5 years ago

So, im making this minigame type of game, and i need all the maps to be different. I tried using a loop to find a new random value, but it makes the game laggy and does not seem very efficient. I found something called Random.new() on the devforum, but have no idea how to use it. here's what i tried so far.

1local maps = game.ReplicatedStorage.Maps:GetChildren()
2local map1 = Random.new(1,#maps)
3local map2 = Random.new(1,#maps)
4local map3 = Random.new(1,#maps)
5script.Map1.Value = map1.Name
6script.Map2.Value = map2.Name
7script.Map3.Value = map3.Name

3 answers

Answered by

Elixcore 1337

5 years ago

Edited 5 years ago

Hey, try this and see if it works fine :)

1local maps = game.ReplicatedStorage.Maps:GetChildren()
2 
3for i = 1, 3, 1 do
4    script["Map"..i].Value = math.random(1, #maps)
5end

Additionally, if you want the maps that have been chosen already to not appear again, you will need to use tables.

here's how I'd do it.

1local maps = game.ReplicatedStorage.Maps:GetChildren()
2 
3 
4for i = 1, 3, 1 do
5    map = math.random(1,#maps)
6    script["Map"..i].Value = map
7    table.remove(maps, map)
8end

This will remove the map that has already been chosen so the next one that gets selected will not be the same.

before editing my answer, I recommend looking at the code context, Feahren. :) Elixcore 1337 — 5y

#maps defines the amount of time the loop iterates based on the amount of children. This way you don’t have to change 3 to how many new maps you add. With additive iteration as well, only two values need to be defined, I suggest writing 1,#maps Ziffixture 6913 — 5y

you got discord? Elixcore 1337 — 5y

I need to talk to you since it's pretty clear you don't understand what the issue is, it's easier to talk in there than here Elixcore 1337 — 5y

View all comments (5 more)

if not, I'll just say it all here. He is selecting specifically 3 random maps, not all of them. he needs only 3 values, not the number of maps that can be randomly chosen, think of it as having many things to choose from but you only need 3 of them, a full loop of all the available things would not work. Elixcore 1337 — 5y

also you had #maps, 1 which would error even if the context was correct because you did not add -1 as the third parameter of the loop so it would not start at all, consider removing the downvote cos what you did was completely wrong. Elixcore 1337 — 5y

@Feahren It is the NUMBER OF TIMES the loop will iterate, not the AMOUNT OF TIME it will take. DeceptiveCaster 3761 — 5y

no one care about my more indepth answer? :( SynthetickDev 188 — 5y

Right, I understand, my bad. I am aware of the semantic error I would’ve caused as well. I clearly resolved that in my previous comment Ziffixture 6913 — 5y

Answered by

SynthetickDev 188

5 years ago

Edited 5 years ago

That my friend is not how the Random class works. Let me explain.

The Random class allows you to create a new random object and you can also specify an optional seed. Such that: rand Random.new([string | float Seed]) using this newly created random object you have the choice of 2 functions one is: int rand:NextInteger(int Min, int Max) which returns a random integer between min and max. The other function you have is: float rand:NextNumber([float Min = 0, float Max = 1]) this returns a random float between min and max which will be 0 to 1 if not specified. The random object also has another function rand rand:Clone() which returns a new random object with the same seed and internal iteration.

So here is how you would use it:

1local rand = Random.new()
2 
3local randomInteger = rand:NextInteger(0, 10)
4 
5print(randomInteger) -- Output: 1, 2, ... 10

And I know this isn't a request site, but for your case you would do this:

01local maps = game.ReplicatedStorage.Maps:GetChildren()
02local rand = Random.new()
03 
04-- You also need to index the "maps" table in order to get the objects, otherwise you would just get a number between 1 and the number of maps
05local map1 = maps[rand:NextInteger(1, #maps)]
06local map2 = maps[rand:NextInteger(1, #maps)]
07local map3 = maps[rand:NextInteger(1, #maps)]
08script.Map1.Value = map1.Name
09script.Map2.Value = map2.Name
10script.Map3.Value = map3.Name

The above code can be simplified to:

01local maps = game.ReplicatedStorage.Maps:GetChildren()
02local rand = Random.new()
03 
04for i = 1, 3 do
05    local map = maps[rand:NextInteger(1, #maps)]
06    local value = script:FindFirstChild("Map"..i)
07    if value then
08        value.Value = map.Name
09    end
10end

I hope this helps!

Great answer, although i prefer something more simple. wish i could accept two. natonator63 29 — 5y

-2

Answered by

AlexanderYar 788

5 years ago

Can't you set a variable to a random integer.

1local map = math.random(1,10)
2If map == 1 then
3Elseif map == 2

ECT.

How to select a NEW random number?

3 answers

Answer this question