Posts: 6,129
Threads: 36
Joined: Jan 2009
Quote:
Originally Posted by Wesley221
pawn Код:
new model = GetVehicleModel( vehicleid );
if(model == 432)
{
if(IsPlayerAdmin(playerid)
{
SendCLientMessage(playerid, COLOR_ARED, "Admin rights confirmed");
return 1;
}
else
{
SetPlayerHealth(playerid,0);
GameTextForPlayer(playerid,"~r~Don't touch my toys",5000,5);
return 1;
}
}
Always open a function and close a function |
Always remember to use casing properly, there's no capital 'L' in SendClientMessage.
Easier to use code:
pawn Код:
public OnPlayerEnterVehicle(playerid, vehicleid, ispassenger)
{
if(GetVehicleModel(vehicleid) == 432 && !IsPlayerAdmin(playerid)) {
SetPlayerHealth(playerid, 0);
GameTextForPlayer(playerid, "~r~Don't touch my toys!", 5000, 5);
}
return 0;
}
You didn't need to create a variable to store the vehicle model, you can just do this, and you didn't need to send the message, as that's not what the OP asked for. Plus, I fixed the indentation - you don't need over 9000 spaces everywhere, it makes your code horrible to read.
