Discussion :

[javatwister]

je tente une amélioration du vieux code mais je peine toujours: selon vous, c'est plus économique ou pas? ça a l'air de marcher en tout cas;

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
var q, bal, p;
 
function clean(d){
	bal=d.getElementsByTagName('*');
	for(j=0;j<bal.length;j++){
		if(!q){
			q=true
		}
		else{
			q=false;
			continue
		}
 
		p=bal[j].parentNode.childNodes;
		for(i=0;i<p.length;i++){alert("ok")
			if(p[i].data && !p[i].data.replace(/\s/g,'')){
				p[i].parentNode.removeChild(p[i])
			};
		q=false;
 
		}
	}
}

[le_chomeur]

Je n'ai pas suivis la première version mais un d.innerHTML = null ( oui c crad je sais ) ne fonctionnerait-il pas ? les noeuds existent-ils toujours en mémoire ??

[javatwister]

ben nan, j'ai testé aussi et ça ne détruit pas le noeud;

mais je viens de voir une connerie dans le code, il faut que je retravaille;

[ah non, ça marche... une alert oubliée ]


disons:

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
var q, bal, p;
 
function clean(d){
	bal=d.getElementsByTagName('*');
	for(j=0;j<bal.length;j++){
		if(!q){
			q=true
		}
		else{
			q=false;
			j++;
		}
 
		p=bal[j].parentNode.childNodes;
		for(i=0;i<p.length;i++){
			if(p[i].data && !p[i].data.replace(/\s/g,'')){
				p[i].parentNode.removeChild(p[i])
			};
		}
	}
}

[Sergejack]

p=bal[j].parentNode.childNodes;

=>

p=bal[j].childNodes;

Mieux ?

C'est dégeux en tout cas ^^'
Et je ferais un test sur le NodeType à la place de .data.

[le_chomeur]

question bète , mais pkoi ne pas faire une fonction avec un appel récursif , avec comme paramètre d'entré le noeud, puis sur chaque noeud a supprimer , on vérifit s'il a des enfants, si oui on rappel la fonction ...

[javatwister]

Sergejack: on parle bien des noeuds frères, non des enfants

le_chomeur: l'une des premières moutures explorait ce principe; après, le but était d'obtenir le moins de boucles / passage par boucle possible, et du coup, la récursivité était moyenne...

[dolfero]

Bonjour,

C'est un vieux sujet mais qui est toujours d'actualité.
J'ai développé un nodeCleaner récursif.
Il est plutôt rapide. Qu'en pensez-vous ?

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
/*****************************************************************
* Fonction récursive de suppression des noeuds texte dans un DOM *
*         Synthaxe : deleteEmptyNodeText(noeudATraiter);         *
*****************************************************************/
 
function deleteEmptyNodeText(pere, fils) {
	if(typeof(fils) == 'undefined') { // Si le 2ème argument (fils) n'est pas renseigné -> 1ère boucle
		fils = pere; // Le fils devient le père
		pere = fils.parentNode; // Et le père devient le père du fils
		deleteEmptyNodeText(pere, fils); // On lance la fonction avec les 2 bons arguments
	} else {
		if(fils.hasChildNodes()) { // Si le fils a des enfants
			deleteEmptyNodeText(fils, fils.firstChild); // On le traite en tant que père avec son 1er enfants
		}
		if(fils.nextSibling) { // Si le fils a un frère
			deleteEmptyNodeText(pere, fils.nextSibling); // On le traite en gardant le même père et son enfant suivant
		}
		if(fils.nodeType == 3 && (fils.nodeValue.replace(/^\s+|\s+$/g, '').length == 0 || fils.nodeValue.charCodeAt(1) == 32)) { // Si il s'agit d'un noeud texte vide
			pere.removeChild(fils); // On le supprime
		}
	}
}