Classe Tile

[^TiGeRShArK^]

Allora...
ieri non avendo un cazzo da fare mi ero messo a parlare col coach riguardo un errore concettuale presente nel gioco.
In pratica i background utilizzavano degli Sprite, mentre, se non sbaglio, gli Sprite dovrebbero essere utilizzati solo per gli oggetti in movimento.
Per questo ho creato una classe Tile che idealmente dovrebbe anche essere "padre" di Sprite in modo che quest'ultima classe aggiunga solo le funzionalità relative al movimento.
di seguito posto i test:

public class TestTile extends TestCase
{
private Tile tile;
private MockEngine engine;


protected void setUp() throws Exception
{
engine = new MockEngine();
tile = new Tile("back000", ".jpg", 0, 0, engine);
}


public void testCreation()
{
assertTrue("Unable to load texture", tile.loaded());
}


public void testOrigin()
{
tile.setOrigin(10, 10);
assertEquals("Error in origin position: wrong x coord", 10, tile.getX());
assertEquals("Error in origin position: wrong y coord", 10, tile.getY());
}


public void testSize()
{
assertEquals("Error in tile size: wrong width", 1024, tile.getWidth());
assertEquals("Error in tile size: wrong heigth", 1024, tile.getHeigth());
}


public void testDraw()
{
Point origin = new Point(tile.getX(), tile.getY());
Rectangle viewArea = new Rectangle(0, 0, engine.getDisplayWidth(), engine.getDisplayHeight());
engine.drawQuad(origin, tile.getWidth(), tile.getHeigth(), tile.getTexture(), viewArea);
assertEquals("Error in drawing Tile", 1, engine.getNumberOfQuadsDrawn());
}
}


invece il codice della classe è il seguente:

public class Tile implements Drawable
{

private boolean loaded;
private Point origin;
private Texture texture;
private Rectangle viewArea;

public Tile(String filename, String type, int originX, int originY, AbstractEngine engine)
{
loadTexture(filename, type);
setOrigin(originX, originY);
viewArea = new Rectangle(0, 0, engine.getDisplayWidth(), engine.getDisplayHeight());
}

public boolean loaded()
{
return loaded;
}

private void loadTexture(String filename, String type)
{
texture = new Texture(filename, type);
loaded = true;
}

public void setOrigin(int x, int y)
{
origin = new Point(x, y);
}

public int getX()
{
return new Float(origin.getX()).intValue();
}

public int getY()
{
return new Float(origin.getY()).intValue();
}

public int getWidth()
{
return texture.getWidth();
}

public int getHeigth()
{
return texture.getHeight();
}

public void draw(AbstractEngine engine)
{
engine.drawQuad(origin, getWidth(), getHeigth(), texture, viewArea);
}

public Texture getTexture()
{
return texture;
}

}


da notare ke il tutto è ancora un pò grezzo e bisogna aggiustare qualkosina...
inoltre finchè non verrà implementato il caricamento di texture che non siano una potenza di due sarà necessario passargli un Rectangle per impostare la viewArea, dato che prendendo le dimensioni dall'immagine molte volte si esce dalle dimensioni delle schermo e si genera un eccezione.
Ordunque.....
Pareri???
Suggerimenti???
Critiche???
inviti gratuiti ad andare a cacare??? :D

cmq fatemi sapere che ne pensate perchè per ora il coach non c'è e quindi ne discuto un pò con voi....
intanto vado a comprare le sigarette... aspetto numerosi i vostri interventi! :D

[VICIUS]

Personalmente la trovo una buona anzi ottima idea. Puoi continuare e fare il commit quando lo ritieni pronto.

ciao ;)

[^TiGeRShArK^]

ormai domani sera... stasera sono troppo fuso! :D

[fek]

Sono solo indecise se chiamarla Tile oppure quello che effettivamente e': un Quad sullo schermo.

I Tile sono in realta' un'altra cosa (piastrelle).

PS. Questo forum e' k-free :)

[^TiGeRShArK^]

scusatemi..
ma x ora non me la sento di lavorare sul codice...
troppi casini :(
vedrò di farlo quando starò un pò meglio...
se intanto volete farlo voi... liberissimi di farlo il codice è praticamente tutto quello che ho postato....
:(

[cdimauro]

Io lo chiamerei BOB (Blitter OBject), in onore dei bei tempi dell'Amiga. :D

Al limite GraphicObject o simile. :-P