How would I make this change brick colors? I have yet to figure it out.

Asked by

5 years ago

Edited 5 years ago

I have this script that if you have a value, you should get a new hair color on a reset or join, but it doesn't work? Line 24 is the issue although there is no output error, I am almost positive it is, any help, heres the script

01local bucnhOfColor = {
02    [1] = "Black", --Default Shirt
03    [2] = "Pine Cone", -- Army green
04    [3] = "rbxassetid://2397305859", --Guess reckless
05    [4] = "rbxassetid://885261392", --Hawaiian shirt
06    [5] = "rbxassetid://3144003341", --JackMercando shirt
07    [6] = "rbxassetid://3152639503", -- Osito Pink Shirt
08    [7] = "rbxassetid://668772580", -- Red Jacket
09    [8] = "rbxassetid://554084413", -- Supreme Worldflags
10    [9] = "rbxassetid://2970602368", -- Tank top
11    [10] = "rbxassetid://2072675468", -- Tommy hilfiger
12    [11] = "rbxassetid://2641146107", -- Tour Jacket
13    [12] = "rbxassetid://604130756", -- Yellow puffer
14    [13] = "rbxassetid://393696145", -- Yellow puffer
15}
16 
17game.Players.PlayerAdded:connect(function(player)
18    player.CharacterAdded:Connect(function(character)       
19        local d = character:WaitForChild("Hat")
20        local val = player:WaitForChild("leaderstats").HairColor.Value
21        if val >= 1 then
22            wait(1)
23            if val >= #bucnhOfColor then
24                d.Handle.BrickColor = BrickColor.new(b ucnhOfColor[#bucnhOfColor])
25            else
26                d.Handle.BrickColor =  bucnhOfColor[val]
27            end
28        end
29    end)
30end)
 View all 30 lines...

use `BrickColor.new()` not `BrickColor.new[]` EteraW 77 — 5y

Use the brackets when referring to the item in the table AltNature 169 — 5y

brickcolor.new is a function, therefore it should have a `:()` beside it not a `:[]` AltNature 169 — 5y

Its still not working even with the () Pooglies 7 — 5y

View all comments (2 more)

Are the asset IDs written from the third key of the dictionary named "bunchOfColors" supposed to exist? And did it work when you obtain your first point? Afterl1ght 321 — 5y

So I had a shirt giver script, and now im trying to make a hair giver, so that is just leftover code until I get this fixed. Pooglies 7 — 5y

REASON

Have you read the output? When you assign a value to a BrickColor property, the new value should be a BrickColor datatype, did you notice? It appears that you're trying to apply a String (text) value to the Handle's BrickColor at line 26.

That being said, the new value should be BrickColor.new(bucnhOfColor[val]) instead.

Now, to prevent further... unwanted events from occurring, you should not include those asset IDs for shirts in that script anymore. Also, I now doubt that you're trying your best to fix the issue on your own, so if you aren't, you should try harder before asking us! :)

Also, if the Handle of the accessory has a Mesh and a texture, the texture (with little to no transparency) may override the Handle's color. You should remove its texture if it has one, or just keep it if you think the texture synchronizes with the color of the Part.

time to fix stuff

As regards what I first mentioned above, here is an abbreviated illustration on how to fix it.

1--yea it's simple
2--Your script from line 23
3if val >= #bucnhOfColor then
4    d.Handle.BrickColor = BrickColor.new(bucnhOfColor[#bucnhOfColor]) --this is correct
5else
6    d.Handle.BrickColor =  BrickColor.new(bucnhOfColor[val]) --this line was wrong, now it should be fixed.
7end

Simple as that!

Tell me if you have more issues arriving after. Cheers!

How would I make this change brick colors? I have yet to figure it out.

1 answer

REASON

time to fix stuff

Answer this question