nodecleaner: le retour

Version imprimable

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:

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;
 
		}
	}
}

17/01/2008, 14h41
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 ??

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 :roll: ]

disons:
Code:

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]) }; } } }

17/01/2008, 16h15
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.
17/01/2008, 16h31
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 ...
17/01/2008, 17h03
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...

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:

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
		}
	}
}