WoWInterface - I admit defeat

WoWInterface (https://www.wowinterface.com/forums/index.php)

- General Authoring Discussion (https://www.wowinterface.com/forums/forumdisplay.php?f=20)

- - I admit defeat (https://www.wowinterface.com/forums/showthread.php?t=48623)

I haven't a clue why all of the sudden it no longer makes sense to me, but after more than a year away - I can't seem to write lua anymore. :(

It refuses to register in my thick skull, leaving me feeling rather inadequate right now (Ok, I have the flu together with abdo flu, so I am feeling quite down.) I've been trying to wrap my head around how others do the scope, event handling, module setup, but it seems to elude me.
Say, p3lim, he handles it well I think, but I just don't understand : https://github.com/p3lim-wow/Inomena...er/Inomena.lua (I am certain if I understand it, I am saved)

Basically; here's what I've been trying to achieve :
- Have my basic UI setup in one add-on, which I had working - but then I made the feverish decision to redo the entire thing.
-- I was going to have it modular again, as it was. -- this is no longer working of course.
-- I wanted to have my core.lua contain the event handlers and utility functions

core.lua is setup and works, however, I seem to have lost the "module" effect, and I can't seem to figure it out. I looked at various other add ons written by people whom I greatly admire - but I just am not seeing the light. Not when it comes to the modular nature, not when it comes to event handling.

It's set up as follows :

Code:

Eavu.toc

cvars.lua -- just cvar edits and setting the UI scale, is actually self sufficient

Eavu.lua -- contains the config, createPanel fund and event handler (which should have been usable by the modules)

modules/autorepair.lua -- works when I include the frame:SetScript(event, method), when I try to get it to fall back on the event handler of eavu.lua - it won't work. I'm also calling a function Eavu.print() which is defined in eavu.lua - but alas. 

modules/panels.lua -- works again

modules/broker.lua -- uses LibCargoShip, doesn't use anything of eavu, positioning is based on panels.lua, so that it can read fine

modules/.... -- haven't gotten to the others

for those interested: https://github.com/moonwitch/Eavu

You need to register events before you can trigger them:

Lua Code:

for k, v in pairs(Eavu) do
 frame:RegisterEvent(k); -- Register all events for which handlers have been defined
end
 
frame:SetScript("OnEvent", function(self, event, ...)
 Eavu[event](self, ...); -- call one of the functions above
end)

I think coding when you're sick is a bad idea, i tried do "programming" once i was drunk, thex next day was a massive omg wtf rollback. :)

However that table should contain the events you want to register:

Lua Code:

local Events = {
 [1] = "ADDON_LOADED",
 [2] = "COMBAT_LOG_EVENT_UNFILTERED",
 [3] = "ETCETC"
}
 
for k, v in pairs(Events) do
 frame:RegisterEvent(v); -- Register all events for which handlers have been defined
end

Also if you use the in pairs loop then you need to register it's v part not the k. (But it depends how your table looks like.)

the only good things about this coding when sick area. I keep losing my train of thought.
b. I am not afraid to delete ENTIRE files (yep, I am missing half an add-on)
and especially c. I actually use github, and "gitted" almost everything.

I keep wondering though :

Lua Code:

local __, Inomena = ...

I think this is : Set _ to the addonname (uhm ok) and Inomena gets made into a table.

Frame handler created just to deal with all the OnEvents

Lua Code:

local handler = CreateFrame('Frame') -- creating frame, nothing special
handler:SetScript('OnEvent', function(self, event, ...) -- on an event, the event name is the function name
self[event](...) -- execute the function which has the same name as the event which fires the function
end)

But then this mastodont .. just wild

Lua Code:

local metatable = { -- create metatable
__call = function(funcs, self, ...) -- no clue on __call 
for __, func in pairs (funcs) do
func(self, ...)
end
end
}

There has got to be an easier way that I *can* understand :P
But I am watching too many events to just do each one separately, all I wanted was to find that one "event handler" that I can drop into my eavu.lua (is loaded first btw) and then use Eavu.EVENT() in the other modules.

Quote:

Originally Posted by Resike (Post 287813)

However that table should contain the events you want to register:

Lua Code:

local Events = {
 [1] = "ADDON_LOADED",
 [2] = "COMBAT_LOG_EVENT_UNFILTERED",
 [3] = "ETCETC"
}
 
for k, v in pairs(Events) do
 frame:RegisterEvent(v); -- Register all events for which handlers have been defined
end

Also if you use the in pairs loop then you need to register it's v part not the k.

Thanks :)

I actually declare my functions like so (Only the Eavu.Print doesn't work, simply because - well the modular aspect got lost somehow. You see how I have to redo the event handler again, even though this is a module?

lua Code:

local __, Eavu = ...
local frame = CreateFrame("Frame")-- sets frame to be used for the hooks, and Eavu to a usable table
 
 
--------------------------------------------
-- Util funcs
--------------------------------------------
function money_to_string(value)
  if value == 0 then return nil end
 
 
  local gold = math.floor(value / 10000)
  local silver = mod(math.floor(value / 100), 100)
  local copper = mod(value, 100)
 
 
  return string.format("|cffffd700%i|r.|cffc7c7cf%02i|r.|cffeda55f%02i|r", gold, silver, copper)
end
 
 
function earned(value)
  return value > 0 and "|cff00ff00Profit|r" or value < 0 and "|cffff0000Loss|r" or nil
end
 
 
function Eavu.MERCHANT_SHOW()
-- Func to AutoRepair from guild if you can repair from guild and haven't exceeded the total amount
  if (CanMerchantRepair()) then -- can we repair from this dude?
    RepairAllItems(CanGuildBankRepair() and GetGuildBankWithdrawMoney() >= GetRepairAllCost())
    Eavu.print('Repaired for:'..GetRepairAllCost())
  end
-- Func to vendor all greys and print out for how much
 
 
  self:UnregisterEvent("MERCHANT_SHOW") -- Unregistering the correct event as well
end
 
 
frame:SetScript("OnEvent", function(self, event, ...)
 Eavu[event](self, ...); -- call one of the functions above
end)
 
 
for k, v in pairs(Eavu) do
 frame:RegisterEvent(k); -- Register all events for which handlers have been defined
end

What p3lim does is create his own implementation of RegisterEvent, where he can pass a handler function as a second argument to it. The code then checks whether a handler is already present for the passed event. If not, it creates a table entry with the event as the key and the handler as the value (the value is of type function). If there is already a handler, and the handler is a function, it changes the table entry so that the key is the event and the value is a new table containing the old and the new handler (so it is an array with 2 values, both are of type function) and sets his metatable to it (more on that below). If the handler is not of type function (it will then be a type table) it just injects the new handler into that table.

The __call metamethod allows a table to be called like a function and defines what the return of that should be. The first argument it gets is the table itself, followed by all the arguments passed in table call. In p3lim's case it tells it to execute all the handlers registered to the given event.

Quote:

Originally Posted by MoonWitch (Post 287809)

(Ok, I have the flu together with abdo flu, so I am feeling quite down.)

Go to bed. For the weekend. THEN look at code. Bleargh!

Quote:

Originally Posted by Torhal (Post 287824)

Go to bed. For the weekend. THEN look at code. Bleargh!

I should, but I feel guilty cuz mom's cleaning ;)

Also: Be nice to Kaelten - I've been stoned ranting about incoherent coding stuff to him for a day or two .... I think I've made him face palm quite a few times

That can be fun, though!

Quote:

Originally Posted by Torhal (Post 287844)

That can be fun, though!

He's a good guy... Clever too - won't admit it though. (At least not to me)