[javascript] problème de récursivité

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [javascript] problème de récursivité

stef_dobermann

Personne n'est parfait ...

Bonjour,

j'ai créé une petite classe en javascript (rien de bien méchant ),
voila une partie du code

Code :

initslideshow function (var1,var2,var3,var4){
this.var1 = var1
this.var2 = var2
this.var3 = var3
this.var4 = var4
//extrait du code pour donner une idée, je posterais plus tard la version complete
}
initslideshow.prototype.runslideshow = function (){
document.image[this.var1].src = thispic[this.i].src
if (++this.i > this.p) {this.i = 0}
settimeout('runslideshow ',this.var4) //fonction non trouvé !!
}

j'ai une erreur sous firefox dans la console d'erreur qui me dit que la fonction n'est pas trouvé !
je ne sais pas comment faire pour appeller récursivement la même fonction en gardant les parametres de celle-ci bien sur.

en faite suivant les pages, je pourrais être amené à instencier plusieurs fois cet classe avec des parametres différents !

---------------
Tout à commencé par un rêve...

Publicité

anapajari

s/travail/glanding on hfr/gs;

normal, runslideshow n'est pas une fonction mais une méthode de ta classe initslideshow ( je trouve les noms particulierement mal choisis ).
Attention, en utilisant un settimeout tu vas sortir du contexte de ton objet, tu ne pourras donc pas utiliser this.runslideshow.

Mais bon, comme j'ai pas la moindre idée de ce que tu veux faire, je trouve que ça sent un peu le paté ton truc.

Message cité 1 fois

stef_dobermann

Personne n'est parfait ...

c'est juste pour faire défiller plusieurs banniere publicitaire sur une page
avec des vitesse différentes.

en ce qui concerne le choix des fonction ou methode, rien de bien grave pour l'instant

si tu as mieux, je suis preneur !

Message édité par stef_dobermann le 02-07-2007 à 14:55:32

---------------
Tout à commencé par un rêve...

anapajari

s/travail/glanding on hfr/gs;

tu peux montrer tout le code utilisé par ton truc.?

stef_dobermann

Personne n'est parfait ...

pas pour l'instant, je suis au boulot, j'ai pas les sources
mais il n'y en a pas plus pour la partie javascript.

il y a juste une image (pour les testes) avec un nom, le nom de l'image est var1 (premier argument de la classe initslideshow)

---------------
Tout à commencé par un rêve...

anapajari

s/travail/glanding on hfr/gs;

nan mais là c'est totalement n'importe quoi ce que tu as écrit.
var2, var3 sont initules mais par contre tu te sers de this.i et this.p qui sont pas initialisées, thispic sort de l'espace donc ça mène forcément à rien ...
Donc montre le vrai code ...
edit: mais à l'instinct je ferais quelque chose dans le genre:

Code :

function slideshow(var1, var2, var3, var4){
this.var1 = var1;
this.var2 = var2;
this.var3 = var3;
this.var4 = var 4;
}
slideshow.prototype.run = function(){
document.image[this.var1].src = thispic[this.i].src
if (++this.i > this.p) {
this.i = 0;
}
}
...
window.onload = function (){
var pouet = new slideshow(...);
setInterval('pouet.run', pouet.var4);
}

non testé et garanti avec des erreurs!

Message édité par anapajari le 02-07-2007 à 17:48:17

stef_dobermann

Personne n'est parfait ...

merci pour ce code, il me parrait correct, je teste ca dès que possible, je te mettrais quand meme le code que j'utilisais !

---------------
Tout à commencé par un rêve...

bignose

anapajari a écrit :

Restons précis, on parle de prototype, pas de classe...

anapajari

s/travail/glanding on hfr/gs;

rancunier :??:

Message cité 1 fois

bignose

anapajari a écrit :

rancunier :??:

Meeeuuuuuh non. Juste précis :lol:

Publicité

stef_dobermann

Personne n'est parfait ...

Voici le script que l'on m'a fournit pour faire ça,
la seul chose est que la personne fait une duplication compléte du script en changeant le nom des variables pour faire tourner 2 ou 3 bannières différentes :

