[Yashas]

Quote:

Originally Posted by Jefff 1. Code: new pInfo[MAX_PLAYERS + 1][PlayerInfo]; 2. I don't know for what is here memcpy Code: pInfo[playerid] = pInfo[MAX_PLAYERS];

memcpy is actually faster that direct assignment when your array is VERY BIG.

Quote:

Originally Posted by Uberanwar Hey Yashas, thank you for this useful & helpful knowledge you've shared to the community. keep it up I'm having an error and a warning though, need help on this.. warning 229: index tag mismatch (symbol "pInfo") error 032: array index out of bounds (variable "pInfo") both on the same line which is ^ that one

I guess its because of the sizeof, try sizeof(pInfo[][]) or something similar. I have lost touch with PAWN and I do not remember. A bit of tweaking should fix the problem.

Quote:

Originally Posted by Day_ This PHP Code: new Float:Coords[5][4] = {     {0.0, 0.1, 0.2, 0.3},     {1.0, 1.1, 1.2, 1.3},     {2.0, 2.1, 2.2, 2.3},     {3.0, 3.1, 3.2, 3.3},     {4.0, 4.1, 4.2, 4.3} }; function(){     new rand = random(sizeof(Coords));     foreach(new i: Player)     {         SetPlayerPos(playerid, Coords[rand][0], Coords[rand][1], Coords[rand][2]);         SetPlayerFacingAngle(playerid, Coords[rand][3]);     }     CreatePickup(1, 2, Coords[rand][0], Coords[rand][1], Coords[rand][2]);     Create3DTextLabel("You are here", -1, Coords[rand][0], Coords[rand][1], Coords[rand][2], 15.0, 0); }  is Slower than this PHP Code: new Float:Coords[5][4] = {     {0.0, 0.1, 0.2, 0.3},     {1.0, 1.1, 1.2, 1.3},     {2.0, 2.1, 2.2, 2.3},     {3.0, 3.1, 3.2, 3.3},     {4.0, 4.1, 4.2, 4.3} }; function(){     new rand = random(sizeof(Coords));     new          Float:X        = Coords[rand][0],         Float:Y        = Coords[rand][1],         Float:Z        = Coords[rand][2],         Float:Ang    = Coords[rand][3]     ;     foreach(new i: Player)     {         SetPlayerPos(playerid, X, Y, Z);         SetPlayerFacingAngle(playerid, Coords[rand][3]);     }     CreatePickup(1, 2, X, Y, Z);     Create3DTextLabel("You are here", -1, X, Y, Z, 15.0, 0); }  The functions are example. I think, because every time at Coords[*] is called, need load the value i and the value from Coords, if you use the array only a time no have differences between speed. Sorry my bad english, someone smarter than me can explain better.

It is well explained here (Click) with proof.

Quote:

Originally Posted by OneDay new gPlayerInfo[MAX_PLAYERS][PlayerData]; new playerInfo[PlayerData]; gPlayerInfo[playerid]=playerInfo; is not function memcpy is pawn faster than function memcpy

IIRC memcpy is faster when the array is pretty huge. The improvement is however negligible since either way its WAY WAY WAY faster than the loop method to reset an array (the speed difference b/w memcpy and direct assignment doesn't matter here - I have explained it in on of my previous replies to SickAttack).

Quote:

Originally Posted by Untonyst If you are looking for a only one symbol, it is better to use a loop. If necessary to use a for loop for the whole string, it is not necessary to know the size of the string. PHP Code: new string[128] = "qw;lasdl;k1l;k2l;4k;"; // :D for (new i; /* none */ ;i++) {     if (string[i] == 'l')     {         // do something         // ...         // loop is complete         break;     }     // cycle is complete     // if symbol is not found     if (string[i] == '\0')     {         break;     } }  or PHP Code: new string[128] = "qw;lasdl;k1l;k2l;4k;",     i; while (string[i++] != '\0') {     if (string[i] == 'l')     {                 // do something         // ...         // loop is complete         break;     } }  For more optimization you can use variables, so often not to refer to the array index. PHP Code: new string[128] = "qw;lasdl;k1l;k2l;4k;",      c,    // is values of the array index      i     // index  ;  while ((c = string[i++]) != '\0') {     if (c == 'l')     {                 // do something         // ...         // loop is complete         break;     } }  If the condition need to compare three numbers, it is possible to make it so PHP Code: new value = 7; if (1 <= value <= 9) {     // do something }

Your loop is horribley slow compared to strfind. Strfind accepts a substring instead of a character but its still faster when you want to search for a character.

Check Natives are faster than PAWN code in this (Click)topic to know more.

Quote:

Originally Posted by OneDay the second code is wrong because ++ but just use strfind

1. strfind is faster than that code
2. His code has a bug. It skips a character.

Quote:

Originally Posted by Crayder No, it's not wrong. Learn PAWN before trying to call someone out like that.

His English isn't good lol

He wanted to say that there is a bug and that code wont work because of i++ but he said it in a completely wrong and misleading way.