[Ciclo 6] Pair Programming task 6.2.3 (fek vs cionci clash of the titans)

[cionci]

commento temporaneamente gli altri due test che riguardano update...

Inserisco questo codice in update di Gem:

    public void update(AbstractTimer timer)
    {
        if(timer.getTime() >= 3500)
            ++currentFrame;        
    }

il test di fek passa...

Ora però devo fare in modo di far passare gli altri test:

    public void testUpdate()
    {
        gem.addFrame(10, 10, 100);        
        
        gem.update(new MockTimer(3500));
        
        assertEquals("current frame must be 1 after an update", 1, gem.getCurrentFrame());                
    }

questo passa senza modifche al codice...

Ora passiamo al secondo:

   public void testUpdateWithThreeFrames()
    {
        gem.addFrame(10, 10, 100);
        gem.addFrame(20, 20, 100);
        
        MockTimer timer = new MockTimer(3500);

        gem.update(timer);
        
        timer.advance(1);
        gem.update(timer);
        
        assertEquals("current frame must be 1", 1, gem.getCurrentFrame());                

        timer.advance(99);
        gem.update(timer);

        assertEquals("current frame must be 2", 2, gem.getCurrentFrame());                
    
    }

Il test modifcato in questo modo non passa...

Devo lavorare sul mio codice che quindi non va bene...

[cionci]

Allora modifico addFrame(0, 0 ,0) nel costruttore in addFrame(0, 0, 3500)...

Aggiungo a gem:

private long frameChangeTimeStamp = 0;

Modifico il codice di update in questo modo:

    public void update(AbstractTimer timer)
    {
        long timeStamp = timer.getTime(); 
        if(timeStamp - frameChangeTimeStamp >= frames.get(getCurrentFrame()).getLength())
        {
            ++currentFrame;
            frameChangeTimeStamp = timeStamp;
        }
    }

il test passa...

Se è tutto ok vado a scrivere il prossimo test...

[cionci]

Il nuovo test è questo:

    public void testOneUpdateForMultipleFrames()
    {
        gem.addFrame(10, 10, 100);
        gem.addFrame(20, 20, 100);
        
        MockTimer timer = new MockTimer(3600);

        gem.update(timer);

        assertEquals("current frame must be 2 after one update", 2, gem.getCurrentFrame());                
    }

Se passa un tempo maggiore di quello necessario al cambio di un solo frame update dovrebbe portare l'animazione comunque al frame corretto...

[fek]

Ed ecco il codice che fa passare il test:

    public void update(AbstractTimer timer)
    {
        long deltaTime = 0;
        
        deltaTime = timer.getTime() - lastTime;
        currentTime += deltaTime;

        int frame = 0;        
        int frameTime = frames.get(frame).getLength();
        
        while (currentTime >= frameTime)
        {
            frameTime += frames.get(++frame).getLength();
        }
        
        lastTime = timer.getTime();
        
        currentFrame = frame;
    }

Non e' il codice migliore del mondo, ma fa il suo dovere. Un momento, ma perche' non ho scritto questo codice passo per passo, riga per riga? Perche' sono stato pigro, avevo gia' scritto esattamente questo algoritmino in passato e mi sentivo sicuro del successo. Infatti... non vi sto a riportare i penosi tentativi casuali per far passare tutti i test e le decine di minuti persi. Tutto perche' avevo dimenticato il += all'interno del loop. Eppure ero sicuro che fosse giusto.

Ho imparato la lezione, la prossima volta mi ricordero' di farlo passo passo anche se sono sicurissimo di quello che sto facendo, impieghero' certamente meno tempo di quanto ne ho impiegato questa volta, cercando di fare tutto in un passo.

[fek]

Test: se la gemma si trova al frame N-1 deve ritornare al frame 0 dopo 100 ms

    public void testCyclingAnimation()
    {
        gem.addFrame(10, 10, 1000);
        
        MockTimer timer = new MockTimer(0);
        
        gem.update(timer);
        timer.advance(3500 + 1000);
        gem.update(timer);
        
        assertEquals("current frame must be 0 after a complete cycle", 0, gem.getCurrentFrame());        
    }

Ci sono due frame nell'animazione, dopo 4500 millisecondi il frame corrente dev'essere di nuovo quello iniziale. Fallisce con un out of bound exception, come era lecito attendersi.

A te. Non fare il mio stesso errore, fallo passo per passo

[cionci]

Non c'è stato bisogno di andare passo-passo perchè ci voleva un solo passo:

    public void update(AbstractTimer timer)
    {
        long deltaTime = 0;
        
        deltaTime = timer.getTime() - lastTime;
        currentTime += deltaTime;

        int frame = 0;        
        int frameTime = frames.get(frame).getLength();
        
        while (currentTime >= frameTime)
        {
            if(frame == frames.size() - 1)
                frame = -1;
            
            frameTime += frames.get(++frame).getLength();
        }
        
        lastTime = timer.getTime();
        
        currentFrame = frame;
    }

