[CICLO 10] Storia 1 - Pagina 4

cdimauro · 10-02-2006, 06:51

OK, per finire il task mi basta passare un Rectangle a NextGemsPanel con le coordinate del box e usare quelle al posto dei valori hard coded che ci sono attualmente.

Però aspetto che la build sia di nuovo verde per mettere mano al codice del task.

Ho ancora qualche minuto prima di andare a lavoro: provo a dare un'occhiata al codice per vedere se riesco a capire cos'è che fa saltare quei test.

cdimauro · 10-02-2006, 07:08

Falso allarme: non so per quale motivo, ma ant faceva riferimento a dei file di test inesistenti, per cui chiaramente falliva.

Buttando tutto il progetto e rieseguendo il CheckOut, la build è verde.

La mia solita sfiga. Fortuna che ho l'ADSL a casa (col 56K sarebbe stato un inferno).

Adesso vado a lavoro. Il task lo finisco stasera.

Scusate per lo spam.

cionci · 10-02-2006, 07:10

Quote:

Originariamente inviato da cdimauro

Falso allarme: non so per quale motivo, ma ant faceva riferimento a dei file di test inesistenti, per cui chiaramente falliva.

Basta fare il clean della build quando succedono queste cose...

fek · 10-02-2006, 11:04

Quote:

Originariamente inviato da Jocchan

Non posso testare il gioco perchè la build machine sarà down (credo) fino a sabato, ma Ufo mi riferisce che non funzionano i game over.

Sabato se tutto va bene, altrimenti domenica. Sabato mi attaccano la DSL nella casa nuova.

Bear with me for a while.

Bonfo · 10-02-2006, 13:35

Cionci...ma per clean intendi quello nel menù di SVN oppure il task ANT ???
Col task ant, se poi faccio i test non riesco più a far partire il gioco dal .bat perchè i file non sono più in bin, main debug o relase.

VICIUS · 10-02-2006, 14:00

Quote:

Originariamente inviato da Bonfo

Cionci...ma per clean intendi quello nel menù di SVN oppure il task ANT ???
Col task ant, se poi faccio i test non riesco più a far partire il gioco dal .bat perchè i file non sono più in bin, main debug o relase.

Nel menu di eclipse "Projects" -> "Clean..."

ciao

^TiGeRShArK^ · 10-02-2006, 14:15

Quote:

Originariamente inviato da Jocchan

Tiger vabbè che sono task facili, ma... sei un mostro (come ovviamente anche Cionci)!

non tanto...
ci ho pensato un pò stanotte quando ero a letto (non ridete... è sempre il momento in cui mi vengono le idee migliori

) e mi sono venuti un paio di refactoring.. mi metto stasera a farli se riesco

(anke xkè non mi piaceva proprio il codice ke avevo fatto in quel modo

)

^TiGeRShArK^ · 10-02-2006, 18:46

refactoring completato, build verde, commit effettuato

^TiGeRShArK^ · 10-02-2006, 18:52

ehm.. una domandina...
ma a livello di giocabilità non è meglio utilizzare per il P1 le freccette + CANC, FINE, PAG e per il P2 W,A,S,D come tasti direzionali e 4,5,6 per le rotazioni???

Ufo13 · 10-02-2006, 19:46

meglio WASD e freccette imho

cionci · 10-02-2006, 21:13

Urge un refactoring...mi sta qui sul collo... Domani ve ne parlo...

cdimauro · 11-02-2006, 06:42

Completato task 4, build verde.

Non è stato introdotto alcune metodo, quindi non c'era bisogno di introdurre nuovi test.

Ho semplicemente aggiunto un parametro Point al costruttore di NextGemsPanel, che viene utilizzato per definire le coordinate del primo punto. Il secondo è definito dalle stesse coordinate, con l'ordinata aumentata di 32.

Si poteva fare diversamente.
Passare un Rectangle, ma un rettangolo mi dà l'idea di una superficie che racchiude uno spazio.
Oppure passare due Point, ognuno legato alla rispettiva gemma.

Io ho scelto la soluzione che ai miei occhi sembra più semplice e veloce per portare a termine il mio task. De gustibus.

x cionci e Vicius: grazie per i suggerimenti.

Ufo13 · 11-02-2006, 08:41

secondo me un test serve...

Dato che in Diamonds non si modifica codice senza avere un test che fallisce basta un test con MockEngine che dimostri che è stato disegnato il pannello nella locazione giusta... A quel punto build rossa e vai con la modifica

