Need help stopping a thread

Question

I have a thread that starts a file download. I tried using the following.
On the mouse click event of a button
thread.interrupt();
in the file download method
if(!thread.isInterrupted())
do stuff
else
thread.interrupted();
The way it stands now, the file download stops and everything seems fine. But I cant restart the thread. I read something about having to return from the run method but I cant get that to work.
Thanks,
Jason

Answer 1

interrupt is not a replacement for the deprecated stop. It performs interruption of waits and blocks. And sets a flag the next time the code in the thread hits a wait or block.
Your if else logic is unsafe wrt the use of interrupt.
You need to change the code to depend on a synchronized object with wait and notify
For example in the thread follow the motionObject:
    public LocateSampleThread(AerotechWrapper aerotechWrapper, double xScale, double yScale, double zScale, double φScale)
        this.aerotechWrapper=aerotechWrapper;
        //this.xScale=xScale;
        //this.yScale=yScale;
        //this.zScale=zScale;
        //this.φScale=φScale;
        this.motionObject=new Object();
        alarmEventListeners=new java.util.Vector();
        recoveredAlarmEventListeners=new java.util.Vector();
    public void run()
        int loopCount=0;
        while(!abort)
            try
                synchronized(motionObject)
                    motionObject.wait();
                    stop=false;
                if(abort) break;
                StateModel.getInstance().setState(StateModel.MOVING);
                try
                    StateModel.getInstance().setActive(StateModel.MOVING,true);
                catch(PropertyVetoException pve)
                StateModel.getInstance().incrementActiveThreadCount();
                System.out.println("LocateSampleThread "+StateModel.getInstance().getActiveThreadCount());
            catch(InterruptedException ie)
...do activity here.
            synchronized(motionObject)
                motionObject.notify();
            try
                StateModel.getInstance().setActive(StateModel.MOVING,false);
            catch(PropertyVetoException pve)
            StateModel.getInstance().clearState(StateModel.MOVING);
            StateModel.getInstance().decrementActiveThreadCount();
            sendStageMotionEvent (new StageMotionEvent (this, false));
    public void moveAndWait () throws InterruptedException
        synchronized (motionObject)
            motionObject.notify();
            motionObject.wait();
    public void moveAndDontWait ()
        synchronized (motionObject)
            motionObject.notify();
    protected Object motionObject=null;in the controlling code
                                try
                                    locateSampleThread.moveAndWait();
                                    //System.out.println("locateSampleThread notify/wait done");
                                    sleep(100);
                                catch(InterruptedException ie)
                                }Notice that the in moveAndWait the notify and wait ion the motionObject inside a synchronized is safe threading because the controlling code that calls moveAndWait will be waiting even before the thread's run(which is synchronized as well) starts going from being notified.