Fonction avec plusieurs instances

Bonjour,

Je patine sur une fonction JS pour generer des calendriers.
Tout fonctionne bien jusqu'au moment ou j'ai plus d'une instance. La, quand je change le mois sur ma premiere instance, c'est la deuxieme instance qui change.
Je ne comprends pas bien la facon dont je dois construire ma fonction.

Pouvez-vous me dire ce qu'il faut faire pour que les methodes appelees depuis la fonction s'appliquent uniquement a l'instance concernée?

Ci-apres un petit extract de la facon dont ma fonction est construite (je vous épargne l'intégralité du code qui fait 500 lignes).

Code:

---

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

function calendar(FieldId) { var me = this; me.SelectedDays = Array(); //etc   GoNextMonth = function(){ //blabla } me.NextMonthButton = document.createElement('div'); me.NextMonthButton.onclick=GoNextMonth(); }

---

PS: J'emploie le mot instance, mais ma fonction est une fonction, pas une classe (je l'appelle par calendar('test'), pas par new calendar('test'))...


Merci!

Il nous faudrait le code de la fonction GoNextMonth() et le code qui utilise le paramètre FieldId pour pouvoir chercher d'où vient le problème.
Si le code est vraiment trop long, un page de test en ligne fera très bien l'affaire !

Merci!
http://www.babna.com/cal/

Voilà c'est mieux :lol:

Dans les fonction GoNextMonth (et toutes les autres du même style d'ailleurs), tu utilises la variable "me". Or, "me" est hors contexte : elle est récupérée de la fonction calendar() et a comme valeur le dernier calendrier créé.
Pour que le script fonctionne il faut remplacer tous les "me" par des "this" dans toutes les fonctions utilisées comme des évènements. Un autre problème vient alors : le "this" ne pointe pas sur le calendrier, mais sur l'objet sur lequel s'est passé l'évènement. Il faut donc que tu trouve un moyen, à partir de cet objet, de retrouver le calendrier auquel il appartient :aie:.

Merci de ta réponse rapide Julien!

Alors j'ai mis des this partout, mais ca ne fonctionne toujours pas.
Cette fois les fonctions ne paraissent fonctionner que lorsqu'elles sont appelees depuis l'extérieur.
Je sais que j'ai merdouillé dans le code mais je ne vois pas ou.

Les this qui etaient a l'origine dans certaines fonctions ont ete remplaces par des "thasse"... Je vais changer ca dans l'ensemble des fonctions mais je veux verifier d'abord que this.GoOver et this.GoOut fonctionnent. (J'ai rajoute un argument dans ces fonctions et remplace thasse par la reference a l'objet correspondant).

Merci encore!

Euh, si tu n'instancie pas d'objet, ton this se réfère à window, c'est pas forcément ce que tu veux faire, c'est pas forcement utile et en plus ça balance pleins de référence dans le scope global

Code:

---

me.NextMonthButton.onclick=GoNextMonth();

---

Dans cette ligne, tu éxectures la fonction alors que tu es censé y faire référence, si elle ne retourne rien :

Code:

---

me.NextMonthButton.onclick == null

---

Merci de ton aide, mais je ne comprends pas ton dernier post.
Peux-tu préciser stp?

Merci!

Quand tu crées un onclick, tu dois faire référence à une fonction. A partir du moment ou tu mets (); après un id objet, javascript va (tenter au moins) l'executer.

Code:

---

1 2 3 4 5 6

function test() { alert('Foo'); }   element.onlick = test; //On fait référence à l'objet (la fonction) test, le click déclenche l'exectution de celle ci.

---

Code:

---

1 2 3 4 5 6 7 8

function test(num) { return function() { alert(num); } }   element.onlick = test(element.offsetHeight); //On execute la fonction, directement, seulement elle retourne une autre fonction, et c'est fonction retournée est encapsulée dans le onclick.

---

Note bien que par défaut une fonction ne renvoie rien.

Je suis désolé, je ne suis pas certain d'y voir plus clair.

Que dois-je faire pour que le script http://www.babna.com/cal/ fonctionne?

Merci!

Bonjour, personnellement j'utilise ce mécanisme :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22

CALENDAR_ARRAY = new Array() en variable globale CALENDAR_ARRAY[0] = new calendar()     function calendar(FieldId, '0') { var me = this; me.SelectedDays = Array(); //etc   GoNextMonth = function(){ //blabla } me.NextMonthButton = document.createElement('div'); me.NextMonthButton.onclick=GoNextMonth(0); }   GoNextMonth(pInstance) { var me = CALENDAR_ARRAY[pInstance] ... }

---

Je viens d'essayer de la facon preconisee dans le POST precedent (merci pour ca), mais j'ai encore mon "me" qui n'est pas reconnu.
C'est en ligne si vous voulez voir...

Merci

Je viens de refaire un autre essai.
Cette fois j'ai une erreur "too much recursion", et mon calendrier affiche l'annee 1508...
C'est sur cette page

http://www.babna.com/cal/

J'ai donc encore deux problemes.
Je vous remercie si vous pouvez m'aider.

Dans http://www.babna.com/cal/, si on fait apparaitre un des deux calendriers, et qu'on va jusqu'en Décembre 2007 (le premier a avoir 6 lignes), la derniere ligne du calendrier ne disparait pas lorsqu'on va sur le mois suivant ou précédent.

Dans ma fonction paint(), je fais pourtant

me.cal.innerHTML = '';

Et j'avais aussi essayé

me.cal.length=0;

avec le meme probleme.

L'autre probleme est avec le bouton du calendrier, qui est cense faire apparaitre ET disparaitre le calendrier.

J'utilise la fonction suivante:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

me.ShowHide = function() { if ( me.cal != null ) { if ( me.cal.style.display != 'none' ) { alert(me.cal.style.display); me.cal.style.display = 'none'; } else { me.cal.style.display = 'block'; } } else { me.paint(); } }

---

J'ai mis un alert() afin de vérifier qu'il reconnait bien ce que je lui demande, ce qui est le cas, mais une fois qu'un des mois ou annees a été cliqué (donc que la fonction paint() a été rappelée, au lieu de le faire disparaitre, il va au mois précédent...

Merci de votre aide!

Thomas

En fait, je crois que ces probleme viennent du fait que ma fonction paint() fait apparaitre une nouvelle instance a chaque fois, mais je ne comprends pas pourquoi ni comment.

Une idee?

Merci

J'ai toujours un problee, mais different cette fois: le DIV est recréé a chaque fois que j'utilise la methode Paint:

http://www.babna.com/cal/

Qqun peut-il m'aider?

Merci!

C'est bon, je supprime le DIV avec removeChild a chaque fois que j'appelle Paint.
Tout est ok...

Merci de votre aide quand meme!