Questo ci consentirà di riutilizzare il codice nella seconda parte in questo modo:
else $ pfCount = count ($ pf); for ($ i = 0; $ i<$pfCount;$i++) if(isDivisible($num, $pf[$i])) return false; return true;
E non appena abbiamo iniziato a lavorare con questo codice, abbiamo osservato che è allineato incautamente. Le parentesi graffe sono a volte all'inizio della linea, altre volte alla fine.
A volte, le schede vengono utilizzate per il rientro, a volte spazi. A volte ci sono spazi tra operandi e operatori, a volte no. E no, questo non è un codice appositamente creato. Questa è la vita reale. Codice reale, non un esercizio artificiale.
// Controlla se un numero è una funzione primaria 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;
Sembra migliore Immediatamente i due Se le dichiarazioni sembrano molto simili. Ma non possiamo estrarli a causa del ritorno dichiarazioni. Se non torniamo, infrangeremo la logica.
Se il metodo estratto restituisce un valore booleano e lo confrontiamo per decidere se dovremmo o non tornare da isPrime (), quello non aiuterebbe affatto. Potrebbe esserci un modo per estrarlo usando alcuni concetti di programmazione funzionale in PHP, ma forse dopo. Possiamo fare qualcosa di più semplice prima.
function isPrime ($ num, $ pf = null) if (! is_array ($ pf)) return checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); else $ pfCount = count ($ pf); per ($ 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;
Estrazione del per il ciclo nel suo complesso è un po 'più semplice, ma quando proviamo a riutilizzare il nostro metodo estratto nella seconda parte del Se possiamo vedere che non funzionerà. C'è questo misterioso $ pf variabile su cui non sappiamo quasi nulla.
Sembra che controlli se il numero è divisibile da un insieme di divisori specifici invece di prendere tutti i numeri fino all'altro valore magico determinato da intval (sqrt ($ num)). Forse potremmo rinominare $ pf in $ divisori.
function isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBetween (2, intval (sqrt ($ num)), $ num); else return check Divisors Between (0, count ($ divisors), $ num, $ divisors); function checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Questo è un modo per farlo. Abbiamo aggiunto un quarto parametro opzionale al nostro metodo di controllo. Se ha un valore, lo usiamo, altrimenti usiamo $ i.
Possiamo estrarre qualcos'altro? Che dire di questo pezzo di codice: intval (sqrt ($ num))?
function isPrime ($ num, $ divisors = null) if (! is_array ($ divisors)) return checkDivisorsBetween (2, integerRootOf ($ num), $ num); else return check Divisors Between (0, count ($ divisors), $ num, $ divisors); function integerRootOf ($ num) return intval (sqrt ($ num));
Non è meglio? Un po '. È meglio se la persona che viene dopo di noi non sa cosa intval () e sqrt () stanno facendo, ma non aiuta a rendere la logica più facile da capire. Perché finiamo il nostro per loop a quel numero specifico? Forse questa è la domanda alla quale dovrebbe rispondere il nostro nome di funzione.
[PHP] // Controlla se un numero è funzione primaria isPrime ($ num, $ divisors = null) if (! Is_array ($ divisori)) return checkDivisorsBetween (2, highestPossibleFactor ($ num), $ num); else return check Divisors Between (0, count ($ divisors), $ num, $ divisors); function highestPossibleFactor ($ num) return intval (sqrt ($ num)); [PHP]
È meglio perché spiega perché ci fermiamo qui. Forse in futuro possiamo inventare una formula diversa per determinare quel numero. La denominazione ha anche introdotto un po 'di incoerenza. Abbiamo chiamato i fattori numerici, che è sinonimo di divisori. Forse dovremmo sceglierne uno e usarlo solo Ti lascerò fare il refactoring del renaming come esercizio.
La domanda è, possiamo estrarre ulteriormente? Bene, dobbiamo provarci finché non cadiamo. Ho citato il lato della programmazione funzionale di PHP alcuni paragrafi sopra. Ci sono due principali caratteristiche di programmazione funzionale che possiamo facilmente applicare in PHP: funzioni di prima classe e ricorsione. Ogni volta che vedo un Se dichiarazione con a ritorno dentro a per loop, come nel nostro checkDivisorsBetween () metodo, penso di applicare una o entrambe le tecniche.
function checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) for ($ i = $ start; $ i < $end; $i++) if (isDivisible($num, $divisors ? $divisors[$i] : $i)) return false; return true;
Ma perché dovremmo affrontare un processo di pensiero così complesso? La ragione più fastidiosa è che questo metodo fa due cose distinte: cicli e decide. Voglio solo andare in bicicletta e lasciare la decisione a un altro metodo. Un metodo dovrebbe sempre fare una cosa sola e farlo bene.
function checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) $ numberIsNotPrime = function ($ num, $ divisor) if (isDivisible ($ num, $ divisor)) return false; ; per ($ i = $ inizio; $ i < $end; $i++) $numberIsNotPrime($num, $divisors ? $divisors[$i] : $i); return true;
Il nostro primo tentativo è stato quello di estrarre la condizione e l'istruzione return in una variabile. Questo è locale, per il momento. Ma il codice non funziona. In realtà il per il ciclo complica un po 'le cose. Ho la sensazione che un po 'di ricorsione aiuterà.
function checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) if ($ current == $ end) return true;
Quando pensiamo alla ricorsività, dobbiamo sempre iniziare con i casi eccezionali. La nostra prima eccezione è quando raggiungiamo la fine della nostra ricorsione.
function checkRecursiveDivisibility ($ current, $ end, $ num, $ divisor) if ($ current == $ end) return true; if (isDivisible ($ num, $ divisor)) return false;
Il nostro secondo caso eccezionale che interromperà la ricorsione è quando il numero è divisibile. Non vogliamo continuare. E questo riguarda tutti i casi eccezionali.
ini_set ('xdebug.max_nesting_level', 10000); function checkDivisorsBetween ($ start, $ end, $ num, $ divisors = null) return checkRecursiveDivisibility ($ start, $ end, $ num, $ divisors); function checkRecursiveDivisibility ($ current, $ end, $ num, $ divisors) if ($ current == $ end) return true; if (isDivisible ($ num, $ divisors? $ divisors [$ current]: $ current)) return false; checkRecursiveDivisibility ($ current ++, $ end, $ num, $ divisors); Questo è un altro tentativo di usare la ricorsione per il nostro problema, ma sfortunatamente, ricorrenti 10.000 volte in PHP portano ad un crash di PHP o PHPUnit sul mio sistema. Quindi questo sembra essere un altro vicolo cieco. Ma se avrebbe funzionato, sarebbe stata una buona sostituzione della logica originale.
Sfida
Quando ho scritto il Golden Master, ho intenzionalmente trascurato qualcosa. Diciamo solo che i test non coprono tutto il codice che dovrebbero. Riesci a individuare il problema? Se sì, come ti avvicineresti?
Pensieri finali
"Estrarre fino allo sfinimento" è un buon modo per analizzare i metodi lunghi. Ti costringe a pensare a piccoli pezzi di codice ea dare ai pezzi uno scopo estraendoli in metodi. Trovo incredibile come questa semplice procedura, insieme alla frequente rinomina, possa aiutarmi a scoprire che un codice fa cose che non avrei mai pensato possibili.
Nel nostro prossimo e ultimo tutorial sul refactoring, applicheremo questa tecnica al gioco a quiz. Spero ti sia piaciuto questo tutorial che si è rivelato un po 'diverso. Invece di parlare su esempi di libri di testo, abbiamo preso del codice reale e abbiamo dovuto combattere con i problemi reali che affrontiamo ogni giorno.
Share
Accentsconagua
