décalage de thread ... help pleaseRésolu

Question

Bonjour,

apr&#232;s plusieurs probl&#232;mes avec les threads, j'ai voulu v&#233;rifier quelque chose de simple. Or m&#234;me ce code source simple ne fonctionne pas comme je veux. Voici le source :

///////// Class main /////////
public class Main {
    /** Creates a new instance of Main */
    public Main() {
    }

    public static void main(String[] args) {
        NewClass thr = new NewClass();
        thr.start();
    }
}

///////////Class NewClass, celle du thread ////////////
public class NewClass extends Thread{
    
    /** Creates a new instance of NewClass */
    public NewClass() {
    }
    public void run()
    {
        while(true)
        {
           oh();
            try
            {            
                sleep(1000);               
            }
            catch(InterruptedException e)
            {
                System.out.println("erreur thread update " + e);
            }
        }
    }
    
    public void oh()
    {
               Timestamp t3 = new Timestamp(0);
                t3.setTime(System.currentTimeMillis());
                System.out.println("update graphik " + t3.toString());
    }
}

Ce que je veux faire est simple. Je veux que toutes les secondes, le thread m'affiche l'heure courante avec la fonction oh(). Le probl&#232;me est qu'&#224; l'ex&#233;cution de ce prog, il se cr&#233; un d&#233;calage. La fonction oh() affiche :
12:08:18.117
12:08:19.118
12:08:20.119

Bref il se cr&#233; un d&#233;calage de 1 millisec toutes les secondes. Cette diff&#233;rence est tr&#232;s faible mais tr&#232;s importante pour moi. Pouvez-vous me corriger mon source pour qu'il m'affiche :
12:08:18.117
12:08:19.117
12:08:20.117

Un grand merci.

cs_Dobel · Accepted Answer

Pour pr&#233;ciser un peu, 



Dans l'exemple au dessus, on triche &#233;videmment en affichant une heure
qui est calcul&#233;e en rajoutant n*1000 &#224; une valeur d'origine.

Mais ce qui est important, c'est que l'affichage est d&#233;clencher de
fa&#231;on tr&#232;s r&#233;guli&#232;re (+- 1ms dans mon test), avec des &#233;carts qui ne se
propagent pas.



On peut donc aussi moins tricher, en affichant l'heure r&#233;elle. Mais le r&#233;sultat est moins beau !



update graphik 2005-05-10 13:48:08.175 -> 1115725688182

update graphik 2005-05-10 13:48:09.175 -> 1115725689175

update graphik 2005-05-10 13:48:10.176 -> 1115725690176

update graphik 2005-05-10 13:48:11.176 -> 1115725691176

update graphik 2005-05-10 13:48:12.176 -> 1115725692177

update graphik 2005-05-10 13:48:13.175 -> 1115725693175

update graphik 2005-05-10 13:48:14.176 -> 1115725694176


le code correspondant :

-----

public class Main {

  /** Creates a new instance of Main */

  public Main() {

  }



  public static void main(String[] args) {

    NewClass thr = new NewClass();

    thr.start();

  }

}



///////////Class NewClass, celle du thread ////////////

class NewClass

    extends Thread {



  /** Creates a new instance of NewClass */

  public NewClass() {

  }



  public void run() {

    int n = 0;

    long t0 = System.currentTimeMillis();

    oh();

    while (true) {

      t0 +=1000;

      try {

        sleep(t0-System.currentTimeMillis());

      }

      catch (InterruptedException e) {

        System.out.println("erreur thread update " + e);

      }

      oh();

    }

  }



  public void oh() {

    Timestamp t3 = new Timestamp(0);

    t3.setTime(System.currentTimeMillis());

    System.out.println("update graphik " + t3.toString()+" -> "+System.currentTimeMillis());

  }

}



