Teleportation Script is No Longer Working /: ?

Asked by

6 years ago

Edited by User#24403 5 years ago

So I have a murder-mystery game, i'm trying to make a script that teleports the player once a TextLabel changes, and then teleports them back to the lobby after it changes back. But for some reason, it will teleport fine, but when it goes to teleport back, it will teleport back to the map spawn point. Can someone look over my script and tell me the cause of this please?

01game.Players.PlayerAdded:Connect(function(player)
02    while true do
03    local Tab = {CFrame.new(-907, 5.4, -307); CFrame.new(-708, 73.4, -338); CFrame.new(-719, 73.4, -243); CFrame.new(-696, 55.4, -288); CFrame.new(-747, 6, -279); CFrame.new(-650, 5, -294); CFrame.new(-712, 8.6, -193); CFrame.new(-903, 5, -425); CFrame.new(-902, 8.6, -176); CFrame.new(-679, 55.4, -366); CFrame.new(-768, 60.2, -394); CFrame.new(-673, 38.6, -373); CFrame.new(-726, 38.6, -253); CFrame.new(-727, 55.4, -330)}
04    local locationselected = Tab[math.random(1, #Tab)]
05    local Spawnplates = {CFrame.new(19.511, 18.27, -42.511); CFrame.new(-31.86, 18.069, -18.085); CFrame.new(36.297, 19.375, -23.067); CFrame.new(-23.937, 18.17, -34.292); CFrame.new(-7.526, 18.27, -45.252); CFrame.new(-31.86, 18.069, -18.085); CFrame.new(37.096, 18.471, 9.724); CFrame.new(-35.261, 18.471, 2.364); CFrame.new(19.511, 18.27, -42.511)}
06    local spawnselected = Spawnplates[math.random(1, #Spawnplates)]
07    player.PlayerGui:WaitForChild("statusbar").TextLabel.Changed:Connect(function()
08    game.Workspace:WaitForChild(player.Name).Head.CFrame = locationselected
09    end)
10    player.PlayerGui:WaitForChild("statusbar").TextLabel.Changed:Connect(function()
11    game.Workspace:WaitForChild(player.Name).Head.CFrame = spawnselected
12end)
13    wait()
14    end
15    end)

you’re handling gui from the server, shouldn’t do that User#19524 175 — 6y

will that fix the problem? SBlankthorn 329 — 6y

If I move it to a local script in startergui, it doesnt even work anymore. SBlankthorn 329 — 6y

That's what i said to him. Completely ignored ;/ Delude_d 112 — 6y

View all comments (4 more)

So does anyone here know if I can just say "Do this changed event the 1st time, do this one the 2nd?" SBlankthorn 329 — 6y

OH User#19524 175 — 6y

TextLabel.Changed:Disconnect() TextLabel.Changed:Connect(function() end) it creates a new connection User#19524 175 — 6y

You can use characteradded in local script. You cannot use playeradded however. xxXTimeXxx 101 — 6y

First off, I want to start by pointing out that your code looks very messy. Indent your code properly.

Here is something you should know about GUI's and Teleportation.

Teleportation should be handled in Server Script and GUI's should be handled in Local Script. In your script, you are trying to do from the Server Script which is not a good idea.

Now, I am not sure what is going on here. I do have couple questions about it that I think you should make clear

Are Tab the positions of the places?

Are SpawnPoint the spawns of the Tab?

Because you are attempting to randomize both the tables and you are trying to move them to both locations? What do you plan on doing if the both random location are different?

Assuming that Tab is SpawnPoint , I will keep going.

This is a general idea on how your code should look like.

Client Sided Script

01-- [Declaration Section]
02--\\ Service Stuff
03local Player            = game:GetService("Players").LocalPlayer;
04local PlayerGui         = Player:WaitForChild("PlayerGui");
05local ReplicatedStorage = game:GetService("ReplicatedStorage");
06 
07--\\ Your User Interfaces goes here. 
08local StatusBar         = PlayerGui:WaitForChild("StatusBar");
09local TextLabel         = StatusBar:WaitForChild("TextLabel");
10 
11--\\ Remote Event 
12local Event             = ReplicatedStorage:WaitForChild("Event");
13 
14--\\ Maps Amount Variable; Maximum number of CFrame goes here. 
15local L_Amount          = 14;
16local S_Amount          = 9;
17 
18-- [Processing Section]
19local function On_Label_Change ()
20    print("Invoked");
21    local Num1          = math.random(1, L_Amount); 
22    local Num2          = math.random(1, S_Amount);
23 
24    if TextLabel.Text  == "Intermission (1 Minute)" then
25        Event:FireServer("Intermission", Num1)
26    elseif TextLabel.Text == "Round In Progress (3 Minutes)" then 
27        Event:FireServer("Finish", Num2);
28    else
29        return;
30    end;
31end;
32 
33-- [Connecting Section]
34TextLabel.Changed:Connect(On_Label_Change);
 View all 34 lines...

Server Sided Script

001-- [Declaration Section]
002--\\ Service Stuff
003local ReplicatedStorage = game:GetService("ReplicatedStorage");
004 
005--\\ Remote Event 
006local Event             = ReplicatedStorage:WaitForChild("Event");
007 
008-- \\ Your tables; declare here. 
009local Locations         = {
010     [1] = CFrame.new(-907, 5.4, -307);
011     [2] = CFrame.new(-708, 73.4, -338);
012     [3] = CFrame.new(-719, 73.4, -243);
013     [4] = CFrame.new(-696, 55.4, -288);
014     [5] = CFrame.new(-747, 6, -279);
015     [6] = CFrame.new(-650, 5, -294);
016     [7] = CFrame.new(-712, 8.6, -193);
017     [8] = CFrame.new(-903, 5, -425);
018     [9] = CFrame.new(-902, 8.6, -176);
019    [10] = CFrame.new(-679, 55.4, -366);
020    [11] = CFrame.new(-768, 60.2, -394);
021    [12] = CFrame.new(-673, 38.6, -373);
022    [13] = CFrame.new(-726, 38.6, -253);
023    [14] = CFrame.new(19.511, 18.27, -42.511);
024};
025 
026local Spawnplates       = {
027    [1]  = CFrame.new(19.511, 18.27, -42.511); 
028    [2]  = CFrame.new(-31.86, 18.069, -18.085); 
029    [3]  = CFrame.new(36.297, 19.375, -23.067); 
030    [4]  = CFrame.new(-23.937, 18.17, -34.292); 
031    [5]  = CFrame.new(-7.526, 18.27, -45.252); 
032    [6]  = CFrame.new(-31.86, 18.069, -18.085); 
033    [7]  = CFrame.new(37.096, 18.471, 9.724); 
034    [8]  = CFrame.new(-35.261, 18.471, 2.364); 
035    [9]  = CFrame.new(19.511, 18.27, -42.511)
036};
037-- [Processing Section]
038local function On_Server_Event (Player, Name, Amount)
039    local HRP           = Player.Character:WaitForChild("HumanoidRootPart");
040 
041    if Name == "Intermission" then
042        print("Intermission");
043        if Amount == 1 then
044            HRP.CFrame  = Locations[1];
045        elseif Amount == 2 then
046            HRP.CFrame  = Locations[2];         
047        elseif Amount == 3 then
048            HRP.CFrame  = Locations[2];
049        elseif Amount == 4 then
050            HRP.CFrame  = Locations[2];
051        elseif Amount == 5 then
052            HRP.CFrame  = Locations[2];
053        elseif Amount == 6 then
054            HRP.CFrame  = Locations[2];
055        elseif Amount == 7 then
056            HRP.CFrame  = Locations[2];
057        elseif Amount == 8 then
058            HRP.CFrame  = Locations[2];
059        elseif Amount == 9 then
060            HRP.CFrame  = Locations[2];
061        elseif Amount == 10 then
062            HRP.CFrame  = Locations[2];
063        elseif Amount == 11 then
064            HRP.CFrame  = Locations[2];
065        elseif Amount == 12 then
066            HRP.CFrame  = Locations[2];
067        elseif Amount == 13 then
068            HRP.CFrame  = Locations[2];
069        elseif Amount == 14 then
070            HRP.CFrame  = Locations[2];
071        else
072            return;
073        end;
074 
075    elseif Name == "Finish" then
076        print("Finish");
077        if Amount == 1 then
078            HRP.CFrame  = Spawnplates[1];
079        elseif Amount == 2 then
080            HRP.CFrame  = Spawnplates[2];
081        elseif Amount == 3 then
082            HRP.CFrame  = Spawnplates[3];
083        elseif Amount == 4 then
084            HRP.CFrame  = Spawnplates[4];
085        elseif Amount == 5 then
086            HRP.CFrame  = Spawnplates[5];
087        elseif Amount == 6 then
088            HRP.CFrame  = Spawnplates[6];
089        elseif Amount == 7 then
090            HRP.CFrame  = Spawnplates[7];
091        elseif Amount == 8 then
092            HRP.CFrame  = Spawnplates[8];
093        elseif Amount == 9 then
094            HRP.CFrame  = Spawnplates[9];
095        else
096            print("no");
097            return;
098        end;
099    end;
100end;
101 
102-- [Connecting Section]
103Event.OnServerEvent:Connect(On_Server_Event);
 View all 103 lines...

Teleportation Script is No Longer Working /: ?

1 answer

First off, I want to start by pointing out that your code looks very messy. Indent your code properly.

Client Sided Script

Server Sided Script

Answer this question