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

Threaded Mode
sizeof error
Hybris
Huge Clucker
***
Posts: 207
Threads: 53
Joined: Dec 2013
Reputation: 0
#1

13.03.2015, 20:58
Hello im getting a strange error that I've never gotten before hoping someone can help me out.
ERROR:
Код:
error 072: "sizeof" operator is invalid on "function" symbols
LINE:
Код:
format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", PlayerName, id, time, reason);
Thanks in advance.
Find
Reply
Hybris
Huge Clucker
***
Posts: 207
Threads: 53
Joined: Dec 2013
Reputation: 0
#2

13.03.2015, 21:03
I'll just post the whole command
Код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new id,time,reason[100],szString[150];
        if(sscanf(params,"dds",id,time,reason)) return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");
        if(!IsPlayerConnected(id)) return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");
        if(Jailed[id] == 1) return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");
		format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", PlayerName, id, time, reason);
		SendClientMessageToAll(-1, szString);
        GetPlayerName(id, PlayerName, sizeof(PlayerName));
	 	GetPlayerName(playerid, GetPlayerName, sizeof(GetPlayerName));
        SetPlayerInterior(id, 3);
		SetPlayerVirtualWorld(id, 10);
		SetPlayerFacingAngle(id, 360.0);
		SetPlayerPos(id, 197.5662, 175.4800, 1004.0);
		SetPlayerHealth(id, 9999999999.0);
		ResetPlayerWeapons(id);
		JailTimer[id] = SetTimerEx("Unjail",time*60000, false, "i", id);
	}
	else{
		return 0;
		}
	return 1;
}
Find
Reply
HazardouS
Huge Clucker
****
Posts: 256
Threads: 1
Joined: Aug 2014
Reputation: 0
#3

13.03.2015, 21:09
pawn Код:
GetPlayerName(id, PlayerName, sizeof(PlayerName));
Why are you doing this AFTER using the variable in the string?

Also:
pawn Код:
if(sscanf(params,"dds",id,time,reason))
is deprecated. Use s[size] instead of just "s". For example:
pawn Код:
if(sscanf(params,"dds[100]",id,time,reason))
Find
Reply
Evocator
Huge Clucker
****
Posts: 470
Threads: 7
Joined: Nov 2013
Reputation: 0
#4

13.03.2015, 21:12
Too much errors, the player name should be fetched before the string formatting. Plus you need to specify sscanf's string specifier's length or your server might crash.

Код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new
			id,
			time,
			reason[100],
			szString[150],
			PlayerName[MAX_PLAYER_NAME+1]
		;
        
        if (sscanf(params, "iis[100]", id, time, reason)) return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");
        if (!IsPlayerConnected(id)) return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");
        if (Jailed[id] == 1) return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");
        
        GetPlayerName(id, PlayerName, sizeof(PlayerName));
		format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", PlayerName, id, time, reason);
		SendClientMessageToAll(-1, szString);

        SetPlayerInterior(id, 3);
		SetPlayerVirtualWorld(id, 10);
		SetPlayerFacingAngle(id, 360.0);
		SetPlayerPos(id, 197.5662, 175.4800, 1004.0);
		SetPlayerHealth(id, 9999999999.0);
		ResetPlayerWeapons(id);
		JailTimer[id] = SetTimerEx("Unjail",time*60000, false, "i", id);
	}
	else return 0;
	return 1;
}
Find
Reply
NoDi522
Huge Clucker
****
Posts: 317
Threads: 65
Joined: Sep 2014
Reputation: 0
#5

