gh-111307 Updating match..case in design documentation #111312
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The documentation still recommended not accurate way to do Structural Pattern Matching (aka `switch case`), and was not up to date as this feature is included in Python 3.10 As suggested, I try to upgrade it python#111307 (comment)
|
All commit authors signed the Contributor License Agreement. |
bedevere-app
bot
added
awaiting review
docs
Eclips4
changed the title
There was a problem hiding this comment.
There is nothing wrong with if ... elif .... The raison d'etre of structural pattern matching is more about destructuring complex objects than it is providing a switch statement. I'd recommend keeping the previous text as it was, and simply adding an example of a match ... case statement after the first paragraph
|
Why do you want to remove the dictionary and getattr solutions? |
|
I explained on the issue why I think deleting the existing options is a mistake. Closing this PR which does that. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The documentation still recommended not accurate way to do Structural Pattern Matching (aka
switch case), and was not up to date as this feature is included in Python 3.10As suggested, I try to upgrade it
#111307 (comment)
📚 Documentation preview 📚: https://cpython-previews--111312.org.readthedocs.build/
match-case. #111307