Just for know is this correct or nah!

[sharp wing]

I'm doing this for check if player has all badges

[grand vale]

I need a bit more context, are you going to check every badge individually?

[sharp wing]

no

like this

[gusty niche]

no, this is really bad

[grand vale]

Oh, oh...

[grand vale]

Yes

[sharp wing]

yeah i know

[grand vale]

Too much nesting.

[sharp wing]

the best option is make a property table

but idk how do it

if it uses for _,v in pairs or other thing idk

[grand vale]

I would go a route of making an array of badge ids and iterate over it. Save in a different var how many of these thw user has and compere the max amount and player's.

[grand vale]

Yes

[sharp wing]

ok

i will make something here and u see if this is correct or nah

[grand vale]

At least that's the best way I can come up with on phone at 2am.

[grand vale]

Copy that

[gusty niche]

yes, but annotate that pairs and ipairs are useless in luau

[void coral]

for loops

use them

[grand vale]

Mostly use Ipairs but pairs for sorted arrays

[sharp wing]

idk how use break

[gusty niche]

is useless in luau

[grand vale]

You shouldn't need to use break in this one?

[bold herald]

bruh instead of using if statement a lot you can just do this instead

[gusty niche]

dont write a code

[bold herald]

why

[void coral]

why use break

[gusty niche]

depends on how you're going to

[sharp wing]

like that thing i did, idk how do it

[void coral]

https://create.roblox.com/docs/tutorials/fundamentals/coding-4/intro-to-for-loops

[gusty niche]

if its just a code show good but dont spoonfeed

[bold herald]

ok

so

[sharp wing]

looking now break doesnt make sense

[void coral]

:GetDescendants() is for objects, not for tables

[bold herald]

put all the badges into the table.

[void coral]

if you want ot loop in a table you simply put the table name

[sharp wing]

so what is for tables?

[bold herald]

and then loop through it

[sharp wing]

ok

[bold herald]

whenever the player added

[sharp wing]

I SAW THAT

[grand vale]

Delete the while loop, and use just Badges

[gusty niche]

:GetDescendants is a method from Instances, which are tables

[void coral]

local tableName = {1, "hi"}
for index, value in tableName do
  -- thread
end

https://www.youtube.com/watch?v=Ma-tXw7ykns

[void coral]

keyword instances

[grand vale]

Just use engine reference or language one...

[grand vale]

It returns a array of the instance's children (descendants)

[sharp wing]

wait guys im confuse

[bold herald]

local allbadge = {...}
local BadgeService = game:Get("BadgeService") <- you get service
game.PlayersAdded... <-- connect for every player
for _, badge in pairs(allbadge)
check if the player join has all the badge or not
BadgeService:UserHasBsadgeAsync(plr.Userid, badge) what are you going to do if you have a badge or not?
end
end)

[grand vale]

You check for the badge inside the for loop.

[sharp wing]

if the player doesnt have all badges the portal doesnt open

[grand vale]

Ah wait

My bad

Telephone shifted text

[grand vale]

Pretty much

[void coral]

use a bool value to determine this

[sharp wing]

ok

[modest valley]

** You are now Level 7! **

[sharp wing]

thanks brotha

[grand vale]

still use pairs

[void coral]

why do you have strings in your table

[sharp wing]

IDK

[grand vale]

Ids

[void coral]

"Badge1" is a number id for you?

[void coral]

[sharp wing]

i love this emoji

[void coral]

dont spoonfeed bro

[sharp wing]

omg

[void coral]

also thats not what he wanted

lmfao

[bold herald]

ok

[void coral]

thanks

[grand vale]

Now, you need to make another variable wich is being incremented by 1 each time the player has a badge

And don't increment when they don't

[bold herald]

ok i will tell it one by one instead

[sharp wing]

k

[sharp wing]

Oh and thanks for trying, seeing here that you made this whole script just to help me

[void coral]

anyway do these steps:

Determine the badges in a table

Mkae a boolean variable that determines if all badges are owned by the player

Use a for index, value in table loop to go through all the badges in the table and make an algorithm that determines if player has all badges and sets the bool variable to true if player has them all, if not then false duh

ultimately execute the algorithm that turns the portal on based if value is true or not.

[bold herald]

are you going to check if the player finally completed all the badge so that the portal automatically opens every second?

that's why you are using while loop?

