How to create a copy of an object without reference?

zork · 02-23-13, 11:47 AM

I have a problem that I cannot solve.

For my UI panel I want a "reset" feature. Thus I have a reset button that can be called to load the database defaults.

The problem I'm having is that as soon as I do this the default data becomes a reference. Thus on a second click on reset it will not work anymore. Because of the reference issue.

Here is my code:

Lua Code:

--object container
  local db = CreateFrame("Frame")
  ns.db = db
  db.default = {}
 
  ---------------------------------------------
  --DEFAULTS
  ---------------------------------------------
 
  --default orb setup
  db.default.orb = {
    --health
    ["HEALTH"] = {
      --filling
      filling = {
        texture     = "Interface\\AddOns\\oUF_Diablo\\media\\orb_filling15",
        color       = { r = 1, g = 0, b = 0, },
        colorAuto   = false, --automatic coloring based on class/powertype
      },
    },--health end
    --power
    ["POWER"] = {
      --filling
      filling = {
        texture     = "Interface\\AddOns\\oUF_Diablo\\media\\orb_filling15",
        color       = { r = 0, g = 0, b = 1, },
        colorAuto   = false, --automatic coloring based on class/powertype
      },
    },--power end
  } --default end
 
  db.getDefaultOrbHealth = function() return db.default.orb["HEALTH"] end
  db.getDefaultOrbPower = function() return db.default.orb["POWER"] end
  db.getDefaultOrb = function() return db.default.orb end
 
  ---------------------------------------------
  --CHARACTER DATA
  ---------------------------------------------
 
  --load character data defaults
  db.loadCharacterDataDefaults = function(type)
    if type then
      if type == "HEALTH" then
        OUF_DIABLO_DB_CHAR[type] = db.getDefaultOrbHealth()
        print(addon..": health orb reseted to default")
      elseif type == "POWER" then
        OUF_DIABLO_DB_CHAR[type] = db.getDefaultOrbPower()
        print(addon..": power orb reseted to default")
      end
    else
      OUF_DIABLO_DB_CHAR = db.getDefaultOrb()
      print(addon..": character data defaults loaded")
    end
    db.char = OUF_DIABLO_DB_CHAR
    --update the orb view
    ns.panel.updateOrbView()
  end

OUF_DIABLO_DB_CHAR is my saved variable per character.

Basically when I click the reset button I call

Code:

  db.loadCharacterDataDefaults("HEALTH")

I cannot copy the table data via ipairs. Assoc tables cannot be looped. The following will return 0:

Code:

