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

Trying to learn about the Pathfinding Service but my MoveTo is not working?

Asked by

ScriptingEngine 5

6 years ago

WARNING: NOT THE PRETTIEST CODE IM JUST TRYING TO LEARN HOW TO USE THE PathfindingService

So the problem is with the MoveTo on the followPath function on line 37

01local pathFindingService = game:GetService("PathfindingService");
02local path = pathFindingService:CreatePath();
03local NPC = script.Parent;
04local NPC_Humanoid = NPC.Humanoid;
05 
06local waypoints;
07local currentWaypointIndex;
08 
09local function findNearestPlayer(position)
10    local lowest = math.huge;
11    local nearestPlayer;
12 
13    for _, v in pairs(game.Players:GetPlayers()) do
14        if v then
15            if v.Character then
16                local distance = v:DistanceFromCharacter(position);
17                if distance <= lowest then
18                    lowest = distance;
19                    nearestPlayer = v;
20                    print(nearestPlayer.Name.." is closest player at a distance of :".. distance );
21                    return nearestPlayer;
22                end
23            end
24        end
25    end
26end
27 
28local function followPath(player)
29    path:ComputeAsync(NPC.HumanoidRootPart.Position,    player.Character.HumanoidRootPart.Position);
30 
31    waypoints = {};
32    if path.Status == Enum.PathStatus.Success then
33        waypoints = path:GetWaypoints();
34        print("lets see if i did a thing");
35        currentWaypointIndex = 1;
36 
37        NPC_Humanoid:MoveTo(waypoints[currentWaypointIndex].Position);
38        print("now it should be moving");
39    else
40        NPC_Humanoid:MoveTo(NPC.HumanoidRootPart.Position);
41    end
42 
43end
44 
45local function onWaypointReached(reached)
46    if reached and currentWaypointIndex < #waypoints then
47        currentWaypointIndex = currentWaypointIndex + 1;
48        NPC_Humanoid:MoveTo(waypoints[currentWaypointIndex].Position);
49    end
50end
51 
52local function onPathBlocked(blockedWaypoint)
53 
54    if blockedWaypoint > currentWaypointIndex then
55        local nearestPlayer = findNearestPlayer(NPC.HumanoidRootPart.Position);
56        followPath(nearestPlayer);
57    end
58end
59 
60 
61while true do
62    wait(15);
63    local nearestPlayer = findNearestPlayer(NPC.HumanoidRootPart.Position);
64 
65    if nearestPlayer then
66        followPath(nearestPlayer);
67    end
68end
69 
70 
71path.Blocked:Connect(onPathBlocked)
72 
73NPC_Humanoid.MoveToFinished:Connect(onWaypointReached)
 View all 73 lines...

It does change the WalkToPoint but the NPC (Clone of my character from in game) does not actually walk to that point I can't figure out why

Here is the WalkToPoint before MoveTo was called

Here it is after the MoveTo

and here is a gif showing the uneventful MoveTo (output should say "now it should be moving" after MoveTo was called)

also sorry for hurting your eyes with this ugly code I just want to figure this out I have been stuck for almost a day now .-. It worked like twice then it just stopped working.

alvinbloxx taught me put a wait at the top of my script so try that User#24403 69 — 6y

Shut up incapaz ScriptingEngine 5 — 6y

Are any of the parts in the character anchored? The humanoid's walkspeed is not 0 and PlatformStand is off, right? User#20279 0 — 6y

Nope, 16 and yes ScriptingEngine 5 — 6y

@ScriptingEngine greatneil80 2647 — 6y

2 answers

Answered by

ihatecars100 502

6 years ago

It's because you're connecting the events after the while loop. As the while loop doesn't let anything else after it run, the events will never connect.

Move the connects to before the while loop, and it should be fine.

01path.Blocked:Connect(onPathBlocked)
02 
03NPC_Humanoid.MoveToFinished:Connect(onWaypointReached)
04 
05while true do
06    wait(5);
07    local nearestPlayer = findNearestPlayer(NPC.HumanoidRootPart.Position);
08 
09    if nearestPlayer then
10        followPath(nearestPlayer);
11    end
12end

Answered by

arshad145 392

6 years ago

Hello,

01local pathFindingService = game:GetService("PathfindingService");
02local path = pathFindingService:CreatePath();
03local NPC = script.Parent;
04local NPC_Humanoid = NPC.Humanoid;
05 
06local waypoints;
07local currentWaypointIndex = 1;
08 
09local function findNearestPlayer(position)
10    local lowest = math.huge;
11    local nearestPlayer;
12 
13    for _, v in pairs(game.Players:GetPlayers()) do
14        if v then
15            if v.Character then
16                local distance = v:DistanceFromCharacter(position);
17                if distance <= lowest then
18                    lowest = distance;
19                    nearestPlayer = v;
20                    print(nearestPlayer.Name.." is closest player at a distance of :".. distance );
21                    return nearestPlayer;
22                end
23            end
24        end
25    end
26end
27 
28local function followPath(player)
29    path:ComputeAsync(NPC.HumanoidRootPart.Position,    player.Character.HumanoidRootPart.Position);
30 
31    waypoints = {};
32    if path.Status == Enum.PathStatus.Success then
33        waypoints = path:GetWaypoints();
34        -- currentWaypointIndex = 1; -- You are declaring the index being 1.
35        print(currentWaypointIndex)
36        NPC_Humanoid:MoveTo(waypoints[currentWaypointIndex].Position);
37    else
38        NPC_Humanoid:MoveTo(NPC.HumanoidRootPart.Position);
39    end
40 
41end
42 
43local function onWaypointReached(reached)
44    if reached and currentWaypointIndex < #waypoints then
45        currentWaypointIndex = currentWaypointIndex + 1;
46    NPC_Humanoid:MoveTo(waypoints[currentWaypointIndex].Position);
47    end
48end
49 
50local function onPathBlocked(blockedWaypoint)
51 
52    if blockedWaypoint > currentWaypointIndex then
53        local nearestPlayer = findNearestPlayer(NPC.HumanoidRootPart.Position);
54        followPath(nearestPlayer);
55    end
56end
57 
58 
59while true do
60    wait(15);
61    local nearestPlayer = findNearestPlayer(NPC.HumanoidRootPart.Position);
62    if nearestPlayer then
63        followPath(nearestPlayer);
64    end
65end
66 
67 
68path.Blocked:Connect(onPathBlocked)
69 
70NPC_Humanoid.MoveToFinished:Connect(onWaypointReached)
 View all 70 lines...

So, having redacted the mistake, I'll proceed to explain.

In the while loop, you are checking for the nearestPlayer and "following" it with the function followPath, for now everything was good, but on Line 24, you were declaring the currentWaypointIndex to be 1, constantly as it is inside of a while loop.

To counter this, we make currentWaypointIndex a "global" in the scope of this script; Line 7.

This should fix your issue, any further questions DM me.

PS:Please accept this as an answer if this helped you.

Trying to learn about the Pathfinding Service but my MoveTo is not working?

2 answers

Answer this question