Am I being stupid or?

Help and Feedback Scripting Support
#1 
game.Players.PlayerAdded:Connect(function(plr)
	for i in pairs(data.sheets[1].data[1].rowData) do
		if data.sheets[1].data[1].rowData[i].values[1].formattedValue == 'AviaJxnny' then
			print(data.sheets[1].data[1].rowData[i].values[2].formattedValue)
		end
	end
end)

Am I being stupid or something because it isnt counting that the player is joining.

#2

Maybe because you only have one Variable in the for loop

for i in pairs
to
for i,v in pairs
#3

Is this excerpt a part of a server script?

#5

No, it isn’t just tried that.

I even tried to do,

game.Players.PlayerAdded:Connect(function()
    print("hi")
end)
#6

Is that a local or regular script?, you should run the code in a regular script

#8

I’m not quite sure what’s causing the script to error - are you able to upload a repro file?

#10

A repro file is a .rbxm file where you include all the items linked to the script’s malfunction.

#11

unfortunately it has private api keys that run the db

#12

Ah, that’s perfectly fine. Regarding the other script:

Was this the only block of code in the script, or was there other code above it? Excessive code and functions such as Instance:WaitForChild() can delay the script, which might have caused your scripts’ failure to print in time.

#13 
game.Players.PlayerAdded:Connect(function(plr)
	for i, element in pairs(data.sheets[1].data[1].rowData) do
		if element.values[1].formattedValue == 'AviaJxnny' then
			print(element.values[2].formattedValue)
		end
	end
end)

Is this the same as your code above right? and if you are making a search an efficient way its using while loop not for loop. Btw maybe you need to make a print debugging to find out the error that sometimes solve bug issues like that idk if you have previous code that generates your problem.

#14

Using a for anything, anything2 in pairs(table) do is the correct way to loop through a table. I’m not sure why you’d use a while loop for that.

1 Like
#15

It always counts if the player joins. To test this, run this:

game:GetService("Players").PlayerAdded:Connect(print)
-- prints the name of all new players who join

if it doesn’t work, that means you have something above it that’s yielding the thread.

1 Like
#16

Because when while loop is used finds the data required and stop searching.
For loop keep searching even if the data is found already.

#17

That doesn’t really make sense. You can just break a for loop once you’ve found what you’re looking for.

1 Like
#18

Yep you can do that but while loop does it for you :wink:

#19
#20