Test Driven Development, Task 14.1.2 [71104 vs redcloud, GIUSTO IN RITARDO]

[71104]

allora, richiesta del task:

Ogni volta che una coppia di gemme viene droppata si guarda il valore del numero di Stone in Arrivo si inserisce un numero pari di Stone all'interno della griglia. L'inserimento avviene dalla prima colonna a sinistra. Ad ogni Stone inserita si scorrono le colonne verso destra. Quando si raggiunge il fondo si riparte dalla colonna più a sinistra. Durante l'inserimento la gravità è settata come "accelerata".

test list:

- Controllo della presenza di incoming stone prima della caduta di una nuova gemspair
- Riempimento con stone delle colonne da sinistra verso destra
- Controllo tra quantità stone e colonne, disposizione su più righe
- Controllo della gravità

adesso fatemi pensare al primo test, mumble...

[71104]

dopo una breve spiegazione a redcloud sul TDD e sul Pair Programming, ecco il primo test:

    public void testCheckIncomingStones()
    {
        playField.addIncomingStones(5);
        assertEquals(playField.getIncomingStones(), 5);
    }

si trova in TestPlayField e richiede l'aggiunta di un metodo per sapere quante incoming stones ci sono in un dato momento.

[71104]

redcloud ha risolto quel test aggiungendo a PlayField:

    public int getIncomingStones()
    {
        return warningBox.getCounter();
    }

come inizio di TDD non c'è male per lui
la prima soluzione che era venuta in mente ad entrambi esaminando il codice di playField era una variabile globale, lui invece ne ha trovata una più semplice

ragazzi, purtroppo devo dire che stasera abbiamo scritto pochissimo codice a causa del fatto che redcloud era al suo primo pair ma anche a causa del fatto che non esaminavo il codice da parecchio; adesso però siamo partiti e abbiamo capito come fare: da adesso andiamo molto più spediti.

per stasera facciamo solo altri due test e relative implementazioni, ma contiamo di finire entro domenica; io domani esco di sera e di pomeriggio sono a casa, se sabato pomeriggio va bene anche a redcloud (lo so, sto chiedendo qualcosa di infame ^_^) possiamo finire anche domani.

nel frattempo ecco il prossimo semplicissimo test:

    public void testInsertNoStones()
    {
        controller.insertStones(0);
        assertFalse(controller.getGrid().isGemAt(0, 0));
    }

serve a testare una prima funzionalità di un ipotetico metodo insertStones che riceve in input il numero di pietre da inserire nella griglia; questo metodo dovrà essere richiamato da GridController.update() se opportuno.

[redcloud]

Oggi pomeriggio ci sarò

[71104]

Originariamente inviato da redcloud

Oggi pomeriggio ci sarò

eh, meglio così, altrimenti non ci sarebbero state più le mie ditine

cmq, soluzione di redcloud all'ultimo test:

    public void insertStones(int stoneNumber)
    {
        ;
    }

mi pare giusto ^^

prossimo test:

    public void testInsertFirstStone()
    {
        controller.insertStones(1);
        assertTrue(controller.getGrid().isGemAt(0, 0));
        assertFalse(controller.getGrid().isGemAt(0, 1));
    }

PS: redcloud, la prossima volta, cioè domani pomeriggio, ti farò un esempio concreto di triangolazione

[redcloud]

Eeeee ormai ho trovato un passatempo produttivo! Addio ogame! Grazie tante 7165512

[71104]

Originariamente inviato da redcloud

Eeeee ormai ho trovato un passatempo produttivo! Addio ogame! Grazie tante 7165512

allora, soluzione di redcloud:

    public void insertStones(int stoneNumber)
    {
        if(stoneNumber==0)
        {
            ;
        }
        else
        {
            grid.insertGem(0, 0, Stone.create(DroppableColor.EMERALD));    
        }
    }

bene bene bene: a domani la triangolazione e la consistenza del colore della pietra con quello indicato nel pattern
e a seguire a ruota l'inserzione delle pietre successive all'ottava.

notte a tutti

[thebol]

stamattina prima di procedere mi puoi conttattare su msn?(sono b0l_padawan, ho gia provato a contattarti)

devo committare una modifica grossa su gridController e voglio farla prima che continuiate a toccarla se no è un macello

[thebol]

la modifica era piccola sono riuscito ad arrangiarmi per i fatti miei ed ho committato. Cmq continuo lo sviluppo del pattern, se riprendete fatemi sapere.

[Ufo13]

Originariamente inviato da 71104

dopo una breve spiegazione a redcloud sul TDD e sul Pair Programming, ecco il primo test:

    public void testCheckIncomingStones()
    {
        playField.addIncomingStones(5);
        assertEquals(playField.getIncomingStones(), 5);
    }

