table.Insert() not saving?

[SharkRayMaster (Dasiwa)]

I have a script that checks a list of disasters to see which one is selectable

Then uses table.insert() to insert the disaster name in a table if it is

Script:

local AvailableMaps = {}
	
	for i, disaster in pairs(Disasters) do
		
		-- Check enabled
		if disaster.Enabled == false then return end
		
		-- Check map specific
		if disaster.isMapSpecific == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then return end
		end
		
		-- Check if only unique map disasters
		
		if UniqueMapDisastersOnly == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then return end
		end
		
		-- Add disaster to table
		
		table.insert(AvailableMaps, disaster.Name)
		
		for i, v in pairs(AvailableMaps) do
			print(v) -- This prints correctly the disasters
		end
		
		
	end
	
	for i, v in pairs(AvailableMaps) do
		print(v) -- this doesn't print anything
	end

Although the table “AvailableMaps” is outside the for loop, printing the things inside the table in the for loops does return the desired disasters, but doing it outside doesn’t print anything, it returns nil

Why does this happen? How could I fix it? HELP

[yesimrealll (SSQ)]

Is there something removing data outside the for loop? If not then maybe try this work around:
1.Create a folder inside the script
2.Create a stringvalue inside the folder with a value of the disaster’s name everytime you add
a disaster’s name

(Optional)
3.Delete all the string value inside the folder if no longer needed.

[SharkRayMaster (Dasiwa)]

There’s nothing removing data from the table, and yes your solution sounds good but I’m wondering why doesn’t it work?

[Dev_Peashie]

There is no clear reason why your script is not working, could you show the Disaster table?

Just to give a simple example this reduced code does the same as yours and yes it SHOULD work:

local AvailableMaps = {}

local Disasters = {
	["Disaster1"] = {Enabled = true, Name = "doom"},
	["Disaster2"] = {Enabled = true, Name = "fire from sky"},
	["Disaster3"] = {Enabled = true, Name = "tacos from sky"},
}

for i, disaster in pairs(Disasters) do
	-- Check enabled
	if disaster.Enabled == false then return end

	-- Add disaster to table
	warn("going to add:", disaster.Name)
	table.insert(AvailableMaps, disaster.Name)
end

-- Print the table
for i, v in pairs(AvailableMaps) do
	print(v)
end

To me, I dont like to use intValues, stringValues everywhere in folders and stuff. Don’t give up with your table based approach, tables are pretty handy

[SharkRayMaster (Dasiwa)]

I’ll explain further, every second ingame the Disaster Event is fired, then in another script, there’s a connection to the event and in that script there is a 20% chance of ocurring (right now its 100% for testing purposes), now here comes the problem part, there’s a custom function to get a random disaster which is the one you see, I’ll show it here:

local function GetRandomDisaster()
	local SelectedDisaster = nil
	local SelectedDisasterName = nil

	AvailableMaps = {}

	for i, disaster in pairs(Disasters) do

		-- Check enabled
		if disaster.Enabled == false then return end

		-- Check map specific
		if disaster.isMapSpecific == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then return end
		end

		-- Check if only unique map disasters

		if UniqueMapDisastersOnly == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then return end
		end

		-- Add disaster to table

		table.insert(AvailableMaps, disaster.Name)

		for i, v in pairs(AvailableMaps) do
			print(v) -- THIS PRINTS "TORNADO"
		end
		
		task.wait(2)

	end
	
	task.wait(3)


	for i, v in pairs(AvailableMaps) do
		print(v) -- THIS DOESN'T PRINT ANYTHING
	end

	SelectedDisaster = AvailableMaps[math.random(1, #AvailableMaps)]
	SelectedDisasterName = SelectedDisaster.Name


	return SelectedDisaster, SelectedDisasterName

end

And this is my disaster table:

local Disasters = {
	["Tsunami"] = {
		Name = "Tsunami",
		isMapSpecific = false,
		SpecificMap = "None",
		Enabled = false,
		Disaster_Function = function() --[[DisasterModule:TsunamiDisaster()]] end},

	["Tornado"] = {
		Name = "Tornado",
		isMapSpecific = false,
		SpecificMap = "None",
		Enabled = true,
		Disaster_Function = function() --[[DisasterModule:TornadoDisaster()]] end}

}

I’m gonna cry

[SharkRayMaster (Dasiwa)]

Well, I don’t know why it wasn’t working but I fixed it by not using "then return end"

Code:

local function GetRandomDisaster()
	local SelectedDisaster = nil
	local SelectedDisasterName = nil

	AvailableMaps = {}

	for i, disaster in pairs(Disasters) do
		
		local map_Enabled = false
		local map_MatchSpecific = true
		local map_MatchUnique = true
		
		-- Check enabled
		if disaster.Enabled == true then
			map_Enabled = true
		end

		-- Check map specific
		if disaster.isMapSpecific == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then
				map_MatchSpecific = false
			end
		end

		-- Check if only unique map disasters

		if UniqueMapDisastersOnly == true then
			if workspace.Map.Map_Name.Value ~= disaster.SpecificMap then
				map_MatchUnique = false
			end
		end

		-- Add disaster to table
		
		if map_Enabled and map_MatchSpecific and map_MatchUnique then
			table.insert(AvailableMaps, disaster.Name)
		end

	end
	
	task.wait(3)


	for i, v in pairs(AvailableMaps) do
		print(v) -- NOW THIS PRINTS ALL THE DISASTERS
	end

	SelectedDisaster = AvailableMaps[math.random(1, #AvailableMaps)]
	SelectedDisasterName = SelectedDisaster.Name

	return SelectedDisaster, SelectedDisasterName

end

[syyIvi (redixii)]

@SharkRayMaster

Well, I don’t know why it wasn’t working but I fixed it by not using "then return end"

using return early in a function just stops the entire thing, so that might be the reason why the value came out as nothin

but, good job in finding the solution!