[POO] Appel d'une fonction d'un objet

**dragonfly** · 30/11/2009, 21h21

Bonjour à tous.

Lorsque j'appelle une fonction définie dans un objet (lui même définie un peu plus haut), j'ai une erreur :

Code :

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

1
2
 
tabM is not defined

Voici mon code :

Code :

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

1
2
3
4
5
 
<table id="tabMorpion" cellspacing="0" onload="var tabM=new morpion(3,3);">
<tbody>
<tr id="caseM1">
<td id="caseM11" onclick="tabM.remplir('X',this.id);" colspan="1">

L'objet étant défini au chargement je ne comprend pas pourquoi il ne le trouve pas lors de l'appel dans le onclick

Merci d'avance.
Dragonfly

**Arnaud F.** · 30/11/2009, 21h38

Bonjour,

l'évènement onload ne se pose pas sur la balise table comme vous avez pu le faire mais bel et bien dans la balise body. Ensuite, on ne peut pas assigner de variable directement dans l'évènement.

Code html :

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
 
<head>
<script type='text/javascript'>
var tabM;
function bodyLoad() {
    tabM = new morpion(3, 3);
}
</script>
</head>
<body onload="bodyLoad()">
<table id="tabMorpion" cellspacing="0">
<tbody>
<tr id="caseM1">
<td id="caseM11" onclick="tabM.remplir('X',this.id);" colspan="1">
<!-- Votre code -->
</body>

**dragonfly** · 02/12/2009, 12h54

Argh, j'avais omis le fait qu'un onload est pas copain avec une table.

Je vais devoir me débrouiller autrement.

En tout cas merci de ta réponse, ca va m'être utile.

**Bovino** · 02/12/2009, 13h19

Envoyé par dragonfly

Argh, j'avais omis le fait qu'un onload est pas copain avec une table.

onload ne s'applique qu'à la balise body et aux éléments remplacés (img, iframe, object, etc.)

Envoyé par Arnaud F.

on ne peut pas assigner de variable directement dans l'évènement.

En fait, techniquement, c'est possible, mais l'attribut onclick génère une fonction anonyme comme contexte d'exécution, donc la variable n'est disponible que dans ce contexte, ce qui n'est bien sûr pas le comportement désiré.
Pour info, même si ta solution est meilleure, il est malgré tout possible d'assigner une variable globale comme ça :

Code :

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

<div onclick="window.mavar = 'variable'">Cliquer</div>

en tout cas, même si window est généralement implicite, éviter absolument la syntaxe :

Code :

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

<div onclick="mavar = 'variable'">Cliquer</div>

**Arnaud F.** · 02/12/2009, 13h25

Envoyé par Bovino

En fait, techniquement, c'est possible, mais l'attribut onclick génère une fonction anonyme comme contexte d'exécution, donc la variable n'est disponible que dans ce contexte, ce qui n'est bien sûr pas le comportement désiré.
Pour info, même si ta solution est meilleure, il est malgré tout possible d'assigner une variable globale comme ça :

Code :

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

<div onclick="window.mavar = 'variable'">Cliquer</div>

en tout cas, même si window est généralement implicite, éviter absolument la syntaxe :

Code :

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

<div onclick="mavar = 'variable'">Cliquer</div>

Merci du tuyau

Je savais pour les fonctions anonymes mais pour les déclarations de variables j'avoue préférer les déclarer dans le <script> directement pour ne pas avoir les rechercher dans les X pages ou pourront être incorporés le JS...

Question d'évolutivité / de maintenabilité tout ça. Mais ça ne regarde que moi

++