!! Just a Realy quick one !! - Printable Version

!! Just a Realy quick one !! - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+---- Forum: Help Archive (https://sampforum.blast.hk/forumdisplay.php?fid=89)
+---- Thread: !! Just a Realy quick one !! (/showthread.php?tid=186697)

!! Just a Realy quick one !! - fubar - 30.10.2010

Hi folks this is going to be a quick answer have a look at this

Код:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys){
	if(!gPlayerUsingLoopingAnim[playerid]) return 0;
	if(IsKeyJustDown(KEY_SPRINT,newkeys,oldkeys)) {
	    StopLoopingAnim(playerid);
        TextDrawHideForPlayer(playerid,txtAnimHelper);}
  if(Enabled[playerid] == 1){
	if (IsPlayerInAnyVehicle(playerid)){
		if((newkeys & KEY_ACTION) && !(oldkeys & KEY_ANALOG_UP)){
  	        switch(GetVehicleModel( GetPlayerVehicleID(playerid) )) {
			case 592,577,511,512,593,520,553,476,519,460,513,487,488,548,425,417,497,563,447,469:
			return 1;}
		    if(CreatedRamp[playerid] == true) DestroyObject(Ramp[playerid]), CreatedRamp[playerid] = false;
			new Float:pX,Float:pY,Float:pZ,Float:vA, Arabam = GetPlayerVehicleID(playerid);
			GetVehiclePos(Arabam, pX, pY, pZ);
			GetVehicleZAngle(Arabam, vA);
			Ramp[playerid] = CreateObject(Pers[playerid], pX + (20.0 * floatsin(-vA, degrees)), pY + (20.0 * floatcos(-vA, degrees)), pZ, 0, 0, vA);
			CreatedRamp[playerid] = true;
			SetTimerEx("destroy", 4000,0,"d",playerid);}}
			return 1;}}

So everything is ok,no errors but i am getting a warning,which i think is coming from this line

Код:

if(!gPlayerUsingLoopingAnim[playerid]) return 0;

and this line is showing up as" : warning 209: function "OnPlayerKeyStateChange" should return a value "

Код:

return 1;}}

I know i have done something wrong,but i cant figure it out

cheers in advance for your help

Re: !! Just a Realy quick one !! - Cameltoe - 30.10.2010

This:

pawn Код:

if(!gPlayerUsingLoopingAnim[playerid]) return 0;

Should probably be like this:

pawn Код:

if(gPlayerUsingLoopingAnim[playerid] == 0) return 0;

Though some indention might help you:

pawn Код:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
    if(gPlayerUsingLoopingAnim[playerid] == 0) return 0;
    if(IsKeyJustDown(KEY_SPRINT,newkeys,oldkeys))
    {
        StopLoopingAnim(playerid);
        TextDrawHideForPlayer(playerid,txtAnimHelper);
    }
    if(Enabled[playerid] == 1)
    {
        if (IsPlayerInAnyVehicle(playerid))
        {
            if((newkeys & KEY_ACTION) && !(oldkeys & KEY_ANALOG_UP))
            {
                switch(GetVehicleModel( GetPlayerVehicleID(playerid) ))
                {
                    case 592,577,511,512,593,520,553,476,519,460,513,487,488,548,425,417,497,563,447,469:
                    return 1;
                }
                if(CreatedRamp[playerid] == true) DestroyObject(Ramp[playerid]), CreatedRamp[playerid] = false;
                new Float:pX,Float:pY,Float:pZ,Float:vA, Arabam = GetPlayerVehicleID(playerid);
                GetVehiclePos(Arabam, pX, pY, pZ);
                GetVehicleZAngle(Arabam, vA);
                Ramp[playerid] = CreateObject(Pers[playerid], pX + (20.0 * floatsin(-vA, degrees)), pY + (20.0 * floatcos(-vA, degrees)), pZ, 0, 0, vA);
                CreatedRamp[playerid] = true;
                SetTimerEx("destroy", 4000,0,"d",playerid);
            }
        }
    }
    return 1;
}

Re: !! Just a Realy quick one !! - Psycho77 - 30.10.2010

//edit: Cameltoe was faster (;

Re: !! Just a Realy quick one !! - Cameltoe - 30.10.2010

Quote:

Originally Posted by Psycho77

try this:

Код:

        return 1;
    }
    return 1;
}

Unreachable code?

Re: !! Just a Realy quick one !! - Psycho77 - 30.10.2010

yeah youґre right..

Re: !! Just a Realy quick one !! - fubar - 30.10.2010

cheers guys for your help it work a treat
Thanx

ps i like my bad indention