PROBLEMS WITH A THREAD!!!

Question

Hi, guys!
Posted this in another forum, but no good answers...
I got a class "GUI" which is calling a method in class FileCopy on a button-click.
Class GUI calls -------> copy in class FileCopy
The FileCopy-class looks kinda like this:
public class FileCopy implements Runnable {
Thread thread = null;
public void copy( ...params...) {
thread = new Thread(this);
// Asign some vars
thread.start();
private void copyFiles(File file, File destination) {
// Copy the file
public void run() {
for (Iteration e = collection.iterator(); e.hasNext();)
copyFile((File) e.next(), destination);
// Kill thread next
The problem is that the thread seems to call the metod copyFiles and then (before the copyFiles-method is done) go to the part in run where I kill the thread. That results in a non-copied file!!!!
I've tried to use the join-method to get the thread to wait for the copyFiles-method to complete, but then the GUI freezes and that's not an option!!!
Plz help!!!
/Andrew

Answer 1

Hi
you are already in the event thread (dispatching thread)
and you don't need to spawn another thread, just handle the event normally
in your event handler.
if you really want to use thread use a Timer to copy (set a low interval)
or
class FileCopy implements Runnable {
FileCopy(params...) {
// Set your file names and params here
private void copyFiles(File file, File destination) {
// Copy the file
public void run() {
for (Iteration e = collection.iterator(); e.hasNext();) {
copyFile((File) e.next(), destination);
// let the thread end normally
// event handler
public void actionPerformed(ActionEvent e) {
Filecopy copy = new FileCopy(params...);
new Thread(copy).start();
I hope this helps