You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Abilities must implement a levelRequirement, however monster specific abilities such as mutliAttack have no use for the levelRequirement field as Monster has no concept of Level, instead implementing a challenge rating.
There is unlikely to be any ability used by both Players and Monsters. Likely a separate implementation is more appropriate (e.g Extra Attack / Multi Attack).
One possibility is to drop levelRequirement from Ability. This is undeseriable as it's main usage is to remind the developer to implement this and consider it.
Another option is to have a Level to represent Monsters (e.g LevelZero), however this will cause issues when pattern matching over levels (see Cleric#clericSpellSlots) where this would not make sense.
The text was updated successfully, but these errors were encountered:
Abilities must implement a
levelRequirement
, however monster specific abilities such as mutliAttack have no use for the levelRequirement field as Monster has no concept of Level, instead implementing a challenge rating.There is unlikely to be any ability used by both Players and Monsters. Likely a separate implementation is more appropriate (e.g Extra Attack / Multi Attack).
One possibility is to drop
levelRequirement
fromAbility
. This is undeseriable as it's main usage is to remind the developer to implement this and consider it.Another option is to have a
Level
to represent Monsters (e.g LevelZero), however this will cause issues when pattern matching over levels (see Cleric#clericSpellSlots) where this would not make sense.The text was updated successfully, but these errors were encountered: