Refactoring involves improving the quality of existing code without changing the system's behavior. It is explicitly
not about adding or changing behavior, but about improving the implementation quality of existing behavior.
A full set of developer tests is required before refactoring can be safely applied. It is critical that the system
behavior be in a known, verifiably correct state before modifying the implementation so that you can improve the
quality without fear that the modified implementation will break something. Refactoring is a safe transformation to
improve code, but it is safe only if there are tests that can verify that the system still works as intended.
Refactoring is initiated when an area that needs improvement is identified in the system by examining either the code
or some other representation of the design. The issues identified are sometimes called "smells."
Here are several smells to look for that might lead to implementation or design refactoring:
-
Duplicated code: Duplicated code makes the system harder to understand and harder to maintain.
-
Large design element or method: Large design elements or methods diminish the ability of people to
understand the code, reduce the potential for reuse, and make developer testing more difficult.
-
Poorly named element: Whether the element be a variable, function, class, or implementation element, its
name should connote what it is so that the code can be maintained.
-
Tight coupling: Each design element should work with minimal concern for the internal aspects of
other design elements. Otherwise, changes to one element can have undesirable effects in other elements.
As you can see from this list, refactoring can improve the "internals" of an element or the interface of the
element. Also, many of the smells are characterized as making the software more difficult to understand; whereas
refactoring is about making the system simpler.
After an issue is identified, a refactoring method can be selected that will improve the situation. There are catalogs
of refactoring methods available that are change patterns that will fix common problems while retaining the behavior of
the system.
These are examples of refactoring methods:
-
Extract Method: Pull out the duplicated code into its own single method or extract part of a large method
into its own method.
-
Extract Class: Pull some cohesive part of a class into its own class to reduce the size of a design element
that is too big.
-
Rename Method, Rename Class, or Rename Variable: Give a more meaningful name to an element to
make it more understandable.
-
Extract Interface: Create a clean interface.
After refactoring has been applied, developer tests are run again to ensure that the system still behaves correctly. It
is important that the system is working correctly after each small refactoring. Although many refactorings can be put
together to drive broad change across the code base, the tests should run correctly between each refactoring applied.
Refactoring must be applied as small behavior-preserving transformations.
As mentioned previously, refactoring requires full developer test coverage of the area under consideration. There are
additional techniques that enable refactoring. Coding standards define a common style and make it easier to refactor
consistently. An attitude of collective code ownership within the team is important. Each developer should feel
that refactoring can be applied across the code base to improve the implementation.
For deeper coverage on this topic, including a listing of "smells" and a catalog of refactorings to respond to them,
see [FOW99].
|