Values in a table always strings?

unlimit · 02-18-12, 01:17 PM

Originally Posted by unlimit

I created a table (??? think that's what it's called) named TRUESYS.MD, that will always have my metadata available.

I then have another function that runs to check if TRYESYS.MD[1] is greater than the current version of my package, and if so, empties the package then using RDXDB.SetPackageMetadata I reassign all of my metadata.

This doesn't seem to work for me, though. Do you see anything blatantly wrong, Sigg? I end up with no metadata at all after emptying the package, even after supposedly reassigning all of my metadata.

lua Code:

TRUESYS.PKG = {
    "true_core",
    "true_unitframes",
    "true_multiframes"
};
 
TRUESYS.MD = {
    "1.1",
    "Unlimit",
    "Demon Soul US",
    "[email protected]",
    "manversuspixel.com",
    "(True) Adaptive User Interface",
    "infoIsShare",
    "infoIsImmutable",
    "infoIsIndelible"
};
 
RDXEvents:Bind("INIT_DATABASE_LOADED", nil, function()
    if not RDXDB.GetPackage(TRUESYS.PKG[1]) then return nil; end
    if not RDXDB.GetPackageMetadata(TRUESYS.PKG[1], "infoVersion") then return nil; end
 
    if RDXDB.GetPackageMetadata(TRUESYS.PKG[1], "infoVersion") < TRUESYS.MD[1] then
        RDX.print("Updating (True) Adaptive User Interface to the latest version!");
        for i=1,3 do
            RDXDB._EmptyPackage(TRUESYS.PKG[i])
            for g=1,9 do
                RDXDB.SetPackageMetadata(TRUESYS.PKG[i], TRUESYS.MD[g], true)
            end
        end
    end
end);
 
RDXEvents:Bind("INIT_DATABASE_LOADED", nil, function()
    -- Define a local to package name and set package metadata
    local core = RDXDB.GetOrCreatePackage(TRUESYS.PKG[1]);
    local unitframes = RDXDB.GetOrCreatePackage(TRUESYS.PKG[2]);
    local multiframes = RDXDB.GetOrCreatePackage(TRUESYS.PKG[3]);
    for i=1,3 do
        for g=1,9 do
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], TRUESYS.MD[g], true)
        end
    end
 
-- Do stuff
 
end);

Originally Posted by sigg

It is because "1.1" is a string.

I suggest that you use version 1, version 2 etc ....

By default, elements in your table are string.

Use the function:
local numb = tonumber(RDXDB.GetPackageMetadata(TRUESYS.PKG[1], "infoVersion"))

Cheers

The problem is, all of those objects in my table are strings, I guess? o.o Which isn't applicable? I'm not sure really what sigg means by using tonumber() as a function to correct the mistake, if someone would be so kind to explain. (I'd ask in the OpenRDX forums, but sigg is only usually on early in the morning. <.<)

SDPhantom · 02-18-12, 01:22 PM

tonumber() takes a string and tries to convert it into a number type value. If it fails in doing this, it returns nil.

For example:
tonumber("1.1") returns the number 1.1
tonumber("unlimit") returns nil

A couple further notes, when you try to create a table with a name of TRUESYS.MD, it actually tries to index a table named TRUESYS and place the table in the field MD. This is because Lua handles "." as an "index of" operator. If the TRUESYS table doesn't exist, it'll throw an error.

The second note, it's perfectly ok to compare strings with <, > , and equivalent operators. This causes a comparison of each byte in the string rather than trying to convert them into numbers. This only works if you use both strings or both numbers as mixing data types in these comparisons would throw an error..

For example:
"1.0"<"1.1" returns true
"1.0"<1.1 errors

unlimit · 02-18-12, 01:28 PM

Originally Posted by SDPhantom

tonumber() takes a string and tries to convert it into a number type value. If it fails in doing this, it returns nil.

For example:
tonumber("1.1") returns the number 1.1
tonumber("unlimit") returns nil

So what he's saying is, I should use tonumber() for anything involving numbers when trying to place it into the metadata of my package?

unlimit · 02-18-12, 02:32 PM

Ah. No, the thing that was blatantly wrong was my use of RDXDB.SetPackageMetadata().

lua Code:

TRUESYS.PKG = {
    "true_core",
    "true_unitframes",
    "true_multiframes"
};
 
    TRUESYS.MD = {
    "1.1",                                              -- infoVersion
    "Unlimit",                                          -- infoAuthor
    "Demon Soul US",                                    -- infoRealm
    "[email protected]",                                  -- infoAuthorEmail
    "manversuspixel.com",                               -- infoAuthorWebsite
    "(True) Adaptive User Interface",                   -- infoComment
    "infoIsShare",
    "infoIsImmutable",
    "infoIsIndelible"
};
 
RDXEvents:Bind("INIT_DATABASE_LOADED", nil, function()
    local version = RDXDB.GetPackageMetadata(TRUESYS.PKG[1], "infoVersion")
    local update
 
    if not RDXDB.GetPackage(TRUESYS.PKG[1]) then return nil; end
    if not version or version < TRUESYS.MD[1] then update = true; end
 
    if update == true then
        RDX.print("Updating (True) Adaptive User Interface to the latest version!");
        for i=1,3 do
            RDXDB._EmptyPackage(TRUESYS.PKG[i], true)
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoVersion", TRUESYS.MD[1]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthor", TRUESYS.MD[2]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorRealm", TRUESYS.MD[3]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorEmail", TRUESYS.MD[4]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorWebsite", TRUESYS.MD[5]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoComment", TRUESYS.MD[6]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsShare", TRUESYS.MD[7]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsImmutable", TRUESYS.MD[8]);
            RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsIndelible", TRUESYS.MD[9]);
        end
    else
        RDX.print("(True) Adaptive user Interface is up to date!");
    end
end);
 
RDXEvents:Bind("INIT_DATABASE_LOADED", nil, function()
    -- Define a local to package name and set package metadata
    local core = RDXDB.GetOrCreatePackage(TRUESYS.PKG[1]);
    local unitframes = RDXDB.GetOrCreatePackage(TRUESYS.PKG[2]);
    local multiframes = RDXDB.GetOrCreatePackage(TRUESYS.PKG[3]);
    for i=1,3 do
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoVersion", TRUESYS.MD[1]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthor", TRUESYS.MD[2]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorRealm", TRUESYS.MD[3]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorEmail", TRUESYS.MD[4]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoAuthorWebsite", TRUESYS.MD[5]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoComment", TRUESYS.MD[6]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsShare", TRUESYS.MD[7]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsImmutable", TRUESYS.MD[8]);
        RDXDB.SetPackageMetadata(TRUESYS.PKG[i], "infoIsIndelible", TRUESYS.MD[9]);
    end
 
    -- do stuff
 
end);