[CICLO 2] Storia 2

[VICIUS]

Grazie al cielo c'è fek
Dunque. Che dici aggiungo questi due task:

5.9 spezzare la classe playarea in due o piu classi. (71104: x giorni)
5.10 auditing del codice e delle classi Test. Aggiungere test per il codice non testato. eliminare test non piu neccesari e modifcare quelli vecchi. (Tutti: 4 giorni)

ciao

[fek]

Perfetto

Se qualcuno ha problemi con lo scrivere i test per iniziare questi taskj, i primi li scriviamo noi.

[fek]

Un'altra cosa. Ho visto che la build e' stata rotta per questi giorni, perche' checkstyle non passava.

Prima di ogni commit, lanciate la build di Ant esterna a Eclipse.

Non c'e' eccezione alla regola.

[cionci]

71104: hai fatto più danni della grandine

[checkstyle] E:\Java\diamonds\trunk\src\it\diamonds\PlayArea.java:14:5: Each var
iable declaration must be in its own statement.
[checkstyle] E:\Java\diamonds\trunk\src\it\diamonds\PlayArea.java:17:5: Each var
iable declaration must be in its own statement.

[VICIUS]

Originariamente inviato da cionci

71104: hai fatto più danni della grandine
[...]

Su dai 71104 metti a posto il codice.

ciao

[fek]

Corretto.

[71104]

Originariamente inviato da fek

PlayArea ad esempio. E' una classe con due responsabilita':

1) Disegna la griglia
2) Gestisce la griglia di celle

veramente per ora non gestisce nulla, la disegna soltanto...

C'e' solo un test. Qui c'e' chiaramente un problema che deve essere risolto prima di continuare. Andare avanti con una classe cosi' importante sostanzialmente non testata e che ha due responsabilita', e' una ricetta sicura per il disastro.

un altro mi era venuto in mente ma non so come realizzarlo: avevo pensato di fare un test in cui la cella di PlayArea veniva creata con degli Extents (che ora vorrei capire perché si chiamano Bounds... ) e controllare che nessuna cella subisse clipping durante il disegno...

[71104]

Originariamente inviato da cionci

71104: hai fatto più danni della grandine

[checkstyle] E:\Java\diamonds\trunk\src\it\diamonds\PlayArea.java:14:5: Each var
iable declaration must be in its own statement.
[checkstyle] E:\Java\diamonds\trunk\src\it\diamonds\PlayArea.java:17:5: Each var
iable declaration must be in its own statement.

è arrivato... -_-'
solo due issues, poteva andare peggio, pensa se avessi programmato col mio solito stile... e poi almeno il mio codice funziona
edit: ed è anche scritto completamente in inglese

[71104]

Originariamente inviato da VICIUS

5.9 spezzare la classe playarea in due o piu classi. (71104: x giorni)

più che altro direi aggiungere una classe per gestire la griglia...

[cisc]

effettivamente PlayArea per il momento disegna solo una griglia di celle, e non gestisce la griglia stessa, penso che l'unica cosa da fare sia dargli un nome più significativo, e aggiungere un'altra classe per la gestione della griglia

[cionci]

Originariamente inviato da 71104

e poi almeno il mio codice funziona

Perchè...il mio no ?

[cisc]

aa, inoltre sono d'accordo con cionci quando dice che il numero di colonne dovrebbe essere dispari

[71104]

Originariamente inviato da cisc

effettivamente PlayArea per il momento disegna solo una griglia di celle, e non gestisce la griglia stessa, penso che l'unica cosa da fare sia dargli un nome più significativo, e aggiungere un'altra classe per la gestione della griglia

ho aggiunto una classe Grid da me, però non so che devo fargli fare per ora: ci ho messo solo un Vector contenente degli Sprite e fa tutto lui...
ora metto un'istanza di Grid in PlayArea.

[71104]

Originariamente inviato da cionci

Perchè...il mio no ?

non so se il tuo funziona o no, ma non so chi sia stato a fare macello con le coordinate; c'è poca comunicazione tra di voi, ognuno sapeva il sistema di coordinate in modo diverso e il programma funzionava per miracolo (anzi, per caso); usatelo spesso il thread della knowledge (lo userò anch'io).

[71104]

http://www.hwupgrade.it/forum/showpo...9&postcount=25

[fek]

Originariamente inviato da 71104

solo due issues, poteva andare peggio, pensa se avessi programmato col mio solito stile... e poi almeno il mio codice funziona

Qual e' il test automatico che mi assicura che funzioni?
Senza test, io assumo che un metodo non funzioni ed il task relativo non sia concluso.

Originariamente inviato da 71104

ho aggiunto una classe Grid da me, però non so che devo fargli fare per ora: ci ho messo solo un Vector contenente degli Sprite e fa tutto lui...
ora metto un'istanza di Grid in PlayArea.

In base a che cosa hai scritto del codice in una classe che non sai che devi fargli fare e soprattutto non hai i test che ti dicono che cosa fargli fare.

E' fondamentale che scriviate i test prima di scrivere il codice di produzione. Non voglio dover arrivare a imporvelo con le cattive (non accettando i commit), preferisco affidarmi al vostro buon senso. Se non avete chiaro come scrivere i test prima, iniziate con un task semplice e domandate. Ma non scrivete codice tanto per scrivere qualcosa, senza i relativi test.

[fek]

Originariamente inviato da 71104

non so se il tuo funziona o no, ma non so chi sia stato a fare macello con le coordinate; c'è poca comunicazione tra di voi, ognuno sapeva il sistema di coordinate in modo diverso e il programma funzionava per miracolo (anzi, per caso); usatelo spesso il thread della knowledge (lo userò anch'io).

Se la colpa e' di qualcuno, e' mia. Non voglio piu' vedere post di questo tipo.

[71104]

Originariamente inviato da fek

Qual e' il test automatico che mi assicura che funzioni?
Senza test, io assumo che un metodo non funzioni ed il task relativo non sia concluso.

infatti io parlavo di Extents e di tutte le correzioni che ho fatto durante la Storia 1

[fek]

ok

[Jocchan]

Il motivo per cui il numero di colonne deve essere pari l'ho spiegato nel topic sul design
Si può anche inserire un numero dispari, ma questo aumenta la difficoltà del gioco, dato che sono necessari parecchi incastri per far quadrare tutto. Facendo un bilancio, è sicuramente sconveniente.