Code :

// Set slideShowSpeed (milliseconds)
var slideShowSpeed = 4000
// Duration of crossfade (seconds)
var crossFadeDuration = 20
// Specify the image files
var Pic = new Array() // don't touch this
// to add more images, just continue
// the pattern, adding to the array below
Pic[0] = '../images/graph/bann/bann1.jpg'
Pic[1] = '../images/graph/bann/bann2.jpg'
// =======================================
// Ne rien éditer au-dessous de cette ligne
// =======================================
var t
var j = 0
var p = Pic.length
var preLoad = new Array()
for (i = 0; i < p; i++){
preLoad[i] = new Image()
preLoad[i].src = Pic[i]
}
function runSlideShow(){
if (document.all){
document.images.SlideShow.style.filter="blendTrans(duration="+crossFadeDuration+" )"
document.images.SlideShow.filters.blendTrans.Apply()
}
document.images.SlideShow.src = preLoad[j].src
if (document.all){
document.images.SlideShow.filters.blendTrans.Play()
}
j = j + 1
if (j > (p-1)) j=0
t = setTimeout('runSlideShow()', slideShowSpeed)
}

Voici le code de mon script :

Code :

</head>
<script>
function initslideshow(imguser,slideShowSpeed,crossFadeDuration,Pic) {
// le constructeur
// propriété publique
this.imguser = imguser
this.slideShowSpeed = slideShowSpeed;
this.crossFadeDuration = crossFadeDuration;
this.Pic = Pic;
this.j = 0
this.preLoad = new Array()
for (this.j = 0; this.j < this.Pic.length; this.j++){
this.preLoad[this.j] = new Image()
this.preLoad[this.j].src = this.Pic[this.j]
}
this.j=0
}
initslideshow.prototype.runslideshow = function(){
if (document.all){
document.images[this.imguser].style.filter="blendTrans(duration="+this.crossFadeDuration+" )"
document.images[this.imguser].filters.blendTrans.Apply()
}
document.images[this.imguser].src = this.preLoad[this.j].src
if (document.all){
document.images[this.imguser].filters.blendTrans.Play()
}
if (++this.j > this.p) this.j=0
alert(this.imguser+'","'+this.j+'","'+this.preLoad+'","'+this.p+'","'+this.crossFadeDuration)
setTimeout('initslideshow.prototype.runslideshow()', this.slideShowSpeed)
}
</script>
<body>
<p><img src="../img/ban/ASDF50ER34_1000.gif" name='SlideShow' width="600" height="75"></p>
<p><img src="../img/ban/SFDF50ER34_1000.gif" name='SlideShow2' width="600" height="75"></p>
<script language="javascript">
//pour faire une banniere qui tourne toute les 2 secondes
Pic = new Array('../img/ban/ASDF50ER34_1000.gif','../img/ban/SFDF50ER34_1000.gif');
var obj = new initslideshow ('SlideShow',"2000","20",Pic);
obj.runslideshow();
//pour faire une banniere qui tourne toute les 4 secondes
Pic = new Array('../img/ban/ERDF50ER34_1000.gif','../img/ban/DRTDF50ER34_1000.gif');
var obj2 = new initslideshow ('SlideShow2',"4000","20",Pic);
obj2.runslideshow();
</script>
</body>

Message édité par stef_dobermann le 03-07-2007 à 14:37:52

---------------
Tout à commencé par un rêve...

bignose

Dans ton setTimeout, change initslideshow.prototype.runslideshow() par obj.runslideshow() (ou obj2 selon la banière que tu veux utiliser).
Dans le corps de ton <body> tu crées les variables obj et obj2. C'est elles que tu dois utiliser pour faire référence aux objets (et pour les puristes , obj et obj2 contiennent bien une référence).

!!!!!!! Les ; manquent presque partout !!!!!!!!!! A corriger.

stef_dobermann

Personne n'est parfait ...

merci pour l'info !

---------------
Tout à commencé par un rêve...

