Block too big?

clowcadia · 05-30-10, 10:52 PM

i get an error, every time that i log in on a new day
do to one function that i will highlight, that created too much tables and gives me an erros of block too big
any ideas why

Code:

-- Author      : Andrew
-- Create Date : 5/28/2010 8:26:28 PM
local addonLoadedCount = 0;
local sessionCount = 0;
local currentDate = date("%m/%d/%y");
local currentTime = date("%H:%M:%S");
local sessionValue; 

local function startSessionInfo()
sessionDate = currentDate

sessionInfo = { [sessionDate] = { sessionCount } }

end

local function newDaySessionInfo()
sessionDate = currentDate

tinsert( sessionInfo, { [sessionDate] = { sessionCount } } )

end

local function onEvent(self,event,...)
----------------------------------------------
 if ( event == "ADDON_LOADED" ) then
addonLoadedCount = addonLoadedCount + 1;
sessionCount=0;
currentDate = date("%m/%d/%y");

  if ( addonLoadedCount == 3 ) then
  print("GVLT Loaded")
   if ( sessionInfo == nil ) then
   startSessionInfo();
   print("New Log File Begun")
   else
    for key,value in pairs(sessionInfo) do
     if ( key == currentDate ) then
     print("Current Day Log Loaded")
     else
     newDaySessionInfo()
     print("New Day Log Begun")
     end
    end
   end   
  end
   
 end
   
 ---------------------------------------------
 if ( event == "GUILDBANKFRAME_OPENED" ) then 
  sessionCount = sessionCount + 1;
  print(sessionCount) 
  
 end
 
 -----------------------------------------------

end

local evFrame = CreateFrame("Frame","evFrame",UIParent);
evFrame:SetScript( "OnEvent", onEvent );
evFrame:RegisterEvent( "ADDON_LOADED" );
evFrame:RegisterEvent( "GUILDBANKFRAME_OPENED" );
evFrame:RegisterEvent( "GUILDBANKFRAME_CLOSED" );

Xrystal · 05-30-10, 11:01 PM

Your problem is that you're doing too many { } entries. Each pair of those braces creates a table and it looks like it is creating tables within tables within tables.

Your best bet is to think up an example of what you want it to look like in the save table and then fit the data into that.

EG.

Code:

SessionData = {
    [Date1] = { ValuesForDate1 },
    [Date2] = { ValuesForDate2 },
    [Date3] = { ValuesForDate3 },
    [Date4] = { ValuesForDate4 },
    [Date5] = { ValuesForDate5 },
    [Date6] = { ValuesForDate6 },
}

This is a 2 deep table .. but looking at what you are wanting to store in there try this and see if this stores the info you want without the errors.

Code:

local function startSessionInfo()
   sessionDate = currentDate
   sessionInfo = { [sessionDate] = sessionCount }
end

Code:

local function newDaySessionInfo()
   sessionDate = currentDate
   tinsert( sessionInfo, { [sessionDate] = sessionCount } )
end

clowcadia · 05-30-10, 11:13 PM

made the changes you have advised. and yet i am running in the same problem
but now i have been reloading, so it should mean it should compare the dates, and yet still says "creating a new date log"(dont remember exact saying in it) and it multiplies the tables

clowcadia · 05-30-10, 11:26 PM

ok it works properly now thanks crystal

clowcadia · 05-30-10, 11:29 PM

nvm still the same probleme

clowcadia · 05-30-10, 11:30 PM

double checking code, is a must .. feel dumb

clowcadia · 05-30-10, 11:36 PM

ok i am not that dumb after all

i tested it with fresh file, and it madea new one as it should, then i changed the date, it made a new one no problme, but once i left it alone to see if it recognises it still begun to create thousands more

the code is identical to yours

Code:

local function startSessionInfo()
sessionDate = currentDate

sessionInfo = { [sessionDate] = sessionCount }

end

local function newDaySessionInfo()
sessionDate = currentDate

tinsert( sessionInfo, { [sessionDate] = sessionCount } )

end

Starinnia · 05-30-10, 11:49 PM

Your call to table.insert means that your pairs loop is iterating over integer keys not the date keys. So you are comparing the current date to 1, 2, 3, etc.

You probably want something like:

Code:

local function newDaySessionInfo()
    sessionDate = currentDate

    sessionInfo[sessionDate] = sessionCount

end

Given the current code I imagine your table looks something like:

Code:

sessionInfo = {
    ["05/31/2010"] = 5,
    [1] = {
        ["06/01/2010"] = 2,
    },
    [2] = {
        ["06/02/2010"] = 2,
    },
}

clowcadia · 05-30-10, 11:52 PM

hmm intresting, i worked with the table[value] before but when i came to pairs function, it was said that pairs works to a limit. where it does not work with subtables such as table[value]

Torhal · 05-31-10, 12:34 AM

The pairs() function works on a data-pair of key/value.

A value can itself be a table.

Example:

Code:

for name, data in pairs(information) do
        for month, week in pairs(data) do
                -- Your code here.
        end
end

This iterates over the key/value pairs in the table named "information", where the keys are strings which represent names and the values are tables - it then iterates over the key/value pairs of the value tables, where the key represents a month and its value represents a week.

Note that this is in no way a real design - I simply chose variable names based on what you're currently dealing with; time.

clowcadia · 05-31-10, 01:05 AM

hmm ok very informative Torhal.
thank you very much
now i got another problem. i got a second subtable made for session count. but that resets every time i reload
and the number doesnt appear in saved variables after 0

source

Code:

-- Author      : Andrew
-- Create Date : 5/28/2010 8:26:28 PM
local addonLoadedCount = 0;
local sessionCount = 0;
local currentDate = date("%m/%d/%y");
local currentTime = date("%H:%M:%S"); 

local function session_Info()
sessionDate = currentDate
sessionCounted = sessionCount
sessionInfo = sessionInfo or {}
sessionInfo[sessionDate] = sessionInfo[sessionDate] or {}
sessionInfo[sessionDate][sessionCounted] = sessionInfo[sessionDate][sessionCount] or {}

sessionInfo[sessionDate][sessionCounted] = {"number"}
print(sessionDate,sessionCounted)

end

local function onEvent(self,event,...)
----------------------------------------------
 if ( event == "ADDON_LOADED" ) then
addonLoadedCount = addonLoadedCount + 1;
currentDate = date("%m/%d/%y");

  if ( addonLoadedCount == 3 ) then
  print("GVLT Loaded")
   if ( sessionInfo == nil ) then
   session_Info();
   print("New Log File Begun")
   else
   session_Info();
   print("Log Begun/Continued")
    if sessionDate == currentDate then
    sessionCount = sessionCount + sessionCounted;
    print(sessionCount)
    else
    sessionCount = 0;
    end
   end     
  end
   
 end
   
 ---------------------------------------------
 if ( event == "GUILDBANKFRAME_OPENED" ) then 
  sessionCount = sessionCount + 1;
  session_Info();
  print("#", sessionCount, "of the day");
  
 end
 
 -----------------------------------------------

end

local evFrame = CreateFrame("Frame","evFrame",UIParent);
evFrame:SetScript( "OnEvent", onEvent );
evFrame:RegisterEvent( "ADDON_LOADED" );
evFrame:RegisterEvent( "GUILDBANKFRAME_OPENED" );
evFrame:RegisterEvent( "GUILDBANKFRAME_CLOSED" );

output

sessionInfo = {
["05/31/10"] = {
{
"number", -- [1]
}, -- [1]
{
"number", -- [1]
}, -- [2]
{
"number", -- [1]
}, -- [3]
[0] = {
"number", -- [1]
},
},