Méthode récursive avec Thread.sleep : StackOverflowError

Bonjour à tous,

Débutant en Java, j'ai un problème récurrent de débordement de pile.

Pour faire de l'animation vectorielle (Swing+JPanel), j'utilise une méthode de ce type :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

private void refresh() { panel.repaint();   try { Thread.sleep(40); refresh(); } catch(InterruptedException ex) { ex.printStackTrace(); } }

---

Cela fonctionne très bien pendant plusieurs minutes mais finit systématiquement par envoyer une erreur de type "java.lang.StackOverflowError".

Ca semble une erreur légitime pour une récursion "infinie", mais étant donné que je demande au programme d'attendre 40ms avant de relancer la fonction je ne comprends pas quel est le problème.

Pourriez-vous m'éclairer sur ce point svp ?
Existe-t-il une meilleure méthode pour rafraîchir l'affichage à une fréquence donnée ?

Merci d'avance !

le fait d'attendre 40ms ne change rien au fait que refresh appelle refresh qui appele refresh qui.....

tu dois créer un thread à part qui effectue le travail dans une boucle et non dans une récursion.

Essaye avec ça :

Code:

---

1 2 3 4 5 6 7

private void refresh() { for (;;) { panel.repaint(); Thread.sleep(40); } }

---

Tu n'auras pas de pb de débordement de pile mais il faut penser à arrêter le Thread

Merci à tous les deux, ça tourne depuis 10min sans pb ! :P

J'ai un peu du mal à comprendre la différence entre les deux par contre. Qu'est-ce que ça change, fondamentalement, pour la jvm ?

Quand tu appelle la méthode refresh(), elle occupe une place dans la pile jusqu'à ce que celle-ci finisse son exécution.

Donc, en appelant la méthode refresh() dans lui-même, la méthode va occuper une autre place avant que la première exécution soit finie, ... d'où le débordement de la pile.

Ok, je comprends mieux. J'avais pas réalisé que la fonction restait dans la pile.

Merci !