Java：揮発性変数が更新されない（getおよびsetメソッドが機能しない）

Question

私はrun()メソッドでプライベート揮発性ダブル値を更新し続けるRunnable "NanoClock"クラスを持っています。

このクラスには、double値を返すgetTime()メソッドもあります。別のクラス（ "Master"）はNanoClockクラスを構築しており、スレッドを作成し、start()メソッドを呼び出す。

これを実行した後、getTime()メソッドを何回か（遅れて）呼び出しますが、値は更新されません。私は間違って何をしていますか？

NanoClock.java：

public class NanoClock implements Runnable { 
    private volatile boolean running; 
    private volatile double time; 
    public NanoClock() { 
     time = System.currentTimeMillis(); 
    } 
    @Override 
    public void run() { 
     running = true; 
     while(running) { 
      try { 
       if(System.currentTimeMillis() > time) { 
        time = System.currentTimeMillis(); 
       } 
       //This returns the updated value continuously when commented out 
       //System.out.println("Time: " + String.format("%.6f", unix_time)); 
       Thread.sleep(2000); 
      } catch(Exception exc) { 
       exc.printStackTrace(); 
       System.exit(1); 
      } 
     } 
    } 
    public double getTime() { 
     return time; 
    } 
    public void end() { 
     running = false; 
    } 
} 

Master.java：

public class Master { 
    public static void main(String[] args) { 
     try { 
      NanoClock nClock = new NanoClock(); 
      Thread clockThread = new Thread(new NanoClock()); 
      clockThread.setPriority(10); 
      clockThread.start(); 
      //MY_ISSUE: This returns the same value every time 
      for(int a = 0; a < 10; a++) { 
       System.out.println("Time: " + nClock.getTime()); 
      } 
      //MY_ISSUE: This cannot stop the while loop - I tested it with 
      //the println in the NanoClock class. 
      nClock.end(); 
      System.out.println("Done!"); 
     catch(Exception e) { 
      e.printStackTrace(); 
      System.exit(1); 
     } 
    } 
} 

Answer 1

でなければなりませんバックギャングに時間を守る。もう1つはnClockです。これはメインスレッドのフォアグラウンドでぼんやりと座っています。

nClock、他のスレッドでRunnableされている必要があります：

Thread clockThread = new Thread(nClock); // not new NanoClock() 

これは、全体のソリューションではないかもしれないが、それは正しい方向への大きな一歩である必要があります。

Answer 2

System.currentTimeMillis()長いを返しますが、あなたは、精度の損失を引き起こす、二重に保管してください。メンバ時間（およびゲッタの戻り値の型）を長く変更すると、期待される結果が得られるはずです。

経験則として、時間単位で作業する場合は、ほとんどの場合、最も適切なデータ型です。浮動小数点数は正確な結果を格納するのには適していません。

Answer 3

以下のコードに2秒かかる場合は、時間が変更されます。

//MY_ISSUE: This returns the same value every time 
for(int a = 0; a < 10; a++) { 
    System.out.println("Time: " + nClock.getTime()); 
} 

しかし、10回の反復を持つforループとsystem.outは1ミリ秒かかるので変更されません。

なぜ2秒ですか？あなたの実行可能コードにThread.sleepがあるからです。

Thread.sleep(2000); 

これは、次の更新が2秒で行われることを意味します。

そして、本当にミリ秒ではなくナノ時間が必要なので、System.currentTimeMillis（）の代わりにSystem.nanoTime（）を使用してください。

---

更新：私のマシンで

public static void main(String args[]) { 
    long start = System.currentTimeMillis(); 
    for(int a = 0; a < 10; a++) { 
     System.out.println("Iterating " + a); 
    } 
    long end = System.currentTimeMillis(); 
    System.out.println("Start = " + start); 
    System.out.println("End = " + end); 
} 

の検索結果を、開始時間と終了時間に差

Iterating 0 
Iterating 1 
Iterating 2 
Iterating 3 
Iterating 4 
Iterating 5 
Iterating 6 
Iterating 7 
Iterating 8 
Iterating 9 
Start = 1499592836298 
End = 1499592836298 

は、コードブロックは、その非常に高速に実行されたことはありません1ミリ秒もかかっていませんでした。タイミングによっては、1ミリ秒かかることがあります。 System.nanoTimeのにそれを変更する

（）

public static void main(String args[]) { 
    long start = System.nanoTime(); 
    for(int a = 0; a < 10; a++) { 
     System.out.println("Iterating " + a); 
    } 
    long end = System.nanoTime(); 
    System.out.println("Start = " + start); 
    System.out.println("End = " + end); 
} 

結果、開始時刻と終了時刻の差があります。あなたの他のスレッドでRunnableは喜んでいるようにそれらの一つは、匿名new NanoClock()です：main()で

Iterating 0 
Iterating 1 
Iterating 2 
Iterating 3 
Iterating 4 
Iterating 5 
Iterating 6 
Iterating 7 
Iterating 8 
Iterating 9 
Start = 1012518090518837 
End = 1012518091012960 

Answer 4

Thread.sleep(2000); 
System.out.println("Time: " + nClock.getTime()); 

forあなたはNanoClockののインスタンスを持っているsleep(2000)