comanda /bonus

[Davidoff7776]

Salut!Mi-am facut comanda /bonus care arata cam asa:

CMD:bonus(playerid, params[]) {
    if(PlayerInfo[playerid][pLevel] < 3)
    {
        PlayerInfo[playerid][pLevel] = 3;
        SetPlayerScore(playerid, PlayerInfo[playerid][pLevel]);
        GivePlayerMoney(playerid, 5000000);
        SendClientMessage(playerid, COLOR_ORANGE, "Ai primit level 3 si $5000000 ca bonus.");
    } else SendClientMessage(playerid, COLOR_ORANGE, "Poti folosi comanda /bonus doar o data.");
    return 1;
}

As dori sa stiu cum vi se pare si daca se mai poate optimiza.

[PauLL]

CMD:bonus(playerid, params[]) {    
    if(PlayerInfo[playerid][pLevel] > 3) return SendClientMessage(playerid, COLOR_ORANGE, "Ai deja level 3 sau ai folosit deja comanda");    
    PlayerInfo[playerid][pLevel] = 3;    
    SetPlayerScore(playerid, PlayerInfo[playerid][pLevel]);    
    GivePlayerMoney(playerid, 5000000);    
    SendClientMessage(playerid, COLOR_ORANGE, "Ai primit level 3 si $5000000 ca bonus.");    
    return 1;
}

	

 

Mai curat asa.

Edited March 3, 2018 by PauLL

[cristiviteza]

CMD:bonus(playerid, params[]) {
    if(PlayerInfo[playerid][pLevel] >= 3) return SendClientMessage(playerid, COLOR_ORANGE, "Poti folosi comanda /bonus doar o data.");
    //-------------------------------------------------------------------------
    else
    {
        PlayerInfo[playerid][pLevel] = 3,	SetPlayerScore(playerid, 3),	GivePlayerMoney(playerid, 5000000);
        //----------------------------------------------------------------------
        return SendClientMessage(playerid, COLOR_ORANGE, "Ai primit level 3 si $5000000 ca bonus.");
	}
}

Poftim si scapi si de return 1;

[PauLL]

Acum 2 ore, cristiviteza a spus:

CMD:bonus(playerid, params[]) {
    if(PlayerInfo[playerid][pLevel] >= 3) return SendClientMessage(playerid, COLOR_ORANGE, "Poti folosi comanda /bonus doar o data.");
    //-------------------------------------------------------------------------
    else
    {
        PlayerInfo[playerid][pLevel] = 3,	SetPlayerScore(playerid, 3),	GivePlayerMoney(playerid, 5000000);
        //----------------------------------------------------------------------
        return SendClientMessage(playerid, COLOR_ORANGE, "Ai primit level 3 si $5000000 ca bonus.");
	}
}

Poftim si scapi si de return 1;

ce-i asta?

Edited March 3, 2018 by PauLL

[Vespa97]

22 hours ago, PauLL said:

ce-i asta?

Nu mai returneaza 1, ci mesajul corespunzator.