DisableAddOn() is only supposed to be on a per-character basis, so should not affect all of your characters.
But if you decide to go with the second approach still, do it this way instead so you're not making unnecessary function calls.
Lua Code:
local _,class == UnitClass("player")
if class == "WARLOCK" or class == "MONK" then
return
end