Discussion :

[ilyespython]

J'essaie de mettre la couleur d'une case en rouge dans un tableau dynamique mais la fonction ne reçoit pas la valeur que j'envoie

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
function create_tableau()
{
var ligne1 = document.getElementById("ligne").value;
var colonne1 = document.getElementById("colonne").value;
var tbl = document.getElementById("dessin");    
    for (var ligne = 0; ligne < ligne1; ligne++) {
        var row = document.createElement("tr");
        for (var colonne = 0 ; colonne < colonne1; colonne++) {
            var cell = document.createElement("td");
            var cellText = document.createTextNode("");
            cell.setAttribute("onclick","Change_case(ligne,colonne);");
            alert(ligne);
            cell.id=""+ligne+colonne;
            cell.appendChild(cellText);
            row.appendChild(cell);
        }
        tbl.appendChild(row);
    }
}
 
function Change_case(ligne,colonne)
{
    num=""+ligne+colonne;
    case_cliquee = document.getElementById(num);
    case_cliquee.style.backgroundColor = 'red';
}

[ProgElecT]

Salut
Ligne 11

Code :

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

            cell.setAttribute("onclick","Change_case(" + ligne +"," + colonne +");");

Me semble t il.

[NoSmoking]

Bonjour,
je me permet une incursion dans cette discussion, qui même si résolue sans aucun remerciement ou autre de la part de son auteur, le manque de politesse devenant chronique, car elle révèle une mauvaise pratique quant à l’affectation d'un événement à un élément.
Je parle de :

Code :

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

1
2
cell.setAttribute("onclick","Change_case(ligne,colonne);");
cell.id=""+ligne+colonne;

pourquoi affecter une ID à l'élément pour la passer à une fonction qui va la récupérer pour pouvoir cibler l'élément, sur lequel on met l'écouteur ....

En gros, plusieurs solutions sont possibles, mais il eut été mieux, à peine, de faire

Code :

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

1
2
3
cell.onclick = function(){
  this.style.backgroundColor = "red";
}

... peut-être plus judicieux de manipuler une classe.

Nota : dans le cas présent la délégation d'événement sur la <table> parent serait une solution plus propre et qui fonctionnerait quelque soit le nombre de cellules crées.


Bon un petit rappel :

	IV-G. Remerciements Il est de bon ton de remercier la personne qui vous a aidé bénévolement et de lui signaler que sa réponse a bien résolu votre problème ou bien répondu à votre question.