runby_pace 0.6.142 → 0.6.143

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2ca16f06cd0e2fb02322c8be33f6c6ae6a043c0d
-  data.tar.gz: 35a76844e0106a92f721030cb17911ed829ef9fb
+  metadata.gz: a168825022bc1fb0538091f2cd05b587d81f7994
+  data.tar.gz: dce210b22b9cdd6d75599b666cf9e08e6da0dd8c
 SHA512:
-  metadata.gz: 09005d4cd8494ded7689d653f66fd43d0b6be001b38773df751a12994aeff87498e9b3806286b0ab39e5e6f1c79330c50e850aab92c213c042c52c994560dbb4
-  data.tar.gz: 6228df8777be6c11e99fa4d1d0c415d628867be9686e366decfb33bfd41424aef5fefaa9105ddd5594e8fead75836f6a7b2f4a7f9eb044acbc7eb120f728c10c
+  metadata.gz: 5bda98da133fa292cfb85e93f9e0998aa695634c6f404678f7cec9ab95c4118bbae636ca321ec7ab1aea318c9e3ba7636cda73a411ea938b0a6ce975d39256b0
+  data.tar.gz: d8019e291d0cc0a7670b76f8fef64937cc6aa9c3b830feb3c4f8b7b9cec0955fe26bea37012392e96b60f89b9ca8af2aa089a11ef7a638823db62301f1504c32

data/lib/runby_pace/runby_time.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Runby
   # Represents a human-readable time in the format MM:ss
   class RunbyTime
@@ -20,7 +22,7 @@ module Runby
       hours = total_seconds.abs.to_i / 60 / 60
       minutes = (total_seconds.abs.to_i / 60) % 60
       seconds = total_seconds.abs.to_i % 60
-      if hours > 0
+      if hours.positive?
         RunbyTime.new format('%d:%02d:%02d', hours, minutes, seconds)
       else
         RunbyTime.new format('%02d:%02d', minutes, seconds)
@@ -38,42 +40,7 @@ module Runby
     end
 
     def self.parse(str)
-      time = str.to_s.strip.chomp
-
-      if time =~ /^\d?\d:\d\d:\d\d$/
-        parts = time.split(':')
-        hours_part = parts[0].to_i
-        minutes_part = parts[1].to_i
-        seconds_part = parts[2].to_i
-      elsif time =~ /^\d?\d:\d\d$/
-        parts = time.split(':')
-        hours_part = 0
-        minutes_part = parts[0].to_i
-        seconds_part = parts[1].to_i
-      elsif time =~ /^\d+$/
-        hours_part = 0
-        minutes_part = time.to_i
-        seconds_part = 0
-      elsif time =~ /^\d+[,. ]\d+$/
-        parts = time.split(/[,. ]/)
-        hours_part = 0
-        minutes_part = parts[0].to_i
-        seconds_part = (parts[1].to_i / 10.0 * 60).to_i
-      else
-        raise 'Invalid time format'
-      end
-
-      raise 'Hours must be less than 24' if hours_part > 23
-      raise 'Minutes must be less than 60 if hours are supplied' if hours_part.positive? && minutes_part > 59
-      raise 'Minutes must be less than 99 if no hours are supplied' if hours_part.zero? && minutes_part > 99
-      raise 'Seconds must be less than 60' if seconds_part > 59
-      hours_part_formatted = ''
-      if hours_part.positive?
-        hours_part_formatted = "#{hours_part.to_s.rjust(2, '0')}:"
-      end
-      time_formatted = "#{hours_part_formatted}#{minutes_part.to_s.rjust(2, '0')}:#{seconds_part.to_s.rjust(2, '0')}"
-
-      RunbyTime.new(time_s: time_formatted, hours_part: hours_part, minutes_part: minutes_part, seconds_part: seconds_part)
+      RunbyTimeParser.parse str
     end
 
     def self.try_parse(str, is_five_k = false)

data/lib/runby_pace/runby_time_parser.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+module Runby
+  # Helper class which parses strings and returns new RunbyTime(s)
+  class RunbyTimeParser
+    def self.parse(str)
+      time = str.to_s.strip.chomp
+
+      if time.match?(/^\d?\d:\d\d:\d\d$/)
+        parts = time.split(':')
+        hours_part = parts[0].to_i
+        minutes_part = parts[1].to_i
+        seconds_part = parts[2].to_i
+      elsif time.match?(/^\d?\d:\d\d$/)
+        parts = time.split(':')
+        hours_part = 0
+        minutes_part = parts[0].to_i
+        seconds_part = parts[1].to_i
+      elsif time.match?(/^\d+$/)
+        hours_part = 0
+        minutes_part = time.to_i
+        seconds_part = 0
+      elsif time.match?(/^\d+[,. ]\d+$/)
+        parts = time.split(/[,. ]/)
+        hours_part = 0
+        minutes_part = parts[0].to_i
+        seconds_part = (parts[1].to_i / 10.0 * 60).to_i
+      else
+        raise 'Invalid time format'
+      end
+
+      raise 'Hours must be less than 24' if hours_part > 23
+      raise 'Minutes must be less than 60 if hours are supplied' if hours_part.positive? && minutes_part > 59
+      raise 'Minutes must be less than 99 if no hours are supplied' if hours_part.zero? && minutes_part > 99
+      raise 'Seconds must be less than 60' if seconds_part > 59
+      hours_part_formatted = ''
+      if hours_part.positive?
+        hours_part_formatted = "#{hours_part.to_s.rjust(2, '0')}:"
+      end
+      time_formatted = "#{hours_part_formatted}#{minutes_part.to_s.rjust(2, '0')}:#{seconds_part.to_s.rjust(2, '0')}"
+
+      RunbyTime.new(time_s: time_formatted, hours_part: hours_part, minutes_part: minutes_part, seconds_part: seconds_part)
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: runby_pace
 version: !ruby/object:Gem::Version
-  version: 0.6.142
+  version: 0.6.143
 platform: ruby
 authors:
 - Ty Walls
@@ -95,6 +95,7 @@ files:
 - lib/runby_pace/run_types/ten_kilometer_race_run.rb
 - lib/runby_pace/runby_range.rb
 - lib/runby_pace/runby_time.rb
+- lib/runby_pace/runby_time_parser.rb
 - lib/runby_pace/speed.rb
 - lib/runby_pace/speed_range.rb
 - lib/runby_pace/version.rb
@@ -104,7 +105,7 @@ homepage: https://github.com/tygerbytes/runby-pace
 licenses:
 - MIT
 metadata:
-  commit-hash: 1b69e5a5f3d05761bb3d5784091d1294ec588ff0
+  commit-hash: 83c260f4c540d7ab5b42c37fa1606905c151d6b9
 post_install_message: 
 rdoc_options: []
 require_paths: