Unable to call paint() from inside method

**Y. Progammer** · 02-28-2010, 08:46 PM

Java Code:

import javax.swing.*;
import java.awt.Graphics;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

public class JPanel01 extends JPanel {

    int xa = 0, ya = 0;
    JButton jButton = new JButton("call painComponent() from inside the method testD()");

    JPanel01() {


        jButton.addActionListener(
                new ActionListener() {

                    public void actionPerformed(ActionEvent event) {
                        if (event.getSource() == jButton) {
                            System.out.println("button");
                            testD();
                        }
                    }
                });
        add(jButton);
    }

    @Override
    public void paintComponent(Graphics g) {
        super.paintComponent(g);
        System.out.println("paint1");
        g.drawLine(xa, ya, xa + 150, ya + 150);
        System.out.println("paint2");
    }

    void testD() {
        for (xa = 100; xa <= 500; xa++) {
            ya = xa;
            System.out.println("testD1");
            repaint();
            System.out.println("testD2");
        }
    }

    public static void main(String args[]) {
        javax.swing.JFrame frame = new javax.swing.JFrame();
        frame.setDefaultCloseOperation(javax.swing.JFrame.EXIT_ON_CLOSE);
        JPanel01 h = new JPanel01();
        frame.add(h);
        frame.setSize(500, 500);
        frame.setVisible(true);
    }
}

plz explain.:confused:

**Fubarable** · 02-28-2010, 10:34 PM

explain what?

**Y. Progammer** · 03-01-2010, 05:16 AM

explain what?

the paintComponent() method is first called when the window appears for the very first rime... paint1 paint 2..

but when I pressed button to call the paintComponent() from inside the method testD()..perhaps it never happened.??:eek:

run:
paint1
paint2
button
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
testD1
testD2
paint1
paint2
paint1
paint2
BUILD STOPPED (total time: 13 seconds)

The output according to me should be ..
testD1
paint1
paint2
testD2

**Fubarable** · 03-01-2010, 12:47 PM

Calling repaint doesn't automatically call paint but rather places a request for paint to the Swing repaint manager. This may or may not honor the request depending on several things including whether or not it knows that the region being requested for a paint is "dirty". Perhaps here the repaint manager is smart enough to know that calling paint/paintComponent is not needed here.

For more details, please look here: Painting in AWT and Swing