[grand vale]

After you do this, after the for loop you just make a simple if statement that checks if x == AmountOfBadges

[grand vale]

Good question

[void coral]

guys yk ure just confusing him more right

[sharp wing]

yes

[sharp wing]

is this what i need to do luciferius?

[void coral]

no

[grand vale]

Yes, but no

[void coral]

i mean yes but in the for loop you have to add 1 to badgecount only when the player has the badge

[bold herald]

make a variable for boolean till the badge count is equal to 5

[sharp wing]

ok

[bold herald]

or use #to get all the length of a table.

[grand vale]

First point, and the second is that the check needs to happen in PlayersAdded

[void coral]

abd delete the while loop

[sharp wing]

oh my god

[grand vale]

A clutter 😅

[sharp wing]

guys i realize that the script is meant to be just for the client

[grand vale]

?

[sharp wing]

yeah

bcz if the script will ecexute for everybody

[grand vale]

It blocks a section of a game, it shouldn't be local based?

[void coral]

local Badges = {...}
local bCount = 0

local bService = game:GetService("BadgeService")
local Players = game:GetService("Players")

Players.PlayerAdded:Connect(function(plr)
  for _, v in pairs(Badges) do
    -- thread
  end
end)

[sharp wing]

yeah but it doesnt change everything?

[void coral]

like this

[grand vale]

Otherwise players will cheat

[sharp wing]

so its just put everything inside a local script?

[void coral]

fuck no

[grand vale]

No

A normal script

[sharp wing]

ok

[void coral]

okay lets do it like this

[bold herald]

if you want only the player who has all the badge then you should send a remote event that opens a portal for that client 🙂

[grand vale]

Yes

[sharp wing]

oh i understand it now

ok i will try do it

[void coral]

Watch this guy

Read this and this

and use this

[sharp wing]

wait

all that

[sharp wing]

or just this?

[bold herald]

local Badges = {...}
local bCount = 0

local bService = game:GetService9"BadgeService")
local Players = game:GetService("Players")

Players.PlayerAdded:Connect(function(plr)
for _, v in pairs(Badges) do
-- thread
end
end) you can do this tho 🙂 but you have to use remote event once the server detected that the player has all the badge

[void coral]

no thats unresourceful

[bold herald]

yeah if it were a simple game it should be fine

[void coral]

no

it also creates bad habits

[bold herald]

alr

[void coral]

it makes 0 sense for server to handle something that is deisgnated for only the client

sure it may intervene when needed

[grand vale]

What I would do.

Have a detector (invisible part with canCollide off but canTouch true) which will be a trigger for us.

When player touches it, run a check.

When the player has sufficient exp/lvl/badges you open the portal via remoteEvent.

[void coral]

but it isnt 99% of the tme

[bold herald]

i rarely use remote event that much tho..

i actually use Module script all the time..

[void coral]

or use a localscript

[sharp wing]

so what did i do

[grand vale]

The check would be done on normal script (server side) to validate it and opening made on localscript

[void coral]

validating things here is deemed useless

its not like the player can just give himself a bagde

or fake it

[void coral]

make a local script check if the player has all the badges every 5 seconds, if they do, stop the thread in any way.

[sharp wing]

i can use debounce?

[void coral]

task.wait()

its your favourite while loop

[sharp wing]

yes it is!

[void coral]

the problem did not include nay other scripts, and i do not know the intentions of para so dont blame me

[grand vale]

That's why I admitted that it's probably a better way, I would make it more complex in a good way. Let's stay simple.

[void coral]

making simple things complex is called overcomplication

as a scripter thats one of your worst enemies

[void coral]

ye

[bold herald]

badgeHandler.__index = badgeHandler


function badgeHandler.new() 
    local self = setmetatable({}, badgeHandler)
    return self 
end