Certo, il codice può essere rimesso in ordine, ma per ora è sufficiente

[cionci]

Passo a scrviere il test per la draw...

[cionci]

    
    public void testDrawTwoFrames()
    {
        gem.addFrame(10, 15, 100);
        gem.addFrame(20, 25, 100);
        
        MockEngine engine = MockEngine.createForTesting(800, 600);
        MockTimer timer = new MockTimer(3500);
        
        gem.update(timer);
        gem.draw(engine);
                
        assertEquals("bad texture drawn", 10.0f, engine.getTextureRect().left(), 0.001);
        assertEquals("bad texture drawn", 15.0f, engine.getTextureRect().top(), 0.001);
        
        timer.advance(100);
        gem.update(timer);
        gem.draw(engine);
        
        assertEquals("bad texture drawn", 20.0f, engine.getTextureRect().left(), 0.001);
        assertEquals("bad texture drawn", 25.0f, engine.getTextureRect().top(), 0.001);
    }

[cionci]

Porto avanti io il test...

[cionci]

Ora bisogna modificare la classe Frame per memorizzare e poter recuperare i valori delle coordinate...
Per fare questo scrivo questo test:

    public void testGetCoordinates()
    {
        Frame frame1 = new Frame(10, 20, 1000);
        assertEquals("erroneous value from getX", 10, frame1.getX());
        assertEquals("erroneous value from getY", 20, frame1.getY());
        
        Frame frame2 = new Frame(15, 5, 2000);
        assertEquals("erroneous value from getX", 15, frame2.getX());
        assertEquals("erroneous value from getX", 5, frame2.getY());
    }

Il codice che soddisfa questo test è questo:

public class Frame
{
    private int length;
    private int x;
    private int y;
    
    public Frame(int x, int y, int length)
    {
        this.x = x;
        this.y = y;
        this.length = length;
    }

    public int getLength()
    {
        return length;
    }
    
    public int getX()
    {
        return x;
    }
    
    public int getY()
    {
        return y;
    }
}

[cionci]

Per fare questo mi basta aggiungere una sola riga in update:

    public void update(AbstractTimer timer)
    {
        long deltaTime = 0;
        
        deltaTime = timer.getTime() - lastTime;
        currentTime += deltaTime;

        int frame = 0;        
        int frameTime = frames.get(frame).getLength();
        
        while (currentTime >= frameTime)
        {
            if(frame == frames.size() - 1)
                frame = -1;
            
            frameTime += frames.get(++frame).getLength();
        }
        
        lastTime = timer.getTime();
        
        currentFrame = frame;
        
        setTextureArea(frames.get(frame).getX(), frames.get(frame).getY());
    }

[cionci]

Il task non è ancora finito, dobbiamo ancora gestire le animazioni della gemma illuminata...ma non so se vi siete resi conto, ma abbiamo implicitamente risolto anche il task 6.2.4 !!!

[Ufo13]

si perchè usi un timer unico... mi accontenterò di fare un po' di refactoring

[cionci]

C'è ancora da includere l'update delle gemme in Grid ed inizializzare le nuove gemme con i giusti frame in GemFactory...

Credo che tu possa partere anche ora... Intanto preparo il test per le gemme illuminate per fek !!!

[cionci]

Originariamente inviato da cionci

inizializzare le nuove gemme con i giusti frame in GemFactory

Anzi, mi sa che questa cosa la dovremmo fare noi...

[cionci]

Il nuovo test per fek:

    public void testBrightenedFrames()
    {
        gem.addFrame(10, 15, 100);
        gem.addFrame(20, 25, 100);
        
        gem.useBrighterImage();
        
        MockEngine engine = MockEngine.createForTesting(800, 600);
        MockTimer timer = new MockTimer(3500);
        
        gem.update(timer);
        gem.draw(engine);
                
        assertEquals("bad texture drawn", 42, engine.getTextureRect().left());
        assertEquals("bad texture drawn", 47, engine.getTextureRect().top());
        
        gem.useNormalImage();
        
        timer.advance(100);
        gem.update(timer);
        gem.draw(engine);
        
        assertEquals("bad texture drawn", 20, engine.getTextureRect().left());
        assertEquals("bad texture drawn", 25, engine.getTextureRect().top());        
    }

[fek]

Rieccomi.

[fek]

Una sola modifica in update():

        setTextureArea(
            frames.get(frame).getX() + (this.bright ? 32 : 0), 
            frames.get(frame).getY());

Poi ho dovuto modificare il test perche' la gemma illuminata e' quella a destra di 32 pixel, non a destra e in basso:

        assertEquals("bad texture drawn", 42, engine.getTextureRect().left());
        assertEquals("bad texture drawn", 15, engine.getTextureRect().top());

Ora non mi resta che aggiungere una chiamata in Grid per fare l'update delle gemme e aggiungere i frame di animazione coretti alla creazione delle gemme stesse.

[cionci]

Il test ce l'hai...telefono alla mia ragazze e dopo apro MSN...

[cionci]

Ok...giusto... Avevo sbagliato il test...