Anyone Know the Lua for Random Numbers in ESO????

[zireko]

I have my code working pretty good the only line of code I can't find or figure out is how to get a random number to pop in.

I'm creating just a simple dice roller for my first addon. The math.random() doesn't seem to work so I even messed around with another random code but couldn't seem to plug it in either.

local Roll = 0 
 
function LootDiceRoll()
    math.random(100)
end

[archpoet]

math.random() returns a value that you're not capturing.

[zireko]

Originally Posted by archpoet math.random() returns a value that you're not capturing.

How would I capture it exactly I'm currently using the tutorial code and was trying to capture it in the counter box where the counter used to be.

[archpoet]

Your question is related more so to a programming fundamental in most languages, than to the ESO API. That is: to capture the return value of any function it is typical to assign it to a variable. Lua is no different in this regard.

Something like:

local num = math.random(100)

...would suffice to assign the return value of math.random() to the variable num, however once stored in a variable it then needs to be returned from the LootDiceRoll() function you've made.

return num

... and finally, inserted into the game UI.

Roll = LootDiceRoll()
YourXMLElement:SetText( Roll )

Good Luck!

[zireko]

I think I understand a little better now. But I'm getting an error. I'll go head and link my xml and lua. If you can see it I can't seem to understand where my error is at. I only get the error when I click on the area that the dice roll is supposed to occur.

XML

<GuiXml>
    <Controls>
        <TopLevelControl name="LootDice" mouseEnabled="true" movable="true">
            <Dimensions x="200" y="42" />
            <Anchor point="CENTER" />
                                  
            <OnMouseDown>
                Roll = LootDiceRoll()
                LootDice:SetText(Roll)
            </OnMouseDown>
 
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)Roll" font="ZoFontWindowTitle" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="LootDice: ">
                    <AnchorFill />
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

LUA

local Roll = 0 
 
function LootDiceRoll()
    local num = math.random(100)
    return num
end

and here are pictures of the addon and the error when I get it.

[archpoet]

I think I see the issue you're having.

LootDice is not the type of element that has the :SetText() method.
Labels do however (i.e. $(parent)Roll,) so try LootDiceRoll:SetText(Roll)

Oh yeah, also... move the

            <OnMouseDown>
                Roll = LootDiceRoll()
                LootDiceRoll:SetText(Roll)
            </OnMouseDown>

to a <controls> inside <label> or just do that in the Lua

LootDiceRoll:SetHandler( "OnMouseDown", function()
    local Roll = LootDiceRoll()
    LootDiceRoll:SetText(Roll)
end)

[archpoet]

Furthermore, you might have a namespace conflict.

You have an element named "LootDiceRoll" and a function named the same.

[zireko]

Ok I've been at this a while lol. Trying different things but nothing is seeming to work. I will link the code how I have it now. If you can could you fix the error and then link the entire code back to me exactly how I have the code here with the fix in it? This will help me better understand where exactly I'm messing up.
XML

<GuiXml>
    <Controls>
        <TopLevelControl name="LootDice" mouseEnabled="true" movable="true">
            <Dimensions x="200" y="42" />
            <Anchor point="CENTER" />   
            
            <OnMouseDown>
                Roll = LootDiceRoll()
                LootDiceRoll:SetText(Roll)
            </OnMouseDown>
            
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)Roll" font="ZoFontWindowTitle" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" text="LootDice: " >
            
                    <AnchorFill />
                
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

LUA

function LootDiceRoll()
    local num = math.random(1,100)
    return num
end

[archpoet]

Yep, namespace conflict: confirmed.

Basically I copied and pasted your code as written and just changed the name of the function to RollDice() in both the lua and the xml and it worked no issue.

function RollDice()
    local num = math.random(1,100)
    return num
end

            <OnMouseDown>
                Roll = RollDice()
                LootDiceRoll:SetText(Roll)
            </OnMouseDown>

In my opinion this is not an issue in strict Lua, (nor do other strongly typed interpreted languages suffer from this issue.)
I feel like this arises because the Game compiles the Lua and the XML elements into globally scoped lexical objects. Which in context makes sense why they would apparently conflict on names.

[zireko]

It's ALIVE IT'S ALIVE lol ty so much for the help now time for the banana dance

[archpoet]

Originally Posted by zireko It's ALIVE IT'S ALIVE lol ty so much for the help now time for the banana dance

LOL. My pleasure, glad to help.

[zireko]

Now I got to figure out how to make it where when you move it somewhere it will save to that location. That way people don't have to move it every time the load up the game or reload the ui. Any ideas where I should start on that. Links are welcomed. O and thank you for all the help again.

[archpoet]

Originally Posted by zireko Now I got to figure out how to make it where when you move it somewhere it will save to that location. That way people don't have to move it every time the load up the game or reload the ui. Any ideas where I should start on that. Links are welcomed. O and thank you for all the help again.

First of all you'll need to setup ZO_SavedVars to save and reload the coordinates.. some info about that here:
http://wiki.esoui.com/AddOn_Quick_Questions

Then you'll want to set a default position on the screen, and OnLoad position the addon there.

Next you'l want to:

LootDice:SetHandler( "OnMoveStop", function()
     -- overwrite default position with the updated position of your addon here
end)

