Skip to content
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

Move Method is incorrectly named #62

Open
LegalizeAdulthood opened this issue Jan 14, 2024 · 0 comments
Open

Move Method is incorrectly named #62

LegalizeAdulthood opened this issue Jan 14, 2024 · 0 comments

Comments

@LegalizeAdulthood
Copy link
Owner

The source file for Move Method says:

Inlines a method body into the class declaration (in a header file), or inversely moves a method's
implementation to a source file leaving the declaration in the header file.

...but this is really Move Implementation to Source File or Move Implementation to Header File which are specific to C++.
The "Move Method" tests should be moved to their appropriate C++ test cases.

The traditional Move Method refactoring involves moving a method from class A to class B, when:

"A method is, or will be, using or used by more features of another class than the class on which it is defined."

This is a holdover from the name of the refactoring in ReFactor!Pro.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant