[CICLO 6/bis] Storia di Natale

[Jocchan]

Dato che a Natale siamo tutti più buoni, oggi apriamo un ciclo "speciale", dalla durata superiore al solito, e che si chiuderà il 6 gennaio 2006 (giorno fino al quale siamo formalmente - ma non effettivamente - in pausa). Si tratterà di un ciclo differente perchè stavolta sarà aperto, e quindi è possibile che dei nuovi task vengano aggiunti in seguito. Quindi, sbirciate spesso questo post... potreste trovare qualche sorpresa di tanto in tanto

Storia: Storia di servizio. Quando la colonna centrale in cui le gemme entrano nell'area di gioco è piena, il gioco deve smettere di crearle, e deve far comparire al centro dello schermo una png con la scritta Game Over. Occorre anche fare in modo che, quando viene premuto un tasto direzionale (sinistra o destra) mentre l'altro opposto è già premuto, venga eliminata l'incertezza sulla colonna di posizionamento della gemma, che deve restare fissa nella colonna in cui si trovava un istante prima della pressione del tasto contrastante.
To be continued...

Punti cardine da tenere a mente durante i lavori:

• Mai fare a gara a chi finisce il task per primo, meglio procedere con calma, altrimenti perderemo molto più tempo in seguito
• Evitiamo di complicarci la vita, esiste di certo una soluzione più semplice di quella che abbiamo pensato di implementare
• MAI aggiungere elementi non richiesti esplicitamente dai task: se mai serviranno, se ne parlerà nelle prossime storie
• Comunichiamo il più possibile, se qualcosa non è chiaro discutiamone tutti i dettagli fino ad eliminare ogni dubbio, anche il più insignificante
• Postare sempre la test list PRIMA di mettere mano al codice

[VICIUS]

Ecco i task per il ciclo natalizio.

Task:
Xmas.1.1: 71104:
Interrompere l'inserimento di nuove gemme quando la colonna centrale si riempie dopo di che si deve mostrare la scritta "Game Over" all'interno della griglia.
Xmas.1.2: cisc: 3 giorni
Aggiungere un ritardo di 300 millisecondi tra l'istante in cui la gemma si ferma e la creazione di una nuova gemma.
Xmas.1.3: cionci: 2 giorni
Ignorare la pressione del tasto direzionale finche quello con direzione opposto rimane premuto.
Xmas.1.4: Ufo13: completo
Portare a 50 millisecondi il repeat delay dopo la prima volta che è stato superato. E riportarlo al suo valore originale quando il tasto viene rilasciato.

ciao

[Jocchan]

Specifico meglio il task 3 per evitare incomprensioni.
Attualmente, se tengo premuto un tasto direzionale (ad esempio destra), ed in seguito premo quello opposto (in questo caso sinistra) senza rilasciare il precedente, spesso la gemma, prima di continuare la sua caduta senza spostarsi lateralmente, ha un comportamento imprevedibile: può rimanere nella stessa colonna, finire in una adiacente, oppure spostarsi temporaneamente e tornare in quella iniziale.
Lo scopo del task è fare in modo che questo non avvenga, e che - finchè entrambi i tasti restano premuti - la gemma rimanga nella stessa colonna, e continui a cadere in questa, senza incertezze (che, nella foga del gioco vero e proprio, possono portare a un posizionamento errato).
La necessità di apportare questa correzione nasce dalla possibilità che il giocatore cambi idea all'ultimo secondo, e decida di posizionare la gemma da un'altra parte: questo evento può comportare la pressione contemporanea di due direzioni contrastanti, e non possiamo permetterci di avere imprecisioni in questa circostanza

[71104]

UAO LA STORIA DI NATALE, A ME A ME!!!
li voglio fare tutti io!!!

comincio col primo ^^
dubbi: la scritta "Game Over" come ce la scrivo? c'è qualche texture? e a che coordinate la scrivo?
ora penso alla test list

[cionci]

Jocchan; mi puoi spiegare emglio il problema che vuoi risolvere ?

a) pressione continua del tasto a dx...pressione del tasto sx, la gemma si deve fermare
b) pressione contemporanea di entrambi i tasti la gemma deve restare ferma...

[VICIUS]

Originariamente inviato da 71104

UAO LA STORIA DI NATALE, A ME A ME!!!
li voglio fare tutti io!!!

comincio col primo ^^
dubbi: la scritta "Game Over" come ce la scrivo? c'è qualche texture? e a che coordinate la scrivo?
ora penso alla test list

Jocchan fra un po prepara la texture tu non devi far altro che metterla al centro della griglia.

ciao

[cisc]

volevo fare la stessa domanda di cionci, cmq, vorrei prenotarmi per il Xmas.1.2, tempo 3 giorni (meglio essere larghi)

[cionci]

Originariamente inviato da cisc

volevo fare la stessa domanda di cionci,

Anche perchè risolvendone uno si cambia l'altro...
Risolvendo il problema a)...se si premono entrambi i tasti la gemma si sposta nella cella corrispondente al tasto premuto prima fra i due (se la pressione va a cavallo fra due polling)

Al massimo possiamo lasciare il b) com'è ora e risolvere il problema a)...

[Jocchan]

Texture committata.

Per il problema della pressione contemporanea, si tratta solo dell'incertezza che si ha quando si preme un tasto opposto a quello già in pressione.
La gemma ogni tanto o si sposta di una colonna, e non deve accadere, oppure si sposta e torna nella precedente.
In realtà bisogna fare in modo che, quando si preme il secondo tasto, oltre a quello premuto, la gemma non abbia questa incertezza.
Se sto tenendo premuto destra e, quando sono nella casella 4, inizio a tenere premuto anche sinistra, la gemma deve continuare a cadere nella casella 4 fino a quando non rilascio uno dei due pulsanti.

[71104]

ho letto il codice attuale e riflettuto, e per la cronaca mi sono anche fatto consigliare da Jocchan; inoltre mi sono fatto un panino con insalata e mortazza, due solitari di Windows, una partita a campo minato, fumato una sigaretta, guardato in aria, fatto una cosa che non si dice, ed infine rieccomi qua sul forum

il problema è il seguente: il codice per catturare l'eccezione e visualizzare uno sprite con scritto "Game Over" dove lo metto, in Grid, in Game o in una classe apposta?

Jocchan dice Grid e io mi trovo d'accordo...

questo è il test che ho fatto (si troverebbe in TestGrid):

    public void testGridStops()
    {
        grid.insertGem(1, 4, gem1);
        grid.getGemUnderControl().dropGem();
        grid.insertGem(0, 4, gem2);
        try
        {
            grid.update();
        }
        catch(IllegalArgumentException e)
        {
            fail("Grid.update is trying to put other gems");
        }
    }

eventualmente poi potrei anche creare una nuova classe GameOverException in modo tale che insertNewGem cattura la IllegalArgumentException causata dall'inserzione di una gemma quando non c'è spazio e la converte lanciando una GameOverException.

poi magari la test listi si potrebbe completare con un test per verificare col MockEngine che Grid.update, una volta catturata l'eccezione, disegni lo sprite "Game Over".

[fek]

Originariamente inviato da 71104

ho letto il codice attuale e riflettuto, e per la cronaca mi sono anche fatto consigliare da Jocchan; inoltre mi sono fatto un panino con insalata e mortazza, due solitari di Windows, una partita a campo minato, fumato una sigaretta, guardato in aria, fatto una cosa che non si dice, ed infine rieccomi qua sul forum

Ma la test list non include una descrizione dettagliata delle tue sessioni onaniste

il problema è il seguente: il codice per catturare l'eccezione e visualizzare uno sprite con scritto "Game Over" dove lo metto, in Grid, in Game o in una classe

Jocchan dice Grid e io mi trovo d'accordo...
apposta?

Allora prima di mettere dell'altro codice in Grid, devi avere un'ottima ragione, ma proprio ottima, e spiegarcela dettagliatamente e poi giustificarla e poi, quando hai finito, devi pensare di nuovo se per caso non c'e' una soluzione migliore.
Tradotto: non mettere nulla in Grid se non strettamente necessario ma anche di piu'.

