no-reply@allegro.cc (AleX-G Squadron) — Sun, 02 Sep 2012 22:00:13 +0000

As i went on with the last game i posted on here, i am having difficulty on adding a boss-like in the game. When the ship.score >= 100 if want to "welcome" the boss in the game. I put this piece of code everywhere and it seems that nothing would solve the problem:

#SelectExpand
 1if(ship.score >= 100)
 2    {
 3    InitComet1(comets1, NUM_COMETS1);
 4    }

Bear in mind that the InitComet1 is another enemy in the game, normally the boss!
The thread with the game code is this one

no-reply@allegro.cc (torhu) — Sun, 02 Sep 2012 22:12:22 +0000

Seems like a good opportunity to learn to use the debugger in msvc. Set a breakpoint on the InitComet1 line (F9 key), press F5 to start debugging. When the score reaches 100, the breakpoint should get triggered. If it doesn't, you need to figure out why. Make sure that the score actually reaches 100, etc. Does it work when you do it unconditionally? That's one thing to try.

If you want use to tell you where the bug is, you have to post more code than that, by the way.

no-reply@allegro.cc (l j) — Sun, 02 Sep 2012 22:12:41 +0000

I would put it were you handle your timer event.

Anyway, does the scoring system work? does the InitComet work?
Does InitComet actually add the enemy to the game?
There is nothing wrong with the code you posted so it must surely be something else.

no-reply@allegro.cc (AleX-G Squadron) — Sun, 02 Sep 2012 22:24:14 +0000

Ok, i added a breakpoint and it triggered.
Added it in the e.type == ALLEGRO_TIMER
The game starts anyway, but no boss appears!!!
If i delete the if statement i added, the boss appears immediately!
The game works perfect, it is just that the boss wont appear when the event is triggered.
I will try different positions and than post here if it works.

I have other 2 demands about the game, so i dont create other threads.
1)How to add High Scores and they save even after you close the game.
2)How to make the enemies "convert colors" when they are hit.

source.cpp

