Dat zal ons in staat stellen om de code in het tweede deel als volgt te hergebruiken:
else $ pfCount = count ($ pf); voor ($ i = 0; $ i<$pfCount;$i++) if(isDivisible($num, $pf[$i])) return false; return true;
En zodra we met deze code begonnen te werken, merkten we dat deze onzorgvuldig is uitgelijnd. Bretels staan soms aan het begin van de lijn, andere keren aan het einde.
Soms worden tabbladen gebruikt voor inspringen, soms spaties. Soms zijn er spaties tussen operand en operator, soms niet. En nee, dit is niet speciaal gemaakte code. Dit is het echte leven. Echte code, geen kunstmatige oefening.
// Controleer of een nummer de primaire functie isPrime ($ 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;
Dat ziet er beter uit. Meteen de twee als uitspraken lijken erg op elkaar. Maar we kunnen ze niet extraheren vanwege de terugkeer statements. Als we niet terugkeren, breken we de logica.
Als de geëxtraheerde methode een booleaanse waarde oplevert en we deze vergelijken om te beslissen of we wel of niet moeten terugkeren isPrime (), dat zou helemaal niet helpen. Er kan een manier zijn om het uit te pakken door enkele functionele programmeerconcepten in PHP te gebruiken, maar misschien later. We kunnen eerst iets eenvoudiger doen.
function isPrime ($ num, $ pf = null) if (! is_array ($ pf)) return checkDivisorsBtween (2, intval (sqrt ($ num)), $ num); else $ pfCount = count ($ pf); voor ($ 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;
De .extracten voor loop als geheel is een beetje eenvoudiger, maar als we onze uitgepakte methode opnieuw proberen te gebruiken in het tweede deel van de als we kunnen zien dat het niet werkt. Er is een mysterieus $ pf variabele waarover we bijna niets weten.
Het lijkt erop dat het controleert of het nummer deelbaar is door een reeks specifieke delers in plaats van alle getallen op te nemen tot de andere magische waarde bepaald door intval (sqrt ($ num)). Misschien kunnen we de naam wijzigen $ pf in $ delers.
function isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBtween (2, intval (sqrt ($ num)), $ num); else return checkDivisorsBetween (0, count ($ divisors), $ num, $ delers); functie checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Dit is een manier om het te doen. We hebben een vierde, optionele parameter aan onze controlemethode toegevoegd. Als het een waarde heeft, gebruiken we het, anders gebruiken we $ i.
Kunnen we nog iets extraheren? Hoe zit het met dit stuk code: intval (sqrt ($ num))?
function isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBtween (2, integerRootOf ($ num), $ num); else return checkDivisorsBetween (0, count ($ divisors), $ num, $ delers); function integerRootOf ($ num) return intval (sqrt ($ num));
Is dat niet beter? Iets. Het is beter als de persoon die achter ons aan komt weet niet wat intval () en sqrt () doen, maar het helpt niet om de logica begrijpelijker te maken. Waarom eindigen we onze voor loop op dat specifieke nummer? Misschien is dit de vraag die onze functienaam zou moeten beantwoorden.
[PHP] // Controleer of een nummer de primaire functie isPrime ($ num, $ divisors = null) if (! Is_array ($ delers)) return checkDivisorsBetween (2, highestPossibleFactor ($ num), $ num); else return checkDivisorsBetween (0, count ($ divisors), $ num, $ delers); function highestPossibleFactor ($ num) return intval (sqrt ($ num)); [PHP]
Dat is beter omdat het verklaart waarom we daar stoppen. Misschien kunnen we in de toekomst een andere formule uitvinden om dat aantal te bepalen. De naamgeving introduceerde ook een beetje inconsistentie. We noemden de getallenfactoren, wat een synoniem is van delers. Misschien moeten we er een kiezen en die alleen gebruiken. Ik zal je de hernoemende refactoring laten maken als een oefening.
De vraag is, kunnen we verder nog extraheren? Welnu, we moeten het proberen tot we erbij neervallen. Ik noemde de functionele programmeerzijde van PHP enkele paragrafen hierboven. Er zijn twee belangrijke functionele programmeerkenmerken die we eenvoudig kunnen toepassen in PHP: eersteklasfuncties en recursie. Wanneer ik een zie als verklaring met een terugkeer in een voor loop, zoals in onze checkDivisorsBetween () methode, denk ik aan het toepassen van een of beide technieken.
function checkDivisorsTussen ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Maar waarom zouden we door zo'n complex denkproces gaan? De meest vervelende reden is dat deze methode twee verschillende dingen doet: het cycli en het beslist. Ik wil dat het alleen maar fietst en laat de beslissing over aan een andere methode. Een methode zou altijd één ding moeten doen en het goed doen.
functie checkDivisorsTussen ($ start, $ end, $ num, $ delers = null) $ numberIsNotPrime = function ($ num, $ divisor) if (isDivisible ($ num, $ deler)) return false; ; voor ($ i = $ start; $ i < $end; $i++) $numberIsNotPrime($num, $divisors ? $divisors[$i] : $i); return true;
Onze eerste poging was om de voorwaarde en de return-verklaring in een variabele te extraheren. Dit is momenteel lokaal. Maar de code werkt niet. Eigenlijk het voor lus compliceert dingen best een beetje. Ik heb het gevoel dat een beetje recursie zal helpen.
functie checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) if ($ current == $ end) return true;
Als we nadenken over recursiviteit, moeten we altijd beginnen met de uitzonderlijke gevallen. Onze eerste uitzondering is wanneer we het einde van onze recursie hebben bereikt.
functie checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) if ($ current == $ end) return true; if (isDivisible ($ num, $ deler)) return false;
Ons tweede uitzonderlijke geval dat de recursie zal doorbreken, is wanneer het getal deelbaar is. We willen niet doorgaan. En dat gaat over alle uitzonderlijke gevallen.
ini_set ('xdebug.max_nesting_level', 10000); function checkDivisorsTussen ($ start, $ end, $ num, $ delers = null) return checkRecursiveDivisibility ($ start, $ end, $ num, $ delers); function checkRecursiveDivisibility ($ current, $ end, $ num, $ divisors) if ($ current == $ end) return true; if (isDivisible ($ num, $ delers? $ delers [$ current]: $ current)) return false; checkRecursiveDivisibility ($ current ++, $ end, $ num, $ delers); Dit is een andere poging om recursie voor ons probleem te gebruiken, maar helaas, 10.000 keer terugkerende in PHP leidt tot een crash van PHP of PHPUnit op mijn systeem. Dus dit lijkt weer een doodlopende weg te zijn. Maar als het zou hebben gewerkt, zou het een mooie vervanging van de oorspronkelijke logica zijn geweest.
Uitdaging
Toen ik de Gouden Meester schreef, heb ik bewust iets over het hoofd gezien. Laten we zeggen dat de tests niet zoveel code bevatten als zou moeten. Zie je het probleem? Zo ja, hoe zou u het benaderen??
Laatste gedachten
"Extract till you drop" is een goede manier om lange methoden te ontleden. Het dwingt je om na te denken over kleine stukjes code en om de stukjes een doel te geven door ze te extraheren in methoden. Ik vind het verbazingwekkend hoe deze eenvoudige procedure, samen met frequente hernoemen, mij kan helpen ontdekken dat sommige codes dingen doen die ik nooit voor mogelijk had gehouden.
In onze volgende en laatste tutorial over refactoring passen we deze techniek toe op het Trivia-spel. Ik hoop dat je deze tutorial leuk vond die een beetje anders bleek te zijn. In plaats van te praten over tekstvoorbeelden, hebben we een aantal echte codes gebruikt en moesten we vechten tegen de echte problemen waarmee we elke dag worden geconfronteerd.
Share
Accentsconagua