-----
Dobel
[Une fois rien, c'est rien; deux fois
rien, ce n'est pas beaucoup, mais pour trois fois rien, on peut d&#233;j&#224;
s'acheter quelque chose, et pour pas cher]

bjdc · Answer

Ton code n'est s'execute ni de facon instantan&#233;e, ni prioriairement sur les autres process, ce qui peut expliquer le decalage.



Pour le deuxieme probleme, tu peux toujours tenter un setPriority(MAX_PROPRITY)


Pour le premier, y'a pas de solution simple. Si tu arrives a calculer
le temps d'execution, tu peux ajuster la duree  du sleep, mais ca
va dependre de la machine sur laquelle le code tourne...

cs_Dobel · Answer

ton code ne tient pas compte du temps de l'affichage. Je te propose une version modifi&#233;e.



On r&#233;pcup&#232;re une valeur t0 de l'horloge, et on synchronise le programme par rapport &#224; t0 +n*1000;





voil&#224; le r&#233;sultat que j'ai obtenu 

A droite, l'heure syst&#232;me au moment de l'affichage dans la console. Ce n'est pas la m&#234;me que l'heure qui est affich&#233;e &#224; gauche 

Les petites fluctuations correspondent &#224; des d&#233;lais d'affichages non constants, mais elles ne sont pas propag&#233;es.

 

update graphik 2005-05-10 13:30:27.078 -> 1115724627087

update graphik 2005-05-10 13:30:28.078 -> 1115724628079

update graphik 2005-05-10 13:30:29.078 -> 1115724629080

update graphik 2005-05-10 13:30:30.078 -> 1115724630080

update graphik 2005-05-10 13:30:31.078 -> 1115724631080

update graphik 2005-05-10 13:30:32.078 -> 1115724632080

update graphik 2005-05-10 13:30:33.078 -> 1115724633079

update graphik 2005-05-10 13:30:34.078 -> 1115724634080







public class Main {

  /** Creates a new instance of Main */

  public Main() {

  }



  public static void main(String[] args) {

    NewClass thr = new NewClass();

    thr.start();

  }

}



///////////Class NewClass, celle du thread ////////////

class NewClass

    extends Thread {



  /** Creates a new instance of NewClass */

  public NewClass() {

  }



  public void run() {

    long t0 = System.currentTimeMillis();

    oh(t0); // on affiche l'heure de t0. Cet affichage prend du temps.

    while (true) {

      t0 +=1000;// prochain rep&#232;re : t0+1000

      try {

       
sleep(t0-System.currentTimeMillis());//on fait une pause jusqu'&#224;
atteindre l'heure t0+1000;

      }

      catch (InterruptedException e) {

        System.out.println("erreur thread update " + e);

      }

      oh(t0);//on affiche t0+1000;

    }

  }



  public void oh(long t) {

    Timestamp t3 = new Timestamp(0);

    t3.setTime(t);

    System.out.println("update graphik " + t3.toString()+" -> "+System.currentTimeMillis());

  }

}




A+


Dobel
[Une fois rien, c'est rien; deux fois rien,
ce n'est pas beaucoup, mais pour trois fois rien, on peut d&#233;j&#224;
s'acheter quelque chose, et pour pas cher]

joebar3333 · Answer

Salut Dobel,
merci pour tes r&#233;ponses pr&#233;cises et rapides. Ca marche ! R&#233;ponse accept&#233;e sans pb.
J'en profite de tes comp&#233;tences pour te diriger vers un de mes posts que je n'ai toujours pas pu r&#233;soudre : http://www.javafr.com/forum.v2.aspx?ID=450878

Si toutefois t'avais une id&#233;e, &#231;a serait g&#233;nial.

Merci encore.

cs_Dobel · Answer

lol, c'est juste que j'ai constat&#233; que &#231;a allait &#234;tre mon 201&#232;me
message, donc je voulais poster quelque chose de pas trop con :D

Dobel
[Une
fois rien, c'est rien; deux fois rien, ce n'est pas beaucoup, mais pour
trois fois rien, on peut d&#233;j&#224; s'acheter quelque chose, et pour pas cher]

joebar3333 · Answer

ok, merci quand meme

Décalage de thread ... help please

6 réponses

Votre réponse

Discussions similaires