Stasera terrò un workshop sul refactoring al PUG Milano. Avete qualche
'codebase' da farmi rifattorizzare per esercizio?
Requisiti:
- deve essere mostrabile in pubblico, niente codice top secret.
- deve essere installabile/eseguibile facilmente 'as is', quindi senza
richiedere di installare altri software, magari perfino non PHP.
- se posso far ricadere qualche beneficio del mio refactoring a favore
della comunità open source tanto meglio - beneficio da valutare a
valle della sessione con una bella pull request ovvio.
- a proposito di pull request, la presenza del codice su GitHub è una
bella comodità.
- altri che potrei dimenticare ora a dispetto della loro ovvietà.
Che dite? Avete qualche dritta? Qualche bel mostriciattolo (leggi
"orribile, schifoso, ma piccolo") da domare una volta per tutte e che
avete sempre rimandato di aggredire...
> Stasera terrò un workshop sul refactoring al PUG Milano. Avete qualche
> 'codebase' da farmi rifattorizzare per esercizio?
> Requisiti:
> - deve essere mostrabile in pubblico, niente codice top secret.
> - deve essere installabile/eseguibile facilmente 'as is', quindi senza
> richiedere di installare altri software, magari perfino non PHP.
> - se posso far ricadere qualche beneficio del mio refactoring a favore
> della comunità open source tanto meglio - beneficio da valutare a
> valle della sessione con una bella pull request ovvio.
> - a proposito di pull request, la presenza del codice su GitHub è una
> bella comodità.
> - altri che potrei dimenticare ora a dispetto della loro ovvietà.
> Che dite? Avete qualche dritta? Qualche bel mostriciattolo (leggi
> "orribile, schifoso, ma piccolo") da domare una volta per tutte e che
> avete sempre rimandato di aggredire...
__/|_ We reject kings, presidents and voting.
/o ) \/ We believe in rough consensus
)__ v _/\ and running code (I.E.T.F. credo)
______________________________________________________________
> Stasera terrò un workshop sul refactoring al PUG Milano. Avete qualche
> 'codebase' da farmi rifattorizzare per esercizio?
> Requisiti:
> - deve essere mostrabile in pubblico, niente codice top secret.
> - deve essere installabile/eseguibile facilmente 'as is', quindi senza
> richiedere di installare altri software, magari perfino non PHP.
> - se posso far ricadere qualche beneficio del mio refactoring a favore
> della comunità open source tanto meglio - beneficio da valutare a
> valle della sessione con una bella pull request ovvio.
> - a proposito di pull request, la presenza del codice su GitHub è una
> bella comodità.
> - altri che potrei dimenticare ora a dispetto della loro ovvietà.
> Che dite? Avete qualche dritta? Qualche bel mostriciattolo (leggi
> "orribile, schifoso, ma piccolo") da domare una volta per tutte e che
> avete sempre rimandato di aggredire...
> __/|_ We reject kings, presidents and voting.
> /o ) \/ We believe in rough consensus
> )__ v _/\ and running code (I.E.T.F. credo)
> ______________________________________________________________
La proposta è nello spirito giusto, grazie!
Dici che ce la faccio a far girare un test verde *prima* di avviare il
refactoring entro 30-60' che ho prima dell'avvio del meeting?
Andrea, certo! *Se* trovo roba open source da rifattorizzare sarò ben
lieto - *se* soddisfatto del refactoring effettuato - di inviare una
pull request al team d'origine.
Premetto ricordandoti che sei un master jedi quindi se qualcuno ce la puo'
fare quello sei tu Luke,
al volo ho visto che non c'è un test unitario su quella funzione ma
ci sono alcuni test che coprono la hydrateAll che a sua volta chiama
hydrateAllData che a sua volta il 'the monster'
> La proposta è nello spirito giusto, grazie!
> Dici che ce la faccio a far girare un test verde *prima* di avviare il
> refactoring entro 30-60' che ho prima dell'avvio del meeting?
> Andrea, certo! *Se* trovo roba open source da rifattorizzare sarò ben
> lieto - *se* soddisfatto del refactoring effettuato - di inviare una
> pull request al team d'origine.
__/|_ We reject kings, presidents and voting.
/o ) \/ We believe in rough consensus
)__ v _/\ and running code (I.E.T.F. credo)
______________________________________________________________
> La proposta è nello spirito giusto, grazie!
> Dici che ce la faccio a far girare un test verde *prima* di avviare il
> refactoring entro 30-60' che ho prima dell'avvio del meeting?
> Andrea, certo! *Se* trovo roba open source da rifattorizzare sarò ben
> lieto - *se* soddisfatto del refactoring effettuato - di inviare una
> pull request al team d'origine.
è semplice semplice ... va tolto l'autoloader custom e usato composer
i test ci sono già ed è una lib php; volendo potresti anche stubbare
qualche servizio esterno
per far vedere come si possa isolare un test.
chiaro che non va bene se vuoi far vedere come un mostro diventa un bel
principe :-)
> è semplice semplice ... va tolto l'autoloader custom e usato composer
> i test ci sono già ed è una lib php; volendo potresti anche stubbare qualche
> servizio esterno
> per far vedere come si possa isolare un test.
> chiaro che non va bene se vuoi far vedere come un mostro diventa un bel
> principe :-)
> ottima idea Jacopo, e ottima proposta Liuggio... solo che quel metodo è un
> vero leviatano.
> Non mi ricordo chi purtroppo (forse Paolo Polce), ma c'era uno speaker
> all'agile day del 2011 che disse:
> in fondo ad un metodo così non dovresti mettere chiusa parentesi graffa,
> ma: "e vissero tutti felici e contenti" :)
se non sbaglio lo disse proprio Cirillo durante il keynote, una delle piu
belle battute degli ultimi tempi.
-- --------------------------------------
davidino
http://davidfunaro.com http://about.me/david.funaro ing.david...@gmail.com
--------------------------------------