lib/url_trimmer.rb in url_trimmer-0.0.1 vs lib/url_trimmer.rb in url_trimmer-0.0.2

@@ -1,20 +1,22 @@
 require_relative "url_trimmer/version"
 require "domain_name"
 
 module URLTrimmer
   class Worker
+    URL_REGEXP = %r(\Ahttps?://([^/]+))
+
     def self.uniq_by_domain(urls)
       urls.map! do |url|
         begin
           url.downcase
         rescue ArgumentError
           url.encode("UTF-8", invalid: :replace, undef: :replace, replace: "").downcase
         end
       end
-      urls.map! { |url| url[%r(\Ahttps?://[^/]+), 0] }
-      urls.compact!
-      urls.uniq! { |url| DomainName(url).domain }
+      urls.select! { |url| url =~ URL_REGEXP }
+      urls.uniq! { |url| DomainName(url[URL_REGEXP, 1]).domain }
+      urls.sort!
       urls
     end
   end
 end