Created
February 19, 2019 18:48
-
-
Save ZASMan/a20bbdafef4ce2d3c8ff6ebce98becb9 to your computer and use it in GitHub Desktop.
code comparison
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# First I had this | |
class Series | |
def initialize(numbers) | |
@numbers = numbers | |
@numbers_array = numbers.split("").map { |num_string| num_string.to_i } | |
end | |
def slices(substring_length) | |
substring_length = substring_length.to_int | |
if substring_length > @numbers_array.length | |
raise ArgumentError.new("Substring length cannot exceed number array length.") | |
end | |
result = [] | |
starting_index = 0 | |
substring_length_doesnt_exceed_string = true | |
while substring_length_doesnt_exceed_string == true | |
substring_array = @numbers_array[starting_index, substring_length] | |
substring = substring_array.join | |
result << substring | |
starting_index = starting_index + 1 | |
if starting_index > @numbers_array.length | |
substring_length_doesnt_exceed_string = false | |
end | |
end | |
# Remove result elements which are less than substring legnth | |
result.reject! { |element| element.length < substring_length } | |
result.compact.flatten.map! { |element| element.to_s } | |
end | |
end | |
# Simplified to this | |
class Series | |
def initialize(numbers) | |
@numbers = numbers | |
@numbers_array = numbers.split("").map { |num_string| num_string.to_i } | |
end | |
def slices(substring_length) | |
substring_length = substring_length.to_int | |
if substring_length > @numbers_array.length | |
raise ArgumentError.new("Substring length cannot exceed number array length.") | |
end | |
@numbers_array.each_cons(substring_length).to_a.map { |array| array.join } | |
end | |
end |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment