Skip to content

Ren - Fire#34

Open
RenCarothers wants to merge 1 commit into
Ada-C14:masterfrom
RenCarothers:master
Open

Ren - Fire#34
RenCarothers wants to merge 1 commit into
Ada-C14:masterfrom
RenCarothers:master

Conversation

@RenCarothers
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@CheezItMan CheezItMan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one commit?

Otherwise this is a great submission, well done, you hit all the learning goals here.

Comment thread lib/linked_list.rb
Comment on lines +21 to 23
# Time Complexity: O(1)
# Space Complexity: O(1)
def add_first(value)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +29 to 31
# Time Complexity: O(n)
# Space Complexity: O(1)
def search(value)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines 46 to 48
# method to return the max value in the linked list
# returns the data value and not the node
def find_max
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +66 to 68
# Time Complexity: O(n)
# Space Complexity: O(1)
def find_min
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +88 to 90
# Time Complexity: O(n)
# Space Complexity: O(1)
def get_first
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +132 to 134
# Time Complexity: O(n)
# Space Complexity: O(1)
def get_at_index(index)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +150 to 152
# Time Complexity: O(n)
# Space Complexity: O(n)
def visit
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +162 to 164
# Time Complexity: O(n)
# Space Complexity: O(1)
def delete(value)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +183 to 185
# Time Complexity: O(n)
# Space Complexity: O(1)
def reverse
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread lib/linked_list.rb
Comment on lines +203 to 205
# Time Complexity: O(n)
# Space Complexity: O(1)
def get_last
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants