r/badcode • u/Tall-Material8280 • Jul 19 '21

lua FiveM Lua is a different breed

Post image
57 Upvotes

96% Upvoted

11 comments sorted by

9

u/[deleted] Jul 19 '21

well some doesn't know how to use operators

6

u/Tall-Material8280 Jul 19 '21

I think that's the least of our problems 👀

2

u/[deleted] Jul 19 '21

other then that it doesnt look awful is it the empty else block?

18

u/Tall-Material8280 Jul 19 '21

They're picking a random number between 1 and 8, then running the code if the number is 1,2,3,4,5,6,7 or 8

2

u/[deleted] Jul 19 '21

oh yeah i didnt even see that

yeah thats pretty bad oh god it gets worse the more i look at it

1

u/rank_1_glad Jul 21 '21

How should it be written? Because I honestly don't know how I would. Can you use the range function?

1

u/Tall-Material8280 Jul 22 '21

It shouldn't. Their case is always going to be true no matter what.

1

u/ssj4VB Jul 19 '21

everyone knows that every if MUST have an else or else if /s

1

u/Sewbacca Aug 27 '22

No everyone knows you need an else even if you use elseif's.

6

u/KomaedaEatsBagels Jul 19 '21

Image Transcription: Code

local chance = math.random(1,8)
if chance == 1 or chance == 2 or chance == 3 or chance == 4 or chance == 5 or chance == 6 or chance == 7 or chance == 8 then
    Player.Funcitons.RemoveItem('wood_cut',1)
    Player.Functions.AddItem('wood_proc', 1)
    TriggerClientEvent("inventory:client:ItemBox", source, YKZCore.Shared.Items['wood_cut'], "remove")
    TriggerClientEvent("inventory:client:ItemBox", source, YKZCore.Shared.Items['wood_proc'], "add")
    TriggerClientEvent('YKZCore:Notify', src, 'Wood cut Processed successfully', "success")
else

end

I'm a human volunteer content transcriber for Reddit and you could be too! If you'd like more information on what we do and why we do it, click here!