Help for a new Addon Programmer

[Lagencie]

Hello Guys,

i feel kind of stupid ... i dont even get the "Writing_your_first_addon" to work ...

LempsAddon = {}
LempsAddon.name = "LempsAddon"
LempsAddon.version = "1.0"
local LempsAddon.inCombat = false

local function LempsAddon.CombatStateChanged(combatState)
	self.inCombat = combatState
	if self.inCombat then
		d("fight")
	else
		d("no fight")
	end
end

local function LempsAddon:Initialize(event, addon)
	self.inCombat = IsUnitInCombat("player")
	EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_PLAYER_COMBAT_STATE, LempsAddon.CombatStateChanged)
end

local function LempsAddon.OnAddOnLoaded(event, addonName)
  if addonName == LempsAddon.name then
    LempsAddon:Initialize()
  end
end

EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_ADD_ON_LOADED, LempsAddon.OnAddOnLoaded)

This is my code but it just doesnt work ... I dont even get where this code would be able to not work ... I see no Text output to the chat (debug) at all

[Dolgubon]

First, did you /reloadui? It's simple, but it can be easy to forget to do. Does your addon appear on the list off addons?

If you did do that, and it's still not working: Did you see any lua errors? If so, read it. It's often a lot of help in figuring out what went wrong and what you need to fix.

If there's no lua error at all, then it's something else. Check to make sure that the name of the addon matches the folder name.

LempsAddon.name = "LempsAddon" -- "LempsAddon MUST be written in exactly the same way as the name of the folder that the addon is contained in.

Also check to make sure that the name of the manifest (.txt) file matches the folder name. Open the manifest file and check that the main.lua (or whatever it's called) is listed in the manifest file. Make sure you aren't missing anything else in it.

If you download pChat, you can also see d() statements placed in the OnAddonLoaded function. If you don't have it, I suggest you do get it, and then place a d() statement right at the start of the function, and one inside the if statement. Place a d() at the start of the CombatStateChanged function too. Basically, you want to see if anything runs, and if something does, but some don't, that can help figure out what's going wrong. (just make sure to take them out later!)

[Ayantir]

Hello, and welcome.

First, I suggest you to use the Lua Highlighter in the forum (the last icon).

Then, few syntaxes mistakes

LempsAddon = {}
LempsAddon.name = "LempsAddon"
LempsAddon.version = "1.0"
local LempsAddon.inCombat = false
 
local function LempsAddon.CombatStateChanged(combatState)
    self.inCombat = combatState
    if self.inCombat then
        d("fight")
    else
        d("no fight")
    end
end
 
local function LempsAddon:Initialize(event, addon)
    self.inCombat = IsUnitInCombat("player")
    EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_PLAYER_COMBAT_STATE, LempsAddon.CombatStateChanged)
end
 
local function LempsAddon.OnAddOnLoaded(event, addonName)
  if addonName == LempsAddon.name then
    LempsAddon:Initialize()
  end
end
 
EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_ADD_ON_LOADED, LempsAddon.OnAddOnLoaded)

• line 1: Your variable is leaked. you should add local before
• line 3: Your variable is unused. please avoid it.
• line 4: It can't work. local must be followed with a new var name. Here you write a sub element of your variable. It should be local inCombat = false or LempsAddon.inCombat = false
• line 6: same problem. local function must be followed with the name a new variable. It's local function CombatStateChanged(combatState) or function LempsAddon.CombatStateChanged(combatState). You also forget the event parameter.
• line 7: Your variable LempsAddon is not an object. self don't work. it's LempsAddon.inCombat = combatState
• line 8: same as line 7
• line 15: same as line 6
• line 16: same as line 7
• line 17: should reflect your choice due to the error in line 6
• line 20: same as line 6/15
• line 22: should reflect the change due to the error in line 15
• line 26: should reflect the change due to the error in line 20

My code (I also added the unregistration of your addon). :

local LempsAddon = {}
LempsAddon.name = "LempsAddon"
LempsAddon.version = "1.0"
LempsAddon.inCombat = false
     
function LempsAddon.CombatStateChanged(event, combatState)
    LempsAddon.inCombat = combatState
    if LempsAddon.inCombat then
        d("fight")
    else
        d("no fight")
    end
end
     
function LempsAddon.OnAddOnLoaded(event, addonName)
    if addonName == LempsAddon.name then
 
        LempsAddon.inCombat = IsUnitInCombat("player")
 
        EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_PLAYER_COMBAT_STATE, LempsAddon.CombatStateChanged)
        EVENT_MANAGER:UnregisterForEvent("LempsAddon", EVENT_ADD_ON_LOADED)
    end
end
     
EVENT_MANAGER:RegisterForEvent("LempsAddon", EVENT_ADD_ON_LOADED, LempsAddon.OnAddOnLoaded)

[Lagencie]

lol, thanks ... so much small things wrong ... works now