Use constant
Use each instead of map
regexp =~ string
shuffle only once