[02manchestera]

Can any one see why these commands are conflicting when i type /airstrike it says /vc [text] but i do have some warning mybe these are ruinning them

Код:

C:\Program Files\Rockstar Games\GTA San Andreas\my server\gamemodes\black.pwn(2223) : warning 225: unreachable code
C:\Program Files\Rockstar Games\GTA San Andreas\my server\gamemodes\black.pwn(3820) : warning 209: function "OnDialogResponse" should return a value
Pawn compiler 3.2.3664	 	 	Copyright © 1997-2006, ITB CompuPhase


2 Warnings.

Код:

if (strcmp(cmdtext, "/airstrike", true) == 0)
	{
	if(spree[playerid] == 5)
	{
		ShowPlayerDialog(playerid,DS,2,"Airstrikes","Vehicle Smasher \nEnemy Wave Attack \nTank Buster \nCarpet Bomb \nNuke \nAir Attack \nAir Bomber \nMegaNuke \nVehicle Nuke \nAircracker bomb \nAirCluster Bomb \nDestructorBomb \nFireCluster \nSplitterMissile \nMOAB ","Ok","Cancel");
		return 1;
	}
	if( strcmp(cmd, "/vc", true)==0)
		tmp = strtok(cmdtext, idx);
		if( strlen(tmp) == 0 ) return SendClientMessage(playerid, COLOR_RED, "USE: /vc [text]");
		if( !IsPlayerInAnyVehicle(playerid) ) return SendClientMessage(playerid, COLOR_RED, "You need to be in a vehicle!");
		new str[100];
		GetPlayerName(playerid, nameee, 16);
		format(str, 100, "VehicleRadio [%i]%s:%s", playerid, nameee, cmdtext[4]);
		new veh = GetPlayerVehicleID(playerid);
		foreach(Player, i)
		{
			if( IsPlayerInVehicle(i, veh) )
			{
				SendClientMessage(i, COLOR_LIGHTGREEN, str);
		  }
		}
		return 1;
	}

[Jeffry]

Try this:

pawn Код:

if (strcmp(cmdtext, "/airstrike", true) == 0)
    {
    if(spree[playerid] == 5)
    {
        ShowPlayerDialog(playerid,DS,2,"Airstrikes","Vehicle Smasher \nEnemy Wave Attack \nTank Buster \nCarpet Bomb \nNuke \nAir Attack \nAir Bomber \nMegaNuke \nVehicle Nuke \nAircracker bomb \nAirCluster Bomb \nDestructorBomb \nFireCluster \nSplitterMissile \nMOAB ","Ok","Cancel");
        return 1;
    }
     else return SendClientMessage(playerid, COLOR_RED, "You dont have a spree."); 
    }

//New command starts here.  


if( strcmp(cmd, "/vc", true)==0)
    {
        tmp = strtok(cmdtext, idx);
        if( strlen(tmp) == 0 ) return SendClientMessage(playerid, COLOR_RED, "USE: /vc [text]");
        if( !IsPlayerInAnyVehicle(playerid) ) return SendClientMessage(playerid, COLOR_RED, "You need to be in a vehicle!");
        new str[100];
        GetPlayerName(playerid, nameee, 16);
        format(str, 100, "VehicleRadio [%i]%s:%s", playerid, nameee, cmdtext[4]);
        new veh = GetPlayerVehicleID(playerid);
        foreach(Player, i)
        {
            if( IsPlayerInVehicle(i, veh) )
            {
                SendClientMessage(i, COLOR_LIGHTGREEN, str);
          }
        }
        return 1;
    }

Hope this helped you.

Jeffry

[aircombat]

that is the right code :

Код:

if (strcmp(cmdtext, "/airstrike", true) == 0)
	{
	if(spree[playerid] == 5)
	{
		ShowPlayerDialog(playerid,DS,2,"Airstrikes","Vehicle Smasher \nEnemy Wave Attack \nTank Buster \nCarpet Bomb \nNuke \nAir Attack \nAir Bomber \nMegaNuke \nVehicle Nuke \nAircracker bomb \nAirCluster Bomb \nDestructorBomb \nFireCluster \nSplitterMissile \nMOAB ","Ok","Cancel");
		return 1;
	}
    else { SendClientMessage(playerid,COLOR_RED,"You Don't Have 5 Killing Spree"); }
    return 1;
    }
	if( strcmp(cmd, "/vc", true)==0)
    {
		tmp = strtok(cmdtext, idx);
		if( strlen(tmp) == 0 ) return SendClientMessage(playerid, COLOR_RED, "USE: /vc [text]");
		if( !IsPlayerInAnyVehicle(playerid) ) return SendClientMessage(playerid, COLOR_RED, "You need to be in a vehicle!");
		new str[100];
		GetPlayerName(playerid, nameee, 16);
		format(str, 100, "VehicleRadio [%i]%s:%s", playerid, nameee, cmdtext[4]);
		new veh = GetPlayerVehicleID(playerid);
		foreach(Player, i)
		{
			if( IsPlayerInVehicle(i, veh) )
			{
				SendClientMessage(i, COLOR_LIGHTGREEN, str);
		  }
		}
		return 1;
	}