Please use built-in enums
andrewvaughan opened this issue · comments
mutagen/mutagen/mp3/__init__.py
Line 36 in f95d3ae
This is a very unusable implementation - namely because it requires a match/case
or similar anti-pattern to be implemented as you've created a fairly whacky method of returning a string representation.
For example, if I simply want to record that the bit-rate mode of an MP3 was CBR - there's no easy way for me to just get CBR
as you've forced the string representation to be BitrateMode.CBR
. I have no idea why anyone would ever want the Enum class in their string representation and is very non-standard.
Either this, or please implement the standard .name
and .value
properties in normal Python enum
to make this possible.
The implementation comes from a time before there was enum in the standard library. Switching to the built-in enum might be a breaking change in some way. But implementing the name
and value
properties makes sense to me.
there's no easy way for me to just get CBR as you've forced the string representation to be BitrateMode.CBR. I have no idea why anyone would ever want the Enum class in their string representation and is very non-standard.
I wouldn't call it "very non-standard". It is exactly what Python's enum.__str__
does as well:
from enum import Enum
Color = Enum('Color', ['RED', 'GREEN', 'BLUE'])
str(Color.RED) # This is "Color.RED"