Refaktorisering

Beskrivning

Seminarium för dig som tittat i Refactoring-menyn och känner på dig att det finns betydligt mer bortom "Rename". Genom konkreta demo och praktiska övningar bekantar vi oss närmare med konkreta handgrepp och tar dessa som utgångspunkt för diskussioner om pragmatik och filosofi.

Tid: 4h

Instruktion (för deltagaren)

Labbkoden finns i form av ett maven-projekt definierat av pom.xml

För att öppna i IntelliJ: välj "Import Project" och markera biblioteket "refactoring-lab", importera som Maven-projekt

Kör enhetstestet i CustomerTest.

Kata 1: Dela upp och organisera om koden i statement-metoden

Vi ärver en halvrörig kod. Vad kan vi göra för att göra koden enklare och lättare att förstå?

Som huvudmål i denna del vill vi extrahera en metod för prisberäkning: charge().

Kodlukter:

Lång metod
Stor klass
Funktionsavundsjuka
Tillfälliga variabler

Del 1.A

Bryt ut switch-satsen ur loopen till en egen metod. Testa skillnaden med att göra det själv och att låta ditt IDE göra det åt dig.

Del 1.B

Flytta den nya metoden till Rental
Byt namn på den nya metoden till exempelvis charge()

Varför gjorde vi denna refaktorisering?

Del 1.C

Gör inline på anropet till den nya metoden charge() i Customer::statement.
Byt ++ mot +=
Summera till en temp-variabel (tmpPoints) och lägg till temp-variabeln till summan
Bryt ut beräkningen av frequentRenterPoints till en egen metod.
Namnge metoden exempelvis getFrequentRenterPoints
Gör inline på tmpPoints variabeln
Flytta den nya metoden till Rental

Del 1.D

Summera till en temp-variabel (tmpResult) och lägg till temp-variabeln till summan
Bryt ut beräkningen av result till en egen metod.
Namnge metoden exempelvis getRentalFigures
Gör inline på tmpResult variabeln
Flytta den nya metoden till Rental

Del 1.E

Pilla isär loopen till en loop per beräkning.
Bryt ut varje loop till en egen metod. Glöm inte att få med den lokala variabeln
EXTRA: Bygg om looparna till att använda streams.

Kata 2: Isolera prisberäkningen.

Vi får reda på att butiken kommer att vilja ha fler typer av priser och kunna byta prissättning på filmerna efterhand. Kan vi förändra koden så att det blir lättare att stödja en sådan funktion?

Del 2.A

Introducera en ny variabel för getDaysRented() (Vi behöver bara daysRented från rental.)
Extrahera en ny metod som exempelvis heter chargeForDaysRented()
Flytta metoden chargeForDaysRented() vidare från Rental till Movie.
Bli av med Rental parameter beroendet i chargeForDaysRented()

Del 2.B

Extrahera getPriceCode() till en lokal variabel
Extrahera tillfällig metod chargeForDaysAndPriceCode(int daysRented, int priceCode)
Gör inline på thisAmount
Extrahera metoden till delegate PriceCalculatorImpl

Del 2.C

Flytta initiering till konstruktor-parameter. Hur ser testet ut nu?
Skapa interface PriceCalculator
Kopiera PriceCalculatorImpl och döp till RegularPriceCalculator
Gör chargeForDaysRentedAndPriceCode i PriceCalculatorImpl publik
Ta bort delar som ej har med REGULAR_PRICE_CODE ur switch-satsen
Byt ut medlemsvariabeln PriceCalculatorImpl mot PriceCalculator interfacet i Movie
Byt ut till RegularPriceCalculator i test-konstruktor

Del 2.D

Gör motsvarande för Children och New Release som gjordes för Regular
Kör testerna

Del 2.E

Lägg till metod för att ändra calculator
Ta bort switch statement
Safe delete på priceCode