Solution #1243
Solution #1243
Conversation
There was a problem hiding this comment.
Great job on implementing the solution! 🎉 The code meets the task requirements and constraints effectively. While there are minor improvements suggested, such as refining the return message for clarity and ensuring the presence of the wearing_a_mask key to avoid potential errors, these do not impact the core functionality. Keep up the good work and consider these suggestions for future enhancements! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
| if visitor["vaccine"]["expiration_date"] < date.today(): | ||
| raise OutdatedVaccineError("Vaccine is outdated.") | ||
|
|
||
| if visitor["wearing_a_mask"] is False: |
There was a problem hiding this comment.
Consider checking if the 'wearing_a_mask' key exists in the visitor dictionary before accessing it. This will prevent a potential KeyError if the key is missing.
| except NotWearingMaskError: | ||
| masks_to_buy += 1 | ||
| if masks_to_buy > 0: | ||
| return f"Friends should buy {masks_to_buy} masks" |
There was a problem hiding this comment.
Consider rephrasing the return message to 'Friends should buy masks' for better readability, especially when masks_to_buy is 1. Alternatively, handle singular and plural cases separately.
No description provided.