Gravity
Gravity copied to clipboard
Use HashSet or flags for choices
Right now, a choice list is a List<T>
. This doesn't ensure that the selected choices are unique, and could lead to strange behavior and/or failed API calls if a choice is added to the list multiple times.
We should use either flag enums instead (and then the choice list can just be of type T
), or the choice list should be a HashSet<T>
instead.
I favor using HashSet
.