[enzulikeS]

Код:

CMD:gotojob(playerid, params[])
{
	if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");
	if(PlayerInfo[playerid][pAdmin] == 0) return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
	{
		ShowPlayerDialog(playerid, DIALOG_GOTOJOB, DIALOG_STYLE_LIST,"Gotojob", "Detective\nDrugs Dealer\nPizza Boy\nBus Driver\nArms Dealer\nGarbageman\nFisherman\nTrucker\nFarmer");
		return 1;
	}
	return 1;
}

D:\server\gamemodes\ExtremeGame.pwn(23571) : warning 225: unreachable code

[git]

Код:

CMD:gotojob(playerid, params[])
{
	if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");
	if(PlayerInfo[playerid][pAdmin] == 0) // return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
	{
                SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
		ShowPlayerDialog(playerid, DIALOG_GOTOJOB, DIALOG_STYLE_LIST,"Gotojob", "Detective\nDrugs Dealer\nPizza Boy\nBus Driver\nArms Dealer\nGarbageman\nFisherman\nTrucker\nFarmer");
		return 1;
	}
	return 1;
}

You were returning the client message and then still carried on the code.

However, I think your code is wrong, do this:

Код:

CMD:gotojob(playerid, params[])
{
	if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");
	if(PlayerInfo[playerid][pAdmin] == 0) // return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
	{
                SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
		return 1;
	}

        else
        {
                ShowPlayerDialog(playerid, DIALOG_GOTOJOB, DIALOG_STYLE_LIST,"Gotojob", "Detective\nDrugs 
                Dealer\nPizza Boy\nBus Driver\nArms Dealer\nGarbageman\nFisherman\nTrucker\nFarmer");
        }

	return 1;
}

[enzulikeS]

fixed, ty.

thanks for the second code, the solve was replacing mine with the second one

[kovac]

It can be simply done like this:

PHP код:

CMD:gotojob(playerid)
{
    if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");
    if(PlayerInfo[playerid][pAdmin] == 0) return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
    ShowPlayerDialog(playerid, DIALOG_GOTOJOB, DIALOG_STYLE_LIST,"Gotojob", "Detective\nDrugs
    Dealer\nPizza Boy\nBus Driver\nArms Dealer\nGarbageman\nFisherman\nTrucker\nFarmer");
    return 1;
} 


[SeanDenZYR]

It can be more easily done by removing:

pawn Код:

return 1;

latest ZCMD Include requires each commands used by its command engine to return a value. But you've already returned values via:

pawn Код:

if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");

pawn Код:

if(PlayerInfo[playerid][pAdmin] == 0) return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");

and by the way, remove these brackets.

Код:

CMD:gotojob(playerid, params[])
{
	if(gPlayerLogged[playerid] == 0) return SCM(playerid, COLOR_LIGHTRED, "You need to log in first.");
	if(PlayerInfo[playerid][pAdmin] == 0) return SCM(playerid, COLOR_LIGHTRED, "Nu ai acces la aceasta comanda.");
	{
		ShowPlayerDialog(playerid, DIALOG_GOTOJOB, DIALOG_STYLE_LIST,"Gotojob", "Detective\nDrugs Dealer\nPizza Boy\nBus Driver\nArms Dealer\nGarbageman\nFisherman\nTrucker\nFarmer");
		return 1;
	}
	return 1;
}