Then when you load the SavedVars again next time instead of using the default positioning, it will load and be positioned in the saved location.

So in summary:

1. Setup Defaults
2. Load SavedVars
3. Do Positioning, Processing, etc.
4. ??
5. Profit.

Happy to Help!

[zireko]

K I think I have it but the game is in Maintenance currently. Here is the code I have so far.

LootDice:SetHandler( "OnMoveStop", function()
     -- overwrite default position with the updated position of your addon here
end)
 
local savedVars = ZO_SavedVars:New(savedVariableLootDice, 1, defaults)
 
local defaults =
{
    offsetX = 0,
    offsetY = 0,
}
 
function RollDice()
    local num = math.random(1,100)
    return num
end
 
EVENT_MANAGER:RegisterForEvent("LootDice", EVENT_ADD_ON_LOADED, OnAddOnLoaded)

[zireko]

K I just got into the game and I know something is wrong I flipped the script around and that made my addon work again but it still doesn't save it and I get an error. Here is what the script I have looks like now.

function RollDice()
    local num = math.random(1,100)
    return num
end
 
EVENT_MANAGER:RegisterForEvent("LootDice", EVENT_ADD_ON_LOADED, OnAddOnLoaded)
 
LootDice:SetHandler( "OnMoveStop", function()
     -- overwrite default position with the updated position of your addon here
end)
 
local savedVars = ZO_SavedVars:New(savedVariableLootDice, 1, defaults)
 
local defaults =
{
    offsetX = 0,
    offsetY = 0,
}

[Harven]

If this is your entire addon code, then you are missing OnAddOnLoaded function. You should call ZO_SavedVars:New in that function.

[zireko]

Ok I took and cleaned up the code and have it looking nice. Now I'm getting an error. Here is the code so far if you can see where I'm messing up let me know. Please show code back to me since I'm new it helps me out a lot because I'm still learning.

function RollDice()
    local num = math.random(1,100)
    return num
end
 
EVENT_MANAGER:RegisterForEvent("LootDice", EVENT_ADD_ON_LOADED, OnAddOnLoaded)
 
LootDice:SetHandler( "OnMoveStop", function())
    local function OnAddOnLoaded(eventCode, LootDice)
    if(LootDice == "<<LootDice>>") then
        local savedVars = ZO_SavedVars:New(LootDice_SavedVariables, 1)
    end
end

[zireko]

I've been playing around with it a lot. I think this is the code that is messed up but I'm not sure how to fix it exactly.

LootDice:SetHandler( "OnMoveStop", function())

[Fathis Ules]

Originally Posted by zireko I've been playing around with it a lot. I think this is the code that is messed up but I'm not sure how to fix it exactly. LootDice:SetHandler( "OnMoveStop", function())

It is

LootDice:SetHandler( "OnMoveStop", function() --[[ CODE GOES HERE]] end)

Assuming LootDice is an existing control of course

Also think when you declare something

function RollDice()
    local num = math.random(1,100)
    return num
end

it is called a global function and may conflict with other global function used by other addons or the client itself

Example if you write

function d()
end

you will overwritte the d() global function and many message of the client or addons won't be processed anymore

it is better to default to local unless you really want the function accessible globally like that

local function RollDice()
    local num = math.random(1,100)
    return num
end

And if you really want it accessible globally usually you put a better naming like MyAddonRollDice so you are sure there is near 0 confict with it

[zireko]

When I set it to local function instead of just function it gives and error but when its a global it works for some odd reason. I'm testing and moving code all around but here is the entire code lua, xml, and txt. So you can see exactly what I'm working with.

LUA - I have the save part of the code commented out so I could test the local.

local function RollDice()
    local num = math.random(1,100)
    return num
end
 
 
--[[
LootDice:SetHandler( "OnMoveStop", function()
    local function OnAddOnLoaded(eventCode, LootDice)
    if(LootDice == "<<LootDice>>") then
        local savedVars = ZO_SavedVars:New(LootDice_SavedVariables, 1)
    end
end)
 
EVENT_MANAGER:RegisterForEvent("LootDice", EVENT_ADD_ON_LOADED, OnAddOnLoaded)
]]--

XML

<GuiXml>
    <Controls>
        <TopLevelControl name="LootDice" mouseEnabled="true" movable="true">
            <Dimensions x="150" y="42" />
            <Anchor point="CENTER" />   
            
            <OnMouseDown>
                Roll = RollDice()
                LootDiceRoll:SetText(Roll)
            </OnMouseDown>
            
            <Controls>
                <Backdrop name="$(parent)BG" inherits="ZO_ThinBackdrop" />
                <Label name="$(parent)Roll" font="ZoFontWindowTitle" color="CFDCBD" wrapMode="ELLIPSIS" verticalAlignment="CENTER" horizontalAlignment="CENTER" text="LootDice" >
            
                    <AnchorFill />
                
                </Label>
            </Controls>
        </TopLevelControl>
    </Controls>
</GuiXml>

TXT

## Title: LootDice - By Rylew
## APIVersion: 100003
## Description: A simple Loot Dice for group runs.
## Version: 1.0
## SavedVariables: LootDice_SavedVariables
 
LootDice.lua 
LootDice.xml