Discussion :

[pouemes]

bonjour jslint
me dit Undeclared 'setTimeout'.
voici mon code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
function test() {
if(window.essai) {
setTimeout("test()",1);
return;
}

commment y remedier?
merci
j'ai aussi Undeclared 'window'
sur la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(window.essai) {

que faire?
merci

[sekaijin]

setTimeout prend 2 paramètres
le premier est une fonction
le second une durée

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
unction test() {
 if(window.essai) {
  setTimeout(test,100);
  console.log('test');
  return;
 }
}

le résultat dans la console

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
>test()
undefined
>essai=true
true
>test()
undefined
(25) test  //<====== ici la fonction a écrit est 25 fois
>essai=false
false

A+JYT

[Watilin]

JSLint est très (trop ?) exigeant. Pour éviter les alertes avec les fonctions globales des navigateurs, uu peux utiliser la directive /*jslint browser*/. Mais du coup il se mettra à causer des problèmes avec 'use strict'. Et les guillemets simples. La morale de l’histoire c’est qu’il ne faut pas se préoccuper de tous les messages de JSLint, il y en a beaucoup…

[pouemes]

donc mon setTimeout est bien declare? test sans () et " ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

setTimeout(test,100);

ou

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

setTimeout("test()",1);

sinon je crois que je vais juste essayer de resoudre ls erreurs que signale dreamweaver
qui demande un use strict
il faut le mettre?
merci

[Watilin]

Le premier paramètre est une fonction, pas une chaîne. Sinon ça appelle eval en interne et… C’est mal.
Le second paramètre est une durée en millisecondes. Si tu mets 1000 ça fait une seconde.

Mets 'use strict' ça ne te coûte rien, si tu as suivi les avertissements de JSLint jusque-là, tu ne devrais pas avoir de problèmes. Tu es gagnant car ça va permettre aux moteurs JS d’optimiser l’interprétation de ton script.

Sous Dreamweaver tu peux coder en JS ? JSLint est inclus dedans ?

[pouemes]

salut watilin
pour l'instant j'ai ca (avec deux images qui presentent un texte l'une est miroir de l'autre ca fonctionne mais je voudrais l'optimiser... si tu peux m'aider (suis juste bidouilleur)
avec jslint j'ai des erreurs
les images sont la
http://pouemes.free.fr/poemes-amour/abe.gif
http://pouemes.free.frpoemes-amour/poeme-abe.gif
merci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
var wdmax = 170;
var hdmax = 145;
var ImAgE = "abe.gif";
var MiRoIr = "poeme-abe.gif";
var wdmin = 0;
var hdmin = 0;
var inc = 5;
var vIt = 50;
var poI = 7000;
var poM = 4000;
var PoE = "PoS";
 
function InVeRsE() {
    "use strict";
    if (window.Am) {
        setTimeout(function() {
            InVeRsE();
        }, 1);
        return;
    }
    if (PoE === "PoS") {
        var wd = document.getElementById("PE").getAttribute("width");
        wd = wd - inc;
        document.getElementById("PE").setAttribute("width", wd);
        if (wd === wdmin) {
            document.getElementById("PE").setAttribute("src", ImAgE);
            inc = -inc;
        }
        if (wd === wdmax) {
            PoE = "pOe";
            inc = -inc;
            setTimeout(function() {
                InVeRsE();
            }, poI);
        } else {
            setTimeout(function() {
                InVeRsE();
            }, vIt);
        }
    } else {
        var hd = document.getElementById("PE").getAttribute("height");
        hd = hd - inc;
        document.getElementById("PE").setAttribute("height", hd);
        if (hd === hdmin) {
            document.getElementById("PE").setAttribute("src", MiRoIr);
            inc = -inc;
        }
        if (hd === hdmax) {
            PoE = "PoS";
            inc = -inc;
            setTimeout(function() {
                InVeRsE();
            }, poM);
        } else {
            setTimeout(function() {
                InVeRsE();
            }, vIt);
        }
    }
}
window.onload = setTimeout(function() {
    "use strict";
    InVeRsE();
}, 3000);

le css

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
#BoItE {
display:table
}
 
#ImAgE {
    display:table-cell;
    vertical-align:middle;
	text-align:center
}

et ma div

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<div id="BoItE" style="width:170px;height:150px" onMouseOver="window.Am=1;" onMouseOut="window.Am=0;">
<div id="ImAgE">
<a href="#"><img id="PE" src="poeme-abe.gif" width="170" height="145" lang="aba"></a></div></div>

[NoSmoking]

Bonjour,

	attention pouemes, on s’éloigne du sujet original de cette discussion qui je le rappelle est « un Undeclared setTimeout »

Il est préférable d'ouvrir une discussion par question afin de ne pas perdre le fil de la discussion et de laissez la possibilité aux visiteurs de bénéficier des réponses en leur offrant un sujet moins dispersé.

PS : il me semble d'ailleurs que tu as déjà une discussion en cours sur ce sujet.

[pouemes]

salut no smoking
c'etait sur du css que j'abandonne car il ne me permet pas de faire ce que je veux je reviens donc a mon javascript que je cherche a optimiser
et c'est bien ce undefined setTimeout et window entre autre que m'annonce jslint
en cherchant sur internet je lis quon peut mettre en haut du script
/*jslint browser:true */
effectivement ca resoudt une partie du probleme mais est-ce une bonne solution?....

[Watilin]

Comme je l’ai déjà dit, tout ce que signale JSLint n’est pas à prendre au pied de la lettre. JSLint est surtout là pour pointer des endroits dans ton code où il y a un danger potentiel sur un aspect du langage que tu ne maîtrises peut-être pas. Ça t’encourage à te documenter et à mieux comprendre les implications de ton code.

À propos des "use strict" : à moins que tu aies une bonne raison de faire autrement, mets-en un seulement au début de ton fichier, pas besoin de le répéter dans chaque fonction. Même si JSLint te dit le contaire.

À propos de la casse (majuscules/minuscules) : c’est une chose que JSLint ne signale pas, mais j’ai l’impression que tu as envie de te casser les petits doigts sur ton clavier ou de devenir chauve en quelques mois. Adopte une convention de codage cohérente et tiens-t’y. Également, utilise des noms de variables entiers, c’est plus confortable pour la relecture. Et si c’est la taille de ton script qui te préoccupe, délègue l’optimisation à un minifieur, mais seulement en dernière étape de ton flux de travail, quand tu n’as plus besoin de toucher au code. Comme le dit l’adage, premature optimization is the root of all evil, en français : l’optimisation prématurée est la source de tous les maux.

Le setTimeout(…, 1) : je te l’ai déjà dit, la durée est en millisecondes. Les moteurs JavaScript actuels fixent un minimum de 10 ms pour éviter que le fil d’exécution soit surchargé. Visuellement, une variation inférieure à un soixantième de seconde, le fameux 60 FPS, apparaît fluide pour l’œil humain. L’intervalle entre deux frames est de 1000/60 soit environ 16,67 ms. Les navigateurs actuels fournissent une fonction réglée sur cet intervalle, et qui en plus optimise la consommation du processeur, par exemple quand la page est en arrière-plan : requestAnimationFrame. Jette un œil, tu verras, c’est super.

Un avertissement à propos de getAttribute et setAttribute : contrairement aux propriétés DOM qui sont un peu « magiques », [gs]etAttribute fonctionne très « bêtement », et ne produit parfois pas les effets qu’on attend. Un exemple avec width :

Code console :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
>> $img = new Image()
<img>
>> $img.width = 500
500
>> typeof $img.width
"number"
>> $img.getAttribute('width')
"500"
>> typeof $img.getAttribute('width')
"string"

On le voit ici, getAttribute renvoie toujours des chaînes, c’est un des effets indésirables.

En pratique, on n’a quasiment jamais besoin de [gs]etAttribute quand on connaît les propriétés DOM équivalentes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
element.setAttribute("width", wd);   -->  element.width = wd;
element.setAttribute("height", hd);  -->  element.height = hd;
element.setAttribute("src", image);  -->  element.src = image;

Et tu n’as pas répondu à ma question : on peut faire du JS sous Dreamweaver ? C’est pas trop contraignant ?

[pouemes]

salut watilin
merci pour ta reponse que je vais essayer de decortiquer
je ne pige pas comment remplacer setattribute et getattribute, je vais chercher sur internet
pour dreamweaver j'avoue que je ne l'utilise que pour le html
par contre il me met en rouge des lignes pour le javascript qd elles ne lui conviennent pas et note qd il n'y a pas use strict...

[Watilin]

Ben c’est pas compliqué, tu prends l’attribut html, dans la plupart des cas c’est le même nom, sauf pour class qui devient className parce que class est un mot-clé réservé en JavaScript.

[pouemes]

salut watilin
tu ne voudrais pas m'ecrire exactement ce que je dois remplacer

[Watilin]

Je l’ai déjà écrit :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
element.setAttribute("width", wd);   -->  element.width = wd;
element.setAttribute("height", hd);  -->  element.height = hd;
element.setAttribute("src", image);  -->  element.src = image;

[pouemes]

bonjour watilin

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById("Po").setAttribute("width", wd);

si je remplace
par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById("Po").element.setAttribute("width", wd);

le script ne fonctionne plus

[Watilin]

Remplace

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById("Po").setAttribute("width", wd);

par

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById("Po").width = wd;

[pouemes]

merci watilin ca marche
j'ai remplace tous les settattibut et meme les get attribut
comme ceci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
        var wd = document.getElementById("P").width;
        wd = wd - inc;
        document.getElementById("Po").width = wd;
        if (wd === wdmin) {
            document.getElementById("Po").src = ImAgE;