try catch et document.getElementById

**rolandcire** · 13/08/2009, 15h59

Bonjour,

Je cherche à améliorer ma manière de coder (et j'en ai besoin !) en faisant une gestion d'erreur via try catch. Or je n'y parviens pas dans le cas d'un document.getElementById(name).
Ex 1 qui fonctionne bien

Code :

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

1
2
3
4
function truc()
 { try{n'importe quoi}
   catch{alert('erreur');}
 }

Ex 2 qui ne fonctionne pas

Code :

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

1
2
3
4
function truc2(name)
 { try{document.getElementById(name);}
   catch{alert('erreur');}
 }

Dans le second cas lors de l'appel de la fonction, si l'argument "name" fourni est une "id" qui n'existe pas,
je n'obtiens aucun message d'erreur via catch !

Par avance, merci

**lifty** · 13/08/2009, 16h35

Il n'y a pas d'erreur tout simplement parce que la méthode getElementById ne retourne jamais d'exceptions. Si il ne trouve pas de correspondance avec l'id passé en paramètre alors il renvoi null et lève pas d'exceptions.

Google -> mot clés : exception javascript getelementbyid -> Premier lien

**Royce** · 13/08/2009, 16h38

Bonjour,

En effet, getElementById() ne gère pas d'exception mais tu peux lui indiquer les exceptions à lever, voir l'exemple ci-dessous.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
try
{
        var v = document.getElementById("sdfs");
        if (v == null) throw "erreur";
}
catch (e)
{
   	alert(e);
}

**rolandcire** · 13/08/2009, 16h44

MERCI de me mettre les yeux en face des trous

Mon niveau n'est pas suffisant. Je n'aurais pas déduit de l'absence de retour d'une exception le fait que le try catch ne fonctionne pas.

Si je comprends bien, je dois donc en revenir à une bonne vieille méthode telle que :

Code :

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

1
2
3
4
5
6
7
 
if (!document.getElementById(name)) {
         alert('erreur...');
}
else
{....
}

Et moi qui espérait pouvoir coder plus proprement qu'avec ces if else.. via les try catch...

Il n'y a vraiment pas de possibilité d'utiliser un try catch en "trichant" un peu ?

Encore merci

**le_chomeur** · 13/08/2009, 16h44

Pour allez un peu plus loin, voici la liste des Exceptions :

* Error - base type for all errors. Never actually thrown by the engine.
* EvalError - thrown when an error occurs during execution of code via eval()
* RangeError - thrown when a number is outside the bounds of its range. For example, trying to create an array with -20 items (new Array(-20)). These occur rarely during normal execution.
* ReferenceError - thrown when an object is expected but not available, for instance, trying to call a method on a null reference.
* SyntaxError - thrown when the code passed into eval() has a syntax error.
* TypeError - thrown when a variable is of an unexpected type. For example, new 10 or "prop" in true.
* URIError - thrown when an incorrectly formatted URI string is passed into encodeURI, encodeURIComponent, decodeURI, or decodeURIComponent.

elles sont assez simple a comprendre , je les laisse donc en anglais

**Bovino** · 13/08/2009, 16h44

Hum... j'ai beau chercher, je vois pas trop l'utilité...

Code :

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

1
2
3
4
var v = document.getElementById("sdfs");
if (!v){
    alert('Cet élément n\'existe pas !'); // Ou toute autre action.
}

**rolandcire** · 13/08/2009, 16h55

Envoyé par Royce

Bonjour,

En effet, getElementById() ne gère pas d'exception mais tu peux lui indiquer les exceptions à lever, voir l'exemple ci-dessous.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
try
 {
        var v = document.getElementById("sdfs");
  	if (v == null) throw "erreur"
}
catch (e)
{
   	alert(e);
}

Ahhhhhhhhhhhhhhhhhh YES LUMINEUX !
Je viens de tester : c'est TOUT BON

Un grand grand MERCI.

@ Bovino : le but pour moi est de pouvoir déclencher une erreur si l'id citée dans document.getElementById("sdfs") n'existe pas dans le HTML. Cela a en effet un impact fort sur mon interface en développement. Mais peut-être n'est-ce pas la bonne méthode ?

MERCI à TOUS, je cherchais depuis hier.

**lifty** · 13/08/2009, 17h10

Il faut alors se poser la question : Pourquoi cet élément n'a t-il pas été généré.
Le client n'a pas d'impact (en théorie) sur le contenu html de ta page. Si tu as codé correctement dans ta page un élément il sera alors affiché, donc le test sur l'existence de cette balise n'a pas lieu d'être.

**rolandcire** · 13/08/2009, 17h16

Envoyé par lifty

Il faut alors se poser la question : Pourquoi cet élément n'a t-il pas été généré.
Le client n'a pas d'impact (en théorie) sur le contenu html de ta page. Si tu as codé correctement dans ta page un élément il sera alors affiché, donc le test sur l'existence de cette balise n'a pas lieu d'être.

OUI, mais justement lever de telles exceptions fait gagnr du temps dans le débuguage de nouveaux templates un peu compliqués. Non ?

**lifty** · 13/08/2009, 17h30

Afficher la Source html -> rechercher : idDeMonElement

ça va me prendre 3 sec de vérifier l'existence d'un element dont je connais l'idée, point besoin de javascript pour ça, si tu lève une exception c'est pour faire un traitement particulier notamment côté client. Mais je te l'accorde c'est possible de faire comme ça aussi

**rolandcire** · 13/08/2009, 17h52

Envoyé par lifty

Afficher la Source html -> rechercher : idDeMonElement

ça va me prendre 3 sec de vérifier l'existence d'un element dont je connais l'idée, point besoin de javascript pour ça, si tu lève une exception c'est pour faire un traitement particulier notamment côté client. Mais je te l'accorde c'est possible de faire comme ça aussi

OUI, cela prend 3 secondes. Le problème pour moi est que ces erreurs sont muettes... et que l'on se retrouve avec une interface "bloquée" à chercher quelle est l'id et la portion de js qui coincent.

**Bovino** · 13/08/2009, 18h03

Envoyé par rolandcire

Et moi qui espérait pouvoir coder plus proprement qu'avec ces if else.. via les try catch...

Il n'y a vraiment pas de possibilité d'utiliser un try catch en "trichant" un peu ?

En quoi "tricher un peu" constitue une manière propre de coder ?
Je reste persuadé que if / else est plus propre que générer une erreur inexistante, en plus cela te permet tout autant de débugger et de voir où se situent les problèmes.

**rolandcire** · 13/08/2009, 18h08

Envoyé par Bovino

En quoi "tricher un peu" constitue une manière propre de coder ?
Je reste persuadé que if / else est plus propre que générer une erreur inexistante, en plus cela te permet tout autant de débugger et de voir où se situent les problèmes.

Bon, ben convaincu !

MERCI
Le sujet pour moi est résolu

try catch et document.getElementById

JavaScript

Discussions similaires

Partager

Partager