[CICLO 12] Storia 1

[VICIUS]

Questa è una di refactoring, bugfix e aggiornamento. Come al solito prima di cominciare qualcosa annunciatelo sul forum e cercate di essere il più chiari possibili magari citando le classi che andrete a toccare.

Bugs:
1. Il layer delle gemme copre quello in cui viene mostrato la png delle combo Ufo13: completato
2. Le png delle combo vengono nascoste appena viene creata una nuova gemspair invece che essere mostrati per un tempo fisso
3. La pausa tra un ciclo di crush e l'altro viene onorata solo tra il primo e il secondo ciclo. thebol
4. Quando si preme il tasto ESC il gioco non esce.
5. Su linux OpenAL usa OSS invece di usare ALSA sul dispositivo default.
...

Refactoring:
1. Portare ad abstract GridTestCase. Togliere le getter. Sistemare i test secondi i cambiamenti della classe. VICIUS: completato
2. Eliminare tutte le Gem.diamond(3500) dai test e usare una createGem(DIAMOND). estendere GridTestCase dovrebbe bastare. VICIUS: completato
3. Togliere CrushBox e WarningBox da Grid ^TiGeRShArK^ + dnarod:completato
...

Varie:
1. Controllare se la versione 0.99 di Lwjgl risolve i problemi di stabilità su linux. VICIUS: completato
2. Provare a modificare il file di configurazione di eclipse per impedire che obj.funzione() venga spezzato. thebol: completato

La storia normale con i nuovi task da implementare sarà postata appena finiti tutti i refactoring.

ciao

[fek]

Ottimo. Al lavoro che siamo vicini alla First Playable

[Ufo13]

Mi occupo del primo Bug. Tempo previsto 2 giorni. Posterò le previsioni il prima possibile

[VICIUS]

Mi son preso i due refacotring che ce li ho mezzi finiti nel mio tree.
Se avete altre idee postate pure che le aggiungiamo.

ciao

[thebol]

Prendo il bug 3

[Jocchan]

Ottimo, così vi vogliamo. Famelici e spietati

[Ufo13]

proporrei come refactoring:

Tirare fuori WarningBox da Grid

[fek]

Originariamente inviato da Ufo13

proporrei come refactoring:

Tirare fuori WarningBox da Grid

Assolutamente si'. Non ci azzecca proprio nulla.

[Ufo13]

e stessa cosa per CrushBox direi

[^TiGeRShArK^]

Prenotazione per il task 3 di refactoring.
Io e dnarod in pair refactoring fisico da casa mia

[fek]

Mi piace di piu' quando cannibalizzate i task

[thebol]

risolto il bug(un assegnazione sbagliata...)

ecco il test

    public void testCrushesDelay()
    {
        int delayCrush = getConfig().getInteger("DelayBetweenCrushes");
        
        insertAndUpdate(createGem(EMERALD), 13, 2);
        insertAndUpdate(createGem(DIAMOND), 12, 2);
        insertAndUpdate(createGem(EMERALD), 11, 2);
        insertAndUpdate(createGem(DIAMOND), 10, 2);
        insertAndUpdate(createGem(DIAMOND_CHEST), 9, 2);
        insertAndUpdate(createGem(EMERALD_CHEST), 8, 2);
        insertAndUpdate(createGem(DIAMOND_CHEST), 7, 2);
        insertAndUpdate(createGem(EMERALD_CHEST), 6, 2);

        insertAndStopGemsPair();
        makeAllGemsFall();

       getController().update(getTimer());


        makeAllGemsFall();        
        oneStepForward();


        for (int i=1; i<delayCrush; i++)
        {
            checkCrush("at iteration:"+ i, 8, 1, 1);
            oneStepForward();     
        }
        
        checkCrush("second crush:",6,1,2);
        
        makeAllGemsFall();
        
        for (int i=0; i<delayCrush; i++)
        {
            checkCrush("at iteration:"+ i, 6, 1, 2);
            oneStepForward();
        }
        
        checkCrush("third crush:", 4, 1, 3);
        
        makeAllGemsFall();
                
        for (int i=0; i<delayCrush; i++)
        {
            checkCrush("at iteration:"+ i, 4, 1, 3);
            oneStepForward();
        }
        
        checkCrush("final update:", 2, 1, 4);
        
        

    }


    private void checkCrush(String string, int numberOfGems, int crushedGemsCounter, int chainCounter)
    {
        assertEquals(string, numberOfGems, getGrid().getNumberOfGems());
        assertEquals(string, crushedGemsCounter, getGrid().getCrushedGemsCounter());
        assertEquals(string, chainCounter, getGrid().getChainCounter());
    }


    private void oneStepForward()
    {
        getTimer().advance(1);
        getController().update(getTimer());
    }

