Thread: Declaring SDL_Rect variables in a class?

Hey all, I've stumbled upon the SDL library, and lately I've been obsessed with c++.

I'm making a simple game with a spaceship that orbits around the sun, currently I'm in the beginning stages. I want to declare a Spaceship Class, and each class I want to have 2 SLD_Rect structs.

As of now I initialize the 2 SDl_Rect structs, and then in another function declare all the structs variables, but when I compile the program it just crashes.
Here is the Space Ship class.

Code:

class Spaceship
{
      public:
                  short int mass;
                  SDL_Rect dest, source;
                  bool initialize(short int locationX, short int locationY, short int imageheight, short int imagewidth)
                  {
                       dest.x = 100;
                       dest.y = 100;
                       dest.w = imagewidth;///this needs to either have image passed into it or needs to have images height and width
                       dest.y = imageheight;
    
                       source.x = 0;
                       source.y = 0;
                       source.w = imagewidth;
                       source.h = imageheight;
                       return 0;
                       }
                       private:
                               short int stoopid;
};

Here is the Main code.

Code:

#include "SDL/SDL.h"
#include "Setupf.h"
#include "Setupf.cpp"//setup sdl video
#include "Imageloadfunc.h"
#include "Imageloadfunc.cpp"//function for loading and formatting images.
#include "Imageloadfunc.h"
#include "string"
#include "SpaceShipClass.cpp"

int main(int argc, char *argv[])//these are needed for sdl to work.
{
    short int height=1000; 
    short int width=1000;
    short int pixels=32;
    SDL_Surface *screen, *image;
    *screen = (Setup(height,width,pixels));///intializes video
    *image = (FormatLoad("Bitty2.bmp"));//loads and formats bitmap
    short int imageheight;
    short int imagewidth;    
    imagewidth = image->w;
    imageheight = image->h//both of these are passed to Original.initialize;
    Spaceship Original;//the first spaceship member
    Original.initialize(100,100,imageheight,imagewidth);
    SDL_BlitSurface(image,&Original.source,screen,&Original.dest);
    SDL_Flip(screen);
    SDL_Delay(4444444);
    return 0;
}

Any help would be really helpful

It could be a number of things. I incorrect Video Surface, or whatever. Have you ran it through a debugger to see what it says?

edit:

Code:

imageheight = image->h//both of these are passed to Original.initialize;

that wont even compile.

You're not allocating space for screen and image.

Code:

bool initialize(short int locationX, short int locationY, short int imageheight, short int imagewidth)
                  {
                       dest.x = 100;
                       dest.y = 100;
                       dest.w = imagewidth;                            
                       dest.y = imageheight;
    
                       source.x = 0;
                       source.y = 0;
                       source.w = imagewidth;
                       source.h = imageheight;
                       return 0;
                       }

I think u there should be "dest.h".

Originally Posted by SONU

I think u there should be "dest.h".

I didn't look close enough, but that shouldn't crash it, just screw up the values used.

FlyingIsFun1217

Originally Posted by Salem

> #include "Setupf.cpp"//setup sdl video
> #include "Imageloadfunc.cpp"//function for loading and formatting images.
> #include "SpaceShipClass.cpp"
Learn how to set up a project which compiles multiple source files. It might be nice and easy now, but shortly it will become cumbersome, then it will become broken.

> *screen = (Setup(height,width,pixels));///intializes video
> *image = (FormatLoad("Bitty2.bmp"));//loads and formats bitmap
Neither of these point to any memory, so you just write over some random memory locations instead.
And what's with the extra ( ) anyway?

Im sorry if I wasn' to clear, Setup() returns a pointer to the screen, and FormatLoad() returns a pointer to the formatted and loaded image.

I think my problem might be that I messed up with the assigning the pointers, I'm not really that good with them, I changed dest.y to dest.h like SONU said, but it still crashes out when I run it.

As for debugging I'm using dev-cpp and I don't get it that well, I understand the breakpoints somewhat,anyone know any tutorials or web sites for it, the book i learned c++ with never really explained it.
Anyone know a website that explains it?

Why aren't you checking return codes/values for any of the SDL functions? How do you know they're working?

I feel so stupid right now.

medievalelks got me thinking what could not be working, and my first thought was that my picture, Bitty2.bmp (its a pun, get it bitty? ) might not be in the folder.

And sure enough when I checked it wasn't there, when I moved it in the program displayed it perfectly.

Well, thanks guys for the help, i would of never spotted that extra dest.y bug, and I probably wouldn't of ever figured out that the bmp wasn't there if it wasn't for medievalelks.