[Swedky]

Quote:

Originally Posted by Konstantinos It's a good tutorial. Although, I'd like to point out that using: pawn Code: stock GetName(playerid) // Code used in previous examples in this tutorial. {     new Nick[MAX_PLAYER_NAME];     GetPlayerName(playerid, Nick, sizeof(Nick));     return Nick; // He returns a chain of text. } is a really bad idea. Let's say in a command you want to use a player's name 5 times so you're going to call GetPlayerName 5 times when it could be done only 1 (by getting the name right there). However, the best way (atleast for me) is to get the name only once on connect and store it to a global array (either packed string or not). Then you can use to pass the name as an argument wherever you want. You also said about foreach instead of for loops for the players and foreach is really great! foreach should be used for the vehicles as well and it's very optimized. Instead of looping 2000 times for vehicles, you can add vehicleid to an Iterator and loop through the valid vehicleids. So let's say you have 2 vehicles in the server, it will only loop 2 times instead of 2000 and imagine if it had to loop many times through the vehicles. PS: It's y_commands and not Y_CMD.

You refer to that something like that:

pawn Code:

CMD:mynick(playerid, params[])
{
    new string[126];
    format(string, sizeof(string), "%s %s %s %s %s.", GetName(playerid), GetName(playerid), GetName(playerid), GetName(playerid), GetName(playerid));
    SendClientMessage(playerid, -1, string);
    return 1;
}

Would it be bad?.

No understood eh very well :S.