Det vil gjøre det mulig for oss å gjenbruke koden i den andre delen slik:
ellers $ pfCount = count ($ pf); for ($ i = 0; $ i<$pfCount;$i++) if(isDivisible($num, $pf[$i])) return false; return true;
Og så snart vi begynte å jobbe med denne koden, så vi at det var uforsiktig. Braces er noen ganger på begynnelsen av linjen, andre ganger på slutten.
Noen ganger brukes faner for innrykk, noen ganger mellomrom. Noen ganger er det mellomrom mellom operand og operatør, noen ganger ikke. Og nei, dette er ikke spesielt opprettet kode. Dette er virkeligheten. Real kode, ikke noe kunstig trening.
// Sjekk om et tall er hovedfunksjonen erPrime ($ num, $ pf = null) if (! Is_array ($ pf)) for ($ i = 2; $ i < intval(sqrt($num)); $i++) if (isDivisible($num, $i)) return false; return true; else $pfCount = count($pf); for ($i = 0; $i < $pfCount; $i++) if (isDivisible($num, $pf[$i])) return false; return true;
Det ser bedre ut. Umiddelbart de to hvis uttalelser ser veldig like ut. Men vi kan ikke trekke dem ut på grunn av komme tilbake uttalelser. Hvis vi ikke kommer tilbake, vil vi bryte logikken.
Hvis den ekstraherte metoden ville returnere en boolsk og vi sammenligner den for å avgjøre om vi skal eller ikke kommer tilbake fra isPrime (), det ville ikke hjelpe i det hele tatt. Det kan være en måte å pakke ut ved å bruke noen funksjonelle programmeringskonsepter i PHP, men kanskje senere. Vi kan gjøre noe enklere først.
funksjon erPrime ($ num, $ pf = null) if (! is_array ($ pf)) return checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); ellers $ pfCount = count ($ pf); for ($ i = 0; $ i < $pfCount; $i++) if (isDivisible($num, $pf[$i])) return false; return true; function checkDivisorsBetween($start, $end, $num) for ($i = $start; $i < $end; $i++) if (isDivisible($num, $i)) return false; return true;
Ekstrahering av til sløyfen som helhet er litt lettere, men når vi prøver å gjenbruke vår ekstraherte metode i den andre delen av hvis vi kan se at det ikke vil fungere. Det er dette mystiske $ pf variabel som vi nesten ikke vet om.
Det ser ut til at det kontrolleres om tallet er delbart med et sett av bestemte divisorer i stedet for å ta alle tall opp til den andre magiske verdien bestemt av intval (sqrt ($ num)). Kanskje vi kunne gi nytt navn $ pf inn i $ divisorene.
funksjon erPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); ellers return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funksjonskontrollDivisorsBetween ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Dette er en måte å gjøre det på. Vi la til en ekstra valgfri parameter til vår kontrollmetode. Hvis den har en verdi, bruker vi den, ellers bruker vi $ i.
Kan vi trekke ut noe annet? Hva med dette koden: intval (sqrt ($ num))?
funksjon erPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBetween (2, integerRootOf ($ num), $ num); ellers return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funksjon integerRootOf ($ num) return intval (sqrt ($ num));
Er det ikke så bra? Noe. Det er bedre hvis personen som kommer etter oss ikke vet hva intval () og sqrt () gjør det, men det hjelper ikke å gjøre logikken enklere å forstå. Hvorfor slutter vi vår til loop på det bestemte nummeret? Kanskje dette er spørsmålet vårt funksjonsnavn skal svare på.
[PHP] // Sjekk om et tall er primærfunksjonen erPrime ($ num, $ divisors = null) if (! Is_array ($ divisors)) return checkDivisorsBetween (2, highestPossibleFactor ($ num), $ num); ellers return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funksjon highestPossibleFactor ($ num) return intval (sqrt ($ num)); [PHP]
Det er bedre som det forklarer hvorfor vi stopper der. Kanskje i fremtiden kan vi finne en annen formel for å bestemme det nummeret. Navngivelsen introduserte også en liten inkonsekvens. Vi kalte tallfaktorene, som er et synonym for divisors. Kanskje vi burde plukke en og bruke det bare. Jeg vil la deg gjøre omdøpet refactoring som en øvelse.
Spørsmålet er, kan vi trekke ut noe videre? Vel, vi må prøve til vi slipper. Jeg nevnte den funksjonelle programmeringssiden av PHP noen få avsnitt over. Det er to hovedfunksjonelle programmeringsegenskaper som vi enkelt kan bruke i PHP: Førsteklasses funksjoner og rekursjon. Når jeg ser en hvis uttalelse med a komme tilbake inne i a til loop, som i vår checkDivisorsBetween () metode, tenker jeg på å bruke en eller begge teknikker.
funksjonskontrollDivisorerBetween ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Men hvorfor skal vi gå gjennom en så kompleks tankeprosess? Den mest irriterende grunnen er at denne metoden gjør to forskjellige ting: Den sykler og bestemmer. Jeg vil bare at den skal sykle og legge avgjørelsen til en annen metode. En metode bør alltid gjøre en ting og gjøre det bra.
funksjonscheckDivisorerBetween ($ start, $ end, $ num, $ divisors = null) $ numberIsNotPrime = funksjon ($ num, $ divisor) hvis (erDivisible ($ num, $ divisor)) return false; ; for ($ i = $ start; $ i < $end; $i++) $numberIsNotPrime($num, $divisors ? $divisors[$i] : $i); return true;
Vårt første forsøk var å trekke ut betingelsen og returoppstillingen i en variabel. Dette er lokalt, for øyeblikket. Men koden virker ikke. Egentlig til sløyfe kompliserer ting ganske mye. Jeg har en følelse av at en liten rekursjon vil hjelpe.
funksjon checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) hvis ($ current == $ end) return true;
Når vi tenker på rekursivitet, må vi alltid begynne med de unike tilfellene. Vårt første unntak er når vi når slutten av vår rekursjon.
funksjon checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) hvis ($ current == $ end) return true; hvis (isDivisible ($ num, $ divisor)) return false;
Vårt andre eksepsjonelle tilfelle som vil bryte rekursjonen er når nummeret er delbart. Vi ønsker ikke å fortsette. Og det handler om alle de unike tilfellene.
ini_set ('xdebug.max_nesting_level', 10000); funksjonskontrollDivisorsBetween ($ start, $ end, $ num, $ divisors = null) return checkRecursiveDivisibility ($ start, $ end, $ num, $ divisors); funksjon checkRecursiveDivisibility ($ current, $ end, $ num, $ divisors) hvis ($ current == $ end) return true; hvis (erDivisible ($ num, $ divisors? $ divisors [$ current]: $ current)) return false; checkRecursiveDivisibility ($ current ++, $ end, $ num, $ divisors); Dette er et annet forsøk på å bruke rekursjon for vårt problem, men dessverre fører tilbakevendende 10.000 ganger i PHP til et krasj av PHP eller PHPUnit på systemet mitt. Så dette ser ut til å være en annen død. Men hvis det ville vært jobbet, ville det vært en fin erstatning av den opprinnelige logikken.
Utfordring
Da jeg skrev den Gyldne Mester, oversett jeg med vilje noe. La oss bare si at testene ikke dekker så mye kode som de burde. Kan du få øye på problemet? Hvis ja, hvordan vil du nærme deg det?
Siste tanker
"Extract till you drop" er en god måte å dissekere lange metoder. Det tvinger deg til å tenke på små stykker kode og å gi brikkene en hensikt ved å trekke dem ut i metoder. Jeg finner det utrolig hvordan denne enkle prosedyren sammen med hyppig omdøping kan hjelpe meg med å oppdage at noen kode gjør ting jeg aldri trodde var mulig.
I vår neste og siste opplæring om refactoring, vil vi bruke denne teknikken til trivia spillet. Jeg håper du likte denne opplæringen som viste seg å være litt annerledes. I stedet for å snakke om eksempler på lærebok, tok vi noen ekte kode, og vi måtte kjempe med de virkelige problemene vi møter hver dag.
Share
Accentsconagua