13.03.2015, 21:12
PHP код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new id,time,reason[100],szString[150];
        if(sscanf(params,"dds",id,time,reason)) return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");
        if(!IsPlayerConnected(id)) return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");
        if(Jailed[id] == 1) return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");
        {
                GetPlayerName(id, PlayerName, sizeof(PlayerName));
         GetPlayerName(playerid, GetPlayerName, sizeof(GetPlayerName));
        format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", PlayerName, id, time, reason);
        SendClientMessageToAll(-1, szString);
        }
        {
                SetPlayerInterior(id, 3);
        SetPlayerVirtualWorld(id, 10);
        SetPlayerFacingAngle(id, 360.0);
        SetPlayerPos(id, 197.5662, 175.4800, 1004.0);
        SetPlayerHealth(id, 9999999999.0);
        ResetPlayerWeapons(id);
        JailTimer[id] = SetTimerEx("Unjail",time*60000, false, "i", id);
    }
    else{
        return 0;
        }
    return 1;
} 
Hey... Try this just fix this identitations it is very hard to edit over here "intext"
Find
Reply
JuanStone
Huge Clucker
****
Posts: 274
Threads: 2
Joined: Apr 2014
Reputation: 0
#6

13.03.2015, 21:17
this is better.

Use the slots to sscanf, so it will be faster and do not need to create so many variables.

PHP код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new reason[100];
        if(sscanf(params, "uis[100]", params[0], params[1], reason))
            return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");
            
        if(!IsPlayerConnected(params[0]) && params[0] == INVALID_PLAYER_ID)
            return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");
            
        if(Jailed[params[0]] == 1)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");
        new name1[24], name2[24];
        GetPlayerName(params[0], name1, sizeof(name1));
        GetPlayerName(playerid, name2, sizeof(name2));
        
        format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", name1, params[0], params[1], reason);
        SendClientMessageToAll(-1, szString);
        
        SetPlayerInterior(params[0], 3);
        SetPlayerVirtualWorld(params[0], 10);
        SetPlayerFacingAngle(params[0], 360.0);
        SetPlayerPos(params[0], 197.5662, 175.4800, 1004.0);
        SetPlayerHealth(params[0], 9999999999.0);
        ResetPlayerWeapons(params[0]);
        JailTimer[params[0]] = SetTimerEx("Unjail", params[1]*60000, false, "i", params[0]);
    }
    return true;
} 
Find
Reply
Hybris
Huge Clucker
***
Posts: 207
Threads: 53
Joined: Dec 2013
Reputation: 0
#7

13.03.2015, 21:26
Quote:
Originally Posted by JuanStone
Посмотреть сообщение
this is better.

Use the slots to sscanf, so it will be faster and do not need to create so many variables.

PHP код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new reason[100];
        if(sscanf(params, "uis[100]", params[0], params[1], reason))
            return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");
            
        if(!IsPlayerConnected(params[0]) && params[0] == INVALID_PLAYER_ID)
            return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");
            
        if(Jailed[params[0]] == 1)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");
        new name1[24], name2[24];
        GetPlayerName(params[0], name1, sizeof(name1));
        GetPlayerName(playerid, name2, sizeof(name2));
        
        format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", name1, params[0], params[1], reason);
        SendClientMessageToAll(-1, szString);
        
        SetPlayerInterior(params[0], 3);
        SetPlayerVirtualWorld(params[0], 10);
        SetPlayerFacingAngle(params[0], 360.0);
        SetPlayerPos(params[0], 197.5662, 175.4800, 1004.0);
        SetPlayerHealth(params[0], 9999999999.0);
        ResetPlayerWeapons(params[0]);
        JailTimer[params[0]] = SetTimerEx("Unjail", params[1]*60000, false, "i", params[0]);
    }
    return true;
} 
That worked but whenever I try to do /unjail now it wont take me out of jail
Код:
CMD:unjail(playerid,params[])
{
	new id;
 	if(pInfo[playerid][pAdminLevel] >= 1)
	 {
		if(sscanf(params,"u",id)) return SendClientMessage(playerid, -1, "*"COL_WHITE" /unjail [playerid]");
		if(!IsPlayerConnected(id)) return SendClientMessage(playerid, -1, "*"COL_RED" Player is not connected.");
		if(Jailed[id] == 0) return SendClientMessage(playerid, -1, "*"COL_RED" Player is not jailed.");
  		Jailed[id] = 0;
		SetPlayerInterior(id, 0);
		SetPlayerVirtualWorld(id, 0);
		SpawnPlayer(id);
		SetPlayerHealth(id, 100);
		KillTimer(JailTimer[id]);
	}
	else
	{
		SendClientMessage(playerid, -1, "*"COL_RED" You do not have the right admin permissions for this command!");
	}
	return 1;
}
It just says player is not jailed also if I wait the specified time like 1 minute it doesnt unjail me.
Find
Reply
JuanStone
Huge Clucker
****
Posts: 274
Threads: 2
Joined: Apr 2014
Reputation: 0
#8

