Ok I've started a 2d game that's supposed to scroll and as it scrolls it'll load the new bits of map, the issue I'm having with that is it's jumping. Everytime the character moves 32 pixels he jumps another 32 and centers on the next tile.
My second issue is that the collision I have setup doesnt register ,it thinks every tile is a wall until Im off the map completely. I only setup the collision on the UP movement to test it otherwise I wouldnt be able to move at all. Weirdest part is the drawing function still see's the floors and draws them properly.
Here is the code I have. I'm using C++ and Allegro 5:
This is for going up (decreasing y). Why do you check at y-2, if you just go to y-1? Why do you check two tiles? And why do you check only one if x is a multiple of 32?
Other issues:
what do you put in levelCurrent? Prove that your tiles are walkable.
when do you set and unset key[] flags? Maybe you move more than once per key press.
I check at y-2 because he moves at a rate of 2 pixels normally. I had changed it just to check a few things.
Tiles have a walkable integer. if its 1 you can walk on it 0 you cant.
Key flags are checked per frame.
int wait = 0;
int direction=0;
int x=0,y=0,levelx=0,levely=0;
int a,b,c;
int d=0;
int onlevel=1;
int view=1;//view: 1=Main menu, 2=Game, 3=Inventory, 4=Character sheet, 5=Pause, 6=Spellbook
struct Object{
unsigned int x, y;
unsigned int width,height;
};
struct Bizzle{
int hp,mp,xp;
int str,dex,intel,con,def,stats;
int fire,water,wind,earth;
int abeast,afly,aundead,adragon,amagic,agiant,ahunger,aplant;
int dbeast,dfly,dundead,ddragon,dmagic,dgiant,dhunger,dplant;
int lodmg,hidmg;
Object Collision;
};
struct Enemy{
int hp,ap,name,graphic,xp,type1,type2,element;
int x,y;
bool moved;
Object Collision;
};
struct Item{
int type; //1=weapon,2=armor,3=consumable,4=parts
int loDmg,hiDmg; //damage buffs
int dex,str,con,intel,def; //stat buffs
int Rdex,Rstr,Rcon,Rintel; //required stats for use
int fire,water,earth,wind; //Weapons: element buffs | Armor: element resistance
int beast,fly,undead,dragon,magic,giant,hunger,plant; //Weapons: type buffs | Armor: type resistance
int hp,mp;
int x,y;
Object Collision;
};
struct Tile{
Object Collision;
unsigned short int walkable;
};
#endif
When you draw the map, why are you adding the players grid position to the map grid position in your checks? Don't you want to draw the entire mini map the whole time?
Ie...
for (b = 0 ; b < 20 ; ++b) {
for (c = 0 ; c < 20 ; ++c) {
if (levelCurrent[c].walkable == 1) {
al_draw_pixel(c , b , al_map_rgb(0,127,255));
}
}
}
Next, to clean up your map drawing, you need to set up a camera and have it follow your player's position on the map.
int camx = 0;
int camy = 0;
// whenever you set the position of your player, reset your cameras position as well
// center on the player's center
camx = player_x + player_width/2 - view_width/2;
camy = player_y + player_height/2 - view_height/2;
// make sure the camera stays over the map
if (camx < 0) {camx = 0;}
if (camy < 0) {camy = 0;}
if (camx > (map_width - view_width)) {camx = map_width - view_width;}
if (camy > (map_height - view_height)) {camy = map_height - view_height;}
Now you draw your walls based on the camera position :
int start_tile_x = camx/tile_width;
int offset_x = -(camx%tile_width);
int start_tile_y = camy/tile_height;
int offset_y = -(camy%tile_height);
int num_tiles_wide = view_width/tile_width + 1;
int num_tiles_tall = view_height/tile_height + 1;
for (b = start_tile_y ; b < start_tile_y + num_tiles_tall ; ++b) {
int drawy = (b - start_tile_y)*tile_height + offset_y;
for (c = start_tile_x ; c < start_tile_x + num_tiles_wide ; ++c) {
int drawx = (c - start_tile_x)*tile_width + offset_x;
if (levelCurrent[c].walkable == 0) {
al_draw_bitmap(wall , drawx , drawy , 0);
}
}
}
I don't know why your code caused the map to 'skip' a tile here and there, but this code shouldn't suffer from the same problem.
Now when you draw your player, you will need to subtract the camera position from the player position to get the screen position :
int player_screen_x = player_x - camx;
int player_screen_y = player_y - camy;
al_draw_bitmap(player , player_screen_x , player_screen_y , 0);
To use this code, you'll have to substitute in values for tile width and height and view width and height. A better idea would be to keep them as variables though instead of hard coded constants in case you ever change your mind about how big your tiles or your view should be. It would also be nice to see better variable names instead of things like b and c and x and y. If you have any questions, just ask ,and I'll explain as best I can.
[/font]
