-
Notifications
You must be signed in to change notification settings - Fork 270
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doubly link list #32
doubly link list #32
Conversation
Great move @TheSTL 👍 The first thing I want you to do is to remove the LRU implementation in a folder named |
@ashokdey |
You forgot to add entries in the README file! 🤒 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review is under process.
display() { | ||
let address = this.head.next; | ||
while (address !== this.tail) { | ||
console.log(address.data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no-console
Also, make sure the code adheres to ESLint rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a display function to print data in a linked list.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the comment to ignore the ESLint rule
🐻 What's inside?
Add doubly link list data structure having following functions:
argument node
Add LRU-cache data structure implementation.
O(1)
O(1)