关于Spring中用quartz定时器在定时到达时同时执行两次的问题

我在使用spring的quartz配置定时任务时,发现每次定时时间到达后,指定的定时方法同时执行两次,而且此方法还是使用的synchronized关键字,每次定时一到,会发现此方法内的System.out输出信息输出两次,说明方法在这时执行了两次,解决方法没有找到更好的,不过有一个方法很有效,我设置了一个静态变量,只要此方法一执行,就将变量由0变为1,执行完再设置为0.如果运行方法前检查此静态变量不为0,则return.开发时注意此静态变量不要让别的方法使用.

下面是定时相关的代码:

package org.openjweb.core.schedule;


import org.apache.log4j.Logger;
import org.apache.lucene.demo.IndexHTML;
//import org.apache.lucene.demo.IndexHTML;
import org.openjweb.core.service.ServiceLocator;
import org.quartz.JobExecutionContext;
import org.quartz.JobExecutionException;
import org.springframework.scheduling.quartz.QuartzJobBean;

public class JobSchedule extends QuartzJobBean
{
static Logger logger = Logger.getLogger(JobSchedule.class);
static long counter = 0;  //计数器
//static long priorTime = System.currentTimeMillis();
static int searchIndexFlag =0;//由doBuildIndex修改，其他方法不要修改此参数

public synchronized void doTimerSchedule()
{
  counter++;
  //System.out.println("测试定时器,总计数:"+String.valueOf(counter));
}

protected void executeInternal(JobExecutionContext arg0) throws JobExecutionException {
  // TODO Auto-generated method stub

}


public synchronized void doBuildIndex()
{
  //定时器同时调用两次的问题需要查找原因
  if(searchIndexFlag>0)return ;//保证同一时刻只有一个定时器运行,通过这种方式保证每次定时时间到时，只执行一个线程
  searchIndexFlag = 1;//锁定
  System.out.println("开始构造索引库....");
     String indexPath = ServiceLocator.getSysConfigService().getStringValueByParmName("luceneIndexDir");
  String filePath = ServiceLocator.getSysConfigService().getStringValueByParmName("searchRoot");
  System.out.println("索引路径为:"+indexPath);
  System.out.println("搜索路径为:"+filePath);
 
  IndexHTML.buildIndex(indexPath,filePath);
  System.out.println("索引库构造完毕!");
  searchIndexFlag = 0;//锁定解除
}

}

使用静态变量标志的方式解决了同一时刻调用两次方法的问题,但我想Spring应该有办法配置只能同一时刻运行一次,是什么造成的定时执行两次呢?大家有没有发现这种问题,欢迎赐教.下面是我在spring的配置:

  <!-- Timer schedule -->
    <bean id="defaultTimerBean" class="org.openjweb.core.schedule.JobSchedule"/>
    <bean id="defaultTimerMethod" class="org.springframework.scheduling.quartz.MethodInvokingJobDetailFactoryBean">
        <property name="targetObject" ref="defaultTimerBean" />
        <property name="targetMethod" value="doTimerSchedule" />
        <property name="concurrent" value="false" /> <!--将并发设置为false-->
    </bean>
    <bean id="defaultTrigger" class="org.springframework.scheduling.quartz.CronTriggerBean">
        <property name="jobDetail" ref="defaultTimerMethod" />
        <!--每三分钟的第一分钟触发-->
        <property name="cronExpression" value="0 1/3 * * * ?" />
    </bean>
   
    <bean id="luceneTimerMethod" class="org.springframework.scheduling.quartz.MethodInvokingJobDetailFactoryBean">
        <property name="targetObject" ref="defaultTimerBean" />
        <property name="targetMethod" value="doBuildIndex" />
        <property name="concurrent" value="false" /> <!--将并发设置为false-->
    </bean>
    <bean id="luceneTrigger" class="org.springframework.scheduling.quartz.CronTriggerBean">
        <property name="jobDetail" ref="luceneTimerMethod" />
        
        <property name="cronExpression" value="0 1/4 * * * ?" />
    </bean>
   
    <bean class="org.springframework.scheduling.quartz.SchedulerFactoryBean">
        <property name="triggers">
            <list>
                <!--作业调度器,list下可加入其他的调度器-->
                <ref bean="defaultTrigger"/>
                <ref bean="luceneTrigger"/> 
            </list>
        </property>
    </bean>

http://blog.csdn.net/baozhengw/archive/2009/04/16/4083247.aspx


JonBLu 发表于2009年4月16日 19:06:49  IP:举报
用的是Tomcat吗？把项目从Tomcat下面的目录移出去试试放在tomcat的webapp下可能会出这种情况LostParadise 发表于2009年4月16日 20:26:07  IP:举报
对于Spring和Quartz来说，你的用法是两个job，所以他们根本不关心他们是否同时运行了。本来concurrent是用来控制同时运行两个相同的job的问题的，但是你的是两个job，所以也用不上。所以。。。。既然你有这样的需求，那只能你自己来解决了。zhenglinshuang 发表于2009年4月20日 18:34:50  IP:举报
如果是这样的话，可能说明两点：1.concurrent = false;具体来说对应的只是job的那个执行方法并发控制;（也就是说某个方法在执行完后才可再执行这个非并发方法）so,你这里用了一个job，两个执行方法，...2.jobDeatail调用bean job时，初始化了不同的的job .attend 发表于2009年5月20日 15:11:49  IP:举报
实现StatefulJob接口就可以了。 public class JobSchedule extends QuartzJobBean implements StatefulJobhjjwind 发表于2009年6月22日 9:27:44  IP:举报
我在开发中也遇到了定时器执行两次得情况……但代码移植到其他环境就正常只执行一次了，定时任务到时间点后连续执行两次应该与SPRING定时器本身的设置无关。

评论

2 楼 xiaoqiang2008 2013-05-30  

执行两次的原因是什么，好像楼主没弄清楚啊！
是不是在web.xml中同时配置了ContextLoaderListener和DispatcherServlet？假如真是这样的话，需要删掉一个配置，因为你相当于配置了两个spring容器，两个容器分别都执行各自的计划任务，所以同一个计划任务被执行了两次。
ContextLoaderListener是整合struts等其他框架时常用的配置，起加载时执行优先级高，可以在struts加载之前就执行；假如项目中只有spring框架，那仅配置DispatcherServlet就行了。
以上只是个人猜测，希望能有助于碰到类似问题的朋友们！

1 楼 wangqiuyi 2011-12-08  

    <bean class="org.springframework.scheduling.quartz.SchedulerFactoryBean">
        <property name="triggers">
            <list>
                <!--作业调度器,list下可加入其他的调度器-->
                <ref bean="defaultTrigger"/>
                <ref bean="luceneTrigger"/> 
            </list>
        </property>
    </bean>

给bean设置个id 应该就没问题了