Need help in ball breaker game

[rahul taiwala]

hello every body i have some painting problem with the game can anybody help.....
here is the code

package javaapplication4;
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
/**
 *
 * @author rahul
 */
public class logic extends JPanel implements MouseMotionListener, MouseListener {
    
brick block[] = new brick[23];
	int px,py,pheight,pwidth,bx,by,bheight,bwidth,height,width,mx,my,speedx,speedy,score,balls,bricks;
	boolean started,title,hovered;
	private Timer timer1,timer2;
        logic(){
            title=true;
            addMouseMotionListener(this);
            addMouseListener(this);
            
                height = 400;
		width = 400;
		pheight = 5;
		pwidth = 50;
		px =height;
		py = width - 50;
                bwidth = 10;
		bheight = 10;
                title=true;
                repaint();
        	timer1 = new Timer(16,new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				bx += speedx;
				by += speedy;
				if (bx + bwidth >= width || bx <= 0) {
					speedx = -speedx;
				}
				if (by + bheight >= py && bx + bwidth >= px && bx <= px + pwidth && by <= py || by <= 0) {
					speedy = -speedy;
				}
				for (int i = 0; i < block.length; i++) {
					if (block[i].visible && by <= block[i].y + block[i].height && bx + bwidth >= block[i].x && bx <= block[i].x + block[i].width && by + bheight >= block[i].y) {
						speedy = -speedy;
						block[i].visible = false;
						score += 10;
						++bricks;
						if (bricks == 23) {
							bx = px + pwidth / 2 - bwidth / 2;
							by = py - bheight;
							bricks = 0;
							speedx = 10;
							speedy = -10;
							drawBricks();
							timer1.stop();
							started = false;
						}
					}
				}
				if (by >= height) {
					--balls;
					if (balls > 0) {
						resetBall();
					}
					timer1.stop();
					started = false;
					repaint();
				}
				repaint();
			}
		});
		timer2 = new Timer(2000,new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				title = true;
				timer2.stop();
			}
		});
               
        }

    @Override
	public void paintComponent(Graphics g) {
           super.paintComponents(g);
           Graphics2D g2d = (Graphics2D)g;
		if(!title){
			for (int i = 0; i < block.length; i++) {
				if (block[i].visible) {
					g2d.setColor(Color.red);
					g2d.drawRect(block[i].x - 1,block[i].y - 1,block[i].width + 1,block[i].height + 1);
					g2d.setColor(Color.green);
					g2d.fillRect(block[i].x,block[i].y,block[i].width,block[i].height);
				}
			}
			g2d.setColor(Color.black);
			g2d.fillRect(px,py,pwidth,pheight);
			g2d.fillArc(bx,by,bwidth,bheight,0,360);
			
			switch(balls) {
				case 0:
					timer1.stop();
					started = false;
					timer2.start();
					repaint();
					break;

				}
		}
	
    }
	public void drawBricks() {
           
		for (int i = 0; i < block.length; i++) {
			block[i] = new brick();
			block[i].width = 35;
			block[i].height = 15;
			block[i].visible = true;
			if (i <= 7) {
				if (i > 0) {
					block[i].x = block[i - 1].x + 45;
				}
				else {
					block[i].x = 10;
				}
				block[i].y = 10;
			}
			else if (i <= 14) {
				if (i > 8) {
					block[i].x = block[i - 1].x + 45;
				}
				else {
					block[i].x = 30;
				}
				block[i].y = 30;
			}
			else {
				if (i > 15) {
					block[i].x = block[i - 1].x + 45;
				}
				else {
					block[i].x = 10;
				}
				block[i].y = 50;
			}
		}
	}

	public void resetBall() {
		bx = px + pwidth / 2 - bwidth / 2;
		by = py - bheight;
		speedx = 10;
		speedy = -10;
	}
    public void mouseMoved(MouseEvent e) {
		mx = e.getX();
		my = e.getY();
		if(!title){
			if (mx - pwidth / 2 <= 0) {
				mx = pwidth / 2;
			}
			if (mx + pwidth / 2 >= width) {
				mx = 400 - pwidth / 2;
			}
			px = mx - pwidth / 2;
			if (!started) {
				bx = px + pwidth / 2 - bwidth / 2;
			}
                }
		repaint();
	}

	public void mousePressed(MouseEvent e) {
	if (title) {
			title = false;
			drawBricks();
			balls = 3;
			score = 0;
			bricks = 0;
			px = width / 2 - pwidth / 2;
			resetBall();
			repaint();
		}
		else if (!title) {
			started = true;
			timer1.start();
		}
		
	}

	public void mouseReleased(MouseEvent e) { }

	public void mouseEntered(MouseEvent e) { }

	public void mouseExited(MouseEvent e) { }

	public void mouseClicked(MouseEvent e) { }

	public void mouseDragged(MouseEvent e) { }

        }

[rahul taiwala]

use paint method instead of paintComponent and its working fine.....

[pete stein]

rahul taiwala wrote:use paint method instead of paintComponent and its working fine.....

This solution sounds very wrong. Sorry but it sounds like you're fixing the wrong thing, but I can't say what since I can't get your code to compile or run due to missing classes. One problem I see that is not causing your main question but can bite you later is that you have program logic within your paint/paintComponent method as indicated by my comment // **** here ****:

   public void paintComponent(Graphics g) {
      super.paintComponents(g);
      Graphics2D g2d = (Graphics2D) g;
      if (!title) {
         for (int i = 0; i < block.length; i++) {
            if (block[i].visible) {
               g2d.setColor(Color.red);
               g2d
                        .drawRect(block[i].x - 1, block[i].y - 1, block[i].width + 1,
                                 block[i].height + 1);
               g2d.setColor(Color.green);
               g2d.fillRect(block[i].x, block[i].y, block[i].width, block[i].height);
            }
         }
         g2d.setColor(Color.black);
         g2d.fillRect(px, py, pwidth, pheight);
         g2d.fillArc(bx, by, bwidth, bheight, 0, 360);

         switch (balls) {
         case 0:
            timer1.stop();  // **** here ****
            started = false;
            timer2.start();
            repaint();
            break; // **** to here ****

         }
      }

   }

and program logic should not be in this method as you do not have full control as to when or if it will be called. Also it is strange to see a repaint() call within a paint/paintComponent method. If the JVM weren't smart, this would cause an infinite recursive cycle.