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

First four parts stay where they are, last part gets removed. Howcome?

Asked by

6 years ago

01local Player = game.Players.LocalPlayer
02local Mouse = Player:GetMouse()
03Mouse.KeyDown:Connect(function(Key)
04    Key = Key:lower()
05    if Key == 'e' then
06        Part = Instance.new('Part', game.Workspace)
07        Part.Size = Vector3.new(2, 12,3)
08        Part.Material = ("Ice")
09        Part.CanCollide = false
10        Part.Anchored = true
11        Part.CFrame = Player.Character.Head.CFrame * CFrame.new(0,-2,-2)
12        wait(1)
13        Part:Destroy()
14    end
15end)
16local Player = game.Players.LocalPlayer
17local Mouse = Player:GetMouse()
18Mouse.KeyDown:Connect(function(Key)
19    Key = Key:lower()
20    if Key == 'e' then
21        wait(0.1)
22        Part = Instance.new('Part', game.Workspace)
23        Part.Size = Vector3.new(2, 12,3)
24        Part.Material = ("Ice")
25        Part.CanCollide = false
26        Part.Anchored = true
27        Part.CFrame = Player.Character.Head.CFrame * CFrame.new(0,-2,-5)
28        wait(1)
29        Part:Destroy()
30    end
31end)
32local Player = game.Players.LocalPlayer
33local Mouse = Player:GetMouse()
34Mouse.KeyDown:Connect(function(Key)
35    Key = Key:lower()
36    if Key == 'e' then
37        wait(0.2)
38        Part = Instance.new('Part', game.Workspace)
39        Part.Size = Vector3.new(2, 12,3)
40        Part.Material = ("Ice")
41        Part.CanCollide = false
42        Part.Anchored = true
43        Part.CFrame = Player.Character.Head.CFrame * CFrame.new(0,-2,-8)
44        wait(1)
45        Part:Destroy()
46    end
47end)
48local Player = game.Players.LocalPlayer
49local Mouse = Player:GetMouse()
50Mouse.KeyDown:Connect(function(Key)
51    Key = Key:lower()
52    if Key == 'e' then
53        wait(0.3)
54        Part = Instance.new('Part', game.Workspace)
55        Part.Size = Vector3.new(2, 12,3)
56        Part.Material = ("Ice")
57        Part.CanCollide = false
58        Part.Anchored = true
59        Part.CFrame = Player.Character.Head.CFrame * CFrame.new(0,-2,-11)
60        wait(1)
61    Part:Destroy()
62    end
63    end)
64local Player = game.Players.LocalPlayer
65local Mouse = Player:GetMouse()
66Mouse.KeyDown:Connect(function(Key)
67    Key = Key:lower()
68    if Key == 'e' then
69        wait(0.4)
70        Part = Instance.new('Part', game.Workspace)
71        Part.Size = Vector3.new(2, 12,3)
72        Part.Material = ("Ice")
73        Part.CanCollide = false
74        Part.Anchored = true
75        Part.CFrame = Player.Character.Head.CFrame * CFrame.new(0,-2,-15)
76        wait(1)
77        Part:Destroy()
78    end
79    end)
 View all 79 lines...

The title didn't let me explain all the details, so what i'm trying to figure out is why does the last part only get destroyed and the first four parts stay where they are when they appear? Alphagoreguy -2 — 6y

It creates lag and just makes the place look messy. Alphagoreguy -2 — 6y

Mouse.KeyDown is deprecated. User#19524 175 — 6y

Also, why are you creating new KeyDown events every time? User#19524 175 — 6y

View all comments (7 more)

On line3, you already created an event listener. User#19524 175 — 6y

What do i use then? Alphagoreguy -2 — 6y

So if i get rid of all the other event listeners will it fix my Destroy problem? Alphagoreguy -2 — 6y

Remove all of the event listens except for line 3. There was no need to create new ones. You shouldn’t be using KeyDown anyways. User#19524 175 — 6y

And on line 8, remove the brackets as they are redundant. User#19524 175 — 6y

Wait, why can't i use KeyDowns? Alphagoreguy -2 — 6y

And i've removed all the other Event listens and now the script doesn't work Alphagoreguy -2 — 6y

1 answer

Answered by

bearpro2 6

6 years ago

Edited 6 years ago

First of all, KeyDown is deprecated. It works in Studio but not in the actual game. Second, you wasted a lot of time making the same variables over and over. Same with all the extra functions.

Instead of using KeyDown, use the UserInputService. Also, don't make the same variables and functions over and over.

A shortened example of what you're doing:

01local player = game.Players.LocalPlayer
02local UIS = game:GetService("UserInputService")
03UIS.InputBegan:Connect(function(input, gpe)
04    if not gpe and input.KeyCode == Enum.Q then
05        part = Instance.new("Part", game.Workspace)
06        part.Anchored = true
07        part.CFrame = player.Character.Head.CFrame * CFrame.new(0, -2, -15)
08        wait(1)
09        part:Destroy()
10        wait()
11        partA = Instance.new("Part", game.Workspace)
12        partA.Anchored = true
13        partA.CFrame = player.Character.Head.CFrame * CFrame.new(0, -2, -15)
14        wait(1)
15        partA:Destroy()
16    end
17end)

These are a couple mistakes. The answer to your question is lag...... I think. Also, even if Keydown wasn't deprecated, its pointless to do the "Key = Key:lower()" part. Hope this helps

Line 5 and 11 have deprecated code. The parent parameter of Instance.new is deprecated. User#19524 175 — 6y

Oh alrighty, thanks mate Alphagoreguy -2 — 6y

First four parts stay where they are, last part gets removed. Howcome?

1 answer

Answer this question