Solution #1503
Solution #1503
Conversation
There was a problem hiding this comment.
Great job on implementing the caching decorator! 🎉 Your code meets all the task requirements and constraints, and it's clear that you've put a lot of effort into ensuring it works correctly with multiple functions and handles caching efficiently. Just a small note: be cautious when using mutable types as keys in dictionaries, as this can lead to unexpected behavior. Consider ensuring that args is always a tuple of immutable types to avoid any hashing issues. Keep up the excellent work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
|
|
||
| @wraps(func) | ||
| def wrapper(*args: Tuple[Any, ...]) -> Any: | ||
| if args in cache_store: |
There was a problem hiding this comment.
Using mutable types as keys in a dictionary can lead to unexpected behavior. Ensure that args is always a tuple of immutable types to avoid issues with hashing.
No description provided.