erreur Internet Explorer 6 - 8

**chochatown** · 06/08/2012, 17h18

Bonjour,

pour les experts de IE

J'ai une erreur sur ma page seulement quand j'y accede avec IE.

les messages d'erreurs sont en peices jointes. Ce qui est ettonant, la ligne d'erreur est dans une fonction reconnu "getElementsByClassname" qui est valide sur tt les navigateurs web...

j'ai l'impression que le deboggueur ne me donne pas la bonne information. Et ce qui est sur c'est que le message d'erreur manque beaucoup d'informations.

Merci pour votre aide!

**arnogues** · 06/08/2012, 17h26

Non cette fonction n'existe pas dans IE6/7/8 donc je te conseille de la coder ou de passer par une librairie genre jquery

**chochatown** · 06/08/2012, 17h37

C'est ce que je suis entrain de faire, l'erreur est dans le code de la fonction que j'ai ajoute, et c'est cette fonction que j'utilise. la ligne d'erreur est dans le code la fonction que j'ai du ajouter puisqu'elle est pas reconnu dans IE. dsl, je pense que j'ai mal explique au debut. le code de la fonction etait teste par son auteur sur tt les navigateurs que je voulais dire et non pas la fonction qui est reconnu.

**NoSmoking** · 06/08/2012, 18h00

Bonjour,
il eut mieux valu joindre le code de la fonction plutôt qu'une image, marci d'y penser par la suite.

- ne pas omettre de déclarer les variables à l'aide du mot clé var.
- ne pas utiliser des mots connu du langage comme match.

essaies de changer le nom de cette variable.

**chochatown** · 06/08/2012, 20h30

Voila je viens de modifier lal fonction, mais autre erreur intervinne apres cette modification! :

ReferenceError: current is not defined
[Break On This Error]

current;