cionci · 11-02-2006, 09:07

Ma se il punto diverso viene passato in Game non si può testare...

cionci · 11-02-2006, 09:10

Piuttosto, ecco la mia proposta di refactoring... Non è semplice perchè bisognerebbe portare molto del codice che è in Game in una classe esterna...e quindi completamente da creare e testare...

Secondo em è arrivata la necessità di creare una classe PlayerField o PlayField in cui si inizializzano e gestiscono tutte le componenti dell'area di gioco di un giocatore... Quindi la classe Game del gioco vedrebbe solo ed esclusivamente questa classe...

Che ne dite ?

cdimauro · 11-02-2006, 10:31

Quote:

Originariamente inviato da Ufo13

secondo me un test serve...

Dato che in Diamonds non si modifica codice senza avere un test che fallisce basta un test con MockEngine che dimostri che è stato disegnato il pannello nella locazione giusta... A quel punto build rossa e vai con la modifica

Sì può anche fare, certamente, ma questo perché sarebbe necessario introdurre almeno un metodo per implementare la funzionalità desiderata.

x cionci: mi sembra una buona idea.

thebol · 11-02-2006, 10:50

Quote:

Originariamente inviato da cionci

Piuttosto, ecco la mia proposta di refactoring... Non è semplice perchè bisognerebbe portare molto del codice che è in Game in una classe esterna...e quindi completamente da creare e testare...

Secondo em è arrivata la necessità di creare una classe PlayerField o PlayField in cui si inizializzano e gestiscono tutte le componenti dell'area di gioco di un giocatore... Quindi la classe Game del gioco vedrebbe solo ed esclusivamente questa classe...

Che ne dite ?

direi fondamentale

cionci · 11-02-2006, 10:54

Ragazzi...allora parto a farlo, ma la cosa sarà alquanto lunga... Intanto parto...ovviamente senza modificare il comportamento attuale e il test lo metto in ignore, così potete continuare dopo di me....

VICIUS · 11-02-2006, 12:12

Quote:

Originariamente inviato da cionci

Piuttosto, ecco la mia proposta di refactoring... Non è semplice perchè bisognerebbe portare molto del codice che è in Game in una classe esterna...e quindi completamente da creare e testare...

Secondo em è arrivata la necessità di creare una classe PlayerField o PlayField in cui si inizializzano e gestiscono tutte le componenti dell'area di gioco di un giocatore... Quindi la classe Game del gioco vedrebbe solo ed esclusivamente questa classe...

Che ne dite ?

È un ottimo refactoring. Potremmo considerarlo parte del task 1 della seconda storia. Qualcuno vuol aiutare cionci in pair ?

ciao

cionci · 11-02-2006, 12:46

Presto aggiornerò la versione sul repository... Per ora ho fatto la gestione della reazione agli input e degli update (spostandoci anche i timeStamp)...

Ora bisogna spostarci l'inizializzazione (quindi l'aggiunta ai vari layer e la posizione delle varie componenti) ed il disegno su schermo...

10-02-2006, 13:35	#65
Bonfo Senior Member Iscritto dal: Nov 2005 Città: Bologna Messaggi: 1303	Cionci...ma per clean intendi quello nel menù di SVN oppure il task ANT ??? Col task ant, se poi faccio i test non riesco più a far partire il gioco dal .bat perchè i file non sono più in bin, main debug o relase. __________________ Software engineer Bonfo's Blog

10-02-2006, 18:46	#68
^TiGeRShArK^ Senior Member Iscritto dal: Jul 2002 Città: Reggio Calabria -> London Messaggi: 12112	refactoring completato, build verde, commit effettuato __________________

10-02-2006, 18:52	#69
^TiGeRShArK^ Senior Member Iscritto dal: Jul 2002 Città: Reggio Calabria -> London Messaggi: 12112	ehm.. una domandina... ma a livello di giocabilità non è meglio utilizzare per il P1 le freccette + CANC, FINE, PAG e per il P2 W,A,S,D come tasti direzionali e 4,5,6 per le rotazioni??? __________________

10-02-2006, 06:51	#61
cdimauro Senior Member Iscritto dal: Jan 2002 Città: Germania Messaggi: 26110	OK, per finire il task mi basta passare un Rectangle a NextGemsPanel con le coordinate del box e usare quelle al posto dei valori hard coded che ci sono attualmente. Però aspetto che la build sia di nuovo verde per mettere mano al codice del task. Ho ancora qualche minuto prima di andare a lavoro: provo a dare un'occhiata al codice per vedere se riesco a capire cos'è che fa saltare quei test.