13.03.2015, 21:38
Try this:

PHP код:
CMD:unjail(playerid, params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        if(sscanf(params, "u", params[0]))
            return SendClientMessage(playerid, -1, "*"COL_WHITE" /unjail [playerid]");

        if(!IsPlayerConnected(params[0]) && params[0] == INVALID_PLAYER_ID)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is not connected.");
            
        if(Jailed[params[0]] == 0)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is not jailed.");

          SetPlayerInterior(params[0], 0);
          SetPlayerVirtualWorld(params[0], 0);
          
        SpawnPlayer(params[0]);
        SetPlayerHealth(params[0], 100);
        KillTimer(JailTimer[params[0]]);
        
        Jailed[params[0]] = 0;
    }
    return true;
} 
If it doesn't work show function "UnJail".
Find
Reply
Hybris
Huge Clucker
***
Posts: 207
Threads: 53
Joined: Dec 2013
Reputation: 0
#9

13.03.2015, 21:48
I don't have the function unjail I only have the command.
Find
Reply
JuanStone
Huge Clucker
****
Posts: 274
Threads: 2
Joined: Apr 2014
Reputation: 0
#10

13.03.2015, 21:54
Try this:

PHP код:
CMD:jail(playerid,params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        new reason[100];
        if(sscanf(params, "uis[100]", params[0], params[1], reason))
            return SendClientMessage(playerid, -1, "*"COL_WHITE" /jail [playerid] [time] [reason]");

        if(!IsPlayerConnected(params[0]) && params[0] == INVALID_PLAYER_ID)
            return SendClientMessage(playerid, -1,"*"COL_RED" Player is not connected.");

        if(Jailed[params[0]] == 1)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is already jailed.");

        new name1[24], name2[24];
        GetPlayerName(params[0], name1, sizeof(name1));
        GetPlayerName(playerid, name2, sizeof(name2));

        format(szString, sizeof(szString), "AdmSys-: %s (ID:%d) has been jailed for %d minutes; Reason: %s", name1, params[0], params[1], reason);
        SendClientMessageToAll(-1, szString);

        Jailed[params[0]] = 1;
        ResetPlayerWeapons(params[0]);
        SetPlayerInterior(params[0], 3);
        SetPlayerVirtualWorld(params[0], 10);
        SetPlayerFacingAngle(params[0], 360.0);
        SetPlayerPos(params[0], 197.5662, 175.4800, 1004.0);
        SetPlayerHealth(params[0], 9999999999.00);
        
        JailTimer[params[0]] = SetTimerEx("Unjail", params[1]*60*1000, false, "i", params[0]);
    }
    return true;
}

CMD:unjail(playerid, params[])
{
    if(pInfo[playerid][pAdminLevel] >= 1)
    {
        if(sscanf(params, "u", params[0]))
            return SendClientMessage(playerid, -1, "*"COL_WHITE" /unjail [playerid]");

        if(!IsPlayerConnected(params[0]) && params[0] == INVALID_PLAYER_ID)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is not connected.");

        if(Jailed[params[0]] == 0)
            return SendClientMessage(playerid, -1, "*"COL_RED" Player is not jailed.");

        SetPlayerInterior(params[0], 0);
        SetPlayerVirtualWorld(params[0], 0);

        SpawnPlayer(params[0]);
        SetPlayerHealth(params[0], 100);
        KillTimer(JailTimer[params[0]]);
        Jailed[params[0]] = 0;
    }
    return true;
}

forward Unjail(playerid);

public Unjail(playerid)
{
    SetPlayerInterior(playerid, 0);
    SetPlayerVirtualWorld(playerid, 0);

    SpawnPlayer(playerid);
    SetPlayerHealth(playerid, 100);
     Jailed[playerid] = 0;
    return true;
} 
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