Discussion :

[arnolem]

Bonjour à tous, je suis en train de debugger des scripts javascript comme rico ou prototype.js

j'ai réussi à faire sauter pas mal de warning mais j'ai encore un truc que je ne comprend pas

Avertissement : variable element hides argument
Fichier source : .../prototype.js
Ligne : 1479, Colonne : 8
Code source :
var element = $(element);

c'est quoi ça

Code :

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

var element = $(element);

et surtout, quelle est l'heure ? merci

[FremyCompany]

Bonjour à tous, je suis en train de debugger des scripts javascript comme rico ou prototype.js

j'ai réussi à faire sauter pas mal de warning mais j'ai encore un truc que je ne comprend pas


c'est quoi ça

Code :

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

var element = $(element);

et surtout, quelle est l'heure ? merci

A ce que j'ai cru comprendre $ est une sorte de document.getElementById made in JS-FramerWork(lequel je sais pas)

L'avertisemment est simple, il s'agit d'un cas de variable d'un niveau supérieur masqué par une variable de niveau inférieur.

ex :

Code :

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

1
2
3
El = function(E) {
    var E = $(E);
}

Code :

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

1
2
3
4
5
6
7
El = function(E) {
    var I = $(E);
    if (I) {
        for (var I=0; I<5; I++) {
        }
    }
}

...

[FremyCompany]

Aujourd'hui, 18h09

[arnolem]

ouai, mais ça ne me dis pas comment corriger les dizaine de bug liés à ça !
Merci quand meme

[FremyCompany]

ouai, mais ça ne me dis pas comment corriger les dizaine de bug liés à ça !
Merci quand meme

Ben faut renommer les variables mal nommées...

Corrections des exemples :

Code :

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

1
2
3
El = function(E) {
    var e = $(E);
}

Code :

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

1
2
3
4
5
6
7
El = function(E) {
    var I = $(E);
    if (I) {
        for (var i=0; i<5; i++) {
        }
    }
}

[arnolem]

Pour te donner un exemple, j'ai ça

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
   round: function(e, options) {
      var e = $(e);
      this._setOptions(options);
 
      var color = this.options.color;
      if ( this.options.color == "fromElement" )
         color = this._background(e);
 
      var bgColor = this.options.bgColor;
      if ( this.options.bgColor == "fromParent" )
         bgColor = this._background(e.offsetParent);
 
      this._roundCornersImpl(e, color, bgColor);
   },

il suffit donc que je supprimer la ligne

Code :

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

      var e = $(e);

ou faut il réellement passer par une autre variable ?

[arnolem]

C'est bon, je n'ai plus aucune erreur ni Warning, par contre, je n'ai toujours pas compris à quoi servais la syntaxe

Code :

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

var e = $(e);

[FremyCompany]

C'est bon, je n'ai plus aucune erreur ni Warning, par contre, je n'ai toujours pas compris à quoi servais la syntaxe

Code :

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

var e = $(e);

C'est une faute... il fallait changer de nom de variable ou écrire

Code :

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

e=$(e) // $ == document.getElementById

[arnolem]

ok, ça j'ai bien compris, mais alors a quoi sert :

Code :

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

var t = $(e);

?