Jump to content
  • Advertisement
Sign in to follow this  
DominicHughes

How to remove the left overs of SDL_FillRect?

This topic is 2517 days old which is more than the 365 day threshold we allow for new replies. Please post a new topic.

If you intended to correct an error in the post then please contact us.

Recommended Posts

Hello I'm trying to figure out how to remove this black crap off my OrangeGhostKnight Surface which you can see clearly in this picture below

UG6jH.png

what would be your suggestions and tips

I've already tried using


SDL_FillRect(Screen,&OrangeGhostKnightRect,(0,0,0));
SDL_BlitSurface(OrangeGhostKnightSurf,NULL,Screen,&OrangeGhostKnightRect);
SDL_SetColorKey(OrangeGhostKnightSurf, SDL_SRCCOLORKEY |SDL_RLEACCEL, SDL_MapRGB(OrangeGhostKnightSurf->format,0, 0, 0));
SDL_Flip(Screen);


SDL_SetColorKey(); Clearly doesn't work in this problem

Any suggestions or tips or am I doing this wrong ?

Share this post


Link to post
Share on other sites
Advertisement
With SDL_FillRect your are explicitly creating that black rectangle.
And also try setting the color key before you blit the surface.

Share this post


Link to post
Share on other sites
its still not working ;(



SDL_SetColorKey(OrangeGhostKnightSurf, SDL_SRCCOLORKEY |SDL_RLEACCEL,BlackColor);


what are like the common problems for SDL_FillRect not working?

Share this post


Link to post
Share on other sites
Hidden
What I meant is that you should entirely remove [color=#282828][font=helvetica, arial, verdana, tahoma, sans-serif]

SDL_FillRect from your code. (or do you want to have a rectangle there?)

[/font]

[font="helvetica, arial, verdana, tahoma, sans-serif"][size="2"][color="#282828"]What does the [/font][color=#282828][font=helvetica, arial, verdana, tahoma, sans-serif]

OrangeGhostKnight surface looks like? (or the base image) Is it black in the background?[/font]


[color=#282828][font=helvetica, arial, verdana, tahoma, sans-serif]

Try this:[/font]


[color=#282828][font=helvetica, arial, verdana, tahoma, sans-serif]

[/font]


[color=#000000][size=2]

SDL_SetColorKey

[color=#666600][size=2]

(

[color=#660066][size=2]

OrangeGhostKnightSurf

[color=#666600][size=2]

,

[color=#000000][size=2]

SDL_SRCCOLORKEY

[color=#666600][size=2]

|

[color=#000000][size=2]

SDL_RLEACCEL

[color=#666600][size=2]

,

[color=#000000][size=2]

SDL_MapRGB

[color=#666600][size=2]

(

[color=#660066][size=2]

OrangeGhostKnightSurf

[color=#666600][size=2]

->

[color=#000000][size=2]

format

[color=#666600][size=2]

,

[color=#006666][size=2]

0

[color=#666600][size=2]

,

[color=#000000][size=2][color=#006666][size=2]

0

[color=#666600][size=2]

,

[color=#000000][size=2][color=#006666][size=2]

0

[color=#666600][size=2]

));


[color=#000000][size=2]SDL_BlitSurface[color=#666600][size=2]([color=#660066][size=2]OrangeGhostKnightSurf[color=#666600][size=2],[color=#000000][size=2]NULL[color=#666600][size=2],[color=#660066][size=2]Screen[color=#666600][size=2],&[color=#660066][size=2]OrangeGhostKnightRect[color=#666600][size=2]);


[color=#000000][size=2]SDL_Flip[color=#666600][size=2]([color=#660066][size=2]Screen[color=#666600][size=2]);


[color=#282828][font=helvetica, arial, verdana, tahoma, sans-serif]

[/font]

Share this post


Link to post
What I meant is that you should entirely remove SDL_FillRect from your code. (or do you want to have a rectangle there?)

What does the OrangeGhostKnight surface look like? (or the base image) Is it black in the background?

Try this:


SDL_BlitSurface(OrangeGhostKnightSurf,NULL,Screen,&OrangeGhostKnightRect);
SDL_SetColorKey(OrangeGhostKnightSurf, SDL_SRCCOLORKEY |SDL_RLEACCEL, SDL_MapRGB(OrangeGhostKnightSurf->format,0, 0, 0));
SDL_Flip(Screen);

Share this post


Link to post
Share on other sites
the orange Coloured character in the image on my first post is that ghostknight and what I want to do is basically change his frames without it overlaying constantly like if it were a trail of images so I'm filling that Orange Coloured character rectangle and then trying to get rid of it because the fill colour is still showing as you can see in the above image

Share this post


Link to post
Share on other sites

what I want to do is basically change his frames without it overlaying constantly like if it were a trail of images


There is no efficient way to do this. As soon as you write pixels to the Screen, the previous pixels are lost. You can track what used to be behind the sprite by copying those pixels first, then drawing them over your sprite with them afterwards. However, reading from video memory is quite slow (read: slower than redrawing the entire screen by several orders of magnitude), and keeping track of the "background pixels" quickly becomes complicated once you have sprites intersecting.

The easiest (and fastest) solution that most games go for is to just redraw the entire scene on top of what was already there.

Share this post


Link to post
Share on other sites

the orange Coloured character in the image on my first post is that ghostknight and what I want to do is basically change his frames without it overlaying constantly like if it were a trail of images so I'm filling that Orange Coloured character rectangle and then trying to get rid of it because the fill colour is still showing as you can see in the above image


Think about this for a second: (this is what you are currently doing)

1) You draw the background.
2) You draw your character.
3) You draw a black rectangle to erase your character.
4) You draw your character again.
5) You complain there is a black rectangle that you just drew two steps before.

What you are wanting to do is "undraw" your character... but that's not possible. You can't undraw something. Instead, you need to draw over it. What you have to do, is draw over the character with the background, and then draw the character again afresh.
You can do that in two different ways:

The easy way: (every loop/frame of your game)
1) Draw the entire background.
2) Draw all the objects over the background.
3) Apply to screen.

Basically, your drawing code looks like this:
1) Erase the entire screen. (so no images from the previous frame shows through)
2) Draw the background.
3) Draw any trees or objects North of the knight.
4) Draw the knight. (so the knight appears over those trees)
5) Draw any trees or objects south of the Knight (so the southern trees appear over the knight, letting him "walk behind" them)
6) Draw the GUI (buttons, menus, healthbars, text, etc...)

Except, you don't actually need to do step 1 ("Erase the entire screen"), because step 2 covers the entire screen anyway.

The hard way: (Don't do this)
1) Draw the background once only.
2) Every time an object moves, calculate and redraw just the piece of the background that covers where the object used to be.
3) Redraw the object.
4) Apply to the screen.

This is called 'Dirty rectangles'. Don't do it like this - it complicates things, and makes things unnecessarily hard while bringing you a benefit that you won't need.
The point of dirty rectangles is to get a little extra speed - but you already have enough speed in your computer, so no matter how slow you think your game is running, the solution is to find the real slowdown, not to use dirty rectangles. There are much easier, safer, and stabler ways to boost a program's speed.

This is how your game should be laid out.

Share this post


Link to post
Share on other sites
Sign in to follow this  

  • Advertisement
×

Important Information

By using GameDev.net, you agree to our community Guidelines, Terms of Use, and Privacy Policy.

GameDev.net is your game development community. Create an account for your GameDev Portfolio and participate in the largest developer community in the games industry.

Sign me up!