这样写线程程序有问题么，附代码

jhsea3do

现在有一个新闻的web接口，我需要每隔20分钟去调用getWebInterfaceContent方法来更新新闻。

不想写线程类，直接在一个方法里面new了一个thread对象。

请问我这样写会有问题么，需不需要用synchronized关键字，注：getWebInterfaceContent是一个static的方法。

1. 
   
2. 
   
3.         /**
   
4.          * WEB_INTERFACE_CONTENT_NEWS
   
5.          */
   
6.         private static String WEB_INTERFACE_CONTENT_NEWS = null;
   
7.        
   
8.         /**
   
9.          * WEB_INTERFACE_CONTENT_NEWS
   
10.          */
    
11.         private static String WEB_INTERFACE_COMMAND_NEWS = "news";
    
12. 
    
13.         /**
    
14.          * 取得最新新闻
    
15.          * @return
    
16.          */
    
17.         synchronized public String getNews() {
    
18.                 if(null == WEB_INTERFACE_CONTENT_NEWS) {
    
19.                         // 设置当前新闻
    
20.                         WEB_INTERFACE_CONTENT_NEWS
    
21.                                 = getWebInterfaceContent(
    
22.                                         WEB_INTERFACE_COMMAND_NEWS);
    
23.                         // 设置每隔一定时间取得最新新闻
    
24.                         Thread t = new Thread() {
    
25.                                 long timeS = System.currentTimeMillis();
    
26.                                 long timeE = System.currentTimeMillis();
    
27.                                 long offset = 20*60*1000;
    
28.                                 public void run() {
    
29.                                         for(;;) {
    
30.                                                 if((timeE - timeS) % offset == 0) {
    
31.                                                         WEB_INTERFACE_CONTENT_NEWS
    
32.                                                                 = getWebInterfaceContent(
    
33.                                                                                 WEB_INTERFACE_COMMAND_NEWS);
    
34.                                                         logger.debug("NEWS, id = " + timeS);
    
35.                                                 }
    
36.                                                 timeE = System.currentTimeMillis();
    
37.                                         }
    
38.                                 }
    
39.                         };
    
40.                         t.start();
    
41.                 }
    
42.                 return WEB_INTERFACE_CONTENT_NEWS;
    
43.         }
    
44. 
    

复制代码

apollolegend

我有点建议:
1)用这种忙线程定时是不是很耗费cpu
2)if((timeE - timeS) % offset == 0)  这种判断太严格,如果由于线程切换丢毫秒那就会出问题

jhsea3do

是了，刚才粗心了，现在改成了时间阻塞方式

1. 
   
2. while(true) {
   
3.                                                 WEB_INTERFACE_CONTENT_NEWS
   
4.                                                         = getWebInterfaceContent(
   
5.                                                                 WEB_INTERFACE_COMMAND_NEWS);
   
6.                                                 logger.debug("NEWS, id = " + timeS);
   
7.                                                 try { Thread.sleep(offset); }
   
8.                                                 catch (Exception ex) {logger.error("thread sleep err", ex);}
   
9.                                         }
   
10. 
    

复制代码

apollolegend

同步这里我想说的是:
读者之间应该不需要互斥.但读写之间要互斥的吧,也就是在更新新闻的时候是不能读新闻(getWebInterfaceContent),所以应该在新闻的读写那里设置读写锁.祥见操作系统中读者写者问题

jhsea3do

getWebInterfaceContent(cmd)是一个static方法，且新闻更新时不影响其返回值，并且返回值不会为null。

getNews()我用synchronized关键字的目的是怕产生多余的线程进程，该方法的目的是永远只有一个线程来更新WEB_INTERFACE_CONTENT_NEWS，默认没有线程来更新，当getNews()方法被第一次调用后才会启动该线程。

perryhg

楼主虽然启用了一个线程来读取新闻，对你主程序来说，你是多开了一个线程，但是对你的读新闻的程序来说，实际上还是单线程处理，并没有别的线程来同时来读取新闻，所以这里的synchronized也没什么必要。

synchronized 主要用于你使用多个线程同时来读取新闻的时候，确保只有一个网络读操作在进行中。

[ 本帖最后由 perryhg 于 2006-5-31 13:23 编辑 ]

jhsea3do

getNews()是WebContentDAO的一个成员方法。

比如我可能在A对象一个方法中new了一个WebContentDAO对象wc0, 又在B对象一个方法中new了一个WebContentDAO对象wc1.

如果我不用synchronized关键字。假如wc0.getNews()和wc1.getNews()先后执行，正好wc0因为网络问题没有及时得到返回值，这个时候WEB_INTERFACE_CONTENT_NEWS仍是null, 正巧wc1也要执行了，结果就new了两个Thread对象。可能么？

wobushiwo

假如是多个线程，和多个新闻文件，判断相同新闻ID互斥是有意义的

但现在多线程去争同一个任务？而且第一个抢到的线程就执行，没抢到的就不执行？

你这样写的多线程有意义吗？

就要要写，也最好符合把 synchronized 块里需要执行的内容减到最少。。。。

perryhg

原帖由 jhsea3do 于 2006-5-31 13:51 发表
getNews()是WebContentDAO的一个成员方法。

比如我可能在A对象一个方法中new了一个WebContentDAO对象wc0, 又在B对象一个方法中new了一个WebContentDAO对象wc1.

如果我不用synchronized关键字。假如wc0.getN ...

照你这么说的确是多线程，但是你仅仅是读取数据的话，也没什么必要用synchronized了，就好比你可以同时用两个浏览器窗口打开同一个网页，这个不会互相影响的吧，线程锁及互斥锁主要用在有写更新的操作里面，避免一个用户读取到别人更改了一半的的数据。

jhsea3do

的确即使生成了多个线程也不会影响news的值。

因为实际运行中要并发大量的getNews()请求，怕影响效率。

现在想了一下，直接在生成thread之前把WEB_INTERFACE_CONTENT_NEWS付一个非null值即可。