Cow tipping - Free simple script

RedM Releases
, ,
1

a quick free script i made its a cow tipping script just run into a cow fast enough and it will tip , why???
cus its funny :slightly_smiling_face:
p.s no cows were harmed in the making of this script or video , they were merely tipped…


cowtipiing.PNG
cowtipiing.PNG1009Ă—698 42.4 KB

mm_cowtip.zip (1.1 KB)
Code is accessible Yes
Subscription-based YNo
Lines (approximately) 400+
Requirements n/a
Support No
2

Love this lol. Is there a way you can add a pushing animation when you’ve “touched” the cows?

3

Cool script, however there are quite a few performance issues with this code.

Firstly, you are running a function every single frame that loops through CPed and does a Vdist on every single ped that isn’t a player.

Vdist is a very very slow way to do distance calculation. The proper way to calculate distance is by using lua’s vector math:

local D = #(vecA - vecB)

Here is a source on that: GitHub - GoatG33k/FiveM-dist-calc-perf-testing: A resource I used for performance testing different FiveM distance calculations

So, let’s backtrack to where I said every single frame. That is an issue with what we’re doing here. We do not need to loop through CPed every frame, or even every 10 milliseconds. I would slow that loop down considerably, to possibly even 300-500ms so the player doesn’t notice its taking too long to loop.

There is also a code structure issue, where in the “Main thread” we are:

  1. Checking if the player is near a cow. This is the very CPU heavy loop through CPed that runs a Vdist on every single ped that isn’t the player.
  2. Checking the player’s velocity, and executing the RagdollCow function, which is:
  3. Checking again if the player is near a cow, which is running that very CPU heavy loop again, when we already know if the player is near a cow. This check is redundant.

There is also no timeout after a successful cow tip. This would increase performance as well.

Now, I’m not meaning to tear your code apart or make you upset in anyway - I’m simply trying to get you on the right track with optimization.

Below I’ve completely refactored your code to be much more optimized and explained what I did in the comments.

-- RedM Script to ragdoll a cow when colliding with it

-- Constants
local COW_MODEL = `a_c_cow` -- Let's hash this right away, so we arent running GetHashKey in a loop for the model check.
local DETECT_DISTANCE = 1.5 -- Distance to detect the cow
local RAGDOLL_TIME = 5000 -- Duration of the ragdoll effect in milliseconds

-- Main thread
CreateThread(function() -- Citizen.CreateThread = CreateThread
    while true do
        -- Slow down the interval A LOT, this is a very expensive loop operation. 
        -- This should be slow enough for a huge performance boost, while still being fast enough to detect cows player is running against.
        Wait(400)

        local playerPed = PlayerPedId()
        local playerPos = GetEntityCoords(playerPed)

        for _, entity in ipairs(GetGamePool('CPed')) do
            -- Let's add the model check here, so we don't have to loop through all entities AND check their distance before we make sure its actually a cow.
            -- I got rid of the IsPedAPlayer check since player's can't be cows (under normal circumstances).
            if GetEntityModel(entity) ~= COW_MODEL then
                -- Skip if the entity is a player or not a cow
                goto continue
            end

            local entityPos = GetEntityCoords(entity)
            local distance = #(playerPos - entityPos) -- This distance check is much, much faster.

            if distance < DETECT_DISTANCE then -- Here we only need to check the distance, since we already know we're dealing with a cow.
                -- Get the player's velocity
                local playerSpeed = GetEntitySpeed(playerPed) -- If we get the speed, we don't need to calculate magnitude on a vector.

                -- If the player is moving fast enough, ragdoll the cow
                if playerSpeed > 5.0 then
                    -- Let's get rid of the RagdollCow function since it only had one line of code after optimization.
                    SetPedToRagdoll(entity, RAGDOLL_TIME, RAGDOLL_TIME, 0, false, false, false)
                    -- Let's introduce a timeout because we don't need to keep checking for a while immediately after ragdolling the cow.
                    Wait(2000) -- Wait 2 seconds before continuing, and then break the for loop because we found our cow.
                    break
                end
            end

            ::continue::
        end
    end
end)
1 Like
4

ah sweet thanks

5

scripts open so u can add it urself if u wish i just made it cus i was bored lol

1 Like
6

Thanks man.

7

Was this added to help with performance? or did the person never add that code? i want this but DEF wan t it only if its been updated. Funny tho!

8

Just copy the code they posted

9

Do you have it on github? i need to learn to use Github better including requesting to add stuff that way. It’ll help me learn how to work it better.

10

the script is open download it and copy the client code that , is posted up there and replace the current one for it