ho dovuto introdurre 2 nuove funzioni se no checkstyle strippava sull'ncss (era 41, e voleva max 30...)

[thebol]

Ho esaminato il bug 2, conviene aspettare che venga tolta crushBox da grid(e magari spostato in gridController) per poi implementare il delay(ora come ora lo si potrebbe introdurre solo in grid.updateGemAnimations(), l'unica funzione di grid chiamata a ogni gridController.update())

ps.va introdotto un parametro di configurazione per quel delay

[^TiGeRShArK^]

sono dnarod, preso sotto la sua ala....mi sta aprendo la mente ....diciamo che ora sto quasi capendo quello che mi imbocca pazientemente.... mi suggeriscono dalla regia che "siamo a buon punto", diciamo che concordo.........se tutto va bene (benissimo) magari si finisce di stasera

[^TiGeRShArK^]

son sempre rod....oppure non sono piu io....mah non so, il mio cervello è prossimo all overflow, quindi non so piu nulla...siamo sulla retta via, ma non abbiamo ancora finito (complicissima la mia incapacita oggettiva, anche se spero di apprendere in fretta); sry per il ritardo, speriamo di terminare domani, io mo torno a casuccia e ripasso la parola al legittimo proprietario di questo pc...ciaoz...

[fek]

Originariamente inviato da thebol

[/code]

ho dovuto introdurre 2 nuove funzioni se no checkstyle strippava sull'ncss (era 41, e voleva max 30...)

Ottimo!

30 e' ancora alto, puoi ridurre ulteriormente la complessita'? Mi posti il test che esercita il bug?

[VICIUS]

Dunque ieri ho fatto alcune prove con la nuova versione di Lwjgl. Devo dire che sono rimasto piuttosto soddisfatto. Niente più crash assurdi. Inoltre hanno risolto il problema con i nomi delle librerie che ci costringeva ad avere su .so uguali con nomi diversi.

Prima di aggiornare i file sul repository però vorrei una conferma da utenti windows e osx. Potete provare la nuova versione e vedere se avete problemi ?

ciao

[VICIUS]

Per quanto riguarda i primi due refactoring c'è un problema. Nel mio tree funziona tutto e checkstyle non dice niente. Come porto la patch ad un checkout liscio ecco che comincia a dare errore sugli attributi. Ieri sera tra una publicità e l'altra ho cercato di capire cosa ci fosse di diverso ma le due cartelle sembravano identiche bit-a-bit.

Ora ho trovato un modo per disabilitare checkstyle in quella esatta porzione di codice tramite due commenti. Prima di procedere a sistemare la patch e fare il commit volevo un via libera anche da voi.

ciao

[fek]

Bene, appena abbiamo l'ok anche dagl'altri possiamo passare al nuovo lwgl visto che sembra piu' stabile.

[thebol]

Originariamente inviato da fek

Ottimo!

30 e' ancora alto, puoi ridurre ulteriormente la complessita'? Mi posti il test che esercita il bug?

mi sa che cè stata un incomprensione =D

era il test (postato nel precedente post) che superava nella prima versione il valore di 40, e ho aggiunto 2 funzioni(riportate sempre nel precedente post).