1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help

Threaded Mode
How to fix this cmd?
SniperZLyfe1
Big Clucker
***
Posts: 74
Threads: 27
Joined: Apr 2017
Reputation: 0
#1

07.09.2017, 17:07
whenever i use /m within a vehicle while on law enforcement faction,nothing happens.

Код:
	if(strcmp(cmd, "/megaphone", true) == 0 || strcmp(cmd, "/m", true) == 0)
	{
	    if(IsPlayerConnected(playerid))
	    {
	        new type = FactionInfo[PlayerInfo[playerid][pMember]][FactionType];
			new tmpcar = GetPlayerVehicleID(playerid);
			GetPlayerName(playerid, sendername, sizeof(sendername));
			new length = strlen(cmdtext);
			while ((idx < length) && (cmdtext[idx] <= ' '))
			{
				idx++;
			}
			new offset = idx;
			new result[64];
			while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
			{
				result[idx - offset] = cmdtext[idx];
				idx++;
			}
			result[idx - offset] = EOS;
			if(!strlen(result))
			{
				SendClientMessage(playerid, COLOR_WHITE, "USAGE: (/m)egaphone [megaphone chat]");
				return 1;
			}
			if(IsACop(playerid) || PlayerInfo[playerid][pMember] == 4 || PlayerInfo[playerid][pLeader] == 4|| IsASoldier(playerid) || IsAnAgent(playerid) || IsAMedic(playerid))
			{
				if(!IsACopCar(tmpcar) && !IsAnAmbulance(tmpcar) && !IsANGCar(tmpcar))
				{
					SendClientMessage(playerid, COLOR_GRAD2, "   You are not in a Government vehicle !");
					return 1;
				}
				else if(IsACop(type))
				{
					format(string, sizeof(string), "[Officer %s:o< %s]", sendername, result);
					ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
				}
				else if(IsAnAgent(playerid))
				{
					format(string, sizeof(string), "[Agent %s:o< %s]", sendername, result);
					ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
				}
				else if(IsAMedic(playerid))
				{
				    format(string, sizeof(string), "[Doctor %s:o< %s]", sendername, result);
				    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
				}
				else if(IsASoldier(playerid))
				{
				    format(string, sizeof(string), "[Soldier %s:o< %s]", sendername, result);
				    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
				}
			}
			else
			{
			    SendClientMessage(playerid, COLOR_GRAD2, "   You are not part of a Faction !");
				return 1;
			}
		}
		return 1;
Find
Reply
10MIN
Banned
Posts: 325
Threads: 5
Joined: Jun 2017
#2

07.09.2017, 17:23
I think this:
PHP код:
if(!IsACopCar(tmpcar) && !IsAnAmbulance(tmpcar) && !IsANGCar(tmpcar)) 
Should be:
PHP код:
if(!IsACopCar(tmpcar) || !IsAnAmbulance(tmpcar) || !IsANGCar(tmpcar)) 
Because: First means: "If tmpcar is a not cop car and not an ambulance and not a ng car..."
Second: "If tmpcar is not a cop car or an ambulance or a ng car..."
PS: Use YCMD / PawnCMD because they are faster at commands. Also use sscanf plugin by ****** because it works veryy very good with them.
Find
Reply
n00blek
Huge Clucker
***
Posts: 249
Threads: 51
Joined: Aug 2017
Reputation: 0
#3

07.09.2017, 17:24
PHP код:
CMD:m(playerid, params[])
{
    if(strcmp(cmd, "/megaphone", true) == 0 || strcmp(cmd, "/m", true) == 0)
    {
        if(IsPlayerConnected(playerid))
        {
            new type = FactionInfo[PlayerInfo[playerid][pMember]][FactionType];
            new tmpcar = GetPlayerVehicleID(playerid);
            GetPlayerName(playerid, sendername, sizeof(sendername));
            new length = strlen(cmdtext);
            while ((idx < length) && (cmdtext[idx] <= ' '))
            {
                idx++;
            }
            new offset = idx;
            new result[64];
            while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
            {
                result[idx - offset] = cmdtext[idx];
                idx++;
            }
            result[idx - offset] = EOS;
            if(!strlen(result))
            {
                SendClientMessage(playerid, COLOR_WHITE, "USAGE: (/m)egaphone [megaphone chat]");
                return 1;
            }
            if(IsACop(playerid) || PlayerInfo[playerid][pMember] == 4 || PlayerInfo[playerid][pLeader] == 4|| IsASoldier(playerid) || IsAnAgent(playerid) || IsAMedic(playerid))
            {
                if(!IsACopCar(tmpcar) && !IsAnAmbulance(tmpcar) && !IsANGCar(tmpcar))
                {
                    SendClientMessage(playerid, COLOR_GRAD2, "   You are not in a Government vehicle !");
                    return 1;
                }
                else if(IsACop(type))
                {
                    format(string, sizeof(string), "[Officer %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsAnAgent(playerid))
                {
                    format(string, sizeof(string), "[Agent %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsAMedic(playerid))
                {
                    format(string, sizeof(string), "[Doctor %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsASoldier(playerid))
                {
                    format(string, sizeof(string), "[Soldier %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD2, "   You are not part of a Faction !");
                return 1;
            }
        }
        return 1;
} 
Find
Reply
SniperZLyfe1
Big Clucker
***
Posts: 74
Threads: 27
Joined: Apr 2017
Reputation: 0
#4

07.09.2017, 17:29
Quote:
Originally Posted by n00blek
Посмотреть сообщение
PHP код:
CMD:m(playerid, params[])
{
    if(strcmp(cmd, "/megaphone", true) == 0 || strcmp(cmd, "/m", true) == 0)
    {
        if(IsPlayerConnected(playerid))
        {
            new type = FactionInfo[PlayerInfo[playerid][pMember]][FactionType];
            new tmpcar = GetPlayerVehicleID(playerid);
            GetPlayerName(playerid, sendername, sizeof(sendername));
            new length = strlen(cmdtext);
            while ((idx < length) && (cmdtext[idx] <= ' '))
            {
                idx++;
            }
            new offset = idx;
            new result[64];
            while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
            {
                result[idx - offset] = cmdtext[idx];
                idx++;
            }
            result[idx - offset] = EOS;
            if(!strlen(result))
            {
                SendClientMessage(playerid, COLOR_WHITE, "USAGE: (/m)egaphone [megaphone chat]");
                return 1;
            }
            if(IsACop(playerid) || PlayerInfo[playerid][pMember] == 4 || PlayerInfo[playerid][pLeader] == 4|| IsASoldier(playerid) || IsAnAgent(playerid) || IsAMedic(playerid))
            {
                if(!IsACopCar(tmpcar) && !IsAnAmbulance(tmpcar) && !IsANGCar(tmpcar))
                {
                    SendClientMessage(playerid, COLOR_GRAD2, "   You are not in a Government vehicle !");
                    return 1;
                }
                else if(IsACop(type))
                {
                    format(string, sizeof(string), "[Officer %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsAnAgent(playerid))
                {
                    format(string, sizeof(string), "[Agent %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsAMedic(playerid))
                {
                    format(string, sizeof(string), "[Doctor %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
                else if(IsASoldier(playerid))
                {
                    format(string, sizeof(string), "[Soldier %s:o< %s]", sendername, result);
                    ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
                }
            }
            else
            {
                SendClientMessage(playerid, COLOR_GRAD2, "   You are not part of a Faction !");
                return 1;
            }
        }
        return 1;
} 
Thanks mate.let me try
Find
Reply
10MIN
Banned
Posts: 325
Threads: 5
Joined: Jun 2017
#5

07.09.2017, 17:31
@n00blek what the hell..... Are you a noob or what
OP: ignore n00blek's answer because he don't know what he is saying....
Find
Reply
SniperZLyfe1
Big Clucker
***
Posts: 74
Threads: 27
Joined: Apr 2017
Reputation: 0
#6

07.09.2017, 17:42
Quote:
Originally Posted by 10MIN
Посмотреть сообщение
@n00blek what the hell..... Are you a noob or what
OP: ignore n00blek's answer because he don't know what he is saying....
So,do you know how to fix it?
Find
Reply
10MIN
Banned
Posts: 325
Threads: 5
Joined: Jun 2017
#7

07.09.2017, 17:45
This:
Quote:
Originally Posted by 10MIN
Посмотреть сообщение
I think this:
PHP код:
if(!IsACopCar(tmpcar) && !IsAnAmbulance(tmpcar) && !IsANGCar(tmpcar)) 
Should be:
PHP код:
if(!IsACopCar(tmpcar) || !IsAnAmbulance(tmpcar) || !IsANGCar(tmpcar)) 
Because: First means: "If tmpcar is a not cop car and not an ambulance and not a ng car..."
Second: "If tmpcar is not a cop car or an ambulance or a ng car..."
PS: Use YCMD / PawnCMD because they are faster at commands. Also use sscanf plugin by ****** because it works veryy very good with them.
Find
Reply
Primes007
Big Clucker
***
Posts: 56
Threads: 8
Joined: Feb 2017
Reputation: 0
#8

07.09.2017, 20:13
Here you go buddy,

PHP код:
CMD:m(playerid)
{
    new type = FactionInfo[PlayerInfo[playerid][pMember]][FactionType];
    new tmpcar = GetPlayerVehicleID(playerid);
    GetPlayerName(playerid, sendername, sizeof(sendername));
    new length = strlen(cmdtext);
    while ((idx < length) && (cmdtext[idx] <= ' '))
    {
        idx++;
    }
    new offset = idx;
    new result[64];
    while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
    {
        result[idx - offset] = cmdtext[idx];
        idx++;
    }
    result[idx - offset] = EOS;
    if(!strlen(result))
    {
        SendClientMessage(playerid, COLOR_WHITE, "USAGE: (/m)egaphone [megaphone chat]");
        return 1;
    }
    if(IsACop(playerid) || PlayerInfo[playerid][pMember] == 4 || PlayerInfo[playerid][pLeader] == 4|| IsASoldier(playerid) || IsAnAgent(playerid) || IsAMedic(playerid))
    {
        if(!IsACopCar(tmpcar) || !IsAnAmbulance(tmpcar) || !IsANGCar(tmpcar))
        {
            SendClientMessage(playerid, COLOR_GRAD2, "   You are not in a Government vehicle !");
            return 1;
        }
        else if(IsACop(type))
        {
            format(string, sizeof(string), "[Officer %s:o< %s]", sendername, result);
            ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
        }
        else if(IsAnAgent(playerid))
        {
            format(string, sizeof(string), "[Agent %s:o< %s]", sendername, result);
            ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
            }
        else if(IsAMedic(playerid))
        {
            format(string, sizeof(string), "[Doctor %s:o< %s]", sendername, result);
            ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
        }
        else if(IsASoldier(playerid))
        {
            format(string, sizeof(string), "[Soldier %s:o< %s]", sendername, result);
            ProxDetector(60.0, playerid, string,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW,COLOR_YELLOW);
        }
    }
    else return SendClientMessage(playerid, COLOR_GRAD2, "   You are not part of a Faction !");
}
CMD:megaphone(playerid)
{
    return cmd_m(playerid);
} 
I don't know if it'll be fully functional (I haven't gone through the code). But, now it'll be easier to debug for you.

Also, Make sure you have zcmd included
PHP код:
#include <zcmd> 
.
Find
Reply
« Next Oldest | Next Newest »


Forum Jump:


Users browsing this thread: 2 Guest(s)
  1. SA-MP Forums Archive
  2. SA-MP Scripting and Plugins
  3. Scripting Help