AND of ||

[jordyvc]

Hey,

ik heb deze code:

CMD:wpickup(playerid, params[])// Rebellen wapen pickup command
{

    if(IsPlayerInRangeOfPoint(playerid, 10.0,1073.1725,-345.5271,73.9922) || (PlayerInfo[playerid][pTeam] == 1))
    {
    	ShowPlayerDialog(playerid, WEAPON_DIALOG_1, DIALOG_STYLE_LIST, "Weapon Pickup", "Deagle \nShotgun \nAK-47 \nRPG ", "Select", "Exit");
    }
    else
    {
        SendClientMessage(playerid, -1, "Your not at the weapon pickup OR you are not the good team!");
    }
	return 1;
}

Maar elk team kan wapens kiezen? En normaal moet alleen team 1 deze command kunnen gebruiken als ze in de buurt zijn.

Weet iemand hoe dit kan komen? Ik heb de lijn vet gezet waarvan ik denk dat er iets mis is.

Mvg,
Jordy Van Camp

[Henkie]

|| betekent OF
&& betekent EN

Je zegt nu dus: De player is op de positie, dus krijgt deze het wapen. Als je && gebruikt, moet de player ook in het team zitten Je moet het dus zo doen:

CMD:wpickup(playerid, params[])// Rebellen wapen pickup command
{

    if(IsPlayerInRangeOfPoint(playerid, 10.0,1073.1725,-345.5271,73.9922) && (PlayerInfo[playerid][pTeam] == 1))
    {
        ShowPlayerDialog(playerid, WEAPON_DIALOG_1, DIALOG_STYLE_LIST, "Weapon Pickup", "Deagle \nShotgun \nAK-47 \nRPG ", "Select", "Exit");
    }
    else
    {
        SendClientMessage(playerid, -1, "Your not at the weapon pickup OR you are not the good team!");
    }
    return 1;
}

[jordyvc]

Ah oke bedankt!!! +REP

[[MM]IKKE]

Deze interesseert je misschien wel:
http://forum.sa-mp.com/showpost.php?...2&postcount=51

Het begin, over short-circuiting.

[faff]

Dit kan ook werken:

CMD:wpickup(playerid, params[])// Rebellen wapen pickup command
{

    if(IsPlayerInRangeOfPoint(playerid, 10.0,1073.1725,-345.5271,73.9922)) 
     {
       if(PlayerInfo[playerid][pTeam] == 1)
         {
        ShowPlayerDialog(playerid, WEAPON_DIALOG_1, DIALOG_STYLE_LIST, "Weapon Pickup", "Deagle \nShotgun \nAK-47 \nRPG ", "Select", "Exit");
         }
        else
        {
          SendClientMessage(playerid, -1, "Your not at the weapon pickup OR you are not the good team!");
        } 
    return 1;
}

[BlackBank3]

Originally Posted by faff

Dit kan ook werken:

CMD:wpickup(playerid, params[])// Rebellen wapen pickup command
{

    if(IsPlayerInRangeOfPoint(playerid, 10.0,1073.1725,-345.5271,73.9922)) 
     {
       if(PlayerInfo[playerid][pTeam] == 1)
         {
        ShowPlayerDialog(playerid, WEAPON_DIALOG_1, DIALOG_STYLE_LIST, "Weapon Pickup", "Deagle \nShotgun \nAK-47 \nRPG ", "Select", "Exit");
         }
        else
        {
          SendClientMessage(playerid, -1, "Your not at the weapon pickup OR you are not the good team!");
        } 
    return 1;
}

Daarbij zul je een error verwachten, omdat je een bracket bent vergeten...

Waarom post je dit, als je er zelf geen bal van snapt?

[Biesmen]

Originally Posted by BlackBank3

Waarom post je dit, als je er zelf geen bal van snapt?

Post count, en mensen die het leuk vinden om zichzelf voor schut te zetten.

[[MM]IKKE]

Trouwens, Grammar Nazi.

Het is "You're not at the weapon pickup"

Ik denk dat deze fout wel belangrijk is, aangezien je je server waarschijnlijk wel een professionele look wilt geven?

[sjvt]

Wat BlackBank3 zeg, en je verbruikt onnodige lines. Waarom moeilijk doen als het super simpel kan..

[legodude]

vooral post count waarschijnlijk 200 posts en niet een rep. kan weinig goeds zijn me dunkt..

http://wiki.sa-mp.com/wiki/Scripting...s_NL#Condities is misschien ook een must-read