Posts: 3,324
Threads: 96
Joined: Sep 2013
For 1437, you just simply need to
properly indent the code.
For the rest you are not using GetPlayerName correctly. This is the correct way to use it:
pawn Код:
public OnPlayerConnect(playerid)
{
// Get the name of the player that connected and display a join message to other players
new name[MAX_PLAYER_NAME], string[24+MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof(name));
format(string, sizeof(string), "%s has joined the server.", name);
SendClientMessageToAll(0xC4C4C4FF, string);
return 1;
}
Source:
https://sampwiki.blast.hk/wiki/GetPlayerName
Posts: 177
Threads: 22
Joined: Jun 2015
Reputation:
0
About 1437 it won't get fixed.
Posts: 3,324
Threads: 96
Joined: Sep 2013
Quote:
Originally Posted by KamalBa
About 1437 it won't get fixed.
|
Wtf does that mean? Are you too lazy to do it? Or do you just not know how? You said that like you refuse to try, you shouldn't be scripting if you're going to refuse to do your own work. That's why I linked you to a good wiki on indenting correctly.
Posts: 177
Threads: 22
Joined: Jun 2015
Reputation:
0
give me full code please ?
Cryder, I know but it won't solved all of the solved expect it
Posts: 3,324
Threads: 96
Joined: Sep 2013
Quote:
Originally Posted by KamalBa
give me full code please ?
|
No, we are not here to give you code. This is the help section. DIY type of shit.
Quote:
Originally Posted by KamalBa
Cryder, I know but it won't solved all of the solved expect it
|
If you do it correctly, YES, it will solve all of the indentation warnings.
Posts: 3,324
Threads: 96
Joined: Sep 2013
Quote:
Originally Posted by KamalBa
Код:
CMD:pm(playerid, params[])
{
new targetid,message[128],string[256];
if(sscanf(params,"us[256]", targetid, message)) return SendClientMessage(playerid,COLOR_YELLOW,"USAGE: /pm [playerid] [message]");
if(!IsPlayerConnected(targetid)) return SendClientMessage(playerid,COLOR_RED,"Player is not online");
if(targetid == playerid) return SendClientMessage(playerid,COLOR_RED,"You Can'nt PM Your Self!");
if(AllowPM[targetid] == 1)
{
format(string, sizeof(string),"PM From %s Message: %s", GetPlayerName(playerid), message);
SendClientMessage(targetid, COLOR_RED, string);
format(string, sizeof(string),"PM send to %s Message: %s", GetPlayerName(targetid), message);
SendClientMessage(playerid, COLOR_RED, string);
}
else return SendClientMessage(playerid, COLOR_RED,"That player is not allowing PM's to send to him, sorry!");
return 1;
}
This command show me 4 warnings how to fix |
Quote:
Originally Posted by justice96
Код:
new pname[MAX_PLAYER_NAME];
GetPlayerName(playerid,pname,sizeof(pname));
|
Quote:
Originally Posted by Crayder
For the rest you are not using GetPlayerName correctly. This is the correct way to use it:
pawn Код:
public OnPlayerConnect(playerid)
{
// Get the name of the player that connected and display a join message to other players
new name[MAX_PLAYER_NAME], string[24+MAX_PLAYER_NAME];
GetPlayerName(playerid, name, sizeof(name));
format(string, sizeof(string), "%s has joined the server.", name);
SendClientMessageToAll(0xC4C4C4FF, string);
return 1;
}
Source: https://sampwiki.blast.hk/wiki/GetPlayerName |
We have both already told you, even showed you how to fix that.
