Scripting Helpers is winding down operations and is now read-only. More info→
Log in to vote
0
To many upvalues error, how to simplify?
Ik this code is terrible. And I could probably make this in just 20 lines. Could anyone help me make it have less code to solve the problem?
001 | local blockbreak = game.ReplicatedStorage.Blockbreakstage |
002 | local bk11 = blockbreak.Destroy1["1destroy"]:Clone() |
003 | local bk12 = blockbreak.Destroy1["2destroy"]:Clone() |
004 | local bk13 = blockbreak.Destroy1["3destroy"]:Clone() |
005 | local bk14 = blockbreak.Destroy1["4destroy"]:Clone() |
006 | local bk15 = blockbreak.Destroy1["5destroy"]:Clone() |
007 | local bk16 = blockbreak.Destroy1["6destroy"]:Clone() |
008 |
009 | local bk21 = blockbreak.Destroy2["1destroy"]:Clone() |
010 | local bk22 = blockbreak.Destroy2["2destroy"]:Clone() |
011 | local bk23 = blockbreak.Destroy2["3destroy"]:Clone() |
012 | local bk24 = blockbreak.Destroy2["4destroy"]:Clone() |
013 | local bk25 = blockbreak.Destroy2["5destroy"]:Clone() |
014 | local bk26 = blockbreak.Destroy2["6destroy"]:Clone() |
015 |
View all 233 lines...
1 answer
Log in to vote
3
What is an upvalue
An upvalue is an external local variable; a local variable used in a function body but declared outside of it. I believe the maximum amount of upvalues is about 60. You have waaaayyyy too many variables here. And yes, your code can severely be simplified.
Loops
Use loops! This will shorten and simplify your code.
This code works. Make sure you put it inside the event listener.
01 | for i = 1, 10 do |
02 | local current_destroy = blockbreak["Destroy" .. i] |
03 |
04 | for j = 1, 6 do |
05 | local current_bk = current_destroy[j .. "destroy"]:Clone() |
06 | current_bk.Parent = target |
07 | wait(waitspeed) |
08 | current_bk:Destroy() |
09 | end |
10 | end |
0
Really good awsner. Helped be a lot! Skydoeskey 108 — 5y
1
-1 for not giving wiki link Robowon1 323 — 5y
0
You need mental help, there is no wiki link on upvalues and their explanations are like worst programmerHere 371 — 5y