Discussion :

[sebac]

Bonjour j'ai exactement le même problème que sur ce post http://www.developpez.net/forums/d79...ml-champ-vide/sauf que tous les if que je fais pour éviter que mon code plante ne marche pas. j'ai fais quelque chose de ce style

Code :

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

if(listeMag.item(j).getElementsByTagName("photo4Mag")==null){mon traitement }

mais mon code continue de planter.
Merci

[supersnail]

Bonjour,

Le "mon traitement" est à effectuer lorsque le champ n'est pas vide?

Parce que là, ton traitement est effectué si le champ est vide

[sebac]

peu importe je veux simplement tester que mon if fonctionne mais mon if n'est pas atteint voici mon code à moi avec le tag qui peut être vide dans mon xml

Code :

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

1
2
3
var itemsDescription = docXML.getElementsByTagName("descriptionSerie");
alert("ici");
 if(itemsDescription==null){alert("toto");}

ici est affiché et jamais le toto

[E.Bzz]

cloud76 t'a donné son point de vue (firstchild)

A+

[supersnail]

Bonjour,

Je peux voir la structure de ton XML?

[sebac]

Pour repondre à E.Bzz ce n'est pas le point de vu de cloud76 qui m'embête c'est juste comment détecter un tagName dont le fils est vide pour que mon code ne plante pas voulant récupérer le dit fils peut importe si je dois faire des if. Sinon voici mon xml et je suis quasiment certain que ce n'est pas lui qui pose problème

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
27
28
29
30
31
32
33
34
35
36
37
  header('Content-type: text/html; charset=UTF-8'); 
	header('Content-Type: text/xml'); 
	echo "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n";
	echo "<serie>\n";
 
 
	$code_serie=$_POST["code_serie"];// "code_serie" paramètre js envoyé par ajax lors de l'appel du fichier xmlSerie.php
	//$code_serie=3;
	$seri=new series();
	$seri->search_id($cnx,$code_serie);
	$code_exam=$seri->get_code_examen();// permettra de récupérer tous les SopinstanceUID et d'instancié la classe serie en js
	$exam=new examen();
	$exam->search_id($cnx,$code_exam);
    //récupération de toutes les images de la série dont le code_serie est passé en paramètre
	$img= new image();
	$tabImg=$img->searchImageByCodeSerie($cnx,$code_serie);// cette fonction retourne un tableau de tableau
 
 
    echo "<code_serie>".$code_serie."</code_serie>\n";
    echo "<serieInstanceUID>".$seri->get_SeriesInstanceUID()."</serieInstanceUID>\n";
    echo "<studyInstanceUID>".$exam->get_StudyInstanceUID()."</studyInstanceUID>\n";
    echo "<descriptionSerie>".$seri->get_description()."</descriptionSerie>\n";
    echo "<modaliteSerie>".$seri->get_nom_modalite()."</modaliteSerie>\n";
    echo "<nbImageSerie>".$seri->get_nb_image()."</nbImageSerie>\n";
 
 
	foreach ($tabImg as $i) 
    { 
 
	  echo "<image>".$i->get_SOPInstanceUID()."</image>\n";
	  echo "<rows>".$i->get_Rows()."</rows>\n";
	  echo "<columns>".$i->get_Columns()."</columns>\n";
 
	}
 
 
    echo "</serie>\n";

[supersnail]

Bonjour,

Bah en fait ton noeud descriptionSerie existe... Donc normal que ton test échoue...
Il faudrait plûtot regarder s'il a des enfants ou pas

Edit: c'est du PHP ton XML

Le XML généré serait plus utile dans ce cas

[RomainVALERI]

Il me semble que la méthode getElementsByTagName ne peut pas renvoyer null. Soit elle renvoie un tableau d'éléments, soit elle renvoie un tableau vide, mais dans aucun cas elle ne renvoie "null". Or un tableau vide n'est pas égal à "null". Du coup le test est plutôt mal barré

[sebac]

voila le test que je fais

Code :

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

1
2
3
alert(itemsDescription.length);
	 var description=itemsDescription.item(0).firstChild.data;
	 if(description==""){alert("toto");}

le premier alert affiche 1 donc il trouve bien un élément descriptionSerie dans mon xml grâce

Code :

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

var itemsDescription = docXML.getElementsByTagName("descriptionSerie");

et lorsque je teste le contenu de ce tagName, toto n'est jamais affiché et ça plante

[E.Bzz]

Code :

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

1
2
var description=itemsDescription.item(0).firstChild.data;
alert('/' + description + '/');

affiche quoi ?

A+

[sebac]

ça n'affiche rien, ça plante

[E.Bzz]

Donc faut "remonter" jusqu'à ce que ça plante plus, puis "redescendre" en testant les différente syntaxes en fonction de la structure de ton XML

Code :

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

alert(itemsDescription.item(0).firstChild);

Et comme déjà demandé par Supersnail, le XML généré serait sûrement d'un grand secoure (par pour moi, mais pour d'autres, sûrement )

A+

[RomainVALERI]

Heu... ^^

C'est par pour instster, mais je ne vois pas le rapport avec un quelconque problème XML dans le contexte décrit : quel que soit le contenu du XML, et même quel que soit la valeur de ta variable docXML, la méthode getElementsByTagName ne renverra jamais "null"...

Fais cet essai :

Code javascript :

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

1
2
var itemsDescription = docXML.getElementsByTagName("descriptionSerie");
 if (itemsDescription.length == 0) alert("coucou");

[E.Bzz]

Heu ...

Il n'est plus question du test, là.

Uniquement d'accéder au contenu de itemsDescription.item(0) il me semble ...

A+

[sebac]

Code :

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

alert(itemsDescription.item(0).firstChild);

plante toujours, sinon voici le xml généré

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?xml version="1.0" encoding="utf-8"?>
<serie>
<code_serie>18</code_serie>
<serieInstanceUID>1.2.826.0.1.3680043.2.406.1.3.305860632.4780.1271941557.6</serieInstanceUID>
<studyInstanceUID>1.2.826.0.1.3680043.2.406.1.2.211008914.16616.1194855120.244</studyInstanceUID>
<descriptionSerie></descriptionSerie>
<modaliteSerie>KO</modaliteSerie>
<nbImageSerie>1</nbImageSerie>
<image>1.2.826.0.1.3680043.2.406.1.4.305860632.4780.1271941557.4</image>
<rows>0</rows>
<columns>0</columns>
</serie>

[E.Bzz]

Ah oui, effectivement, si tu testes justement avec le cas qui, au final ne devra pas être affiché, forcément ça complique

ajoute le test RomainVALERI (qui évitera l'appel à l'alert() )

EDIT :

Il n'est plus question du test, là.

Donc, si

A+

[sebac]

voici la solution

Code :

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

1
2
3
4
5
6
7
var description = itemsDescription.item(0).childNodes.length && // enfants présents ?
		  itemsDescription.item(0).firstChild.data; // si oui, on récupère les données
		  // sinon, description vaut false
 
if(description) {
// do things
}

[sekaijin]

Attention de ne pas confondre NULL et UNDEFINED
pour savoir si quelque chose existe utiliser

Code :

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

if ('undefined' == typeof varname ) {...

A+JYT

[RomainVALERI]

Attention de ne pas confondre NULL et UNDEFINED
pour savoir si quelque chose existe utiliser

Code :

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

if ('undefined' == typeof varname ) {...

A+JYT

Tout-à-fait ^^

On peut même rencontrer cette autre syntaxe :

Code javascript :

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

if (undefined === varname) {...}