Solution #1400
Solution #1400
Conversation
There was a problem hiding this comment.
Great job on implementing the caching decorator! 🎉 Your code meets all the task requirements and constraints effectively. Just a couple of minor points to note: ensure you're using cache_result[args] for dictionary access since args is already a tuple, and remember to use Callable with a capital 'C' for type hints. Keep up the excellent work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| def cache(func: Callable) -> Callable: | ||
| # Write your code here | ||
| pass | ||
| def cache(func: callable) -> Callable: |
There was a problem hiding this comment.
The type hint for func should be Callable with a capital 'C'. The correct import from typing is Callable, not callable.
|
|
||
| def function_cache(*args) -> Any: | ||
| if args not in cache_result: | ||
| cache_result[*args] = func(*args) |
There was a problem hiding this comment.
The syntax cache_result[*args] is incorrect for dictionary key assignment. It should be cache_result[args] since args is already a tuple.
| print("Calculating new result") | ||
| else: | ||
| print("Getting from cache") | ||
| return cache_result[*args] |
There was a problem hiding this comment.
Similar to the previous comment, use cache_result[args] instead of cache_result[*args] to access the cached result.
No description provided.