Use of string constants instead of enum variables
The contract defines rarity categories as string constants for the rarity levels of 'common', 'rare', and 'legendary'.
Using string constants might increase the possibility of human error and reduce code readability.
Consider using enum to define rarity levels. Enums will enhance code readability and decrease the probability of mistakes related to rarity definitions and logic related to the rarity.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.