invert check so we fail faster

there's no reason to to_sym the string if it doesn't match the regexp
anyway

actionpack/lib/action_dispatch/routing/mapper.rb

@@ -1765,7 +1765,7 @@ def name_for_action(as, action) #:nodoc:
               # and return nil in case it isn't. Otherwise, we pass the invalid name
               # forward so the underlying router engine treats it and raises an exception.
               if as.nil?
-                candidate unless @set.named_routes.key?(candidate) || candidate !~ /\A[_a-z]/i
+                candidate unless candidate !~ /\A[_a-z]/i || @set.named_routes.key?(candidate)
               else
                 candidate
               end