10-02-2006, 07:08	#62
cdimauro Senior Member Iscritto dal: Jan 2002 Città: Germania Messaggi: 26110	Falso allarme: non so per quale motivo, ma ant faceva riferimento a dei file di test inesistenti, per cui chiaramente falliva. Buttando tutto il progetto e rieseguendo il CheckOut, la build è verde. La mia solita sfiga. Fortuna che ho l'ADSL a casa (col 56K sarebbe stato un inferno). Adesso vado a lavoro. Il task lo finisco stasera. Scusate per lo spam.

10-02-2006, 19:46	#70
Ufo13 Senior Member Iscritto dal: Nov 2005 Messaggi: 1545	meglio WASD e freccette imho

10-02-2006, 21:13	#71
cionci Senior Member Iscritto dal: Apr 2000 Città: Vicino a Montecatini(Pistoia) Moto:Kawasaki Ninja ZX-9R Scudetti: 29 Messaggi: 53971	Urge un refactoring...mi sta qui sul collo... Domani ve ne parlo...

11-02-2006, 06:42	#72
cdimauro Senior Member Iscritto dal: Jan 2002 Città: Germania Messaggi: 26110	Completato task 4, build verde. Non è stato introdotto alcune metodo, quindi non c'era bisogno di introdurre nuovi test. Ho semplicemente aggiunto un parametro Point al costruttore di NextGemsPanel, che viene utilizzato per definire le coordinate del primo punto. Il secondo è definito dalle stesse coordinate, con l'ordinata aumentata di 32. Si poteva fare diversamente. Passare un Rectangle, ma un rettangolo mi dà l'idea di una superficie che racchiude uno spazio. Oppure passare due Point, ognuno legato alla rispettiva gemma. Io ho scelto la soluzione che ai miei occhi sembra più semplice e veloce per portare a termine il mio task. De gustibus. x cionci e Vicius: grazie per i suggerimenti.

11-02-2006, 08:41	#73
Ufo13 Senior Member Iscritto dal: Nov 2005 Messaggi: 1545	secondo me un test serve... Dato che in Diamonds non si modifica codice senza avere un test che fallisce basta un test con MockEngine che dimostri che è stato disegnato il pannello nella locazione giusta... A quel punto build rossa e vai con la modifica

11-02-2006, 09:07	#74
cionci Senior Member Iscritto dal: Apr 2000 Città: Vicino a Montecatini(Pistoia) Moto:Kawasaki Ninja ZX-9R Scudetti: 29 Messaggi: 53971	Ma se il punto diverso viene passato in Game non si può testare...

11-02-2006, 09:10	#75
cionci Senior Member Iscritto dal: Apr 2000 Città: Vicino a Montecatini(Pistoia) Moto:Kawasaki Ninja ZX-9R Scudetti: 29 Messaggi: 53971	Piuttosto, ecco la mia proposta di refactoring... Non è semplice perchè bisognerebbe portare molto del codice che è in Game in una classe esterna...e quindi completamente da creare e testare... Secondo em è arrivata la necessità di creare una classe PlayerField o PlayField in cui si inizializzano e gestiscono tutte le componenti dell'area di gioco di un giocatore... Quindi la classe Game del gioco vedrebbe solo ed esclusivamente questa classe... Che ne dite ?

11-02-2006, 10:54	#78
cionci Senior Member Iscritto dal: Apr 2000 Città: Vicino a Montecatini(Pistoia) Moto:Kawasaki Ninja ZX-9R Scudetti: 29 Messaggi: 53971	Ragazzi...allora parto a farlo, ma la cosa sarà alquanto lunga... Intanto parto...ovviamente senza modificare il comportamento attuale e il test lo metto in ignore, così potete continuare dopo di me....

11-02-2006, 12:46	#80
cionci Senior Member Iscritto dal: Apr 2000 Città: Vicino a Montecatini(Pistoia) Moto:Kawasaki Ninja ZX-9R Scudetti: 29 Messaggi: 53971	Presto aggiornerò la versione sul repository... Per ora ho fatto la gestione della reazione agli input e degli update (spostandoci anche i timeStamp)... Ora bisogna spostarci l'inizializzazione (quindi l'aggiunta ai vari layer e la posizione delle varie componenti) ed il disegno su schermo...

Strumenti
Mostra una versione stampabile Invia questa pagina per email