STL and BITMAPs

STL and BITMAPs

Don Freeman

Member #5,110

October 2004

Is it possible to use STL with allegro BITMAP structures?

such as:

list<BITMAP> images; //or list<BITMAP*> images;
list<BITMAP>::iterator iter // or list<BITMAP*>::iterator iter;
images.insert(iter,*(load_bitmap("SomeImage.bmp",NULL))); // or images.insert(iter,load_bitmap("SomeImage.bmp",NULL));

I've tried but can't seem to get it to work correctly. I am not very experienced with using STL... I can get it to store an image, but when you add another one, it replaces all the others in the list!:o

Thanks as always,
Donald

--
"Everyone tells me I should forget about you, you don’t deserve me. They’re right, you don’t deserve me, but I deserve you."
"It’s so simple to be wise. Just think of something stupid to say and then don’t say it."

X-G

Member #856

December 2000

BITMAP *

--
Since 2008-Jun-18, democracy in Sweden is dead. | 悪霊退散！悪霊退散！怨霊、物の怪、困った時は　ドーマン！セーマン！ドーマン！セーマン！　直ぐに呼びましょう陰陽師レッツゴー！

Rampage

Member #3,035

December 2002

list<BITMAP*> images

[edit]

Iiiii'mmmmm ssssslllllooooowwwww.

-R

Don Freeman

Member #5,110

October 2004

So, basically no... The allegro BITMAP structure just holds the basic layout...it also holds the image data, but in an array...

I am confused. I would have to write a "wrapper" class around BITMAP? And if so, how would I handle creation/deletion properly?

X-G

Member #856

December 2000

Store BITMAP *. Iterate through the list and destroy_bitmap them. That's the easiest way.

Rampage

Member #3,035

December 2002

The allegro functions work with pointers to BITMAP structures, not with the structures themselves. You just have to do the same with the STL functions, you must work with pointers.

-R

Trezker

Member #1,739

December 2001

There is no problem.

Creation:
std::list<BITMAP*> bmps;
BITMAP *bmp=create_bitmap(...);
bmps.push_back(bmp);

Removal:
for(std::list<BITMAP*>::iterator i=bmps.begin(); i!=bmps.end(); ++i)
destroy_bitmap(*i);
bmps.clear();

I think that's right...

[My github]

Don Freeman

Member #5,110

October 2004

This is how I get it to SORTA work...
But, like I said...when the Forest image is loaded...bye-bye to the Plains image....

1//...
2//allegro startup code...
3//...
  list<BITMAP*> *images = new list<BITMAP*>;
  list<BITMAP*>::iterator it;
  it = images->begin();
  BITMAP *tmp = load_bitmap("WarLand/GFX/Terrain/Plains.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images->insert(it,tmp);
  destroy_bitmap(tmp);
  it++;
  tmp =  load_bitmap("WarLand/GFX/Terrain/Forest.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images->insert(it,tmp);
  destroy_bitmap(tmp);
  clear_keybuf();
  int x = 0;
  while ( !keypressed() )
  {
it = images->begin();
x+=40;
if ( x > 160 )
    x = 0;
if ( it == images->end() )
{
    it = images->begin();
}
++it;
draw_sprite(screen,(*it),x,0);
rest(1);
  }
  delete images;
39return 0;
40}END_OF_MAIN()

Rampage

Member #3,035

December 2002

Don't you have to create the iterator after filling the list? You don't need an iterator to add elements to a list, just to get them out.

-R

Trezker

Member #1,739

December 2001

Well DUH! You're destroying it. ::)

[My github]

Don Freeman

Member #5,110

October 2004

Doesn't matter...both do the same thing...(creating the iterator before or after)

Rampage

Member #3,035

December 2002

Oh yeah, Trezker is right! You are destroying the bitmap after the insertion! Your list contains only invalid pointers.

[edit]

X-G mentioned the solution in his second post on the topic.

-R

Don Freeman

Member #5,110

October 2004

Yeah...I'm special::) We can do that on Mars...8-)

Ok....here is the "working" code...

1///////////////////////////////////////////////////////////////////////////////////////////////////////////////
2#include <list>
3using std::list;
4int main(int argc, char *argv[])
5{
  allegro_init();
  set_color_depth(32);
  set_gfx_mode(GFX_AUTODETECT_WINDOWED,640,480,0,0);
  set_color_conversion(COLORCONV_TOTAL);
  text_mode(-1);
  install_timer();
  install_keyboard();
  install_mouse();
  ///////////////////////////////////////////////////
  list<BITMAP*> *images = new list<BITMAP*>;
  list<BITMAP*>::iterator it;
  it = images->begin();
  BITMAP *tmp = load_bitmap("WarLand/GFX/Terrain/Plains.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images->insert(it,tmp);
  //destroy_bitmap(tmp);
  it++;
  tmp =  load_bitmap("WarLand/GFX/Terrain/Forest.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images->insert(it,tmp);
  //destroy_bitmap(tmp);
  clear_keybuf();
  int x = 0;
  while ( !keypressed() )
  {
for ( it = images->begin(); it != images->end(); ++it )
{
    draw_sprite(screen,(*it),x,0);
    x+=40;
}
x = 0;
rest(1);
  }
  for ( it = images->begin(); it != images->end(); ++it )
  {
destroy_bitmap((*it));
  }
  images->clear();
  delete images;
  clear_keybuf();
  return 0;
53}END_OF_MAIN();

Thanks again to all of you:-*,
Donald

Trezker

Member #1,739

December 2001

1//...
2//allegro startup code...
3//...
  list<BITMAP*> images;
  list<BITMAP*>::iterator it;
6 
  BITMAP *tmp = load_bitmap("WarLand/GFX/Terrain/Plains.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images.push_back(tmp);
13 
  tmp =  load_bitmap("WarLand/GFX/Terrain/Forest.bmp",NULL);
  if ( !tmp )
printf("Cound not create BITMAP structure...\n");
  else
printf("Found image and you should see it...\n");
  images.push_back(tmp);
20 
  clear_keybuf();
  int x = 0;
  while ( !keypressed() )
  {
      for(it=images.begin(); it!=images.end(); ++it)
      {
          x+=40;
    if ( x > 160 )
        x = 0;
      draw_sprite(screen,(*it),x,0);
      }
rest(1);
  }
  for(it=images.begin(); it!=images.end(); ++it)
  {
      destroy_bitmap(*it);
  }
  images.clear();
39return 0;
40}END_OF_MAIN()

EDIT: This thread is fast.

[My github]

Don Freeman

Member #5,110

October 2004

Trezker: So you would call delete on the bitmap object? Yes...I should use the push_back instead...

Rampage

Member #3,035

December 2002

You should use destroy_bitmap(), not delete...

-R

Trezker

Member #1,739

December 2001

Oops, no delete.
Must call destroy_bitmap function.
I'm too used to C++ objects.

[My github]