try catch et document.getElementById

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:

---

1 2 3 4

function truc() { try{n'importe quoi} catch{alert('erreur');} }

---

Ex 2 qui ne fonctionne pas

Code:

---

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

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

Bonjour,

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

Code:

---

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); }

---

MERCI de me mettre les yeux en face des trous :mouarf:

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:

---

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... :oops:

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

Encore merci

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

Citation:

---

* 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 :)

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

Code:

---

1 2 3 4

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

---

Citation:

---

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:

---

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 :ccool:
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. :D:D

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.

Citation:

---

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 ?

:mouarf: Afficher la Source html -> rechercher : idDeMonElement :aie:

ç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 :mrgreen:

Citation:

---

Envoyé par lifty

:mouarf: Afficher la Source html -> rechercher : idDeMonElement :aie:

ç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 :mrgreen:

---

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. ;)

Citation:

---

Envoyé par rolandcire

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

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

---

:koi:
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.

Citation:

---

Envoyé par Bovino

:koi:
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 !:D MERCI
Le sujet pour moi est résolu