eventualmente poi potrei anche creare una nuova classe GameOverException in modo tale che insertNewGem cattura la IllegalArgumentException causata dall'inserzione di una gemma quando non c'è spazio e la converte lanciando una GameOverException.

Non mi piace come idea, perche' le eccezioni servono solo per comunicare condizioni eccezionali nell'esecuzione del programma, ovvero condizioni che non dovrebbero mai accadere in una normale esecuzione (sono per l'appunto eccezionali). Il Game Over e' una condizione normale alla fine del gioco, quindi quest'informazione non andrebbe veicolata per mezzo di un'eccezione.

[Ufo13]

prendo il secondo task

[Jocchan]

Originariamente inviato da Ufo13

prendo il secondo task

E' già prenotato da cisc

[Ufo13]

Allora il quarto

[cionci]

Originariamente inviato da Jocchan

Se sto tenendo premuto destra e, quando sono nella casella 4, inizio a tenere premuto anche sinistra, la gemma deve continuare a cadere nella casella 4 fino a quando non rilascio uno dei due pulsanti.

Ok...allora vedo di mantenere la situazione com'è per la situazione b) e correggo la situazione a)...

[cionci]

Mi prendo 2 giorni di tempo...

L'acceptance test list sarà come segue:

- se un tasto freccia è premuto continuamente e premo il tasto freccia opposta la gemma deve stare ferma

- se due tasti freccia vengono premuti due tasti freccia e la pressione risulta a cavallo fra due aggiornamenti la gemma deve tornare alla posizione iniziale (quindi mantennedo la condizione attuale)

[cionci]

Test list per il task Xmas.1.3:

    public void testLeftAndRightBothPressed()
    {
        grid.insertGem(2, 4, gem);
        grid.setGemUnderControl(gem);
        input.generateKey(KeyCode.vk_Left);
        input.generateKey(KeyCode.vk_Right);
        grid.reactToInput(input, timer);
        assertTrue("Gem may not move when Left & Right key are both pressed",
            grid.isGemAt(2, 4));
    }


    public void testLeftAndRightSequenceInTwoPollings()
    {
        grid.insertGem(2, 4, gem);
        grid.setGemUnderControl(gem);
        input.generateKey(KeyCode.vk_Left, timer.getTime());
        grid.reactToInput(input, timer);
        timer.advance(grid.getRepeatDelay() - 1);
        input.generateKey(KeyCode.vk_Right, timer.getTime());
        grid.reactToInput(input, timer);
        assertTrue("Gem may not move when Left & Right key are both pressed",
            grid.isGemAt(2, 4));
    }
    
    
    public void testLeftStillPressedAndRightPressed()
    {
        grid.insertGem(2, 4, gem);
        grid.setGemUnderControl(gem);
        input.generateKey(KeyCode.vk_Left, timer.getTime());
        grid.reactToInput(input, timer);
        timer.advance(grid.getRepeatDelay() + 1);
        input.generateKey(KeyCode.vk_Right, timer.getTime());
        grid.reactToInput(input, timer);
        assertTrue("Gem may not move when Left key is repeated & Right key is pressed",
            grid.isGemAt(2, 3));
        
    }
    
    
    public void testRightStillPressedAndLeftPressed()
    {
        grid.insertGem(2, 4, gem);
        grid.setGemUnderControl(gem);
        input.generateKey(KeyCode.vk_Right, timer.getTime());
        grid.reactToInput(input, timer);
        timer.advance(grid.getRepeatDelay() + 1);
        input.generateKey(KeyCode.vk_Left, timer.getTime());
        grid.reactToInput(input, timer);
        assertTrue("Gem may not move when Right key is repeated & Left key is pressed",
            grid.isGemAt(2, 5));        
    }

[fek]

Ma vi state mangiando quattro task al giorno voi? E' cosi' che mi piace, duri, cattivi e produttivi.

E per premiarvi, vi diamo altri task di Natale

[cionci]

Fatto il commit del task Xmas.1.3

[VICIUS]

Siete delle belve.

ciao