Quote:
Originally Posted by Grumbles
Yeah, that's exactly what I mean. I'm using OnPlayerCommandText as the command client.
|
Okay. Then first I'd highly recommend using a command processor such as SmartCMD or Pawn.CMD, they are advanced but still great for beginners (YCMD, which is my choice, is GREAT as well, but may be a small bit more confusing).
Then, since you want to do a comparison with all of your commands... you'll want to do something similar to below;
pawn Код:
mathMin(inp1,inp2)
return (inp1 > inp2 ? inp2 : inp1);
LevenshteinDistance(strs[], strt[])
{
new n = strlen(strs), m = strlen(strt);
new d[128][128];
if (n == 0)
return m;
if (m == 0)
return n;
for (new i; i <= n; i++)
d[i][0] = i;
for (new j; j <= m; j++)
d[0][j] = j;
for (new j = 1; j <= m; j++)
{
for (new i = 1; i <= n; i++)
{
if (strs[i - 1] == strt[j - 1])
d[i][j] = d[i - 1][j - 1];
else
d[i][j] = mathMin(mathMin(d[i - 1][j] + 1, d[i][j - 1] + 1), d[i - 1][j - 1] + 1);
}
}
return d[n][m];
}
// CHANGE THIS. 20 should be changed to your number of commands.
#define NUMBER_OF_COMMANDS 20
new commands[NUMBER_OF_COMMANDS][] = {
"help",
"kill",
"getplayer",
"spawncar"
// ALL OF YOUR COMMANDS
};
GetClosestCommand(string[])
{
new match, // matching command with least changes
distance, // number of changes to make command
least = 1000; // random high number
for(new i; i < sizeof(commands); i++)
{
if(strfind(commands[i], string, true) != -1)
{
distance = LevenshteinDistance(string, commands[i]);
if(least > distance)
{
match = i;
least = distance;
}
}
}
return commands[match];
}
`GetClosestCommand` will decide which of your commands take the least number of changes from the given typed command and return it as a string.
For OnPlayerCommandText this may look like this;
pawn Код:
public OnPlayerCommandText(playerid,cmdtext[])
{
// all commands here
new response[128];
format(response, 128, "Error: This command didn't exist, did you mean /%s?", GetClosestCommand(cmdtext));
SendClientMessage(playerid, -1, response);
return 0; // this is still going to show the UnKnown command message unless you change this to return 1, however this this will break other scripts... Another reason to switch processors.
}
Again, you should definitely stop using OnPlayerCommandText. Switch to a better command processor and this entire process is a bit simpler. The strcmp method is SLOW AS HELL and has other issues. It's an outdated method that should've never existed.
