Drawing alpha

[Daslee]

Hello. Im trying to draw a graphics from RGBA color, but i have a little problem. I want to make it move to right and when it reaches 100 at x coordinates, it starts decreasing his alpha. But now what i made, it moves to right until x <= 100 and when reaches 100 it stucks and after few seconds disappears. Here is my code at paint(Graphics g) method:

if(bomb.isLaunched() == true){
			bomb.x +=3;
			g.setColor(bomb.getColor());
			if(bomb.getX() >= 100){
				for(int i=0; i<2550; i++){
					int alpha = (int)(2550 - i) / 10;
					bomb.setColor(new Color(255, 0, 0, alpha));
					g.setColor(bomb.getColor());
					g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
				}
				bomb.x = 0;
				bomb.setLaunched(false);
			}else{
				g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
			}
		}

And bomb class (It's defined at the top of my main frame. Bomb bomb = new Bomb(0, 0, new Color(255, 0, 0, 255));):

import java.awt.Color;

public class Bomb {
	int x, y;
	Color clr;
	boolean launched;
	
	public Bomb(int x, int y, Color clr){
		this.x = x;
		this.y = y;
		this.clr = clr;
		launched = false;
	}
	
	public int getX(){
		return x;
	}
	public int getY(){
		return y;
	}
	public Color getColor(){
		return clr;
	}
	public void setColor(Color clr){
		this.clr = clr;
	}
	public boolean isLaunched(){
		return launched;
	}
	public void setLaunched(boolean tf){
		this.launched = tf;
	}
}

[DarrylBurke]

Moved from New to Java

db

[sakratt]

paintMethod(Graphics g) {
     if(bomb.isLaunched()){ // you don't have to manually compare booleans
            bomb.x +=3; // personally I would update the game logic in another method
            g.setColor(bomb.getColor());
            if(bomb.getX() >= 100){
                    int alpha = bomb.getColor().getAlpha() - 1;
                    bomb.setColor(new Color(255, 0, 0, alpha));
                    g.setColor(bomb.getColor());
                    g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
                }
                bomb.x = 0;
                bomb.setLaunched(false);
            }else{
                g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
            }        
}

I'm no expert but I'm pretty sure this is much more efficient than doing a loop that loops 2550 rounds each time the game updates. It will get the alpha of the bomb and decrease it by 1 each time the game updates. What happened in your method was that it updated the bomb's alpha inside the for loop without actually redrawing it.

[Daslee]

paintMethod(Graphics g) {
     if(bomb.isLaunched()){ // you don't have to manually compare booleans
            bomb.x +=3; // personally I would update the game logic in another method
            g.setColor(bomb.getColor());
            if(bomb.getX() >= 100){
                    int alpha = bomb.getColor().getAlpha() - 1;
                    bomb.setColor(new Color(255, 0, 0, alpha));
                    g.setColor(bomb.getColor());
                    g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
                }
                bomb.x = 0;
                bomb.setLaunched(false);
            }else{
                g.fillOval(bomb.getX(), bomb.getY(), radius, radius);
            }        
}

I'm no expert but I'm pretty sure this is much more efficient than doing a loop that loops 2550 rounds each time the game updates. It will get the alpha of the bomb and decrease it by 1 each time the game updates. What happened in your method was that it updated the bomb's alpha inside the for loop without actually redrawing it.

Thanks, your code helped me. Now i fixed my problem. :))