loading Bitmaps into a vector

loading Bitmaps into a vector

Mark Oates

Member #1,146

March 2001

ok here's the code (using OpenLayer):

1#define FA_ALL FA_RDONLY | FA_HIDDEN | FA_SYSTEM | FA_LABEL | FA_DIREC | FA_ARCH
2 
3class graphics_database_class
4{
5public:
   vector<Bitmap> image;
   
   int draw(int t, float xx, float yy, float zz)
   {
        if ((t >= image.size()) || (t < 0) || (!tile[t])) return 0;
        draw_image(&image[t], xx, yy, zz);
        return 1;
   }
14 
   void load()
   {
       al_ffblk info;
       string gimp_string = "";
  
       if (al_findfirst("graphics/tiles/*.bmp", &info, FA_ALL) == 0)
       {
           do
           {
             gimp_string = "graphics/tiles/" + ToString(info.name);
             image.push_back(Bitmap(gimp_string.c_str()));
             image[image.size()-1].Load(gimp_string.c_str());
           } while (al_findnext(&info) == 0);
       }
       else allegro_message("no bitmap images found");
  
       al_findclose(&info);
   }     
33};

it's a class that contains all the bmp images in the "graphics/tiles/" directory and puts each one into the "vector<Bitmap> image", giving them each an index number, and allowing you to draw them using the index number. But it doesn't work because when displaying the images, they are all the last image that was loaded. What's going on?

--
Visit CLUBCATT.com for cat shirts, cat mugs, puzzles, art and more <-- coupon code ALLEGRO4LIFE at checkout and get $3 off any order of 3 or more items!

AllegroFlare • AllegroFlare Docs • AllegroFlare GitHub

HoHo

Member #4,534

April 2004

Have you tried printing out gimp_string in the load() function?

__________
In theory, there is no difference between theory and practice. But, in practice, there is - Jan L.A. van de Snepscheut
MMORPG's...Many Men Online Role Playing Girls - Radagar
"Is Java REALLY slower? Does STL really bloat your exes? Find out with your friendly host, HoHo, and his benchmarking machine!" - Jakub Wasilewski

Mark Oates

Member #1,146

March 2001

the gimp_string gets the right address for each image. I also tried slapping in if (!image[image.size()-1]) allegro_message("blurb!"); and the images load without error.

It's like they are being overwritten with the wrong Bitmap address or something.

--
Visit CLUBCATT.com for cat shirts, cat mugs, puzzles, art and more <-- coupon code ALLEGRO4LIFE at checkout and get $3 off any order of 3 or more items!

AllegroFlare • AllegroFlare Docs • AllegroFlare GitHub

Don Freeman

Member #5,110

October 2004

I would try to load them into a dumby BITMAP * first to test they are correct and then add them to the vector.

Example:

1//////////////////////////////////////////////////////////////////////////////////////////////////
2void load()
3{
al_ffblk info;
string gimp_string = "";
6 
  BITMAP *pBMP = NULL;
8 
if (al_findfirst("graphics/tiles/*.bmp", &info, FA_ALL) == 0)
{
  do
  {
    // get file name
    gimp_string = "graphics/tiles/" + ToString(info.name);
    // load the image to the pointer
    pBMP = load_bitmap(gimp_string.cstr(),NULL);
    // push the image pointer into the vector
    image.push_back(pBMP);
    // clean up the temp pointer so no mem leak
    destroy_bitmap(pBMP);
    pBMP = NULL;
    // I don't know what you are doing here...
    // looks like loading the image twice?
    //image[image.size()-1].Load(gimp_string.c_str());
  }
  while (al_findnext(&info) == 0){}
}
else 
  allegro_message("no bitmap images found");
al_findclose(&info);
31}
32//////////////////////////////////////////////////////////////////////////////////////////////////

This should work...thought some of the functions you are using such as the gimp functions, I've never used or seen...so I am guessing at how they work.

Also...you may not need to destroy the bitmap pointer, as I can't remeber how allegro handles the memory. I would try it with it first....and if you get an access error, then remove the destroy_bitmap(pBMP); and pBMP = NULL;

--
"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."

HoHo

Member #4,534

April 2004

It seems to be correct but as you say there must be a problem somewhere

I'll just list some stupid ideas

Have you checked the value of image.size()?
Does the Bitmap::Load function work correctly?
Are the image frames really different from each other?
Where do you initialize the image vector an what size does it has right before calling the load function?

If everything else fails use a debugger to step through your program

Don Freeman

Member #5,110

October 2004

And if you are using the image.Load(...) function above to load the next image...that is your error!

If you are trying:

image[image.size()-1].Load(gimp_string.c_str());

then image.size()-1 would be the image you just loaded!

Edit:
This is because of the push_back function. Everything is added to the end.

Mark Oates

Member #1,146

March 2001

if I comment out image[image.size()-1].Load(gimp_string.c_str()); then all the images are white. So the idea is that I push_back(*garbage*) then re-load the image with that last line.

HoHo:
I double checked your list and everything is there, each image is different (see attached), the vector has a size of 5 and starts out empty.

--
Visit CLUBCATT.com for cat shirts, cat mugs, puzzles, art and more <-- coupon code ALLEGRO4LIFE at checkout and get $3 off any order of 3 or more items!

AllegroFlare • AllegroFlare Docs • AllegroFlare GitHub

HoHo

Member #4,534

April 2004

Does it happen to be that in Bitmap class you store BITMAP pointers and for some reason they are pointing to the exact same image? Perhaps some static or uninitialized variable causes it.

Could you post more source? Bitmap class definition, constructor and Bitmap::Load functions would probably help.

Mark Oates

Member #1,146

March 2001

it's OpenLayer's Bitmap...

--
Visit CLUBCATT.com for cat shirts, cat mugs, puzzles, art and more <-- coupon code ALLEGRO4LIFE at checkout and get $3 off any order of 3 or more items!

AllegroFlare • AllegroFlare Docs • AllegroFlare GitHub

Don Freeman

Member #5,110

October 2004

Are you loading the images after initializing allegro? You may not know, but even objects that are created before allegro is initialized (if they contain BITMAPs) will cause the BITMAPs to be garbage.

Also, I agree that more source will reveal the problem...it looks like there is something somewhere else too that is causing this.

Edit:

Quote:

if (!image[image.size()-1]) allegro_message("blurb!");

This does not neccessarly mean that the image is correct. The way the vector is optimized is that it adds at least 2 to 3 extra spots. The vector may not be growing correctly.

HoHo

Member #4,534

April 2004

Quote:

it's OpenLayer's Bitmap...

heh, I should have thougt of that
Then I would start checking if the pointers in the vector are the same. If they are then there is something quite wrong.

Quote:

This does not neccessarly mean that the image is correct

There are two separate things: size and capacity. Usually, when vector resizes itself its capacity increases 50% but its size doesn't.