print(# db.default.orb["HEALTH"])

Any idea?

Tuller uses sth like

Lua Code:

local function copyDefaults(tbl, defaults)
    for k, v in pairs(defaults) do
        if type(v) == 'table' then
            tbl[k] = copyDefaults(tbl[k] or {}, v)
        elseif tbl[k] == nil then
            tbl[k] = v
        end
    end
    return tbl
end

Maybe I'm missing sth...

Oh...his GetDefault funnction is looking like:

Lua Code:

function tullaRange:GetDefaults()
    return {
        normal = {1, 1, 1},
        oor = {1, 0.3, 0.1},
        oom = {0.1, 0.3, 1}
    }
end

Hmmm...so this could be an idea. I do not create a default table variable in first place that could get referenced later on.

I will try that.

Perfect that did it!

What I'm now doing is:

Lua Code:

--default orb setup
  function db:GetOrbDefaults()
    return {
      --health
      ["HEALTH"] = {
        --filling
        filling = {
          texture     = "Interface\\AddOns\\oUF_Diablo\\media\\orb_filling15",
          color       = { r = 1, g = 0, b = 0, },
          colorAuto   = false, --automatic coloring based on class/powertype
        },
      },--health end
      --power
      ["POWER"] = {
        --filling
        filling = {
          texture     = "Interface\\AddOns\\oUF_Diablo\\media\\orb_filling15",
          color       = { r = 0, g = 0, b = 1, },
          colorAuto   = false, --automatic coloring based on class/powertype
        },
      },--power end
    } --default end
  end

pelf · 02-23-13, 12:32 PM

I feel like I remember seeing "default settings" tables in code, before. I assume they're usually used to initialize, but they could just as easily be used to reset. Nice.

ravagernl · 02-23-13, 12:43 PM

One thing you could also do if your options are not very complicated is make use of metatables:

http://www.lua.org/pil/13.4.3.html

Then when resetting, just loop over the options database table and set all keys to nil.

Haleth · 02-23-13, 04:08 PM

I use the following function to copy a table by value:

Code:

local function copyTable(source, target)
	for key, value in pairs(source) do
		if type(value) == "table" then
			target[key] = {}
			copyTable(value, target[key])
		else
			target[key] = value
		end
	end
end

pelf · 02-23-13, 04:23 PM

If that works, then what is this talking about?

I cannot copy the table data via ipairs. Assoc tables cannot be looped.

Haleth · 02-23-13, 05:19 PM

I don't see the problem there. Why not use:

Code:

 for key, value in pairs(db.default.orb["HEALTH"]) do
	...
 end

Any table with only keys in it rather than indexes will have #table return zero. That doesn't mean you can't iterate it.

zork · 02-23-13, 06:17 PM

Ah damn. I think I tried to iterate over the table length. I think I mixed sth up when using # table.

The loop Haleth posted actually does work.

Thanks.

*edit*

Thanks a bunch it is working now!

semlar · 02-23-13, 06:23 PM

You can use either pairs or next to loop over a dictionary table.

Lua only returns a length for numerically indexed tables (or at least the highest consecutive index).

pelf · 02-24-13, 01:19 AM

I love it when I go down another path almost to completion to avoid a problem I didn't have when I'm coding. Always fun

.

Rilgamon · 02-24-13, 02:53 AM

Just incase you missed it ... CopyTable is a global function since quite some time now

Lua Code:

function CopyTable(settings)
    local copy = {};
    for k, v in pairs(settings) do
        if ( type(v) == "table" ) then
            copy[k] = CopyTable(v);
        else
            copy[k] = v;
        end
    end
    return copy;
end

http://wowprogramming.com/utils/xmlb...L/UIParent.lua

Haleth · 02-24-13, 03:44 AM

Oh, interesting. There's quite a few useful function in UIParent.lua. However I noticed this...

Code:

    if ( tempName == "LeftButton" ) then
        tempName = "BUTTON1";
    elseif ( tempName == "RightButton" ) then
        tempName = "BUTTON2";
    elseif ( tempName == "MiddleButton" ) then
        tempName = "BUTTON3";
    elseif ( tempName == "Button4" ) then
        tempName = "BUTTON4";
    elseif ( tempName == "Button5" ) then
        tempName = "BUTTON5";
    elseif ( tempName == "Button6" ) then
        tempName = "BUTTON6";
    elseif ( tempName == "Button7" ) then
        tempName = "BUTTON7";
    elseif ( tempName == "Button8" ) then
        tempName = "BUTTON8";
    elseif ( tempName == "Button9" ) then
        tempName = "BUTTON9";
    elseif ( tempName == "Button10" ) then
        tempName = "BUTTON10";
    elseif ( tempName == "Button11" ) then
        tempName = "BUTTON11";
    elseif ( tempName == "Button12" ) then
        tempName = "BUTTON12";
    elseif ( tempName == "Button13" ) then
        tempName = "BUTTON13";
    elseif ( tempName == "Button14" ) then
        tempName = "BUTTON14";
    elseif ( tempName == "Button15" ) then
        tempName = "BUTTON15";
    elseif ( tempName == "Button16" ) then
        tempName = "BUTTON16";
    elseif ( tempName == "Button17" ) then
        tempName = "BUTTON17";
    elseif ( tempName == "Button18" ) then
        tempName = "BUTTON18";
    elseif ( tempName == "Button19" ) then
        tempName = "BUTTON19";
    elseif ( tempName == "Button20" ) then
        tempName = "BUTTON20";
    elseif ( tempName == "Button21" ) then
        tempName = "BUTTON21";
    elseif ( tempName == "Button22" ) then
        tempName = "BUTTON22";
    elseif ( tempName == "Button23" ) then
        tempName = "BUTTON23";
    elseif ( tempName == "Button24" ) then
        tempName = "BUTTON24";
    elseif ( tempName == "Button25" ) then
        tempName = "BUTTON25";
    elseif ( tempName == "Button26" ) then
        tempName = "BUTTON26";
    elseif ( tempName == "Button27" ) then
        tempName = "BUTTON27";
    elseif ( tempName == "Button28" ) then
        tempName = "BUTTON28";
    elseif ( tempName == "Button29" ) then
        tempName = "BUTTON29";
    elseif ( tempName == "Button30" ) then
        tempName = "BUTTON30";
    elseif ( tempName == "Button31" ) then
        tempName = "BUTTON31";
    end

pelf · 02-24-13, 08:44 AM

Well, if Button4 ever changes to RightRightButton, we'll be ready.

Thread Tools
Show Printable Version Email this Page
Display Modes
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode