Variable from IntValue error

Help and Feedback Scripting Support
#1

I’m trying to set the variable minCount to the value of an IntValue called minCount in a part. This is the relevant code:

bases = {}
basesFolder = game.Workspace.bases

for i, v in ipairs(basesFolder:GetChildren()) do
	bases[i] = v
end

for v in bases do
	minCount = v.minCount.Value
end

Instead of setting the variable to the IntValue, it returns “attempt to index number with ‘minCount’” and the script stops. How do I fix this?

5 Likes
#2

Change IntValue to the NumberValue because IntValues are intended for integer values only.

2 Likes
#3

The error is exactly the same, nothing changed.

#4

why dont you set local:

local mincount
#5

This is a server sided script. How on earth would using local variables fix anything?
Edit: tested and to nobody’s surprise, nothing changed.

#6

Can you show us what the folder looks like in the explorer?

#7

reworked your script

bases = {}
basesFolder = game.Workspace.bases

for i, v in ipairs(basesFolder:GetChildren()) do
    bases[i] = v
end

for i, v in pairs(bases) do
    if v:FindFirstChild("minCount") and v.minCount:IsA("IntValue") then
        minCount = v.minCount.Value
    else
        warn("Not Found " .. v.Name)
    end
end
#8

Yea noticed the for in loops always get 2 variables so on your second loop you only have 1 meaning v = number index instead of the bade object. Changing it to for i’v … fixes it

#9

Depending on the situation I don’t recommend using pairs since the order of operation isn’t guranteed to go in the number order. These days the ipairs and pairs are no longer needed and the syntax of for i, v in list do is completely valid :smiley:

#10

If you need the mincount to be personalized for each object:

bases = {}
basesFolder = game.Workspace.bases

for i, v in ipairs(basesFolder:GetChildren()) do
	bases[i] = v
end

for v in bases do
	local minCount = tonumber(math.floor(v.minCount.Value))
end

else:

bases = {}
basesFolder = game.Workspace.bases

for i, v in ipairs(basesFolder:GetChildren()) do
	bases[i] = v
end

for v in bases do
	minCount = tonumber(math.floor(v.minCount.Value))
end
#11

What’s the exact line that errored? I’m thinking the part of the script you provided isn’t actually the problem

#12

It’s difficult to solve your problem because you provide very little context. Can you provide explorer images/the whole script?

#13

As mentioned here, the issue comes from the lack of defining a 2nd variable in the second for loop.

-- The "v" is the problem.
for v in bases do
	minCount = v.minCount.Value
end

The first variable defined in a for in loop will always be an index which is the number. So if for example v is 0 the code is trying to run 0.minCount.Value which throws the error.

-- This works now
for i, v in bases do
	minCount = v.minCount.Value
end

By defining index to be i lets the for in loop define v as the actual base instance which is more valid. To be safe you should introduce some FindFirstChild checks to actually make sure minCount can be found.

for i,v in bases do
	local foundMinCount = v:FindFirstChild("minCount")
	if foundMinCount and foundMinCount:IsA("IntValue") then
		minCount = foundMinCount.Value
	end
end

Hope this clears it up better for everyone :smiley:

#14

This needs a full explanation on what this is doing and how everything is set up.
Going off what you said was the error: “attempt to index number with minCount”

That sound like you picked the wrong .Value …
Are you sure you manually added that as a number and not a int?

#15

Oh yeah; can’t believe my eyes missed that. I suppose because of the lack of an i before the v, the for loop is simply taking v to be a variable name for i, thus v has become the index. And because indexes are numbers, this results in the error. Nicely spotted

2 Likes
#16

This topic was automatically closed 14 days after the last reply. New replies are no longer allowed.