Refactoring gratis

[Jacopo Romei]

Ciao!

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...

Grazie!

--
Jacopo Romei
http://www.sviluppoagile.it/
http://twitter.com/jacoporomei
http://www.anonimarmonisti.com/

[liuggio]

Ciao non vorrei essere troppo 'cattivo' ma faresti un gran piacere alla comunità se semplificassi/rifattorizzassi
the monster:
https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php#L352-L607

2013/2/14 Jacopo Romei <jro...@gmail.com>

--
Hai ricevuto questo messaggio perché sei iscritto al gruppo "PUG Roma" di Google Gruppi.
Visita questo gruppo all'indirizzo http://groups.google.com/group/pug-roma?hl=it-IT.

--
 __________________liuggio_________________________________
 
  __/|_      We reject kings, presidents and voting.
/o )   \/    We believe in rough consensus
)__ v _/\          and running code  (I.E.T.F. credo)
______________________________________________________________

[Andrea Giuliano]

Ovviamente poi ci mandi il link della PR :)

--
Andrea Giuliano

[Jacopo Romei]

> Ciao non vorrei essere troppo 'cattivo' ma faresti un gran piacere alla
> comunità se semplificassi/rifattorizzassi
> the monster:
> https://github.com/doctrine/doctrine2/blob/master/lib/Doctrine/ORM/Internal/Hydration/ObjectHydrator.php#L352-L607

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.

[liuggio]

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'

2013/2/14 Jacopo Romei <jro...@gmail.com>

--
Hai ricevuto questo messaggio perché sei iscritto al gruppo "PUG Roma" di Google Gruppi.
Visita questo gruppo all'indirizzo http://groups.google.com/group/pug-roma?hl=it-IT.

[Eugenio Pombi]

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" :)

ho provato a cercare se ci fosse già uno straccio di test da cui partire nella suite di doctrine, ma non ho trovato nulla. Quindi alla tua domanda:

> 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?

ti direi che se ci riesci allora puoi provare anche a camminare sull'acqua e moltiplicare pizze e birre ;)

è una bella sfida

--
Hai ricevuto questo messaggio perché sei iscritto al gruppo "PUG Roma" di Google Gruppi.
Visita questo gruppo all'indirizzo http://groups.google.com/group/pug-roma?hl=it-IT.

--
Eugenio Pombi
Developer

tel:      +39.329.5455797
mail:    eux...@gmail.com
twitter: @euxpom
skype: euxeux

it.linkedin.com/pub/eugenio-pombi/3b/28b/99a

[leonardo proietti]

>Stasera terrò un workshop sul refactoring al PUG Milano. Avete qualche
>'codebase' da farmi rifattorizzare per esercizio?

https://github.com/ideatosrl/GeoAdapter

è 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 :-)

Ciao
Leonardo

[Jacopo Romei]

Grazie a tutti, ora vediamo!

Intendo improvvisare e vediamo quanto tempo consumeremo.
Il Leviatano non è ancora fuori dal radar... ;-)

2013/2/14 leonardo proietti <leonardo...@gmail.com>:

> --
> Hai ricevuto questo messaggio perché sei iscritto al gruppo "PUG Roma" di
> Google Gruppi.
> Visita questo gruppo all'indirizzo
> http://groups.google.com/group/pug-roma?hl=it-IT.
>
>

[Jacopo Romei]

Avevo pensato anche ad uno dei file di phpMyAdmin.

Non abbiamo qualcosa del genere? Un mostriciattolo procedurale da
convertire all'OO....

:-)

[David Funaro]

2013/2/14 Eugenio Pombi <eux...@gmail.com>

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.da...@gmail.com
--------------------------------------