Avant :

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
    /*
            Developed by Robert Nyman, http://www.robertnyman.com
            Code/licensing: http://code.google.com/p/getelementsbyclassname/
        */    
        s.getElementsByClassName = function (className, tag, elm){
			var getElementsByClassName = function(){}; // limit scope of function
 
            if (document.getElementsByClassName) {
                getElementsByClassName = function (className, tag, elm) {
                    elm = elm || document;
                    var elements = elm.getElementsByClassName(className),
                        nodeName = (tag)? new RegExp("\\b" + tag + "\\b", "i") : null,
                        returnElements = [],
                        current;
                    for(var i=0, il=elements.length; i<il; i+=1){
                        current = elements[i];
                        if(!nodeName || nodeName.test(current.nodeName)) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            else if (document.evaluate) {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    var classes = className.split(" "),
                        classesToCheck = "",
                        xhtmlNamespace = "http://www.w3.org/1999/xhtml",
                        namespaceResolver = (document.documentElement.namespaceURI === xhtmlNamespace)? xhtmlNamespace : null,
                        returnElements = [],
                        elements,
                        node;
                    for(var j=0, jl=classes.length; j<jl; j+=1){
                        classesToCheck += "[contains(concat(' ', @class, ' '), ' " + classes[j] + " ')]";
                    }
                    try    {
                        elements = document.evaluate(".//" + tag + classesToCheck, elm, namespaceResolver, 0, null);
                    }
                    catch (e) {
                        elements = document.evaluate(".//" + tag + classesToCheck, elm, null, 0, null);
                    }
                    while ((node = elements.iterateNext())) {
                        returnElements.push(node);
                    }
                    return returnElements;
                };
            }
            else {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    var classes = className.split(" "),
                        classesToCheck = [],
                        elements = (tag === "*" && elm.all)? elm.all : elm.getElementsByTagName(tag),
                        current,
                        returnElements = [],
                        match;
                    for(var k=0, kl=classes.length; k<kl; k+=1){
                        classesToCheck.push(new RegExp("(^|\\s)" + classes[k] + "(\\s|$)"));
                    }
                    for(var l=0, ll=elements.length; l<ll; l+=1){
                        current = elements[l];
                        match = false;
                        for(var m=0, ml=classesToCheck.length; m<ml; m+=1){
                            match = classesToCheck[m].test(current.className);
                            if (!match) {
                                break;
                            }
                        }
                        if (match) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            return getElementsByClassName(className, tag, elm);
        };

apres enlever les var et les match :

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
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
        /*
            Developed by Robert Nyman, http://www.robertnyman.com
            Code/licensing: http://code.google.com/p/getelementssbyclassname/
        */    
        s.getElementsByClassName = function (className, tag, elm){
			getElementsByClassName = function(){}; // limit scope of function
 
            if (document.getElementsByClassName) {
                getElementsByClassName = function (className, tag, elm) {
                    elm = elm || document;
                    elementss = elm.getElementsByClassName(className),
                        nodeName = (tag)? new RegExp("\\b" + tag + "\\b", "i") : null,
                        returnElements = [],
                        current;
                    for(iount=0, il=elementss.length; iount<il; iount+=1){
                        current = elementss[iount];
                        if(!nodeName || nodeName.test(current.nodeName)) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            else if (document.evaluate) {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    classes = className.split(" "),
                        classesToCheck = "",
                        xhtmlNamespace = "http://www.w3.org/1999/xhtml",
                        namespaceResolver = (document.documentElement.namespaceURI === xhtmlNamespace)? xhtmlNamespace : null,
                        returnElements = [],
                        elementss,
                        node;
                    for(jount=0, jl=classes.length; jount<jl; jount+=1){
                        classesToCheck += "[contains(concat(' ', @class, ' '), ' " + classes[jount] + " ')]";
                    }
                    try    {
                        elementss = document.evaluate(".//" + tag + classesToCheck, elm, namespaceResolver, 0, null);
                    }
                    catch (e) {
                        elementss = document.evaluate(".//" + tag + classesToCheck, elm, null, 0, null);
                    }
                    while ((node = elementss.iterateNext())) {
                        returnElements.push(node);
                    }
                    return returnElements;
                };
            }
            else {
                getElementsByClassName = function (className, tag, elm) {
                    tag = tag || "*";
                    elm = elm || document;
                    classes = className.split(" "),
                        classesToCheck = [],
                        elementss = (tag === "*" && elm.all)? elm.all : elm.getElementsByTagName(tag),
                        current,
                        returnElements = [],
                        matche;
                    for(kount=0, kl=classes.length; kount<kl; kount+=1){
                        classesToCheck.push(new RegExp("(^|\\s)" + classes[kount] + "(\\s|$)"));
                    }
                    for(lount=0, ll=elementss.length; lount<ll; lount+=1){
                        current = elementss[l];
                        matche = false;
                        for(mount=0, ml=classesToCheck.length; mount<ml; m+=1){
                            matche = classesToCheck[mount].test(current.className);
                            if (!matche) {
                                break;
                            }
                        }
                        if (matche) {
                            returnElements.push(current);
                        }
                    }
                    return returnElements;
                };
            }
            return getElementsByClassName(className, tag, elm);
        };

**NoSmoking** · 06/08/2012, 20h52

pourquoi avoir modifié l'original ?

Code :

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

1
2
3
var getElementsByClassName = function (className, tag, elm){
	if (document.getElementsByClassName) {
		getElementsByClassName = function (className, tag, elm) {

devient

Code :

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

1
2
3
4
s.getElementsByClassName = function (className, tag, elm){
	var getElementsByClassName = function(){}; // limit scope of function
        if (document.getElementsByClassName) {
               getElementsByClassName = function (className, tag, elm) {

**chochatown** · 06/08/2012, 23h09

C'est juste pour des tests. Je limite la fonction. Mais, là mon erreur s'affiche dans les deux cas. Cette ligne existe ou non...

Je n'ai fait que enlever les var et changer le nom de match et ça donne une nouvelle erreur.

**chochatown** · 06/08/2012, 23h33

Et voila un Warning de Closure-compiler :

JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2870 character 0
currente;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2889 character 0
elementss;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2890 character 0
node;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2913 character 0
currente;
^
JSC_USELESS_CODE: Suspicious code. This code lacks side-effects. Is there a bug? at line 2915 character 0
matche;

**NoSmoking** · 07/08/2012, 11h32

Ce qui m'interpèle le plus c'est le s.getElementsByClassName = ..., il faudrait voir le contexte dans lequel tu l'utilises mais je pense qu'il faut que tu reviennes à l'utilisation "standard" de la fonction, sans modification, il me semble que cette fonction à déjà fait ses preuves.

**chochatown** · 09/08/2012, 22h20

Finalement, le bug est resolu.

Pour des raisons inconnu lors de la comparaison :

Code :

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

match = classesToCheck[m].test(current.className)

Donc j'ai modifié cette ligne pour verifier les objets et leurs types comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
					//match = classesToCheck[m].test(current.className);	
					tmp1 = classesToCheck[m];
					if(typeof current=='object')
						{
							if(typeof current.className=='string')
								{
								tmp2 = current.className;
								cmpr = tmp1.test(tmp2);
								if(cmpr == true)
								match = true;
								}
						}

Resolu! IE sucks!

erreur Internet Explorer 6 - 8

JavaScript

Discussions similaires

Partager

Partager