Discussion :

[sagopa]

Bonjour,

J'aimerais supprimer les noeuds ayant pour tag "script" et attribut class="script"

Par exemple, je voudrais supprimer tout le code suivant:

Code :

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

1
2
3
4
 
<script type="text/javascript" class="script">
function();
</script>

Pour cela, j'ai créé la fonction suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
function suppression_element()
{
	//Get the node to delete
	var nodeToDelete = document.getElementsByTagName("script");
	//If this node exists then delete it
	for (i = 0 ; i < nodeToDelete.length ; i++)
	{
		if (nodeToDelete[i].getAttribute('class') == "script")
		{
			nodeToDelete[i].parentNode.removeChild(nodeToDelete[i]);
		}
	}
}

Sur IE ça ne marche pas du tout. Et sur Firefox, si j'ai plusieurs éléments à supprimer , il sort de la boucle for après la première suppression.

Je n'ai pas d' énormes connaissances en JavaScript, si vous pouviez m'aider je vous serais reconnaissant.

En outre, si vous vous demandez pourquoi je veux supprimer les noeuds ayant pour tag "script" c'est parce que, en faite, j'ai un cgi qui tourne sur Apache. Ce cgi crée la page web vu depuis le navigateur. Mon cgi tourne en continue, cad en continue il envoie des données. Ces données sont le code JavaScript à exécuter par le navigateur. Ainsi j'ai peur que le code source devient énorme et qu'il prenne beaucoup de mémoire. Je me trompe ?

[SpaceFrog]

quand tu supprimes un noeud le length change
il faut donc toujours commencer par le dernier ...

Code :

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

1
2
3
4
5
6
7
while (nodeToDelete.length>0)
	{
		if (nodeToDelete[nodeToDelete.length-1].className == "script")
		{
			nodeToDelete[nodeToDelete.length-1].parentNode.removeChild(nodeToDelete[nodeToDelete.length-1]);
		}
	}

[sagopa]

Merci beaucoup!

En effet,il faut commencer par la fin. Et en utilisant className au lieu de .getAttribute('class'), ça fonctionne sur IE également.

Voici le code final:

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
 
function timer()
{
	objectSetTimeout = setTimeout("deleteCode()", 300000);
}
 
function deleteCode()
{
	//Get the node to delete
	var nodeToDelete = document.getElementsByTagName("script");
	//If this node exists then delete it
	for (i = nodeToDelete.length - 1 ; i >= 0  ; i--)
	{ 
		if (nodeToDelete[i].className == "script")
		{
			nodeToDelete[i].parentNode.removeChild(nodeToDelete[i]);
			nodeToDelete[i] = null;
		}
	}
 
	timer();
}

Avec un timer, tous les 5 min j'effectue la suppression.

[sagopa]

J'ai un petit souci.
La suppression que je désire s'effectue correctement, mais pour chaque suppression, un retour à la ligne est inseré.

Ainsi est-il possible d'éviter ceci?
Merci.

[SpaceFrog]

regarde dans le code généré quel est le type de Noeud qui te fait un retour ligne et vire le de la même façon ...