si trova in TestPlayField e richiede l'aggiunta di un metodo per sapere quante incoming stones ci sono in un dato momento.

Potete aggiungere ancora un test su questa cosa per favore?

Passa anche se gli fai ritornare 5 direttamente

[Bonfo]

Quando io e RedCloud avevamo iniziato questo task Ufo aveva chiesto se era possibile non complicare Grid e GrodController.
In grid è già presente il metodo insert insertStoneInColumn()...perchè non usate e modficate quello??

SOprattutto questo metodo è accedibile direttamente da playField:

playField.getGridController().getGrid().insertStoneInColumn()

Il che vi risparmierebbe il getIncomingStones().

Pensateci se è una cosa che vi piace o no.
A me piace moooolto

[71104]

eh Bonfo, sto metodo non l'avevamo neanche notato ^^
grazie per avercelo fatto notare, il codice di Diamond Crush è doventato piuttosto vasto e quando si pensa ai test tocca pure cercare i metodi giusti in mezzo a tanti...

[71104]

Originariamente inviato da Ufo13

Potete aggiungere ancora un test su questa cosa per favore?

Passa anche se gli fai ritornare 5 direttamente

giustissimo, l'avevo anche fatto notare a redcloud ma non ci eravamo preoccupati di essere troppo pignoli perché si trattava di un metodo semplicissimo; l'altro test lo metterei se non fosse che forse getIncomingStones va eliminato, come suggerito da Bonfo...
vedremo dove ci porterà il codice.

[71104]

scusate, ma il metodo insertStoneInColumn perché l'avete messo visto che non lo usa nessuno...? (Eclipse non trova riferimenti)
era stato messo apposta per questo task?

[71104]

Originariamente inviato da 71104

scusate, ma il metodo insertStoneInColumn perché l'avete messo visto che non lo usa nessuno...? (Eclipse non trova riferimenti)
era stato messo apposta per questo task?

redcloud sta modificando l'implementazione di insertStones affinché utilizzi il metodo insertStoneInColumn al posto di insertGem; nel frattempo ecco il prossimo test:

    public void testInsertNextStones()
    {
        controller.insertStones(2);
        assertTrue(controller.getGrid().isGemAt(0, 0));
        assertTrue(controller.getGrid().isGemAt(0, 1));
        assertFalse(controller.getGrid().isGemAt(0, 2));
    }

redcloud, questa volta ti tocca il for

[redcloud]

Per superare

public void testInsertNextStones()
{
    controller.insertStones(2);
    assertTrue(controller.getGrid().isGemAt(0, 0));
    assertTrue(controller.getGrid().isGemAt(0, 1));
    assertFalse(controller.getGrid().isGemAt(0, 2));
}

ho utilizzato

public void insertStones(int stoneNumber)
{
    for(int i=0; i<stoneNumber; i++)
    {
        grid.insertGem(0, i, Stone.create(DroppableColor.EMERALD));
    }
}

nella classe GridController

[71104]

bene, adesso vediamo i colori:

    public void testStoneColor()
    {
        controller.insertStones(8, pattern);
        for(int i = 0; i < 8; i++) {
            assertEquals(grid.getGemAt(0, i).getColor(), pattern.getDroppableColor(i));
        }
    }

[fek]

Originariamente inviato da 71104

scusate, ma il metodo insertStoneInColumn perché l'avete messo visto che non lo usa nessuno...? (Eclipse non trova riferimenti)
era stato messo apposta per questo task?

Guarda, neppure ci penso, Eclipse segnala un metodo che non e' usato da nessuna parte e io lo uccido automaticamente.

Se tutti dovessimo lasciare metodi che seriviranno in futuro, il codice diventa grande il doppio. Nel repository voglio solo codice che e' usato, e che e' testato.

[71104]

ho spostato tutti i test di questo task in TestIncomingStones, tranne quello per getIncomingStones che si trova ancora in mezzo a quelli per PlayField.

[71104]

Originariamente inviato da fek

Guarda, neppure ci penso, Eclipse segnala un metodo che non e' usato da nessuna parte e io lo uccido automaticamente.

Se tutti dovessimo lasciare metodi che seriviranno in futuro, il codice diventa grande il doppio. Nel repository voglio solo codice che e' usato, e che e' testato.

ok ^^
oltrettutto redcloud ha anche provato ad usarlo in insertStones, ma non è riuscito a farlo funzionare...

EDIT: lo eliminerò dopo perché la sua eliminazione comporta anche qualche correzione in alcuni test che lo usano (non per testarlo però, almeno non tutti); per ora quello che ci preme è di finire quest'altro test perché tra poco devo uscire O.o'