[Yashas]

#1 Your processor is case-sensitive. iZCMD case-sensitive version is 7x faster than your code.

I am sure that you still don't understand the difference between using natives and writing plain PAWN code.

#2 Even if you compare your code with iZCMD non-case-sensitive (ZCMD is non-case-sensitive), yours is still slightly slower! (just 1.0097x slower but anyway its slower inspite of doing less work than iZCMD)

#3 The correct spelling of re-ce-ived is "received" not "recieved"

#4 Your code doesn't even work! Do you know what static means? The data is saved after the function ends. You are not adding a null character at the end of the "function". Shouldn't it be zero by default? No! Static arrays are global arrays with limited scope so its not filled with zeros unlike local arrays which are filled with zeros when they are added to the heap.

#5 iZCMD code is almost identical to your code. You have used a 'switch' whereas iZCMD uses 'if..else'.

And to finish the reply with another blow, if else is faster than switch in this case!

Here is your fixed code:

Код:

#define CMD:%1(%2) \
			forward cmd_%1(%2); public cmd_%1(%2)

public OnPlayerCommandText(playerid, cmdtext[])
{
    #if defined OnPlayerCommandRecieved
		if (!OnPlayerCommandRecieved(playerid, cmdtext))
		    return 1;
	#endif

	static function[31] = "cmd_";

	for (new i = 1; ; i++)
	{
	    switch (cmdtext[i])
	    {
	        case ' ':
	        {
		        if (cmdtext[(i+1)] == EOS)
		        {
		            function[((i-1) + 3)] = 0;
		            break;
		        }

				#if defined OnPlayerCommandPerformed
					return OnPlayerCommandPerformed(playerid, function[4], cmdtext[(i+1)], CallLocalFunction(function, "is", playerid, cmdtext[(i+1)]));
				#else
					return CallLocalFunction(function, "is", playerid, cmdtext[(i+1)]);
				#endif
	        }

	        case EOS:
	        {
	            function[((i-1) + 4)] = 0;
				break;
	        }

	        default:
	        {
				function[((i-1) + 4)] = cmdtext[i];
	        }
	    }
	}

	#if defined OnPlayerCommandPerformed
		return OnPlayerCommandPerformed(playerid, function[4], "\1", CallLocalFunction(function, "is", playerid, "\1"));
	#else
		return CallLocalFunction(function, "is", playerid, "\1");
	#endif
}

#if defined OnPlayerCommandReceived
   forward OnPlayerCommandReceived(playerid, cmdtext[]);
#endif

#if defined OnPlayerCommandPerformed
	forward OnPlayerCommandPerformed(playerid, cmdtext[], params[], response);
#endif

#6 If I really want to make iZCMD even faster, I can do it by making insane optimizations which murder the readablity completely.

Код:

static prefix[] = {'c', 'm', 'd'}, funcname[32] ="_";

No need for jumping between indexs if that is used ^

Can add shitton of assembly code to get rid of all redudent index calculations.

I can approximate that I can make iZCMD code 50% faster but its simply ridiculous to do all those kind of optimizations.