[CICLO 2] Storia 1

[VICIUS]

Hai fatto benissimo
Quel assertNotEquals non so come mi sia finito nei sorgenti. Giuro

ciao

[71104]

VICIUS, nei test della classe Extents hai usato un costruttore non definito per un oggetto Gem che usa un nuovo parametro Extents; ho definito il costruttore ma non è che tante volte manca un parametro Config?
ce lo metto?

[fek]

Originariamente inviato da 71104

VICIUS, nei test della classe Extents hai usato un costruttore non definito per un oggetto Gem che usa un nuovo parametro Extents; ho definito il costruttore ma non è che tante volte manca un parametro Config?
ce lo metto?

Il test che ha scritto Vicius richiede un costruttore con parametro di tipo Config?

[71104]

Originariamente inviato da fek

Il test che ha scritto Vicius richiede un costruttore con parametro di tipo Config?

no, solo Extent, ma io quando inizializzo Gem devo chiamare un costruttore di una super-class; mi sa che un parametro Config ce lo devo mettere per forza perché Sprite lo vuole in tutte le versioni e non lo gestisce (credo) se è null.

[fek]

Originariamente inviato da 71104

no, solo Extent, ma io quando inizializzo Gem devo chiamare un costruttore di una super-class; mi sa che un parametro Config ce lo devo mettere per forza perché Sprite lo vuole in tutte le versioni e non lo gestisce (credo) se è null.

Tu devi solo scrivere quello che richiede il test e nient'altro.
Se il test richiede di passare solo Extents, fai quello.
Se Sprite richiede un Config, devi guardare il codice e probabilmente dovrai fare del refactoring affinche' anche Sprite accetti un Extents.

Il modo piu' sicuro per procedere e' aggiungere un test che richieda un costruttore di Sprite che accetti un Extents e poi proseguire di li' con l'implementazione. A quel punto puoi cambiare uno per volta tutti i test che accettano Config e fargli accettare Extents, controllando ogni volta che tutti i test passino.
Quando hai finito, puoi eliminare il costruttore che accetta Config e tenere solo quello che accetta Extents. Poi passi l'Extents da Gem a Sprite ed il gioco e' fatto.

Il trucco qui e' di fare un passo per volta, avendo sempre i test sul verde quando fai refactoring, cambiando solo pochissimo codice ad ogni passo e continuando a testare.

Se provi a fare tutto il refactoring in un colpo solo, prevedo che sarai costretto a debuggare, e passerai molto tempo con test che falliscono o che non compilano. Alla fine impiegherai piu' tempo che facendo un passo per volta e andando sul sicuro.

[RaouL_BennetH]

perdonate l'intromissione ma per la prima volta ho lanciato JUnit da eclipse e volevo chiedervi:

è normale che riporta una marea di failure? (compila ma è luce rossa?)

[71104]

Originariamente inviato da RaouL_BennetH

perdonate l'intromissione ma per la prima volta ho lanciato JUnit da eclipse e volevo chiedervi:

è normale che riporta una marea di failure? (compila ma è luce rossa?)

è successo anche a me; no, non è normale, probabilmente devi impostare bene il java.library.path (cavolo, come sto diventando esperto, sono COMMOSSO )

per impostarlo vai sulla finestrona delle configurazioni di Run o Debug e nella configurazione di JUnit vai su arguments; poi tra gli argomenti della VM scrivi:

-Djava.library.path=lib/win32

[RaouL_BennetH]

Originariamente inviato da 71104

è successo anche a me; no, non è normale, probabilmente devi impostare bene il java.library.path (cavolo, come sto diventando esperto, sono COMMOSSO )

per impostarlo vai sulla finestrona delle configurazioni di Run o Debug e nella configurazione di JUnit vai su arguments; poi tra gli argomenti della VM scrivi:

-Djava.library.path=lib/win32

Bene,mi ci metto domani allora,sono davvero a pezzi stasera @@.
P.S.:mi sa che sul mio linux "win32" fa incacchiare il sistema

[^TiGeRShArK^]

Originariamente inviato da RaouL_BennetH

perdonate l'intromissione ma per la prima volta ho lanciato JUnit da eclipse e volevo chiedervi:

