Acest lucru ne va permite să reutilizăm codul în a doua parte astfel:
altceva $ pfCount = count ($ pf); pentru ($ i = 0; $ i<$pfCount;$i++) if(isDivisible($num, $pf[$i])) return false; return true;
Și de îndată ce am început să lucrăm cu acest cod, am observat că este aliniat neglijent. Bretele sunt uneori la începutul liniei, alteori la sfârșit.
Uneori, filele sunt folosite pentru indentare, uneori pentru spații. Uneori există spații între operand și operator, uneori nu. Și nu, acesta nu este un cod special creat. Aceasta este viața reală. Cod real, nu un exercițiu artificial.
// Verificați dacă un număr este funcția prime isPrime ($ num, $ pf = null) if (! Is_array ($ pf)) pentru ($ 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;
Asta arata mai bine. Imediat cele două dacă afirmațiile arată foarte asemănătoare. Dar nu le putem extrage datorită întoarcere declarații. Dacă nu vom reveni, vom rupe logica.
Dacă metoda extrasă va returna un boolean și o vom compara pentru a decide dacă ar trebui sau nu să revenim isPrime (), care nu ar ajuta deloc. Este posibil să existe o modalitate de a le extrage folosind câteva concepte de programare funcționale în PHP, dar poate mai târziu. Putem face mai întâi ceva mai simplu.
funcția isPrime ($ num, $ pf = null) if (! is_array ($ pf)) întoarcere checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); altceva $ pfCount = count ($ pf); pentru ($ 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;
Extragerea pentru buclă ca un întreg este un pic mai ușor, dar când încercăm să reutilizăm metoda noastră extrasă în a doua parte a dacă vedem că nu va funcționa. Există acest misterios $ pf variabilă despre care nu știm aproape nimic.
Se pare că verifică dacă numărul este divizibil de un set de divizori specifici, în loc să se ia toate numerele până la valoarea cea mai mică determinată de intval (sqrt ($ num)). Poate putem redenumi $ pf în $ divizori.
funcția isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) întoarcere checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); altceva return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funcția checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) pentru ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Aceasta este o modalitate de ao face. Am adăugat un parametru opțional, opțional, metodei noastre de verificare. Dacă are valoare, îl folosim, altfel vom folosi $ i.
Putem extrage orice altceva? Ce zici de această bucată de cod: intval (sqrt ($ num))?
funcția isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBetween (2, integerRootOf ($ num), $ num); altceva return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funcția integerRootOf ($ num) retur intval (sqrt ($ num));
Nu e mai bine? Oarecum. Este mai bine dacă persoana care vine după noi nu știe ce intval () și sqrt () fac, dar nu ajută la înțelegerea mai ușoară a logicii. De ce ne oprim pentru buclă la acel număr specific? Poate că aceasta este întrebarea pe care ar trebui să o răspundă numele funcției noastre.
[PHP] // Verificați dacă un număr este principala funcție isPrime ($ num, $ divisors = null) if (! Is_array ($ divisors)) return checkDivisorsBetween (2, highestPossibleFactor ($ num), $ num); altceva return checkDivisorsBetween (0, count ($ divisors), $ num, $ divisors); funcția highestPossibleFactor ($ num) întoarcere intval (sqrt ($ num)); [PHP]
Asta e mai bine, deoarece explică de ce ne oprim acolo. Poate că în viitor vom putea inventa o altă formulă pentru a determina numărul respectiv. Denumirea a introdus și o mică inconsistență. Am numit numerele factorilor, care este un sinonim al divizoarelor. Poate ar trebui să alegem una și să folosim doar asta. Vă voi lăsa să faceți ca refacerea redenumitului să fie un exercițiu.
Întrebarea este, putem extrage mai departe? Ei bine, trebuie să încercăm să scăpăm. Am menționat partea de programare funcțională a PHP câteva paragrafe de mai sus. Există două caracteristici principale de programare funcționale pe care le putem aplica cu ușurință în PHP: funcții de primă clasă și recursivitate. Ori de câte ori văd un dacă declarație cu întoarcere în interiorul a pentru bucla, ca în cazul nostru checkDivisorsBetween () metodă, mă gândesc la aplicarea uneia sau a ambelor tehnici.
funcția checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) pentru ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Dar de ce ar trebui să trecem printr-un astfel de proces de gândire complex? Motivul cel mai enervant este că această metodă are două lucruri distincte: ciclurile și decizia. Vreau doar să ciclu și să las decizia la o altă metodă. O metodă trebuie să facă întotdeauna un singur lucru și să o facă bine.
funcția ($ num, $ divisor) if (este divizibilă ($ num, $ divisor)) return false; ; pentru ($ i = $ start; $ i < $end; $i++) $numberIsNotPrime($num, $divisors ? $divisors[$i] : $i); return true;
Prima noastră încercare a fost să extragem condiția și declarația de returnare într-o variabilă. Acest lucru este local, pentru moment. Dar codul nu funcționează. De fapt, pentru buclă complică lucrurile destul de puțin. Am sentimentul că o mică recursivitate vă va ajuta.
Funcția checkRecursiveDivisibility ($ curent, $ sfârșit, $ num, $ divisor) if ($ curent == $ end) return true;
Când ne gândim la recursivitate trebuie să începem întotdeauna cu cazurile excepționale. Prima noastră excepție este când ajungem la sfârșitul recursului nostru.
Funcția checkRecursiveDivisibility ($ curent, $ sfârșit, $ num, $ divisor) if ($ curent == $ end) return true; dacă (esteDivizibil ($ num, $ divisor)) return false;
Cel de-al doilea caz excepțional care va rupe recursiunea este atunci când numărul este divizibil. Nu vrem să continuăm. Și asta e despre toate cazurile excepționale.
ini_set ('xdebug.max_nesting_level', 10000); funcția checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) retur checkRecursiveDivisibility ($ start, $ end, $ num, $ divisors); funcția checkRecursiveDivisibility ($ curent, $ end, $ num, $ divisors) dacă ($ curent == $ end) return true; dacă (esteDivizibil ($ num, $ divisors? $ divisors [$ current]: $ current)) return false; checkRecursiveDivisibility ($ curent ++, $ end, $ num, $ divisors); Aceasta este o altă încercare de a folosi recursul pentru problema noastră, dar, din păcate, repetarea de 10.000 de ori în PHP duce la un accident de PHP sau PHPUnit pe sistemul meu. Deci, pare a fi un alt punct mort. Dar dacă ar fi funcționat, ar fi fost o înlocuire plăcută a logicii originale.
Provocare
Când am scris Maestrul de Aur, am uitat intenționat ceva. Să spunem doar că testele nu acoperă codul așa cum ar trebui. Puteți observa problema? Dacă da, cum v-ați apropia?
Gândurile finale
"Extras până când picătură" este o modalitate bună de a diseca metode lungi. Ea te forțează să te gândești la mici bucăți de cod și să dai piesele un scop prin extragerea lor în metode. Mi se pare uimitor cum această procedură simplă, împreună cu redenumirea frecventă, mă poate ajuta să descopăr că unele coduri fac lucruri pe care nu le-am crezut niciodată.
În următorul nostru și ultimul tutorial despre refactorizare, vom aplica această tehnică jocului trivia. Sper că ți-a plăcut acest tutorial care sa dovedit a fi puțin diferit. În loc să vorbim despre exemple de manuale, am luat un adevărat cod și trebuia să ne luptăm cu problemele reale cu care ne confruntăm în fiecare zi.
Share
Accentsconagua
