Discussion :

[mapmip]

bonjour,
j'essaie de minifier mon code javascript avec http://jscompress.com/ mais j'ai l'erreur :Unexpected token operator, expected punc.
Pourtant firefox ne me signale aucune erreur !
Comme faire ?
Merci d'avance

[SpaceFrog]

Comment faire ?

M'envoyer assez rapidement une boule de cristal que je devine ton code ...

[Invité]

Faites un tour sur jslint pour être certain que le code que vous tentez de compresser est valide.

[mapmip]

le code est enorme !
comment faire pour reperer la ligne d'erreur ???
connaissez vous un minifier qui permette de reperer la ligne d'erreur ?

[SpaceFrog]

a mon avis il manque un ; en fin de ligne quelque part ...

[mapmip]

POURTANT FIREBUG NE DETECTE AUCUNE ERREUR !

[SpaceFrog]

ce qui est normal car en syntaxe indentée avec retour ligne, les ; sont facultatifs, mais ils deviennent obligatoires pour la compression

[Invité]

le code est enorme !
comment faire pour reperer la ligne d'erreur ???
connaissez vous un minifier qui permette de reperer la ligne d'erreur ?

Minifier (2) et vérifier la syntaxe (1) sont deux opérations différentes. Je ne connais pas d'outils qui fasse les deux.
Mais, il vaut mieux faire (1) avant de faire (2). Minifier du code syntaxiquement faux est source d'erreur comme vous pouvez le constater.
Firebug n'est pas un validateur de syntaxe javascript. Et firefox se contente d'exécuter au mieux ce qu'on lui demande de faire à travers le code, en corrigeant à la volée les erreurs les plus grossières.

[mapmip]

mince , j'ai trouvé,
le minifier n'aime pas les valeurs par défaut des parametres de fonction :

Code :

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

1
2
function setBackgroundColor(element,color='blue') {
   }

va bugger car le parametre color ne doit pas avoir de valeur par défaut il faut corriger par

Code :

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

1
2
function setBackgroundColor(element,color) {
   }

Pourtant ca m'arrange de mettre une valeur par défaut ! Une solution ?

[SpaceFrog]

Cette syntaxe n'est pas correcte en js

Code :

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

1
2
function setBackgroundColor(element,color='blue') {
   }

Essaye plutot quelque chose du genre

Code :

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

1
2
3
4
5
6
7
8
 
function SetDef(arg, def) {
   return (typeof arg == 'undefined' ? def : arg);
}
 
function setBackgroundColor(element,color){
  color = SetDef(color, 'blue');
}