è normale che riporta una marea di failure? (compila ma è luce rossa?)

rifai l'update...
ti conviene..
non vorrei ke tu abbia bekkato la versione incasinata del repository..
ora funge tutto..

[RaouL_BennetH]

Originariamente inviato da ^TiGeRShArK^

rifai l'update...
ti conviene..
non vorrei ke tu abbia bekkato la versione incasinata del repository..
ora funge tutto..

l'update l'ho fatto proprio pochi secondi fa,e se faccio partire l'applicazione mi funziona.Solo se provo a lanciare JUnit mi da na marea di errori.

[71104]

Originariamente inviato da RaouL_BennetH

Bene,mi ci metto domani allora,sono davvero a pezzi stasera @@.
P.S.:mi sa che sul mio linux "win32" fa incacchiare il sistema

ah e che ne sapevo
allora anziché win32 semplicemente metti linux:

-Djava.library.path=lib/linux

dovrebbe essere così se nn sbaglio...
ciao

[^TiGeRShArK^]

allora devi settare quel maledetta java.property..
(ma solo a me funge tutto senza settarlo??? )

[VICIUS]

Originariamente inviato da 71104

no, solo Extent, ma io quando inizializzo Gem devo chiamare un costruttore di una super-class; mi sa che un parametro Config ce lo devo mettere per forza perché Sprite lo vuole in tutte le versioni e non lo gestisce (credo) se è null.

Non avevo pensato a config quando ho scritto il test. Puoi tranquillamente aggiunger un uovo parametro al costruttore. Visto che ci sei modifica anche createGemForTesting e usa quello nel test.

ciao

[fek]

Originariamente inviato da VICIUS

Non avevo pensato a config quando ho scritto il test. Puoi tranquillamente aggiunger un uovo parametro al costruttore. Visto che ci sei modifica anche createGemForTesting e usa quello nel test.

ciao

Config e Extents comunicano la stessa informazione ma con significato leggermente diverso. E' una duplicazione. O uno o l'altro.

[VICIUS]

Originariamente inviato da fek

Config e Extents comunicano la stessa informazione ma con significato leggermente diverso. E' una duplicazione. O uno o l'altro.

Se non ho capito male Config potrebbe in un futuro portare anche altre informazioni. Il problema è che ora non ci serve a niente. Come ha fatto a finire nei sorgenti.?
Io per ora sposterei tutto il codice di Config da qualche parte dove non possa fare male.

ciao

[cionci]

Ma Extents chi l'ha fatto ?

[71104]

Originariamente inviato da cionci

Ma Extents chi l'ha fatto ?

lo sto facendo io... ^^

perciò allora che devo, fare, cerco di togliere Config dai costruttori di Sprite?
inoltre in Sprite ho aggunto un membro privato di tipo Extents che viene ricevuto dal costruttore e che può anche essere null (in tal caso lo sprite non ha extents e non vengono fatti controlli sul movimento); dite che va bene?

[71104]

altra cosa, VICIUS: il significato dei parametri del costruttore di Extents sarebbe (nell'ordine) left, top, width, height, ho interpretato bene?

[VICIUS]

Originariamente inviato da 71104

lo sto facendo io... ^^

perciò allora che devo, fare, cerco di togliere Config dai costruttori di Sprite?

Procedi pure. Tanto per ora Config non ti serve a niente.

Originariamente inviato da 71104

inoltre in Sprite ho aggunto un membro privato di tipo Extents che viene ricevuto dal costruttore e che può anche essere null (in tal caso lo sprite non ha extents e non vengono fatti controlli sul movimento); dite che va bene?

Per l'implementazione hai carta bianca. Pero cerca di attenerti ai test. Evitare di fare il controllo se è null un campo non è richiesto quindi non lo devi implemetnare.

ciao

[71104]

ragazzi, Config non è inutile in realtà perché Sprite, Texture e altre classi la usano per leggere le dimensioni del display, secondo me *non va* tolta, e anzi sapete una cosa? mi sa che la metto pure nel costruttore di Extents visto che le dimensioni del display mi servono per convertire le coordinate...