#SelectExpand
   1#include 
   2#include 
   3#include 
   4#include 
   5#include 
   6#include "objects.h"
   7
   8// Buttons
   9enum BUTON{A,S,D,W,SPACE};
  10bool buton[5] = {false,false,false,false,false};
  11
  12// Globals
  13const int FPS = 80, width = 1000, height = 500;
  14const int NUM_BULLETS = 50;
  15const int NUM_COMETS = 10;
  16const int NUM_COMETS1 = 2;
  17int x = width / 2;
  18int y = height / 2;
  19
  20// Protorypes
  21void InitShip(SpaceShip &ship);
  22void DrawShip(SpaceShip &ship);
  23void MoveShipUp(SpaceShip &ship);
  24void MoveShipDown(SpaceShip &ship);
  25void MoveShipLeft(SpaceShip &ship);
  26void MoveShipRight(SpaceShip &ship);
  27
  28void InitBullet(Bullet bullet[], int size);
  29void DrawBullet(Bullet bullet[], int size);
  30void FireBullet(Bullet bullet[], int size, SpaceShip &ship);
  31void UpdateBullet(Bullet bullet[], int size);
  32void CollideBullet(Bullet bullet[], int bSize, Comet comets[], int cSize, SpaceShip &ship);
  33void CollideBullet1(Bullet bullet[], int bSize, Comet1 comets[], int cSize, SpaceShip &ship);
  34
  35void InitComet(Comet comets[], int size);
  36void DrawComet(Comet comet[], int size);
  37void StartComet(Comet comet[], int size);
  38void UpdateComet(Comet comet[], int size);
  39void CollideComet(Comet comets[], int cSize, SpaceShip &ship);
  40
  41void InitComet1(Comet1 comets1[], int size);
  42void DrawComet1(Comet1 comet1[], int size);
  43void StartComet1(Comet1 comet1[], int size);
  44void UpdateComet1(Comet1 comet1[], int size);
  45void CollideComet1(Comet1 comets1[], int cSize, SpaceShip &ship);
  46
  47int main()
  48{
// Initialization
al_init();
al_init_primitives_addon();
al_init_image_addon();
al_init_font_addon();
al_init_ttf_addon();
  55
// Installations
al_install_keyboard();
al_install_mouse();
  59
// Variables
SpaceShip ship;
Bullet bullets[NUM_BULLETS];
Comet comets[NUM_COMETS];
Comet1 comets1[NUM_COMETS1];
  65
// I don't think you are actually using this variable
int fireCooldown = 0;
  68
bool quit = false;
bool draw = true;
bool GameOver = false;
  72
// ALLEGRO variables
ALLEGRO_DISPLAY *window = al_create_display(width, height);
ALLEGRO_BITMAP *icon = al_load_bitmap("image/1.png");
ALLEGRO_FONT *font16 = al_load_ttf_font("fonts/1.ttf", 80, ALLEGRO_ALIGN_CENTRE);
ALLEGRO_TIMER *timer = NULL;
ALLEGRO_EVENT_QUEUE *event = NULL;
  79
event = al_create_event_queue();
timer = al_create_timer(1.0 / FPS);

// Events
al_register_event_source(event, al_get_keyboard_event_source());
al_register_event_source(event, al_get_mouse_event_source());
al_register_event_source(event, al_get_display_event_source(window));
al_register_event_source(event, al_get_timer_event_source(timer));
  88
// Window title, icon and mouse hide
al_set_window_title(window, "Cigarette Life");
al_set_display_icon(window, icon);
al_hide_mouse_cursor(window);

// Player initializations
srand(time(NULL));
InitShip(ship);
InitBullet(bullets, NUM_BULLETS);
InitComet(comets, NUM_COMETS);
  99
// Start timer
al_start_timer(timer);
 102
 103
// The drawing
while(!quit)
{
  // Call of the event
  ALLEGRO_EVENT e;
  al_wait_for_event(event, &e);
 110
  // Drawing
  if(e.type == ALLEGRO_EVENT_TIMER)
  {
    draw = true;
  if(ship.score >= 100)
  {
  InitComet1(comets1, NUM_COMETS1);
  }
  if(buton[A])
    MoveShipLeft(ship);
  if(buton[D])
    MoveShipRight(ship);
  if(buton[S])
    MoveShipDown(ship);
  if(buton[W])
    MoveShipUp(ship);
 127
  // Why didn't you apply the same logic to firing your bullets?
if(!GameOver)
{
    UpdateBullet(bullets, NUM_BULLETS);
    StartComet(comets, NUM_COMETS);
    UpdateComet(comets, NUM_COMETS);
    StartComet1(comets1, NUM_COMETS1);
    UpdateComet1(comets1, NUM_COMETS1);
  CollideBullet(bullets, NUM_BULLETS, comets, NUM_COMETS, ship);
  CollideComet(comets, NUM_COMETS, ship);
  CollideBullet1(bullets, NUM_BULLETS, comets1, NUM_COMETS, ship);
  CollideComet1(comets1, NUM_COMETS1, ship);
  
  if(ship.lives <= 0)
  {
    GameOver = true;
  }
}
 146
  if(buton[SPACE])
  {
     // also it's button, you got it right in the comments though.
     // Or is this a different language?
     if(fireCooldown == 0)
     {
        // We'll decrement this variable later
        // This block will only get executed if fireCooldown equals 0
        fireCooldown = 25;
        // Now that we've assigned 10 to fireCooldown we can be sure the next bullet is not going to be fired for a couple of frames
        FireBullet(bullets, NUM_BULLETS, ship);    
     }
  }
  
  // Let's get the ship ready for the next shot
  if(fireCooldown > 0)
     fireCooldown--;
  }
 165
  // Key down or up
  switch(e.type)
  {
    case ALLEGRO_EVENT_KEY_DOWN:
      switch(e.keyboard.keycode)
      {
      case ALLEGRO_KEY_A:
        buton[A] = true;
        break;
 175
      case ALLEGRO_KEY_S:
        buton[S] = true;
        break;
 179
      case ALLEGRO_KEY_D:
        buton[D] = true;
        break;
 183
      case ALLEGRO_KEY_W:
        buton[W] = true;
        break;
 187
  case ALLEGRO_KEY_P:
    GameOver = !GameOver;
    break;
 191
      case ALLEGRO_KEY_SPACE:
        buton[SPACE] = true;
        // Now you are going to fire a bullet each time you press your spacebar, so let's move this out of here
        // FireBullet(bullets, NUM_BULLETS, ship);
        break;
 197
      case ALLEGRO_KEY_ESCAPE:
        quit = true;
        break;
      }
  }
 203
  switch(e.type)
  {
    case ALLEGRO_EVENT_KEY_UP:
      switch(e.keyboard.keycode)
      {
      case ALLEGRO_KEY_A:
        buton[A] = false;
        break;
 212
      case ALLEGRO_KEY_S:
        buton[S] = false;
        break;
 216
      case ALLEGRO_KEY_D:
        buton[D] = false;
        break;
 220
      case ALLEGRO_KEY_W:
        buton[W] = false;
        break;
 224
      case ALLEGRO_KEY_SPACE:
        buton[SPACE] = false;
        break;
      }
  }
 230
 231
  // Close display when X is pressed
  if(e.type == ALLEGRO_EVENT_DISPLAY_CLOSE)
  {
    quit = true;
  }
 237
  // Mouse movement
  else if(e.type == ALLEGRO_EVENT_MOUSE_AXES)
  {
    ship.y = e.mouse.y;
  }
  
  // Bad idea, an ALLEGRO_EVENT is an union
  // Meaning that multiple different members can take the same place in memory
  // Other events might 'overwrite' e.mouse.button, causing it to falsely evaluate to true and this is more likely than you might think as it caused the firing to seem irregular with this not being commented
  /*else if(e.mouse.button & 1)
  {
    FireBullet(bullets, NUM_BULLETS, ship);
  }*/
  if(e.type == ALLEGRO_EVENT_MOUSE_BUTTON_DOWN)
  {
     if(e.mouse.button & 1)
     {
        buton[SPACE] = true; // Ugly hack but it works
     }
  }
  else if(e.type == ALLEGRO_EVENT_MOUSE_BUTTON_UP)
  {
     if(e.mouse.button & 1)
     {
        buton[SPACE] = false;
     }
  }
 265
  if(draw && al_event_queue_is_empty(event))
  {
    draw = false;
  if(!GameOver)
  {
  
    DrawShip(ship);
    DrawBullet(bullets, NUM_BULLETS);
    DrawComet(comets, NUM_COMETS);
    DrawComet1(comets1, NUM_COMETS1);
 276
    al_draw_textf(font16, al_map_rgb(255,0,0), x + 300, 10, ALLEGRO_ALIGN_CENTRE, "Lifes: %i Points %i", ship.lives, ship.score);
  }
  else
  {
    al_draw_textf(font16, al_map_rgb(255,0,0), x, 100, ALLEGRO_ALIGN_CENTRE, "Game Over");
    al_draw_textf(font16, al_map_rgb(255,0,0), x, 200, ALLEGRO_ALIGN_CENTRE, "Final Score: %i", ship.score);
  }
 284
    al_flip_display();
    al_clear_to_color(al_map_rgb(0,0,0));
  }
 288
}
 290
// Destroy
al_destroy_event_queue(event);
al_destroy_bitmap(icon);
al_destroy_timer(timer);
al_destroy_display(window);
 296}
 297
 298void InitShip(SpaceShip &ship)
 299{
ship.x = 20;
ship.y = height / 2;
ship.ID = PLAYER;
ship.lives = 3;
ship.speed = 7;
ship.boundx = 126;
ship.boundy = 0;
ship.score = 0;
 308}
 309
 310void DrawShip(SpaceShip &ship)
 311{
al_draw_filled_rectangle(ship.x - 10, ship.y-15, ship.x + 35, ship.y, al_map_rgb(255,128,0)); 
al_draw_filled_rectangle(ship.x + 35, ship.y-15, ship.x + 130, ship.y, al_map_rgb(255,255,255)); 
al_draw_line(ship.x + 30, ship.y-15, ship.x + 30, ship.y, al_map_rgb(255,255,50), 2.0); 
 315}
 316
 317void MoveShipUp(SpaceShip &ship)
 318{
ship.y -= ship.speed;
if(ship.y < 15)
  ship.y = 15;
 322}
 323
 324void MoveShipDown(SpaceShip &ship)
 325{
ship.y += ship.speed;
if(ship.y > height)
  ship.y = height;
 329}
 330
 331void MoveShipLeft(SpaceShip &ship)
 332{
ship.x -= ship.speed;
if(ship.x < 10)
  ship.x = 10;
 336}
 337
 338void MoveShipRight(SpaceShip &ship)
 339{
ship.x += ship.speed;
if(ship.x > 100)
  ship.x = 100;
 343}
 344
 345void InitBullet(Bullet bullet[], int size)
 346{
for(int i = 0; i < size; i++)
{
  bullet[i].ID = BULLET;
  bullet[i].speed = 10;
  bullet[i].live = false;
}
 353}
 354
 355void DrawBullet(Bullet bullet[], int size)
 356{
for(int i = 0; i < size; i++)
{
  if(bullet[i].live)
    al_draw_filled_circle(bullet[i].x, bullet[i].y, 5, al_map_rgb(128,128,128));
}
 362}
 363
 364void FireBullet(Bullet bullet[], int size, SpaceShip &ship)
 365{
for(int i = 0; i < size; i++)
{
  if(!bullet[i].live)
  {
    bullet[i].x = ship.x + 120;
    bullet[i].y = ship.y - 7;
    bullet[i].live = true;
    break;
  }
}
 376}
 377
 378void UpdateBullet(Bullet bullet[], int size)
 379{
for(int i = 0; i < size; i++)
{
  if(bullet[i].live)
  {
    bullet[i].x += bullet[i].speed;
    if(bullet[i].x > width)
    {
      bullet[i].live = false;
    break;
    }
  }
}
 392}
 393
 394void CollideBullet(Bullet bullet[], int bSize, Comet comets[], int cSize, SpaceShip &ship)
 395{
for(int i = 0; i < bSize; i++)
{
  if(bullet[i].live)
  {
    for(int j = 0; j < cSize; j++)
    {
      if(comets[j].live > 0)
      {
        if(bullet[i].x > (comets[j].x - comets[j].boundx) &&
          bullet[i].x < (comets[j].x + comets[j].boundx) &&
          bullet[i].y > (comets[j].y - comets[j].boundy) &&
          bullet[i].y < (comets[j].y + comets[j].boundy))
        {
          bullet[i].live = false;
          comets[j].live -= 1;
          ship.score += 5;
        }
      }
    }
  }
}
 417}
 418
 419void CollideBullet1(Bullet bullet[], int bSize, Comet1 comets1[], int cSize, SpaceShip &ship)
 420{
for(int i = 0; i < bSize; i++)
{
  if(bullet[i].live)
  {
    for(int j = 0; j < cSize; j++)
    {
      if(comets1[j].live > 0)
      {
        if(bullet[i].x > (comets1[j].x - comets1[j].boundx) &&
          bullet[i].x < (comets1[j].x + comets1[j].boundx) &&
          bullet[i].y > (comets1[j].y - comets1[j].boundy) &&
          bullet[i].y < (comets1[j].y + comets1[j].boundy))
        {
          bullet[i].live = false;
          comets1[j].live -= 1;
          if(comets1[j].live == 0)
          {
            ship.score += 25;
          }
        }
      }
    }
  }
}
 445}
 446
 447void InitComet(Comet comets[], int size)
 448{
for(int i = 0; i < size; i++)
{
  comets[i].ID = ENEMY;
  comets[i].live = 2;
  comets[i].speed = 2;
  comets[i].boundx = 20;
  comets[i].boundy = 30;
}
 457}
 458
 459void DrawComet(Comet comets[], int size)
 460{
for(int i = 0; i < size; i++)
{
  if(comets[i].live > 0)
  {
    al_draw_filled_ellipse(comets[i].x, comets[i].y, 20, 30, al_map_rgb(255,0,0));
    al_draw_ellipse(comets[i].x, comets[i].y, 10, 20, al_map_rgb(0,0,0), 5.0);
  }
}
 469}
 470
 471void StartComet(Comet comets[], int size)
 472{
for(int i = 0; i < size; i++)
{
  if(comets[i].live == 0)
  {
    if(rand() % 500 == 0)
    {
      comets[i].live += 1;
      comets[i].x = width;
      comets[i].y = 30 + rand() % (height - 60);
      break;
    }
  }
}
 486}
 487
 488void UpdateComet(Comet comets[], int size)
 489{
for(int i = 0; i < size; i++)
{
  if(comets[i].live > 0)
  {
    comets[i].x -= comets[i].speed;
 495
    if(comets[i].x < 0)
    {
      comets[i].live = 0;
    }
  }
}
 502}
 503
 504void CollideComet(Comet comets[], int cSize, SpaceShip &ship)
 505{
for(int i = 0; i < cSize; i++)
{
  if(comets[i].live > 0)
  {
    if(comets[i].x - comets[i].boundx < ship.x + ship.boundx &&
      comets[i].x + comets[i].boundx > ship.x - ship.boundx &&
      comets[i].y - comets[i].boundy < ship.y + ship.boundy &&
      comets[i].y + comets[i].boundy > ship.y - ship.boundy)
    {
      ship.lives--;
      comets[i].live = 0;
    }
    else if(comets[i].x < 10)
    {
      ship.lives--;
      comets[i].live = 0;
    }
  }
}
 525}
 526
 527
 528void InitComet1(Comet1 comets1[], int size)
 529{
for(int i = 0; i < size; i++)
{
  comets1[i].ID = ENEMY;
  comets1[i].live = 5;
  comets1[i].speed = 1;
  comets1[i].boundx = 20;
  comets1[i].boundy = 30;
}
 538}
 539
 540void DrawComet1(Comet1 comets1[], int size)
 541{
for(int i = 0; i < size; i++)
{
  if(comets1[i].live > 0)
  {
    al_draw_filled_ellipse(comets1[i].x, comets1[i].y, 20, 30, al_map_rgb(255,255,255));
    al_draw_ellipse(comets1[i].x, comets1[i].y, 10, 20, al_map_rgb(0,0,0), 5.0);
  }
}
 550}
 551
 552void StartComet1(Comet1 comets1[], int size)
 553{
for(int i = 0; i < size; i++)
{
  if(comets1[i].live == 0)
  {
    if(rand() % 500 == 0)
    {
      comets1[i].live += 5;
      comets1[i].x = width;
      comets1[i].y = 30 + rand() % (height - 60);
      break;
    }
  }
}
 567}
 568
 569void UpdateComet1(Comet1 comets1[], int size)
 570{
for(int i = 0; i < size; i++)
{
  if(comets1[i].live > 0)
  {
    comets1[i].x -= comets1[i].speed;
 576
    if(comets1[i].x < 0)
    {
      comets1[i].live = 0;
    }
  }
}
 583}
 584
 585void CollideComet1(Comet1 comets1[], int cSize, SpaceShip &ship)
 586{
for(int i = 0; i < cSize; i++)
{
  if(comets1[i].live > 0)
  {
    if(comets1[i].x - comets1[i].boundx < ship.x + ship.boundx &&
      comets1[i].x + comets1[i].boundx > ship.x - ship.boundx &&
      comets1[i].y - comets1[i].boundy < ship.y + ship.boundy &&
      comets1[i].y + comets1[i].boundy > ship.y - ship.boundy)
    {
      ship.lives = 0;
      comets1[i].live = 0;
    }
    else if(comets1[i].x < 10)
    {
      ship.lives = 0;
      comets1[i].live = 0;
    }
  }
}
 606}

