TODO Official Thread

[Bonfo]

Ne abbiamo troppi, e quasi tutti in italiano!
Propongo di ammazzarli uno alla volta, altrimenti rimango lì in eterno.

Come procedere:
1) capire cosa cavolo chiede il TODO;
2) capire se la richiesta ha ancora senso ( potrebbe essere lì dal medioevo);
3) eseguire il TODO.

Propongo il primo:
cercare di correggere questo test (TestGameLoop.java - line 240)

Il test che controlla il disegno del GameOver c'è già (testGameOverMessageIsDrawn in TestPlayField).
Cnacello?

[71104]

Originariamente inviato da Bonfo

Cancello?

LOL!
no dai, siamo seri prima cerchiamo di capire il problema...

[Bonfo]

Originariamente inviato da 71104

LOL!
no dai, siamo seri prima cerchiamo di capire il problema...

Certo... se no perchè avrei aperto il Thread

[71104]

*aehm*... l'ho decommentato per vedere se falliva e noto che quel test è talmente vecchio che neanche compila più...

si, in effetti ripensandoci sarebbe meglio evitare troppi sbattimenti e cancellarlo a piè pari; se poi si presenta un bug lo ritesteremo...

[fek]

Originariamente inviato da 71104

*aehm*... l'ho decommentato per vedere se falliva e noto che quel test è talmente vecchio che neanche compila più...

si, in effetti ripensandoci sarebbe meglio evitare troppi sbattimenti e cancellarlo a piè pari; se poi si presenta un bug lo ritesteremo...

Ma no, facciamo che riscrivi un test per quel metodo.

[71104]

Originariamente inviato da fek

Ma no, facciamo che riscrivi un test per quel metodo.

*quale* metodo?
ne vengono chiamati almeno 6 o 7 là dentro, ciascuno testato separatamente

conta che l'ho decommentato e l'ho fatto compilare con qualche modifica, ecco la nuova versione:

    // TODO: cercare di correggere questo test
    public void testGameOversAreDisplayingCorrectly()
    {
        MockEngine engine = (MockEngine)environment.getEngine();
        LayerManager layerManager = gameLoop.getLayerManager();
        gameLoop.doOneStep();
        engine.clearDisplay();
        layerManager.drawLayers(engine);
        int numberOfQuadsDrawn = engine.getNumberOfQuadsDrawn();
        gameLoop.getPlayFieldOne().getGridController().getGemsPair().getPivot().getFallingObject().drop();
        gameLoop.getPlayFieldOne().getGridController().getGemsPair().getSlave().getFallingObject().drop();
        gameLoop.getPlayFieldTwo().getGridController().getGemsPair().getPivot().getFallingObject().drop();
        gameLoop.getPlayFieldTwo().getGridController().getGemsPair().getSlave().getFallingObject().drop();
        gameLoop.doOneStep();
        engine.clearDisplay();
        layerManager.drawLayers(engine);
        assertEquals("Game over message must be shown", numberOfQuadsDrawn + 2,
            engine.getNumberOfQuadsDrawn());
    }

[71104]

comunque mi sembra che quel test serva semplicemente a testare moooolto dall'alto che i due sprite "GameOver" appaiano; provo a scrivere un test analogo...

[Bonfo]

A me falliva!
Il che vuol dire che il test è sbagliato, perchè i gameOver dono displayati correttamente.
In più, come già detto, un test sul display del gameOver c'è già.

[71104]

Originariamente inviato da Bonfo

A me falliva!

pure a me infatti, ho detto che l'ho fatto compilare, non che passasse

In più, come già detto, un test sul display del gameOver c'è già.

DEO GRATIAS

e allora che facciamo, canniamo?

[fek]

Canna pure allora.

[71104]

fatto; chi s'è visto s'è visto

ok, il secondo adesso lo affronteremo con serietà

[Bonfo]

Eccolo:
check if reinitialization of GridController can be eliminated (TestGemCreationInGrid.java - line 22)

Ricordo i tre punti:
1) capire il TODO
2) capire se ha ancora senso
3) risolvere

Ehm.. questo non l'ho ancora letto.

[Bonfo]

Come siamo messi a TODO??

[fek]

Trasforma i TODO in Refactor This, cosi' spariscono con il programma Refactor This. Ogni commit, un Refactor This o volano i revert

[71104]

non tutti i TODO però necessitano di refactoring: alcuni (come il primo) vanno semplicemente eliminati e basta. direi che quelli da eliminare gli segnaliamo qui, tutti gli altri li convertiamo in REFACTOR THIS