strings-case 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc47bee251774eb57217a3fbac0028bd94ad6b44697f364c3d9fbb9e5f23c78c
-  data.tar.gz: 6d2f3fc1a27f70387723cb362fa665bdb243dc948306d5d6e5021bcadaf316b2
+  metadata.gz: bec785e9ba89be06265c1bc10dc3e9523aedd7381f318439251f7100ff03d4f4
+  data.tar.gz: 7d3291105499505fcc5782e18f9c278b687a1b2dbd88b4342e5641601757dd3e
 SHA512:
-  metadata.gz: 5c2a23a788a31480cdb8b099d39d5ec7ab58e9bbb8291810b4f89a1d34447a96f7883d599f056b91b85e8df19fbaf3c5503c3aa17e54efe543422ee1a4c23a3d
-  data.tar.gz: 771199eab86b3a8c5e350b7b14787fe9e0b0645b583cf33465d46975dd4c97a542e02426014775fe46db287e3dac9932cbad67d68e30fc22ee504cea7d58e08c
+  metadata.gz: 7bee902c8bd54279c1d70e8592117eaf173f2c2920b0b7caa4d0e5006faf795771e82f834078bc1022b581d16df4b0fb9e60af928395c03bceb2265bac323d07
+  data.tar.gz: c533aaf5d4f0cbe6f88a18f66aef6f25254dac9ee753c2e4941a02c8cb4456a82b7bc09e68f05d9f7fcc26063fbef9ffd91366bfd76c489b7cf28e5bc34215ae

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Change log
 
+## [v0.3.0] - 2019-12-07
+
+### Added
+* Add performance tests
+
+### Changed
+* Change to double parsing speed and halve object allocations
+
 ## [v0.2.0] - 2019-11-23
 
 ### Fixed
@@ -9,5 +17,6 @@
 
 * Initial implementation and release
 
+[v0.3.0]: https://github.com/piotrmurach/strings-case/compare/v0.1.0...v0.3.0
 [v0.2.0]: https://github.com/piotrmurach/strings-case/compare/v0.1.0...v0.2.0
 [v0.1.0]: https://github.com/piotrmurach/strings-case/compare/v0.1.0

data/README.md

@@ -265,7 +265,7 @@ Strings::Case.titlecase("HTTP response code", acronyms: ["HTTP"])
 
 Though it is highly discouraged to pollute core Ruby classes, you can add the required methods to `String` class by using refinements.
 
-For example, if you wish to only extend strings with `wrap` method do:
+For example, if you wish to only extend strings with `snakecase` method do:
 
 ```ruby
 module MyStringExt

data/lib/strings/case.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "strscan"
+
 require_relative "case/version"
 
 module Strings
@@ -8,9 +10,11 @@ module Strings
     UP_LETTERS = ("A".."Z").freeze
     DOWN_LETTERS = ("a".."z").freeze
     DELIMITERS = [" ", "\n", "\t", "_", ".", "-", "#", "?", "!"].freeze
+    DELIMS = Regexp.union(DELIMITERS)
     NONALPHANUMERIC = (32..127).map(&:chr) -
       (DIGITS.to_a + DOWN_LETTERS.to_a + UP_LETTERS.to_a + DELIMITERS)
-    UPCASE = /(?<!\p{Lu})\p{Lu}$/.freeze
+    NONALPHAS = Regexp.union(NONALPHANUMERIC)
+    UPPERCASE = /^(\p{Ll}|\p{Digit})\p{Lu}/.freeze
     LOWERCASE = /\p{Lu}(?=\p{Ll})/.freeze
 
     class Error < StandardError; end
@@ -258,45 +262,45 @@ module Strings
     def split_into_words(string, sep: nil)
       words = []
       word = []
-      last = string.length - 1
-
-      string.each_char.with_index do |char, i|
-        combine = word[-1].to_s + char
+      scanner = StringScanner.new(string)
 
-        if combine =~ UPCASE
+      while !scanner.eos?
+        if scanner.match?(UPPERCASE)
+          char = scanner.getch
           if word.size <= 1 # don't allow single letter words
             word << char
           else
+            word << char
             words << word.join
-            word = [char]
+            word = []
           end
-        elsif combine =~ LOWERCASE
-          letter = word.pop
+        elsif scanner.match?(LOWERCASE)
+          char = scanner.getch
           if word.size <= 1 # don't allow single letter words
-            word << letter << char
+            word << char
           else
             words << word.join
-            word = [letter, char]
+            word = [char]
           end
-        elsif DELIMITERS.include?(char)
+        elsif scanner.match?(DELIMS)
+          char = scanner.getch
           words << word.join unless word.empty?
-          if i.zero? && char == sep
+          if scanner.pos == 1 && char == sep
             words << ""
+          elsif scanner.eos? && char == sep
+            word = [""]
           else
             word = []
           end
-        elsif NONALPHANUMERIC.include?(char)
+        elsif scanner.skip(NONALPHAS)
           # noop
         else
-          word << char
-        end
-
-        if last == i
-          word = [""] if char == sep
-          words << word.join unless word.empty?
+          word << scanner.getch
         end
       end
 
+      words << word.join unless word.empty?
+
       words
     end
     module_function :split_into_words

data/lib/strings/case/version.rb

@@ -2,6 +2,6 @@
 
 module Strings
   module Case
-    VERSION = "0.2.0"
+    VERSION = "0.3.0"
   end
 end

data/spec/perf/parsecase_spec.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+require "rspec-benchmark"
+require "active_support"
+
+RSpec.describe Strings::Case do
+  include RSpec::Benchmark::Matchers
+
+  it "changes case" do
+    expect {
+      Strings::Case.snakecase("fooBarBaz")
+    }.to perform_slower_than {
+      ActiveSupport::Inflector.underscore("fooBarBaz")
+    }.at_most(2).times
+  end
+
+  it "allocates no more than 100 objects" do
+    expect {
+      Strings::Case.snakecase("fooBarBaz")
+    }.to perform_allocation(25)
+  end
+end

data/tasks/spec.rake

@@ -18,6 +18,11 @@ begin
     RSpec::Core::RakeTask.new(:integration) do |task|
       task.pattern = "spec/integration{,/*/**}/*_spec.rb"
     end
+
+    desc "Run performance specs"
+    RSpec::Core::RakeTask.new(:perf) do |task|
+      task.pattern = "spec/perf{,/*/**}/*_spec.rb"
+    end
   end
 
 rescue LoadError

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: strings-case
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - Piotr Murach
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-23 00:00:00.000000000 Z
+date: 2019-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -67,6 +67,7 @@ files:
 - lib/strings/case.rb
 - lib/strings/case/extensions.rb
 - lib/strings/case/version.rb
+- spec/perf/parsecase_spec.rb
 - spec/spec_helper.rb
 - spec/unit/camelcase_spec.rb
 - spec/unit/constcase_spec.rb