Well, having a lot of localscripts and they all are active, might unnecessarily be taking up performance. It is recommended that you handle things that work similarly in a single script rather than a script for each similar/semi-similar thing.
Since it seems like all the buttons function the same in your case, except probably to where the player is being teleported, you should have, for example, either an IntValue in each button, or a key for each button, in a table in the script with the value being the Id of the instance that you want the player to be teleported to. In that case, you can have a single script function with the exact same code for each button, by making it look for the Id that button has (wants to teleport the player to) in either the IntValue as a child of that button, or by a key used in a table to retrive that button's Id. The keys could be the names of the buttons, since that won't matter in the gameplay.
1 comment
There will be some overhead in setting up a script enviroment. As long as you can maintain the code then there should be no real issues. That being said making one script per button is just supid.