Something wrong?

[-Doc-]

Is something wrong there?

function isPlayerMoving(player) 
    return (not isPedInVehicle(player) and (getPedControlState(player, "forwards") or getPedControlState(player, "backwards") or getPedControlState(player, "aim_weapon") or getPedControlState(player, "left") or getPedControlState(player, "right") or getPedControlState(player, "accelerate") or getPedControlState(player, "brake_reverse") or getPedControlState(player, "enter_exit") or getPedControlState(player, "enter_passenger"))) 
end 
  

[ALw7sH]

Yes

you have to do

(not isPedInVehicle(player) and getPedControlState(player, "forwards") == true) and true or (not isPedInVehicle(player) and getPedControlState(player, "backwards") == true) and true etc...

Just use that method

local controls = {"forwards","backwards","aim_weapon","left","right","accelerate","brake_reverse","enter_exit","enter_passenger"} 
function isPlayerMoving(player) 
    if not isPedInVehicle(player) then 
        for k,v in ipairs(controls) do 
            if getPedControlState(player, v) then 
                return true 
            end 
        end 
    end 
    return false 
end 

[-Doc-]

Thx ill try.

[-Doc-]

Look in pm i sended my script.

[Addlibs]

you have to do

(not isPedInVehicle(player) and getPedControlState(player, "forwards") == true) and true or (not isPedInVehicle(player) and getPedControlState(player, "backwards") == true) and true etc...

That's pretty incorrect. You don't have to do that.

[ALw7sH]

you have to do

(not isPedInVehicle(player) and getPedControlState(player, "forwards") == true) and true or (not isPedInVehicle(player) and getPedControlState(player, "backwards") == true) and true etc...

That's pretty incorrect. You don't have to do that.

There's nothing called incorrect, it called a proper way ...

[Addlibs]

1) By 'incorrect', I did not mean 'incorrect way' I mean that your statement was incorrect - you do not have to add 'and true'

2) The proper way is the most efficient way — don't add unnecessary bytes by adding redundant parts to a script. That is called being inefficient.

[IIYAMA]

The function getPedTask gives more accurate results in what the player is doing and will be more 'efficiënt'...

https://wiki.multitheftauto.com/wiki/GetPedTask