Discussion :

[destructive]

salut tout le monde,

j'ai voulu écrire une fonction qui classifie des lignes d'un tableau en utilisant la couleur du texte contenu dans l'une de leurs cellules ainsi :

1/ la fonction va scanner toutes les lignes de tableaux dans la page et examinera dans chaque ligne une cellule identifiée par sa classe.
2/ la fonction va détecter la couleur du texte que contient cette cellule, si la couleur est le rouge par exemple elle va cacher toute la ligne
voilà ce que j'ai écrit :

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
function detectcolumn(identificateur, coulor) {
   var i;
 
   var tabTD = document.getElementsByTagName("tr");
   var n = tabTD.length;
 
  for (i=0; i<n; i++)
  {
    var textcolor = document.getElementsByTagName("td").fgColor;
    if (tabTD[i].getElementsByTagName("td").className=="identificateur") {
	if (textcolor=="coulor")
    {
     tabTD[i].style.display = (tabTD[i].style.display=="none")?"":"none";
    }
	}
  }
	   }

mais ça marche pas !!

quelqu'un peut m'aider ?
merci

[E.Bzz]

Code :

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

var textcolor = document.getElementsByTagName("td")[i].style.color;

sinon, getElementsByTagName retourne un tableau (d'où le [i ]).

A toi de la caser dans une boucle
Sinon, utilise les id des <td> ...

EDIT : ben non, c'est déjà le cas


A+

[destructive]

j'ai essayé mais ça marche pas encore, même en modifiant le code ainsi :

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
function detectcolumn(identificateur, coulor) {
   var i;
 
   var tabTD = document.getElementsByTagName("tr");
   var n = tabTD.length;
 
  for (i=0; i<n; i++)
  {
 
   if (tabTD[i].getElementsByTagName("td").className=="identificateur") {
   var textcolor = document.getElementsByTagName("td")[i].style.color;
	if (textcolor=="coulor")
    {
 
     tabTD[i].style.display = (tabTD[i].style.display=="none")?"":"none";
    }
	}
  }
  i++
}

ça donne rien !!

[marcha]

Salut,

Tes paramètres ne devraient pas être entre guillemets dans ta fonction.
d'autre part i est l'index utilisé pour la liste des tr, il ne faut pas l'utiliser
comme tu le fait avec la liste de td.

Voici un réarrangement de ton code qui me semble plus juste (non-testé)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function detectcolumn(identificateur, couleur) { 
  var trs = document.getElementsByTagName("tr");
  for(var row=0;row<trs.length;row++) {
    if (trs[row].className!=identificateur) continue;
    var tds = trs[row].getElementsByTagName("td");
    for(var col=0;col<tds.length;col++) {
      if(tds[col].style.color!=couleur) continue;
      trs[row].style.display = (trs[row].style.display=="none") ? "" : "none";
      break; // si plusieurs td de couleur dans cette ligne, éviter de la réafficher ?
    }
  }
}

Pour que cela fonctionne au niveau de la détection des couleurs
il faut impérativement que les td aient un attribut style='color: ...' et
non une couleur définie par le css.

EDIT: Posté pendant la réponse de E.Bzz

[E.Bzz]

Pas besoin de 2 boucles : getElementsByTagName("td") est un accès "direct" ...

[marcha]

Pas besoin de 2 boucles : getElementsByTagName("td") est un accès "direct" ...

D'après ce que j'ai compris du problème, seuls les TD d'une couleur donnée
et qui sont dans un TR d'un className donné doivent provoquer le masquage
de la ligne, non ?

[E.Bzz]

EDIT: Posté pendant la réponse de E.Bzz

Oui.
Et complémentaires les réponses : toi tu as laissé les 2 boucles, et moi j'ai zappé le css qui sera pas pris en compte

Pour le reste, on est en phase

A+

[E.Bzz]

Code :

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

1
2
3
4
5
6
7
8
function detectcolumn(identificateur, coulor) {
  var i; 
for (i=0; i<document.getElementsByTagName("td").length; i++)  
{  var textcolor = document.getElementsByTagName("td")[i].style.color; 
if (document.getElementsByTagName("td")[i].className==identificateur) {  
if (textcolor==coulor)  {  
document.getElementsByTagName("td")[i].style.display = (document.getElementsByTagName("td")[i].style.display=="none")?"":"none"; } 
 }  }  }

Version light :

Code :

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

1
2
3
4
5
6
7
8
9
function detectcolumn(identificateur, coulor) {
  var i; 
for (i=0; i<document.getElementsByTagName("td").length; i++)  {  
with(document.getElementsByTagName("td")[i]) {
var textcolor = style.color; 
if (className==identificateur) {  
if (textcolor==coulor)  {  
style.display = (style.display=="none")?"block":"none"; 
}  }  }  }  }

Mais je doute que tu ais une couleur "coulor" (oups compris : les "" en trop !!)

... et l'inverse de "none" est "block" et pas ""

A+

EDIT : qq retours à la ligne ça aide ...