Re: is that OK if i make the Thread object as member variable?

From:

Wisdo <wisdo@hf.webex.com>

Newsgroups:

comp.lang.java.programmer

Date:

Mon, 21 Aug 2006 12:10:07 +0800

Message-ID:

<ecbbn2$uge$1@news.yaako.com>

hiwa ??????:

Wisdo ??????????????????:

Hi ALL,

I write a Listener class, and let the Thread obj
as member variable, when initilize and destroy
call corresponding thread method.

have it some implicit problem ?
whether i need add synchronized keyword into
some places?

please give your salted suggestion.

-------- code begin ----------------------
public Listener extends Runnable {

    private Thread th;

    public void initilize() {
            th = new Thread(this);
            th.setName(getClass().getName());
            th.setDaemon(true);
            th.start();
    }

    public void run() {
        //listen evnet and perform operation
    }

    public void destry() {
        th.stop();
    }
}
---------- code end --------------------

Thanks
-Wisdo

Your code is wrong in a few places.
Show your real usage. Post a small demo code that is generally
compilable, runnable and could reproduce your problem. See:
http://homepage1.nifty.com/algafield/sscce.html and
[url=http://riters.com/JINX/index.cgi/Suggestions_20for_20Asking_20Questions_20on_20Newsgroups]this
wiki.[/url]

thanks for the inform.

the new code, is any synchronzied suggestion?

package test.misc;

public class Listener implements Runnable {

   private Thread th;

   public void initilize() {
         th = new Thread(this);
         th.setName(getClass().getName());
         th.setDaemon(true);
         th.start();
   }

   public void run() {
   while(true) System.out.println("get event, processing....");
   }

   public void destroy() {
   th.stop();
   }


   public static void main(String args[]) throws InterruptedException {
   Listener lst = new Listener();
   lst.initilize();

   Thread.sleep(10000);
   lst.destroy();
   }
}

Regards
-Wisdo