I was suggesting that whatever Module.new returned had its own function to begin movement instead of passing it to a function as a parameter. In retrospect, seeing as this is a wrapper for PathfindingService, this wouldn’t really be easy without rewriting some crucial parts of the code.
2 Likes
Would you like me to make an example Object Oriented pathfinding module?
Its a lot better for intuition and it saves having to make path an argument for every function.
1 Like
Please refer to:
1 Like
Hi, when i use the array the npc stops before it gets to the first checkpoint no matter how close I put the checkpoint, this does not happen using the single checkpoint point method.
Julian
1 Like
Noted. Thanks for the report. I will get back to you when the bug has been fixed. Although it might be on ROBLOX’s end with pathfinding. Either or, it should be fixed soon.
You really don’t need to wait for Replicated storage. I’ve never had to.
It is better practice and I rather use WaitForChild rather than indexing it right away as ReplicatedStorage may not have loaded in yet. Besides that is part of an example of how to use it. If you are not satisfied with the way I set it up, then that is your fault and not my issue to take care of.
Its just personally bad practice ¯_(ツ)_/¯
Then you can’t view what’s inside replicated storage, IVE never seen replicated storage or any service not exist;
Your not really understanding my point nor know bad/good practices. WaitForChild is actually a way better practice than just flat out indexing it. Many other alternatives can be used as well, such as game:GetService. As said before, if you don’t like the example I put, then that is your own problem and not my problem to deal with.
Could you please explain more thoroughly what you mean?
It is more of the fact that it is better practice not to just index services using game.Service. Your not understanding my point at all. It is my personal preference to use GetService and WaitForChild instead of indexing. Also, using GetService or WaitForChild is indeed a better practice. As I have said like a million times, if you don’t like it that it not my fault, it is yours.
Once again, this is not an issue with my module. If you don’t like it, change it. That’s why it is considered an example. I don’t really know how else to explain it.
1 Like
I’m confused about your choice to use a constructor function but not call any functions from the value it returns… is this supposed to add support for multiple paths at once?
As @OminousVibes0 said, constructor functions are generally reserved for Object-Oriented Programming and it doesn’t really seem to be contributing anything here.
It would be helpful if you could add some documentation for every function of the module and not just the “extra” ones.
1 Like
Thanks for reminding me. Totally forgot to document all the other non-extra functions. Don’t know how I missed that. Thanks again for the reminder. Ill do that soon as I haven’t really reviewed this module in for ages.
2 Likes
Big fan of your EZ pathfinding Modules, I am currently using it for my Telltale Engine!
1 Like
I’m glad to hear people are using this in their games! Good luck to you and your Telltale Engine!
Would this work with parts or only NPCs?
This works with parts also however it uses a different method with pathfinding.
@LucasTutoriaisSaimo @Iifebit
I think both of you were doing it wrong.
You should not use WaitForChild() with game services because they will load pretty fast nor using it by just index it by doing “game.ReplicatedStorage”, but instead using GetService(), it works the same as FindFirstChildOfClass() but for services, although it’s fine not to use GetService() because GetService() is mainly used for services that aren’t guaranteed to be instantiated and WaitForChild() is practically useless in this situation because WaitForChild() is used on things that is a child or a descendant of service, not a service itself.
I think this is the best way and the best practice of doing it:
local ReplicatedStorage = game:GetService("ReplicatedStorage")
local Module = require(ReplicatedStorage:WaitForChild("MainModule"))
or
local Module = require(game:GetService(“ReplicatedStorage”):WaitForChild(“MainModule”))
if you are not thinking of using ReplicatedStorage anytime soon.
1 Like
Could you add an option to choose between animations (Rthro, Cartoony, Astronaut etc. etc.)
This is just a wrapper with extra functionality so this isn’t really necessary. Later on I might create a community GitHub for those willing to add extensions.
Sorry for the late reply.