diff options
| author | Sadie Powell <sadie@witchery.services> | 2024-02-29 21:56:14 +0000 |
|---|---|---|
| committer | Sadie Powell <sadie@witchery.services> | 2024-02-29 21:56:14 +0000 |
| commit | 1c1645096f953aa06b5809be6b302015b5bc73d6 (patch) | |
| tree | be03513aa05c44ea017ca4ecb1e6c497dd231857 /src | |
| parent | e0ac5509b4020cb6e78c6b7c591ef2e7f6314ced (diff) | |
If a user runs an invalid command try to suggest a valid one.
Diffstat (limited to 'src')
| -rw-r--r-- | src/command.cpp | 54 | ||||
| -rw-r--r-- | src/misc.cpp | 36 |
2 files changed, 81 insertions, 9 deletions
diff --git a/src/command.cpp b/src/command.cpp index c9372c40f..e7912864c 100644 --- a/src/command.cpp +++ b/src/command.cpp @@ -201,11 +201,53 @@ void Command::OnSyntaxError(CommandSource &source, const Anope::string &subcomma source.Reply(MORE_INFO, Config->StrictPrivmsg.c_str(), source.service->nick.c_str(), source.command.c_str()); } +namespace +{ + void HandleUnknownCommand(CommandSource& source, const Anope::string &message) + { + // Try to find a similar command. + size_t distance = Config->GetBlock("options")->Get<size_t>("didyoumeandifference", "4"); + Anope::string similar; + auto umessage = message.upper(); + for (const auto &[command, info] : source.service->commands) + { + if (info.hide || command == message) + continue; // Don't suggest a hidden alias or a missing command. + + size_t dist = Anope::Distance(umessage, command); + if (dist < distance) + { + distance = dist; + similar = command; + } + } + + bool has_help = source.service->commands.find("HELP") != source.service->commands.end(); + if (has_help && similar.empty()) + { + source.Reply(_("Unknown command \002%s\002. \"%s%s HELP\" for help."), message.c_str(), + Config->StrictPrivmsg.c_str(), source.service->nick.c_str()); + } + else if (has_help) + { + source.Reply(_("Unknown command \002%s\002. Did you mean \002%s\002? \"%s%s HELP\" for help."), + message.c_str(), similar.c_str(), Config->StrictPrivmsg.c_str(), source.service->nick.c_str()); + } + else if (similar.empty()) + { + source.Reply(_("Unknown command \002%s\002. Did you mean \002%s\002?"), message.c_str(), similar.c_str()); + } + else + { + source.Reply(_("Unknown command \002%s\002."), message.c_str()); + } + } +} + void Command::Run(CommandSource &source, const Anope::string &message) { std::vector<Anope::string> params; spacesepstream(message).GetTokens(params); - bool has_help = source.service->commands.find("HELP") != source.service->commands.end(); CommandInfo::map::const_iterator it = source.service->commands.end(); unsigned count = 0; @@ -222,10 +264,7 @@ void Command::Run(CommandSource &source, const Anope::string &message) if (it == source.service->commands.end()) { - if (has_help) - source.Reply(_("Unknown command \002%s\002. \"%s%s HELP\" for help."), message.c_str(), Config->StrictPrivmsg.c_str(), source.service->nick.c_str()); - else - source.Reply(_("Unknown command \002%s\002."), message.c_str()); + HandleUnknownCommand(source, message); return; } @@ -233,10 +272,7 @@ void Command::Run(CommandSource &source, const Anope::string &message) ServiceReference<Command> c("Command", info.name); if (!c) { - if (has_help) - source.Reply(_("Unknown command \002%s\002. \"%s%s HELP\" for help."), message.c_str(), Config->StrictPrivmsg.c_str(), source.service->nick.c_str()); - else - source.Reply(_("Unknown command \002%s\002."), message.c_str()); + HandleUnknownCommand(source, message); Log(source.service) << "Command " << it->first << " exists on me, but its service " << info.name << " was not found!"; return; } diff --git a/src/misc.cpp b/src/misc.cpp index b44176ecd..b40c69844 100644 --- a/src/misc.cpp +++ b/src/misc.cpp @@ -20,6 +20,7 @@ #include "sockets.h" #include <cerrno> +#include <numeric> #include <sys/stat.h> #include <sys/types.h> #ifndef _WIN32 @@ -747,3 +748,38 @@ Anope::string Anope::Random(size_t len) buf.append(chars[rand() % sizeof(chars)]); return buf; } + +// Implementation of https://en.wikipedia.org/wiki/Levenshtein_distance +size_t Anope::Distance(const Anope::string &s1, const Anope::string &s2) +{ + if (s1.empty()) + return s2.length(); + if (s2.empty()) + return s1.length(); + + std::vector<size_t> costs(s2.length() + 1); + std::iota(costs.begin(), costs.end(), 0); + + size_t i = 0; + for (const auto c1 : s1) + { + costs[0] = i + 1; + size_t corner = i; + size_t j = 0; + for (const auto &c2 : s2) + { + size_t upper = costs[j + 1]; + if (c1 == c2) + costs[j + 1] = corner; + else + { + size_t t = upper < corner ? upper : corner; + costs[j + 1] = (costs[j] < t ? costs[j] : t) + 1; + } + corner = upper; + j++; + } + i++; + } + return costs[s2.length()]; +} |
