-
-
Notifications
You must be signed in to change notification settings - Fork 29
Glasgow | Sheetal Kharab | Module-Complexity | Sprint 2 | Implement an LRU cache in Python #39
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
base: main
Are you sure you want to change the base?
Changes from 5 commits
b2a67d4
4aa063a
5212626
aad8ad2
60478b5
cf30a3f
ec754f8
c5eee37
bceca8f
51ded70
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,25 @@ | ||
| /** | ||
| * Finds common items between two arrays. | ||
| * | ||
| * Time Complexity: | ||
| * Space Complexity: | ||
| * Optimal Time Complexity: | ||
| * Time Complexity:O(n+m) it build set and loop through arrays once | ||
| * Space Complexity: store second array in set | ||
| * Optimal Time Complexity:O(m+n) | ||
| * | ||
| * @param {Array} firstArray - First array to compare | ||
| * @param {Array} secondArray - Second array to compare | ||
| * @returns {Array} Array containing unique common items | ||
| */ | ||
| export const findCommonItems = (firstArray, secondArray) => [ | ||
| ...new Set(firstArray.filter((item) => secondArray.includes(item))), | ||
| ]; | ||
| // export const findCommonItems = (firstArray, secondArray) => [ | ||
| // ...new Set(firstArray.filter((item) => secondArray.includes(item))), | ||
| // ]; | ||
| // Refactored to use a Set for faster lookups, making the code more efficient | ||
| export const findCommonItems = (firstArray, secondArray) => { | ||
| const secondArraySet = new Set(secondArray); | ||
| const resultSet = new Set(); | ||
| for (const element of firstArray) { | ||
| if (secondArraySet.has(element)) { | ||
| resultSet.add(element); | ||
| } | ||
| } | ||
| return [...resultSet]; | ||
| }; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,21 +1,35 @@ | ||
| /** | ||
| * Find if there is a pair of numbers that sum to a given target value. | ||
| * | ||
| * Time Complexity: | ||
| * Space Complexity: | ||
| * Optimal Time Complexity: | ||
| * Time Complexity: o(n2) the function use 2 nested loop | ||
| * Space Complexity:o(1)no additional significant memory used | ||
| * Optimal Time Complexity: O(n) — in the refactored version using a Set for lookups | ||
| * | ||
| * @param {Array<number>} numbers - Array of numbers to search through | ||
| * @param {number} target - Target sum to find | ||
| * @returns {boolean} True if pair exists, false otherwise | ||
| */ | ||
| // export function hasPairWithSum(numbers, target) { | ||
| // for (let i = 0; i < numbers.length; i++) { | ||
| // for (let j = i + 1; j < numbers.length; j++) { | ||
| // if (numbers[i] + numbers[j] === target) { | ||
| // return true; | ||
| // } | ||
| // } | ||
| // } | ||
| // return false; | ||
| // } | ||
|
|
||
| export function hasPairWithSum(numbers, target) { | ||
| for (let i = 0; i < numbers.length; i++) { | ||
| for (let j = i + 1; j < numbers.length; j++) { | ||
| if (numbers[i] + numbers[j] === target) { | ||
| return true; | ||
| } | ||
| const numbersNeeded = new Set(); // stores numbers we've already seen | ||
|
|
||
| for (const num of numbers) { | ||
| const requiredNumber = target - num; | ||
| if (numbersNeeded.has(requiredNumber)) { | ||
| return true; // found a pair! | ||
| } | ||
| numbersNeeded.add(num); // remember current number | ||
| } | ||
| return false; | ||
|
|
||
| return false; // no pair found | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,75 @@ | ||
| class Node: | ||
| def __init__(self, key, value): | ||
| self.key = key | ||
| self.value = value | ||
| self.prev = None | ||
| self.next = None | ||
|
|
||
| class LruCache: | ||
| def __init__(self, limit: int): | ||
| if limit <= 0: | ||
| raise ValueError("Cache limit must be greater than zero") | ||
|
|
||
| self.limit = limit | ||
| self.cache = {} | ||
| self.head = None | ||
| self.tail = None | ||
| self.size = 0 | ||
|
|
||
|
|
||
| #to remove a node frpm linkedlist | ||
| def _remove_node(self, node: Node): | ||
| if node.prev: | ||
| node.prev.next = node.next | ||
| else: | ||
| self.head = node.next | ||
|
|
||
| if node.next: | ||
| node.next.prev = node.prev | ||
| else: | ||
| self.tail = node.prev | ||
|
|
||
| node.prev = node.next = None | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. A prudent tidy up step: don't let the removed node be able to get back "into" the list.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks handled in _remove_node by clearing prev and next.
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @OracPrime Please review it, sorry for delayed response. |
||
|
|
||
|
|
||
| #Add a node to the head of the linked list | ||
| def _add_to_head(self, node: Node): | ||
| node.next = self.head | ||
| node.prev = None | ||
|
|
||
| if self.head: | ||
| self.head.prev = node | ||
| self.head = node | ||
|
|
||
| if not self.tail: | ||
| self.tail = node | ||
|
|
||
|
|
||
| def get(self, key): | ||
| if key not in self.cache: | ||
| return None | ||
| node = self.cache[key] | ||
| # Move node to head (most recently used) | ||
| self._remove_node(node) | ||
| self._add_to_head(node) | ||
| return node.value | ||
|
|
||
|
|
||
| def set(self, key, value): | ||
| if key in self.cache: | ||
| node = self.cache[key] | ||
| node.value = value | ||
| self._remove_node(node) | ||
| self._add_to_head(node) | ||
| else: | ||
| node = Node(key, value) | ||
| self.cache[key] = node | ||
| self._add_to_head(node) | ||
| self.size += 1 | ||
|
|
||
| if self.size > self.limit: | ||
| # Evict LRU (tail node) | ||
| lru = self.tail | ||
| self._remove_node(lru) | ||
| del self.cache[lru.key] | ||
| self.size -= 1 | ||
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.
typos even in comments can reduce confidence in the reader!
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.
sorry, I will be carefull next time.