From 88acb6bcfc29121a414f7a2bbf85c39dd110e2f9 Mon Sep 17 00:00:00 2001 From: Diana Date: Sun, 29 Mar 2020 13:35:04 -0700 Subject: [PATCH 1/2] passing max sub array test using kadane's algorithm --- lib/max_subarray.rb | 22 +++++++++++++++++++--- lib/newman_conway.rb | 2 +- test/max_sub_array_test.rb | 2 +- test/newman_conway_test.rb | 2 +- 4 files changed, 22 insertions(+), 6 deletions(-) diff --git a/lib/max_subarray.rb b/lib/max_subarray.rb index 5204edb..04dde23 100644 --- a/lib/max_subarray.rb +++ b/lib/max_subarray.rb @@ -1,8 +1,24 @@ # Time Complexity: ? # Space Complexity: ? +# Kadane's Algorithm def max_sub_array(nums) - return 0 if nums == nil - - raise NotImplementedError, "Method not implemented yet!" + return nil if nums.length == 0 + + max_so_far = nums[0] + max_ending_here = 0 + + nums.length.times do |i| + max_ending_here = max_ending_here + nums[i] + + if max_so_far < max_ending_here + max_so_far = max_ending_here + end + + if max_ending_here < 0 + max_ending_here = 0 + end + end + + return max_so_far end diff --git a/lib/newman_conway.rb b/lib/newman_conway.rb index 4c985cd..74323c4 100644 --- a/lib/newman_conway.rb +++ b/lib/newman_conway.rb @@ -1,6 +1,6 @@ -# Time complexity: ? +# Time complexity: O(n) # Space Complexity: ? def newman_conway(num) raise NotImplementedError, "newman_conway isn't implemented" diff --git a/test/max_sub_array_test.rb b/test/max_sub_array_test.rb index 3253cdf..e27e1ca 100644 --- a/test/max_sub_array_test.rb +++ b/test/max_sub_array_test.rb @@ -1,6 +1,6 @@ require_relative "test_helper" -xdescribe "max subarray" do +describe "max subarray" do it "will work for [-2,1,-3,4,-1,2,1,-5,4]" do # Arrange input = [-2,1,-3,4,-1,2,1,-5,4] diff --git a/test/newman_conway_test.rb b/test/newman_conway_test.rb index 537d376..6a42001 100644 --- a/test/newman_conway_test.rb +++ b/test/newman_conway_test.rb @@ -1,6 +1,6 @@ require_relative "test_helper" -describe "Newman Conway Tests" do +xdescribe "Newman Conway Tests" do it "works with 13" do # Arrange input = 13 From e28569fc9d5c488c64d71272756b50cc9491f85f Mon Sep 17 00:00:00 2001 From: Diana Date: Sun, 29 Mar 2020 15:08:56 -0700 Subject: [PATCH 2/2] newman conway complete using memoization --- lib/max_subarray.rb | 5 ++--- lib/newman_conway.rb | 19 +++++++++++++++---- test/newman_conway_test.rb | 2 +- 3 files changed, 18 insertions(+), 8 deletions(-) diff --git a/lib/max_subarray.rb b/lib/max_subarray.rb index 04dde23..7620a0c 100644 --- a/lib/max_subarray.rb +++ b/lib/max_subarray.rb @@ -1,6 +1,5 @@ - -# Time Complexity: ? -# Space Complexity: ? +# Time Complexity: O(n) +# Space Complexity: O(1) # Kadane's Algorithm def max_sub_array(nums) return nil if nums.length == 0 diff --git a/lib/newman_conway.rb b/lib/newman_conway.rb index 74323c4..656b5e9 100644 --- a/lib/newman_conway.rb +++ b/lib/newman_conway.rb @@ -1,7 +1,18 @@ +# Time complexity: O(n) +# Space Complexity: O(n) +def newman_conway(num) # return a string of numbers, in O(n) using memoization + raise ArgumentError if num == 0 + return "1" if num == 1 + result = "1 1" + memo = [0, 1, 1] + n = 3 + while n <= num + memo[n] = memo[memo[n-1]] + memo[n-memo[n-1]] + result += " #{memo[n]}" -# Time complexity: O(n) -# Space Complexity: ? -def newman_conway(num) - raise NotImplementedError, "newman_conway isn't implemented" + n += 1 + end + + return result end \ No newline at end of file diff --git a/test/newman_conway_test.rb b/test/newman_conway_test.rb index 6a42001..537d376 100644 --- a/test/newman_conway_test.rb +++ b/test/newman_conway_test.rb @@ -1,6 +1,6 @@ require_relative "test_helper" -xdescribe "Newman Conway Tests" do +describe "Newman Conway Tests" do it "works with 13" do # Arrange input = 13