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

Why is this error coming up when I run my code? And how can I stop it?

Asked by

Longjohnnn 28

4 years ago

Edited 4 years ago

I keep on getting this error when I run this code, but I have no idea why.

01Move.OnServerEvent:Connect(function(person, key, Value, MouseValue)-- Key detection for all players
02    print(Value, CanMove1, grounded)
03    if Value == true and CanMove1 == true and grounded == false then --and MouseValue == false then
04                print(key)
05        if key == W then
06            print(person.name)
07            Name = game.Workspace:WaitForChild(person.name.."Tempman")
08            CanMove2 = true
09            print("Should be true")
10        end if key == A then
11            print(person.name)
12            Name = game.Workspace:WaitForChild(person.name.."Tempman")
13            CanMove3 = true
14        end if key == S then
15            print(person.name)
16            Name = game.Workspace:WaitForChild(person.name.."Tempman")
17            CanMove4 = true
18        end if key == D then
19            print(person.name)
20            Name = game.Workspace:WaitForChild(person.name.."Tempman")
21            CanMove5 = true
22        end if key == Space then
23            print(Space)
24            Name = game.Workspace:WaitForChild(person.name.."Tempman")
25            CanMove6 = true
26        end if key == R then
27            Name = game.Workspace:WaitForChild(person.name.."Tempman")
28            Name:Destroy()
29            function spawnman(person)-- change this when you're gonna do more keys
30        end
31    elseif Value == false then
32        if key == W then
33            CanMove2 = false
34        elseif key == A then
35            CanMove3 = false
36        elseif key == S then
37            CanMove4 = false
38        elseif key == D then
39            CanMove5 = false
40        end
41        if key == "Mouse" then
42            print(MouseValue)
43            Name = game.Workspace:WaitForChild(person.name.."Tempman")
44            --local Mousex = Vector3.new(0,MouseValue.X * -82 + 180, 0)
45            --local Var = Name.Union.Orientation
46            local Var2 = Name.Union.Orientation + one
47            Name.Union.Orientation = Var2
48            --Var -= Vector3.new(0,MouseValue.X * -82 + 180, 0)
49            --Name.Union.Orientation = Var + Vector3.new(0,MouseValue.X * -82 + 180, 0)
50            Name.Head.Orientation = Vector3.new(MouseValue.Y * -80,0, 0) + Name.Union.Orientation
51            --Name.Focus.Orientation = Name.Head.Orientation
52            Name.Focus.CFrame = Name.Head.CFrame * CFrame.new(4,2.5,6)
53            --local Orient = Name.Focus.Orientation
54            Name.Neck.Orientation = Name.Union.Orientation
55            --Name.Head.Orientation = Orient
56            one = Vector3.new(0,MouseValue.X * -85 + 180, 0) - Name.Union.Orientation
57        end
58    end
59end
 View all 59 lines...

The error is on the last end. Here is the error

ServerScriptService.Spawn/Move:197: Expected 'end' (to close 'function' at line 129), got <eof>; did you forget to close 'then' at line 169?

This code is extremely inefficient. The error in layman's terms means that you forgot to close an open scope, and is instead met with the End Of File. You didn't close your if-block from line 3; there are meant to be four 'end' keywords present. Ziffixture 6913 — 4y

Yeah I thought there would probably be a more efficient way to do that, but how? Longjohnnn 28 — 4y

Oh wait I see it. Wow that's terrible. Longjohnnn 28 — 4y

oh wait Longjohnnn 28 — 4y

View all comments (3 more)

this script is for all players in the server hence why I'm not using elseif Longjohnnn 28 — 4y

I think it would be a good idea to use one script for all the players movement right? Longjohnnn 28 — 4y

Please send me your full code and the :FireServer() code happy_gagarara12 13 — 4y

1 answer

Answered by

happy_gagarara12 13

4 years ago

Edited 4 years ago

01Move.OnServerEvent:Connect(function(person, key, Value, MouseValue)-- Key detection for all players
02    print(Value, CanMove1, grounded)
03    if Value == true and CanMove1 == true and grounded == false then --and MouseValue == false then
04        print(key)
05        if key == W then
06            print(person.name)
07            Name = game.Workspace:WaitForChild(person.name.."Tempman")
08            CanMove2 = true
09            print("Should be true")
10        end if key == A then
11            print(person.name)
12            Name = game.Workspace:WaitForChild(person.name.."Tempman")
13            CanMove3 = true
14        end if key == S then
15            print(person.name)
16            Name = game.Workspace:WaitForChild(person.name.."Tempman")
17            CanMove4 = true
18        end if key == D then
19            print(person.name)
20            Name = game.Workspace:WaitForChild(person.name.."Tempman")
21            CanMove5 = true
22        end if key == Space then
23            print(Space)
24            Name = game.Workspace:WaitForChild(person.name.."Tempman")
25            CanMove6 = true
26        end if key == R then
27            Name = game.Workspace:WaitForChild(person.name.."Tempman")
28            Name:Destroy()
29            function spawnman(person)-- change this when you're gonna do more keys
30            end
31                end
32        elseif Value == false then
33            if key == W then
34                CanMove2 = false
35            elseif key == A then
36                CanMove3 = false
37            elseif key == S then
38                CanMove4 = false
39            elseif key == D then
40                CanMove5 = false
41            end
42            if key == "Mouse" then
43                print(MouseValue)
44                Name = game.Workspace:WaitForChild(person.name.."Tempman")
45                --local Mousex = Vector3.new(0,MouseValue.X * -82 + 180, 0)
46                --local Var = Name.Union.Orientation
47                local Var2 = Name.Union.Orientation + one
48                Name.Union.Orientation = Var2
49                --Var -= Vector3.new(0,MouseValue.X * -82 + 180, 0)
50                --Name.Union.Orientation = Var + Vector3.new(0,MouseValue.X * -82 + 180, 0)
51                Name.Head.Orientation = Vector3.new(MouseValue.Y * -80,0, 0) + Name.Union.Orientation
52                --Name.Focus.Orientation = Name.Head.Orientation
53                Name.Focus.CFrame = Name.Head.CFrame * CFrame.new(4,2.5,6)
54                --local Orient = Name.Focus.Orientation
55                Name.Neck.Orientation = Name.Union.Orientation
56                --Name.Head.Orientation = Orient
57                one = Vector3.new(0,MouseValue.X * -85 + 180, 0) - Name.Union.Orientation
58            end
59        end
60    end
61end)
 View all 61 lines...

Here, you forgot "end)" at the end of your code. Hope It worked

Edit: you also forgot to close 'then'

Why is this error coming up when I run my code? And how can I stop it?

1 answer

Answer this question