0

I am trying to create a maze generation script using the module listed below. I'm having this strange problem were the grid variable has a value, but when return grid is called, it returns and empty table? Here is the code to the module script.

local ServerStorage = game:GetService("ServerStorage")
local cellTemplate = ServerStorage:WaitForChild("Cell")

local Maze = {}
local cellObject = {}

local grid = {}

Maze.Size = 25

Maze.CellWidth = 9
Maze.CellLength = 9

Maze.Offset = Vector3.new(0,0,0)

function Maze.new(x,z) -- Create a new cell
    local newCell = {}
    
    setmetatable(newCell,{ -- Allows us to create functions for a cell easier
        __index = cellObject
    })
    
    newCell.X = math.clamp(x,1,Maze.Size)
    newCell.Z = math.clamp(z,1,Maze.Size)
    
    newCell.Visited = false
    
    newCell.Model = cellTemplate:Clone()
    newCell.Base = newCell.Model:WaitForChild("Base")
    
    newCell.Model.Name = newCell.Model.Name.. "_".. newCell.X.. "-".. newCell.Z
    
    newCell.Walls = {
        ["Forward"] = newCell.Model:WaitForChild("Forward");
        ["Backward"] = newCell.Model:WaitForChild("Backward");
        ["Left"] = newCell.Model:WaitForChild("Left");
        ["Right"] = newCell.Model:WaitForChild("Right");
    }
    
    if not grid[x] then grid[x] = {} end -- We might not have anything on that x axis yet; inserts it into the table
    
    grid[x][z] = newCell
    
    print(grid)
    
    return newCell
end

function Maze.RenderAll() -- Render every cell; exists for more readibility
    for _,cellRow in pairs(grid) do -- Loop through every cell row
        for _,cell in pairs(cellRow) do -- Loop through every cell in the row
            cell:Render() -- Render the cell
        end
    end
end

function Maze.Grid() -- Allows other scripts to get the grid but not modify it
    return grid
end

function Maze.FilterUnvisited(cells) -- Takes in a table and returns one with only the cells in the table that are unvisited
    local unvisited = {}
    
    for _,cell in pairs(cells) do -- Loop through every cell in the table passed
        if not cell.Visitied then -- The cell hasn't been visited
            table.insert(unvisited,cell)
        end
    end
    
    return unvisited
end

function Maze.GetCell(x,z)
    local cell
    
    if grid[x] and grid[x][z] then
        cell = grid[x][z]
    else
        cell = nil
    end
    
    return cell
end

function cellObject:Render() -- Render the cell
    self.Model:SetPrimaryPartCFrame(CFrame.new(Vector3.new(self.X * Maze.CellLength,0,self.Z * Maze.CellWidth) + Maze.Offset)) -- Move the cell to the correct position
    
    if self.Visited then -- We have gone through the cell
        self.Model.PrimaryPart.Color3 = Color3.new(0,1,0) -- Show that the cell has been visited; used for debugging
    end
    
    self.Model.Parent = workspace
end

function cellObject:Neighbours() -- Returns the cell's neigbours
    local neighbours = {}
    
    -- Order: Right Left Up Down
    
    if grid[self.X + 1] and grid[self.X + 1][self.Z] then -- A cell with +1 X exists
        table.insert(neighbours,grid[self.X + 1][self.Z])
    end
    
    if grid[self.X - 1] and grid[self.X - 1][self.Z] then -- A cell with -1 X exists
        table.insert(neighbours,grid[self.X - 1][self.Z])
    end
    
    if grid[self.X][self.Z + 1] then -- A cell with +1 Z exists
        table.insert(neighbours,grid[self.X][self.Z + 1])
    end

    if grid[self.X][self.Z - 1] then -- A cell with -1 Z exists
        table.insert(neighbours,grid[self.X][self.Z - 1])
    end
    
    return neighbours
end

