correctly name shuffle to indicate that it is modifying the deck
get rid of discards entirely
I actually think the constants should be in the card class
nice card generation with id