return badgeHandler``` just make oop structure xd

[sharp wing]

ok

[grand vale]

I would make it save to player array/dictionary/class with player related vars to avoid any/all unnecessary checks. At least that's my reasoning.

Let's leave it be for debate.

[void coral]

honestly checking if the player has all the badges every 5 seconds is straight bs but wtv

the guys learning

let him be

[bold herald]

i am still learning tho please teach me how to get into level 2 scripting

[sharp wing]

help me to get on lvl 1

[bold herald]

because of my bad naming convetions, etc.

[sharp wing]

i have 1 year of script my life is in a while loop

[bold herald]

honestly, i'll get into level 2 scripting sooner because i am taking my cs next month

[sharp wing]

local life = false

[bold herald]

damn that's sad

i am taking my first year as cs next month btw

so good luck with me

[sharp wing]

and i have 4 years of studio and my builds look trash too

[sharp wing]

relax im bad bcz im lazy

[grand vale]

Let's stop complaining

[bold herald]

are you into networking system?

[sharp wing]

yeah i will not learn crying

[grand vale]

Yyyy... I was bored. I program in cpp as hobby.

[bold herald]

you can make good games with cpp bro

[modest valley]

** You are now Level 10! **

[grand vale]

I learnt it for UE but the documentation is shit, and I don't want to pay for courses

I think about turning Unity

[grand vale]

Yes

[bold herald]

is learning OOP early good for cs?

i learn OOP in python aswel

and lua

[grand vale]

Basically do a localscript that checks every 5 sec if the player has x badges

[sharp wing]

like this

[grand vale]

Idk, I'm not even in higher edu yet. I learnt python, lua, cpp, dart (for flutter) and swift

[sharp wing]

and i was struggling with html some days ago

[grand vale]

Scrap the check

[grand vale]

It isnt even a programming language

[bold herald]

i do HTML, CSS, JAVASCRIPT, + CSS FRAMEWORK THO 💀

[sharp wing]

what?

[bold herald]

it was a bootstrap i think

[bold herald]

it's a hypertext markup language bro

not a real programming language.

[sharp wing]

ah hell nah

and some people call me a scripter

[vivid fractal]

https://tenor.com/view/fairy-fairymf-poop-fairygodmother-slander-gif-22756406

[bold herald]

gtg now

[sharp wing]

bye

luciferius

[grand vale]

It's a coding language as it doesn't use logic

[grand vale]

I meant the one with touched

[sharp wing]

no

[grand vale]

We will stay simple

[sharp wing]

the touch is for tp the player for another place

[grand vale]

Ah

[sharp wing]

its ok

i didnt explained it

[grand vale]

No, wait

[sharp wing]

?

[bold herald]

lets do vibe coding bro

[grand vale]

I will sketch you a crude structure of how it is supposed to look like

[grand vale]

Hell nah

[bold herald]

i was joking let's make a fun game or simple game bro 🙂

[bold herald]

how long have you been scripting?

i was like 5 months but i guess still not enough to reach l2

[grand vale]

//your vars and array

Player:Added
-> For loop
->-> check if the player has badge if so increment a variable by one, otherwise do no.
->the rest

Arrows are the indents

[sharp wing]

please explain with code im dumb

[grand vale]

So for loop and the rest is inside player:added and the check is inside the for loop

[grand vale]

Truthfully couple of years but didn't have any serious undertakings in roblox studio, that's why I don't have anything to back my S2 claim.

[sharp wing]

im using a local script brotah

[grand vale]

Yes

You should

[sharp wing]

i cant use playeradded

[grand vale]

We don't want to overcomplicate

[grand vale]

You cant?

Oh lol

https://tenor.com/view/kirby-i-forgot-i-forgor-gif-22449575

Well

That changes things

[sharp wing]

what i do now

[bold herald]

local allbadge = {...}
local badgeservice = game:GetService("BadgeService")


for _, badge in pairs(allbadge)
    local badgeCounter = 0
    if badgeservice:UserHasBadgeAsync(plr.Userid, badge)
        badgeCounter += 1
    end
end 


if badgeCounter == #allbadge then
    print("has all the badge whatever")
else

print("the player doesnt have all the badge")
end ```

[grand vale]

Learn remoteEvents

[bold herald]

no need to use playeradded whne you are using local script..

[grand vale]

Ah right

Good point you can use localplayer

3:42 am here am I

[sharp wing]

but is that right or not

[grand vale]

Should be g

[bold herald]

dont use remote event if it is not necessary.. since you are opening a portal i think you dotn have to use remote event

you can just do it on the client

[grand vale]

Yes, you need to learn sterelization then

If the value provided is the one you expect

Hard asf

[grand vale]

This should be useable

[bold herald]

just do transparency 1 to the portal if he has it simple

[grand vale]

And cancolide off

Or tweens depending on what you want