function cellObject:RandomNeighbour()
    local neighbours = self:Neighbours() -- Gets the neigbours of the current cell
    
    if #neighbours > 0 then
        return neighbours[math.random(1,#neighbours)] -- Returns a random neigbour
    else
        return nil
    end
end

function cellObject:WallArray() -- Returns an array of the walls instead of a table
    local wallArray = {}
    
    wallArray[1] = self.Walls.Forward
    wallArray[2] = self.Walls.Right
    wallArray[3] = self.Walls.Left
    wallArray[4] = self.Walls.Backward
    
    return wallArray
end

function cellObject:Join(cell) -- Joins 2 cells together
    local wallPosition = self.Base.Position:Lerp(cell.Base.Position,.5) -- This will return the position of the wall (excluding the Y)
    
    local cell1Array = self:WallArray()
    local cell2Array = cell:WallArray()
    
    for wallIndex,wall in pairs(cell1Array) do
        if wall.Position.X == wallPosition.X and wall.Position.Z == wallPosition.Z then -- Its the right wall
            wall.Transparency = 1
            wall.CanCollide = false
            
            cell2Array[4 - (wallIndex - 1)].Transparency = 1
            cell2Array[4 - (wallIndex - 1)].CanCollide = false
            
            break -- We don't need to loop anymore, since we've already removed the walls
        end
    end
end

function cellObject:Unjoin(cell) -- Unjoins 2 cells
    local wallPosition = self.Base.Position:Lerp(cell.Base.Position,.5) -- This will return the position of the wall (excluding the Y)
    
    local cell1Array = self:WallArray()
    local cell2Array = cell:WallArray()
    
    for wallIndex,wall in pairs(cell1Array) do
        if wall.Position.X == wallPosition.X and wall.Position.Z == wallPosition.Z then -- Its the right wall
            wall.Transparency = 0
            wall.CanCollide = true
            
            cell2Array[4 - (wallIndex - 1)].Transparency = 0
            cell2Array[4 - (wallIndex - 1)].CanCollide = true
            
            break -- We don't need to loop anymore, since we've already added the walls
        end
    end
end

return Maze

Below is the piece of code I'm having diffuculty with.

function Maze.Grid() -- Allows other scripts to get the grid but not modify it
    return grid
end

I have tested printing the grid variable and it's not {} but when I call the function in the script below, I always get {}! I have no idea why this is. Is this a bug in roblox or am I being stupid?

local ServerScriptService = game:GetService("ServerScriptService")
local Maze = require(ServerScriptService:WaitForChild("Maze"))

local latestX = 0
local latestZ = 0

function CreatePath(cell)
    local nextCell = cell:RandomNextCell(false)
    
    if nextCell then -- We have a cell next to the current one
        print("Joining cells:",cell,nextCell)
        
        cell:Join(nextCell)
        cell.Visited = true
        
        cell:Render()
        
        wait(.01)
        CreatePath(nextCell)
    else -- It must be the end of the maze
        print("Path generated!")
        print("Maze end: ".. cell.Model.Name)
        
        cell.Base.Color = Color3.new(1,0,0)
    end
end

function GenerateMaze()
    local cell = Maze.new(latestX + 1,latestZ + 1) -- Create a new cell
    
    cell:Render() -- Render it
    
    print("Created cell")
    
    latestZ += 1
    
    if latestZ > Maze.Size then -- It has exceeded the max size, move on to a new row
        latestX += 1
        latestZ = 0
    end
    
    if #Maze.Grid() < Maze.Size ^ 2 then
        wait(.01)
        GenerateMaze()
    else
        print("Grid completed, generating path...")
        
        CreatePath(Maze.GetCell(1,1))
    end
end

GenerateMaze()
0

1 Answer 1

0

You are returning a cached value, or well i don't find out other reason.

You may do (and it's not editable):

function Maze.Grid(i, l)
    local passed = 1
    local response = {}
    for index, v in pairs(grid) do
        response[index] = i and passed >= i and l and passed <= l and v or not (i or l) and v
        -- basically get more than `i` if it, and minor than `l` if it passed
        i = i + 1
    end
    return response
end

It should work, I don't tested it sorry.

3
  • I hope this solve the problem, great day.
    – Corotyest
    Feb 22, 2022 at 0:25
  • 1
    Sorry for the late response, but I have stopped working on the project and forgot about this this post. Still, thanks for the response and I will keep this in mind for any future project I make! Apr 2, 2022 at 17:08
  • 1
    P.S. Have a great day as well! Apr 2, 2022 at 17:09

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.