The boss has equal things as the other enemy, just that some parameters change.
Life is 5 speed is 1 and you earn more.
I added a Pause button with P

no-reply@allegro.cc (l j) — Sun, 02 Sep 2012 23:33:47 +0000

#SelectExpand
  1void StartComet1(Comet1 comets1[], int size)
  2{
for(int i = 0; i < size; i++)
{
  // I changed the == to != and now it works as expected
  if(comets1[i].live != 0)
  {
    if(rand() % 500 == 0)
    {
      comets1[i].live += 5;
      comets1[i].x = width;
      comets1[i].y = 30 + rand() % (height - 60);
      break;
    }
  }
}
 17}

AleX-G Squadron said:

I have other 2 demands about the game, so i dont create other threads.

Demands sounds rather domineering in English. I would use an alternative like question if you don't want to sound bossy.

High scores will require 2 things.

File input/output
Sorting values

File input/output can be done using allegro, the standard C library or the standard C++ library.
Sorting can be done using the C++ standard template library or the standard C library.

Making enemies convert colors can be done by drawing them in different colors?
You'll need a counter for each comet/enemy or something to keep track of when they were last hit.

no-reply@allegro.cc (AleX-G Squadron) — Mon, 03 Sep 2012 00:34:37 +0000

Ok, sorry!
It worked, but there is a very irritating thing.
The boss like thing disappears in the middle of the road!

Game Boss-Like