Re: Painting in swing. repaint doesn't work.
On Jun 16, 8:34 pm, SadRed <cardinal_r...@yahoo.co.jp> wrote:
s1 = new Szescian();
s1.setLocation(0,0);
s1.setOpaque(true);
s1.setVisible(true);
s1.setSize(panelSzescian.getSize());;
s1.setOpaque(true);
panelSzescian.add(s1); // a JPanel to which i add s1
s1.uruchom(); //start the thread
public void uruchom(){
Runnable r = new Szescian();
Thread t = new Thread(r);
t.start();
}
}
Your 's1' and 'r' are completely different objects.
This much is true. The uruchom method should just do:
(new Thread(this)).start();
I guess the OP's intent with this design is to have a component that
auto-updates on a timer. There should really be some facility for
terminating the thread later on when the component no longer needs
updating. I suppose the component can detect in the run loop or paint
method if it's been told to deactivate or is no longer visible or
whatever; maybe handling that is planned but unimplemented by the OP
yet, or omitted from the code posted here to save space. Right now it
looks like it updates only fifty times and then stops anyway, which is
really odd, but that may again be temporary until a more robust system
is in place to determine when to stop the updating. I suppose the code
we've seen is a test to get the updating working first, and then
refine into the final code later.
The more usual design would probably be to have an external Runnable
calling repaint on a set of components registered with it as needing
automatic updating. I'm not sure also of the effect of calling repaint
from arbitrary threads. I checked the JComponent docs and it doesn't
say that it should be done on the event-dispatch thread, at least not
in the repaint method descriptions themselves. If so you'd need to use
a SwingUtilities.invokeLater on a runnable that calls repaint once on
the component.
Another thing to consider is whether the compiler or JVM is choking on
identifiers with too high a consonant-to-vowel ratio. ;)