[CaRa]

Quote:

Originally Posted by AndreT Oy, I have another few things to say here... 1) Your command does not require sscanf parsing. It has one parameter only! pawn Код: CMD:changeemail(playerid, params[]) {     if(isnull(params))     {         // Output the message about how the usage of the command really is!     }     // ...     return true; } Ok, i didn't know that, thanks. 2) Your "var" array is a bit too small to contain larger e-mail addresses as their size, by rfcs, is allowed to up to 64 characters I believe. pawn Код: new var[64+1]; I get it. 3) You don't actually need multiple arrays to store these strings, you don't use any of them at the same time. So don't waste memory! And don't get the same player's name TWICE. pawn Код: new string[128], playername[MAX_PLAYER_NAME]; GetPlayerName(playerid, playername, sizeof(playername)); format(string, sizeof(string), "users/%s.ini", playername); // new hFile = ...;

I forgot to remove playername2 ... Can you delete the array that waste my memory ?

Код:

 new string3[128];
			new playername3[MAX_PLAYER_NAME];
			GetPlayerName(playerid, playername3, sizeof(playername3));
			format(string3, sizeof(string3), "users/%s.ini", playername3);
            new File: hFile = fopen(string3, io_write);
			if (hFile)
			{
			    strmid(PlayerInfo[playerid][pEmail], params, 0, strlen(params), 255);
			    new var[64+1];
				format(var, 32, "Email=%s\n", PlayerInfo[playerid][pEmail]);fwrite(hFile, var);
			}
			fclose(hFile);