getElementById is not a function

Version imprimable

Bonjour,

J'ai une erreur, "getElementById is not a function" que je ne comprend pas.

voici ma fonction (ajax) qui supprime un document.

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
function supprimer_archive(num,document)
{
    if (confirm('supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
    if (confirm('êtes-vous bien sûr de vouloir supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
var num = num;
var document = document;
var dossier = 'archives';
 
    xhr.open('POST','./php/script_archive_supprimer.php',true);
    xhr.onreadystatechange = function()
        {
            if ((xhr.readyState == 1)||(xhr.readyState == 2)||(xhr.readyState == 3))
            {
            }
            else if (xhr.readyState == 4)
            {
                // supprimer le tr
                var node = document.getElementById('tr'+num);
                var parent = node.parentNode;
                parent.removeChild(node);
                // afficher message de confirmation
                alert('l\'archive « '+xhr.responseText+' » a été supprimée de la base.');
            }
        }
    xhr.setRequestHeader('Content-type','application/x-www-form-urlencoded');
    var data = 'document='+escape(document)+'&dossier='+escape(dossier);
    xhr.send(data);
}

le document est bien supprimé. pas de problème.
mais ce petit bout de code :

Code:

1
2
3
4
5
 
 // supprimer le tr
var node = document.getElementById('tr'+num);
var parent = node.parentNode;
parent.removeChild(node);

est sensé me supprimer la ligne de mon tableau.
bien sûr elle n'est pas supprimée et message d'erreur dans la console :
"getElementById is not a function" !!

merci pour votre aide...

30/01/2009, 16h28
vermine

Bonjour,

Tu as plusieurs formulaires dans ta page ?
Car dans document.getElementById(...);, document fait référence au formulaire. Or toi, tu lui passes une valeur pour la variable document. Que lui passes-tu ?
30/01/2009, 17h52
kryogen

non aucun formulaire... je fais référence à un <tr> d'une table...
30/01/2009, 18h09
Auteur
bonjour,

Code:

var num = num;

c'est curieux ceci... Ta variable num est un paramètre, inutile de la déclarer par la suite. D'ailleurs cette ligne ne déclenche pas une erreur ?

Pour supprimer une ligne d'un tableau utilise deleteRow() c'est plus efficace ;)

Code:

document.getElementById('idTableau').deleteRow(indexLigne);

en reprenant ton code :
Code:

1 2 3 var ligne = document.getElementById('tr'+num); document.getElementById('idDuTableau').deleteRow(ligne.rowIndex);
kryogen : le tr ciblé possède bien un id ?

sinon pour Vermine : getElementById() est une méthode de document exclusivement, donc la syntaxe de kryogen est juste. Par ailleurs un id est unique dans la page donc que l'objet soit dans un formulaire ou pas, le résultat sera le même ;)
30/01/2009, 19h16
Bovino

Citation:

Code:

var num = num;

c'est curieux ceci...

Oui, mais le plus... "exotique", ce serait surtout :

Code:

var document = document;

:roll::roll::roll:
Effectivement, quand tu fais ensuite

Code:

document.getElementById

tu parles de quoi, de la variable ou de l'objet document ???

Il ne faut JAMAIS donner ce genre de nom à une variable !!!
30/01/2009, 19h31
Auteur

Citation:

Envoyé par Bovino

Oui, mais le plus... "exotique", ce serait surtout :

Code:

var document = document;

:roll::roll::roll:
Effectivement, quand tu fais ensuite

Code:

document.getElementById

tu parles de quoi, de la variable ou de l'objet document ???

Il ne faut JAMAIS donner ce genre de nom à une variable !!!

ouh là, oui, je ne l'ai pas vu celle-là :bug:

Je comprends mieux la remarque de Vermine maintenant :D
31/01/2009, 03h19
kryogen

ok ok :oops:
je ne suis qu'un modeste novice en javascript... pitié :)
bon merci en tout cas je vais tout corriger avec toutes vos remarques...
pour les var num = num, je pensais (à tord) qu'il fallait les redéclarer... :oops:
merci encore mes bons maîtres :king:
bon week end !
31/01/2009, 12h33
Auteur

Déjà pour bien corriger le code dis nous à quoi fait référence ta variable document ;)
Quels sont les paramètres que tu transmets à ta fonction supprimer_archive() lorsque tu l'appelles ?
31/01/2009, 13h32
kryogen

num correspond à la ligne (du tableau listant les documents d'un dossier sur le serveur) à supprimer, exemple : <tr id="tr1"> (id généré en PHP).
document correspond au nom du document que je vais supprimer physiquement sur le serveur de manière asynchrone (exemple : 000001-20080210100245-exemple.pdf).
voilà...

C'est bon c'est corrigé !
Il est vrai que la remarque de la vermine n'était pas sans intérêt.
En effet y avait une pauvre confusion entre la variable document et document.getElementById.
Faut le reconnaître c'est une horrible erreur de débutant (ce que je suis, faut pas se voiler la face).
donc voici le code qui marche trop cool génial :

Code:

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
///////////////////////////////////////////
// supprimer une archive //////////////////
///////////////////////////////////////////
 
function supprimer_archive(num,intitule)
{
    if (confirm('supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
    if (confirm('êtes-vous bien sûr de vouloir supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
var dossier = 'archives';
 
    xhr.open('POST','./php/script_archive_supprimer.php',true);
    xhr.onreadystatechange = function()
        {
            if ((xhr.readyState == 1)||(xhr.readyState == 2)||(xhr.readyState == 3))
            {
            }
            else if (xhr.readyState == 4)
            {
                // supprimer le tr
                var node = document.getElementById('tr'+num);
                var parent = node.parentNode;
                parent.removeChild(node);
                // afficher message de confirmation
                alert('l\'archive « '+xhr.responseText+' » a été supprimée de la base.');
            }
        }
    xhr.setRequestHeader('Content-type','application/x-www-form-urlencoded');
    var data = 'intitule='+escape(intitule)+'&dossier='+escape(dossier);
    xhr.send(data);
}

merci à tous :king:
bonne continuation !
youhou ! :mouarf:

ps : notez bien la disparition de var num = num ;)