[Bakr]

For a short answer I say yes.

On the other hand, I think this post is worth a read:

Quote:

Originally Posted by AndreT I agree that beginners need to be told about more efficient ways to handle commands, but the command "processors" that use CallLocalFunction only become favorable when there are more commands than just a few. CallLocalFunction, too, uses string comparisons, but does it using binary search resulting in logarithmic compexity. But it does not search from amongst your commands, but all public functions registered with the AMX (everything such as native callbacks, timers, etc. are included in the search). So for just a few commands I would favor using string comparisons in OnPlayerCommandText. Especially when the code is created by or for beginners who could find the idea of callback hooking or includes confusing!

Even with that said, I still agree that the other options should be presented at the time you also show using OnPlayerCommandText. Maybe just have an additional warning box stating that there are more efficient ways. As pointed out, there are numerous pages that would need modifying, and you cannot register on the Wiki anymore (AFAIK), so you would have to get a select few to do that.

The reason I say don't strip the OnPlayerCommandText method completely is I believe they should know the different approaches they can take. That also makes it their responsibility on whether or not to choose the better method.