-
Notifications
You must be signed in to change notification settings - Fork 0
Chapter06 Compose Method #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
jiwhunkim
wants to merge
65
commits into
master
Choose a base branch
from
chapter06
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
Example: extract method
Example: extract method refactor
Example: No Local Variables
Example: No Local Variables refactor
Example: Using Local Variables
Example: Using Local Variables refactor
Example: Reassigning a Local Variable
Example: Reassigning a Local Variable refactor
Example: Reassigning a Local Variable inject Collection Closure Method
Example: Reassigning a Local Variable outstanding variable pass initial value
Example: Reassigning a Local Variable outstanding variable pass initial value refactor
Example: Reassigning a Local Variable clear up the way the outstanding variable is initialized
Example: Inline Method
Example: Inline Method refactor
Example: Inline Temp
Example: Inline Temp refactor
Example: Replace Temp with Query
Example: Replace Temp with Query refactor
Example: Replace Temp with Query 2
Example: Replace Temp with Query 2 extract the right-hand side of the assignment
Example: Replace Temp with Query 2 replace the first reference to the temp
Example: Replace Temp with Query 2 remove the temp assignment
Example: Replace Temp with Query 2 extract discount_factor
Example: Replace Temp with Query 2 remove the temp assignment
Example Replace Temp with Chain
Example Replace Temp with Chain refactor
Example Replace Temp with Chain 2 create Select method
Example Replace Temp with Chain 2 creating a Method Chained solution is to create a method that creates the Select instance and adds an option
Example Replace Temp with Chain 2 change the method that adds options to return self so that it can be chained
Example Replace Temp with Chain 2 rename the add_option method to something that reads more fluently, such as "and".
example with extract method 1 extract a method for the base price
example with extract method 1 extract a method for the quantity discount
example with extract method 1 extract a method for the shipping
example with split temporary variable
example with split temporary variable refactor
example with split temporary variable
example with split temporary variable changing the name of the temp
example with split temporary variable second assignment of the temp
Explain Remove Assignments to Parameters
Explain Remove Assignments to Parameters refactor Use a temporary variable instead
Example Remove Assignments to Parameters
Example Remove Assignments to Parameters refactor
Example Replace Method with Method Object
Example Replace Method with Method Object refactor
Explain Substitute Algorithm
Explain Substitute Algorithm refactor
Example Replace Loop with Collection Closure Method
Example Replace Loop with Collection Closure Method refactor
Example Replace Loop with Collection Closure Method employee office
Example Replace Loop with Collection Closure Method employee office refactor
Example Replace Loop with Collection Closure Method manager office
jiwhunkim
added a commit
that referenced
this pull request
Aug 31, 2019
Example Replace Loop with Collection Closure Method manager office refactor
Example Replace Loop with Collection Closure Method manager office refactor
Example Replace Loop with Collection Closure Method employee total salary
Example Replace Loop with Collection Closure Method employee total salary use inject
Explain Extract Surrounding Method
Explain Extract Surrounding Method refactor
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.
No description provided.