[CICLO 3] Test Driven Development Task 3.1.2 (cisc vs fek)

[fek]

E' stato un po' tribolato ma abbiamo finito

Tutti i test passano, l'applicazione sembra ok. Tutto in ordine.

C'e' un po' di refactoring da fare qua e la', qualche nome di metodo e variabile non mi convince, ma sostanzialmente ci siamo.

Solo una piccola duplicazione mi fa dormire poco tranquillo: le classi Rectangle e Bounds si assomigliano terribilmente. Sara' il caso di eliminare questa duplicazione e usare una per implementare l'altra? Ereditarieta' o Incapsulazione? Chi si vuole occupare di questo refactoring?

[71104]

ereditarietà secondo me: Bounds che deriva da Rectangle concettualmente ce la vedo bene
se vuoi lo faccio io domani (potrò occuparmene nel tardo pomeriggio e di sera).

[cisc]

sono d'accordo per l'ereditarietà anche io

[fek]

Risposta errata per entrambi

"Favour composition over inheritance". E' un tipico caso composizione.

http://www.design-nation.net/en/archives/000423.php

[cisc]

mi veniva proprio istintiva l'ereditarietà, specialmente a vedere tutti i metodi di Rectangle implementati anche in Bounds, concettualmente però Bounds non è un Rectangle (aaa, il prof di prog orientata agli oggetti quante volte ci ha fatto sto genere di esempi), quindi effettivamente c'ha ragione (come sempre ) fek....

[71104]

e quindi che dobbiamo fare?
mettiamo una istanza di Rectangle all'interno di Bounds?

[fek]

Originariamente inviato da cisc

mi veniva proprio istintiva l'ereditarietà, specialmente a vedere tutti i metodi di Rectangle implementati anche in Bounds, concettualmente però Bounds non è un Rectangle (aaa, il prof di prog orientata agli oggetti quante volte ci ha fatto sto genere di esempi), quindi effettivamente c'ha ragione (come sempre ) fek....

Esatto, Bounds non e' un Rectangle, o meglio non segue il principio di sostituzione: non ha senso usare un Bounds ovunque sia necessario un Rectangle.

Quindi Composizione.

Un esempio nel mondo reale e' un Macchina: una Macchina "e' composta da un" Motore, ma "non e'" un Motore.

71104: Si', metti un'istanza di Rectangle dentro Bounds e poi deleghi i metodi.

Lo vogliamo fare stasera assieme io e te? Ti faccio vedere come farlo passettino per passettino sfruttando i test che abbiamo gia', un altro esempio di come vengano comodi i test. Potrebbe essere interessante.

[71104]

Originariamente inviato da fek

Lo vogliamo fare stasera assieme io e te? Ti faccio vedere come farlo passettino per passettino sfruttando i test che abbiamo gia', un altro esempio di come vengano comodi i test. Potrebbe essere interessante.

per me va bene, quando torno a casa apro un thread e se ci sei rispondi che iniziamo =)

[fek]

Stasera sono stato precettato dalla ragazza. Facciamo il refactoring domani sera? Non e' urgentissimo comunque.

[71104]

Originariamente inviato da fek

Stasera sono stato precettato dalla ragazza. Facciamo il refactoring domani sera? Non e' urgentissimo comunque.

ah ok, allora vada per domani sera