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

How to make sure no BrickColor is repeated when changing a group of brick's BrickColor randomly?

Asked by

9 years ago

01door = script.Parent  
02local list = game.ServerStorage.Colors:GetChildren()
03local list2 = game.Workspace.Colors4.pads:GetChildren()
04local pm = game.ServerStorage.pm.Value
05function onChatted(msg, recipient, speaker) 
06local num = 0
07 
08 
09local source = string.lower(speaker.Name) 
10msg = string.lower(msg) 
11 
12if (msg == ":randomize") then
13 
14repeat
15 
16list2[math.random(#list2)].BrickColor = list[math.random(#list)].Value
17num = num + 1
18print(num)
19wait()
20until num == 9
21end
22num = 0
23end
24 
25function onPlayerEntered(newPlayer) 
26newPlayer.Chatted:connect(function(msg, recipient) onChatted(msg, recipient, newPlayer) end) 
27end
28 
29game.Players.ChildAdded:connect(onPlayerEntered) 
 View all 29 lines...

The script I have works fine but it has 1 problem. I do not want the color to be repeated onto any of the bricks. I tried using variables in a earlier version of this script but that did not work.

Thanks if you decide to help out, thanks if you just read this. - UltChowsk

2 answers

Answered by

TheDeadlyPanther 2460

9 years ago

01door = script.Parent  
02local list = game.ServerStorage.Colors:GetChildren()
03local list2 = game.Workspace.Colors4.pads:GetChildren()
04local pm = game.ServerStorage.pm.Value
05function onChatted(msg, recipient, speaker) 
06    local num = 0
07    --------------------------
08    function colorCheck(tab,val) -- function, so you can do multiple times
09        for _,v in pairs (tab) do
10            if v == val.Value then
11                return false
12            else
13                return true
14            end
15        end
16    end
17    ------------------------------
18    local source = string.lower(speaker.Name) 
19    msg = string.lower(msg) 
20    -----------------------------------
21    local colors = {} -- table for brickcolors
22    -----------------------------------
23    if (msg == ":randomize") then
24 
25    repeat
26    ---------------------------------
27    function change() -- so you can do multiple times
28        local c = list[math.random(#list)]
29        if colorCheck(colors,c) == true then
30            list2[math.random(#list2)].BrickColor = c.Value
31        else
32            change()
33        end
34    end
35    change()
36    --------------------------------
37    num = num + 1
38    print(num)
39    wait()
40    until num == 9
41    end
42    num = 0
43end
44 
45function onPlayerEntered(newPlayer) 
46newPlayer.Chatted:connect(function(msg, recipient) onChatted(msg, recipient, newPlayer) end) 
47end
48 
49game.Players.ChildAdded:connect(onPlayerEntered) 
 View all 49 lines...

I put it in and got a stack overflow. UltChowsk 85 — 9y

Any idea how to prevent that? UltChowsk 85 — 9y

It may be becausen you used up all of the brickcolors available. TheDeadlyPanther 2460 — 9y

I have 9 seperate colors. Thats what the local variable list is for, I am trying to get it to pick from those 9 to change 9 different bricks without any repeats. UltChowsk 85 — 9y

View all comments (2 more)

Then it should work. TheDeadlyPanther 2460 — 9y

I got a stack overflow every time I tried. UltChowsk 85 — 9y

Answered by

UltChowsk 85

9 years ago

I fixed it, I used another script to reset the values and it all works fine now. :D

How to make sure no BrickColor is repeated when changing a group of brick's BrickColor randomly?

2 answers

Answer this question