Update builder.md #433
Open
Update builder.md #433
Conversation
This file contains 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
Cite advantage about code extension use-case and the fact that Rust does not support default values for function parameters.
marcoieni
reviewed
Apr 2, 2025
| @@ -73,6 +73,9 @@ Prevents proliferation of constructors. | |||
|
|
|||
| Can be used for one-liner initialisation as well as more complex construction. | |||
|
|
|||
| When the target struct is updated with new fields, the builder can be updated | |||
| to leave client code correct without changes. | |||
There was a problem hiding this comment.
I'm not sure what this means
There was a problem hiding this comment.
Imagine you have a custom type (the target struct) that you want to extend with new behaviour. You would like to add new behaviour and use it in new code, without modifying existing legacies. Since the builder pattern encapsulates the construction logic, it's possible to update the builder so that new fields can be used in new code while safely ignored in older usages.
marcoieni
reviewed
Apr 2, 2025
Comment on lines
+87
to
+89
| many other languages because Rust lacks overloading and default values for | ||
| function parameters. Since you can only have a single method with a given name, | ||
| having multiple constructors is less nice in Rust than in C++, Java, or others. |
There was a problem hiding this comment.
this change looks good 👍
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.
Cite advantage about code extension use-case and the fact that Rust does not support default values for function parameters.