正确更新静态变量的正确方法

时间:2013-05-09 16:16:08

标签: java

我有一个静态变量,我在类的开头加载。我想每小时更新一次变量。问题是这样做的正确方法是什么?

我尝试这样做的方法如下,但它需要将静态变量更新为每个构造函数的方法:

import java.util.Date;

public class MyClass {

    private static String globalString = "";

    // initialize lastUpdate with two hours back to make sure first update happens
    private static Date lastUpdate = new Date(System.currentTimeMillis() - (2 * (3600 * 1000)));


    MyClass() {
        updateGlobalString();

        // DO MORE STUFF HERE...
    }


    MyClass(String string) {
        updateGlobalString();

        // DO MORE STUFF HERE...
    }

    private synchronized void updateGlobalString() {
        // check if we need to update
        if (lastUpdate.before(new Date(System.currentTimeMillis() - (3600 * 1000)))) {

            // DO THINGS TO UPDATE globalString HERE...

            lastUpdate = new Date();
        }
    }
}

还有其他想法/更好的方法吗?

2 个答案:

答案 0 :(得分:5)

您应该使用某种计时器来进行更新。

例如,使用ScheduledExecutorService每小时运行一次任务,从而更新该字段。像这样:

public class MyClass {

    private static volatile String globalString;
    private static ScheduledExecutorService exec = Executors.newSingleThreadScheduledExecutor();

    static {
        exec.scheduleAtFixedRate(new Runnable() {
            public void run() {
               // DO THINGS TO UPDATE globalString HERE...
            }
        },
        0, 1, TimeUnit.HOUR);
    }

    // Rest of class, without having to worry about the updateGlobalString
    // method, or the lastUpdate variable, or anything like that
    ...
}

请注意,由于多个线程正在访问变量,因此您需要确保代码是线程安全的。 (上面的计时器示例肯定是这种情况,但您当前的方法也可能就是这种情况。)

简单地确保看到更新的最简单方法是将globalString变量标记为volatile,但根据类的使用方式,其他方法可能更合适。

答案 1 :(得分:1)

一个简单的解决方案是使用Executors框架:

public class MyClass {
   public static String globalString = "";

   private static ScheduledExecutorService s = Executors.newSingleThreadScheduledExecutor();

   //static block of code
   static {
     s.scheduleAtFixedRate(new Runnable() {
        public void run() {
           globalString = System.currentTimeMillis() + "";
        }
     }, 0, 1, TimeUnit.HOUR);
   } 
}