anapajari

s/travail/glanding on hfr/gs;

marchera pas voir ma 1ere reponse.
Dans runslideshow tu es sur une méthode de ton objet, celui-ci est donc représenté par this et non par le nom de la variable y faisant référence. Pour faire ce que suggère bignose, il te faudrait connaitre le "nom" de la variable faisant référence à ton objet dans la méthode.
Perso je ferais le setTimeOut hors de runslideshow.

Code :

initslideshow.prototype.runslideshow = function(){
if (document.all){
document.images[this.imguser].style.filter="blendTrans(duration="+this.crossFadeDuration+" )"
document.images[this.imguser].filters.blendTrans.Apply()
}
document.images[this.imguser].src = this.preLoad[this.j].src
if (document.all){
document.images[this.imguser].filters.blendTrans.Play()
}
if (++this.j > this.p) this.j=0
alert(this.imguser+'","'+this.j+'","'+this.preLoad+'","'+this.p+'","'+this.crossFadeDuration)
}
...
var obj = new initslideshow ('SlideShow',"2000","20",Pic);
setInterval(obj.runslideshow(), obj..slideShowSpeed);

Message cité 1 fois

stef_dobermann

Personne n'est parfait ...

Le script mis précédemment est le code de base, comme dit dans les précédent post,"je suis à mon travail, je ne peux vous fournir le code initiale... " voila qui est fait

en ce qui concerne le code ci-dessus, il ne fonctionne qu'une fois !
l'image change bien au bout de 2 secondes, mais c'est tout !!
le script n'a pas l'air de continuer, aucune erreur dans la console javascript de firefox !

---------------
Tout à commencé par un rêve...

anapajari

s/travail/glanding on hfr/gs;

euh tu as bien mis setInterval comme moi ou, par le plus grand des hasards, tu ne l'aurais pas remplacer par un setTimeOut

Message édité par anapajari le 04-07-2007 à 15:33:42

stef_dobermann

Personne n'est parfait ...

oups !

---------------
Tout à commencé par un rêve...

stef_dobermann

Personne n'est parfait ...

tiens j'ai cette erreur qui apprait à chaque fois que le script est exécuté :

Code :

Erreur : this.preLoad[this.j] has no properties
Fichier source : http://localhost/banniere/src/Banmultiple.htm
Ligne : 28

c'est cette ligne :

Code :

document.images[this.imguser].src = this.preLoad[this.j].src

je ne comprend pas pourquoi :?:

mais sinon l'image change

merci anapajari

je part bosser, à plus tard

Message édité par stef_dobermann le 04-07-2007 à 15:43:00

---------------
Tout à commencé par un rêve...

bignose

anapajari a écrit :

Seulement voila, des applications utilisant cette méthode pour faire tourner des dizaines d'objets de façon asynchrone dans un browser j'en ai déja fait pas mal... Alors je reconnais que c'est pas la méthode qu'on utiliserait avec des langages comme c++, c# ou java, mais ici on est en javascript et dans un browser. Il faut donc utiliser ce qu'on a sous la main.

Comme je l'ai déja dit dans un post précédent, en javascript tout ce qu'i n'est pas défini dans une fonction quelle qu'elle soit via le mot clé var est défini dans l'objet intrinsèque Global. Dès lors, et par conséquent, ceci fonctionne :

function truc()
{...}

function truc.prototype.run()
{...

setTimeout(A.run(), 1000);
...
}

var A = new truc();

Pourquoi ?

Parce que A est défini en dehors de toute fonction. Elle est donc créée dans l'objet Global. En utilisant setTimeout(A.run(), 1000), on s'adresse à la variable A définie dans l'objet Global (évidement, si on a défini une variable locale A, ça marchera pas). L'objet Global est unique tant qu'on ne ferme pas le browser. Tout ceci n'est qu'une propriété du langage javascript, rien de plus. Et c'est aussi une des ses faiblesses, car quand les gens ne déclarent pas leurs variables dans leurs fonctions via le mot clé var, ils ne se rendent souvent pas compte que ces variables se retrouvent toutes dans l'objet Global, avec tous les problèmes de collisions et de débugage que ça implique.

