[CICLO 3] Test Driven Development task 3.2.6 (fpitt vs cisc)

[cisc]

allora, il task è:

Spostare la gestione della gravita e le collisioni dei diamanti nella classe che gestisce la griglia

che mi sembra un po impreciso, e modificherei (secondo l'interpretazione suggeritami da fek) in:

Spostare la gestione della gravita e le collisioni dei diamanti a livello di cella


pensiamo prima alle collisioni:
quindi, il primo test:

    public void testCellBottomCollision()
    {
        int gridColumns = 8;
        int gridRows = 14;
        
        Bounds bounds = new Bounds(40f, 40f, 256f, 448f, Config.createForTesting());
        Grid grid = new Grid(gridRows, gridColumns, bounds);
        
        Gem gem = new Gem("diamond", bounds);
        int yStep = (int)(float)bounds.getHeight() / gridRows;

        grid.setGravity(gem.getGemSize());   
        
        grid.insertGem(13,4,gem);
        
        grid.insertGem(11,4,gem);
        grid.moveAllGems(0, 2 * yStep);
        assertTrue (grid.getGemAt(12,4).isDropped());
    }

a te fpitt, io vado a mangiare

EDIT: ho modificato il test

[cisc]

l'avevo detto che ero pieno di impegni in questi giorni..ed infatti, fpitt, ti devo lasciare, ci aggiorniamo a domani sera

[fpitt]

Ok cisc, a domani sera

[fek]

Ragazzi, avete lasciato tutta la notte la build rotta. Questo e' molto grave.

[fek]

Qualcuno puo' fare il revert degl'ultimi due commit per favore? La build e' rotta e questo non deve MAI essere lasciato stare per lungo tempo perche' significa che nessuno puo' lavorare.

[cionci]

Ora funziona...

[cionci]

Per la collisione fra diamanti, è molto semplice...
Non stravolgete il codice perchè basta modificare una sola funzione...

Ci sarebbe anche un altro test da fare... L'inserimento su una cella già occupata deve portare ad una eccezione...

[fek]

Grazie per l'averlo aggiustato

Originariamente inviato da cionci

Per la collisione fra diamanti, è molto semplice...
Non stravolgete il codice perchè basta modificare una sola funzione...

Mi raccomando ragazzi, sempre la soluzione piu' semplice.

Ci sarebbe anche un altro test da fare... L'inserimento su una cella già occupata deve portare ad una eccezione...

Mi sembrava io e 71104 avessimo scritto il test sull'inserimento in una cella gia' occupata.

[cionci]

fek: io non aggiustato niente
Allora me lo sono perso

[fek]

Hmmm, allora sara' un problema della build machine. Quando arrivo a casa ci guardo.

[cisc]

io avevo pensato a modificare il Bounds di Gem ad un quadrato intorno alla cella corrente, che viene ridotto in caso di presenza di altri diamanti vicini, cmq, il test che non passa al momento è nel package it.diamonds.tests.ignore, la build machine considera anche questo package?

[cionci]

Io invece penserei di farla il più semplice possibile (vediamo intanto di far passare il test)... Guarda updateGemPosition... Come faresti a far passare il test (solo il test) ? Basta un if...
PS: consiglio di invertire l'ordine dei for per cominciare a disegnare le gemme a partire dal fondo della griglia
PPS: non è detto che la gemma vada mossa prima...

[cisc]

vabbè, in TDD si andrà un passo alla volta, come dici tu cionci, io parlavo della soluzione finale..

[fek]

Originariamente inviato da cisc

vabbè, in TDD si andrà un passo alla volta, come dici tu cionci, io parlavo della soluzione finale..

La soluzione finale uscira' un passo per volta e dal refactoring. Ora preoccupatevi del test.

[cisc]

cmq cionci, non mi sembra che si possa risolvere con un if, dopo tutto nel test io chiedo alla gemma se ha colliso.

[cisc]

modifico il test, per rendere lo sviluppo più regolare:

    public void testCellBottomCollision()
    {
        int gridColumns = 8;
        int gridRows = 14;
        
        Bounds bounds = new Bounds(40f, 40f, 256f, 448f, Config.createForTesting());
        Grid grid = new Grid(gridRows, gridColumns, bounds);
        
        Gem gem = new Gem("diamond", bounds);
        int yStep = (int)(float)bounds.getHeight() / gridRows;

        grid.setGravity(gem.getGemSize());   
        
        grid.insertGem(13,4,gem);
        
        grid.insertGem(11,4,gem);
        grid.moveAllGems(0, 2 * yStep);
        assertTrue (grid.gemIsDropped(12.4));
    }

a te fpitt

[fek]

Non mi e' chiaro il significato del metodo moveAllGems(). Chi lo puo' spiegare meglio?

Non mi e' neppure chiaro il significato di swapGrid.

[cionci]

Muove tutte le gemme presenti nel grid dello step scelto...in pratica tutte le gemme, se non sono dropped, reagiscono all'input (viene appunto richiamato da reactToInput)...

[cisc]

già, una spiegazione su swapGrid sarebbe utile...

[fek]

Originariamente inviato da cionci

Muove tutte le gemme presenti nel grid dello step scelto...in pratica tutte le gemme, se non sono dropped, reagiscono all'input (viene appunto richiamato da reactToInput)...

Non ho capito, scusami

Il codice non lo chiarisce molto, in alcune parti e' un po' oscuro. In particolare non e' chiaro da dove derivi la necessita' di avere una seconda griglia alternativa e non si possa fare la stessa operazione sulla griglia principale.

Facciamo un po' di refactoring assieme?