Encore une fois, je le répète (qu'on vienne pas me relancer là-dessue) c'est juste exploiter une propriété du javascript (qui est en fait simplement un moyen de faire du multi-thread), avec d'autres langages on fera évidement autrement.

Alors on dira, quel bordel ! Comment déguger un truc comme ça ? Et je répondrai : en ayant de bonnes habitudes de programmation, en appliquant des best practices de qualité, ça ne pose aucun problème. Les gens n'ayant aucune discipline de programmation auront de toute façon des problèmes, avec cette méthode ou avec une autre. Le code présenté ici par stef_doberman n'est en aucun cas de qualité suffisante pour pouvoir utiliser cette méthodologie efficacement, loin s'en faut. Mais le cas étant simple, ça ne pose pas de problème.

En tout cas, je déconseille totalement l'utilisation de setInterval, sauf à ceux à qui il reste encore quelques cheveux à s'arracher. Car là c'est vraiment indébugable (et je parle par expérience malheureusement).

Message cité 1 fois

0x90

→

Code :

lala.prototype.run = function() {
    // blah blah blah
 
    setTimeout(
        function(obj) { return function() {obj.run();}; }(this)    
    , 1000);
}

Jmet pas la déco (et y'a d'autres manières de le faire), mais l'idée est là , on a aucunement besoin de saloper l'espace global pour faire ça [:spamafote]

---------------
Me: Django Localization, Yogo Puzzle, Chrome Grapher, C++ Signals, Brainf*ck.

anapajari

s/travail/glanding on hfr/gs;

bignose a écrit :

function truc()
{...}
function truc.prototype.run()
{...

setTimeout(A.run(), 1000);
...
}
var A = new truc();

J'ai le droit de vomir?
En dehors de l'erreur sur la déclaration d'une méthode via prototype, ce que tu as écrit traduit une profonde incompréhension de la notion d'objet.
Cela cantonne l'utilisation du prototype "truc" au seul objet A.
Exemple:

Code :

function truc( p ){
this.prop = p;
}
truc.prototype.run = function(){
alert(A.prop);
}
var A = new truc('toto');
var B = new truc('tata');
B.run(); // affiche toto

ça n'a strictement rien à voir avec utiliser "une propriété du langage javascript", c'est juste faire n'importe quoi [:spamafote] .
Et ça va totalement à l'encontre de ce que tu écris ensuite:

bignose a écrit :

en ayant de bonnes habitudes de programmation, en appliquant des best practices de qualité, ça ne pose aucun problème. Les gens n'ayant aucune discipline de programmation auront de toute façon des problèmes, avec cette méthode ou avec une autre.

enfin:

bignose a écrit :

En tout cas, je déconseille totalement l'utilisation de setInterval, sauf à ceux à qui il reste encore quelques cheveux à s'arracher. Car là c'est vraiment indébugable (et je parle par expérience malheureusement).

c'est vrai qu'il vaut mieux faire un méthode appelé via setInterval et dans cette méthode refaire un nouveau setInterval à chaque appel... totalement logique [:w3c compliant]

Message édité par anapajari le 05-07-2007 à 13:28:47

stef_dobermann

Personne n'est parfait ...

sur ce coup, je suis d'accord avec anapajari

---------------
Tout à commencé par un rêve...

Publicité

FORUM HardWare.fr

Programmation

HTML/CSS

[javascript] problème de récursivité

Sujets relatifs
Problème de copier coller de données depuis un classeur fermé.	pbl javascript comment nommer un chemin d'objet par une variable
Problème de condition	Batch Probleme Echo sur serveur distant
menu vertical javascript que l'on peut cacher	Javascript : Donner focus à un <ul>
Problème saut ligne RSS => HTML via XSL	Gros probleme map sous IE
problème pour mettre un fond dans <body>	problème d'onglet
Plus de sujets relatifs à : [javascript] problème de récursivité

Page générée en 0.128 secondes