Discussion :

[kacedda]

Bonjour,

Désolé pour la prose . Je suis débutant en xslt et je souhaiterai juste une correction de mon code, celui-ci fonctionne.

J'ai un fichier XML structuré ainsi, ce fichier est fourni par un soft tiers, je ne peux pas modifier la structure du xml:

Code :

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

1
2
3
4
5
6
7
8
9
		<ElementA attributElA="unechaine">
			<ElementB>
				<ElementC1>IndexRechercher</ElementC1>
				<ElementC2>ValeurACopier</ElementC2>
			</ElementB>
			<ElementB>
				<ElementC1>IndexNonrechercher</ElementC1>
				<ElementC2>ValeurAIgnorer</ElementC2>
			</ElementB>

J'ai 1 à n élément ElementA.
Dans chaque ElementA, j'ai 1 à n élément ElementB.
Dans chaque ElementB, j'ai uniquement et toujours 2 éléments nommés ElementC1 et ElementC2

Je souhaite récupérer un fichier plat contenant:
- l'attribut attributElA des éléments ElementA
et
- la valeur de l'élément ElementC2.
Ce binôme ne doit être récupérer que si ElementC1 est égale à une certaine valeur.


Voici ma méthode:

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
<?xml version="1.0" encoding="UTF-8"?>
<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
	<xsl:output method="text"/>
 
	<xsl:template match="/">
 
	<xsl:for-each select="ElementA">
 
	<xsl:variable name="myElement" select="."/>
 
		<xsl:for-each select="ElementB">
 
			<xsl:if test="ElementC1='IndexRechercher'">
 
				<xsl:value-of select="$myElement/@attributElA"/>
				<xsl:value-of select="ElementC2"/>
 
			</xsl:if>			
		</xsl:for-each>
	</xsl:for-each>
	</xsl:template>
</xsl:stylesheet>

Cela fonctionne chez moi, et j'ai le résultat souhaité. C'est juste que j'aimerai améliorer mes connaissances et avoir vos critiques sur mon script, je ne suis pas sur d'avoir bien compris le fonctionnement de la variable par exemple et si je l'utilise bien.

Merci

[tsuji]

Je m'en doute que ça marche sans toutefois préciser un peu plus que je suis sûr vous vous avez juste épargné de le faire ici.

J'ai 1 à n élément ElementA.

On peut déduire donc ElementA ne peut pas être la racine du document et en tout cas il doit y avoir une élément englobant des éléments A. En conséquence, ceci

<xsl:template match="/">
<xsl:for-each select="ElementA">

ne peut pas marcher. Si le parent des ElementA est la racine du document, il doit être au moins comme ça pour qu'il marche.

Code :

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

1
2
<xsl:template match="/racine">
	<xsl:for-each select="ElementA">

A part ça, le template est bien propre : si on veut critiquer, il manque un peu de mis-en-forme plus raisonable pour les sorties, et puis, je préfererais un peu plus de mettre la variable directement l'attribut au lieu de l'élément lui-mê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
<xsl:template match="/racine">
    <xsl:for-each select="ElementA">
        <xsl:variable name="myAttribute" select="@attributElA"/>
        <xsl:for-each select="ElementB">
            <xsl:if test="ElementC1='IndexRechercher'">
                <xsl:value-of select="$myAttribute"/>
                <xsl:text>, </xsl:text>
                <xsl:value-of select="ElementC2"/>
                <!-- nouvelle ligne style windows, sinon on s'adapte un peu -->
                <xsl:text>&#x0d;&#x0a;</xsl:text>
            </xsl:if>			
        </xsl:for-each>
    </xsl:for-each>
</xsl:template>

[thelvin]

Je dirais aussi que le if n'est pas nécessaire et reflète le manque d'habitude des prédicats :

Code :

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

1
2
3
4
5
6
<xsl:for-each select="ElementB">
  <xsl:if test="ElementC1='IndexRechercher'">
    <!-- traitement -->
  </xsl:if>
  <!-- et rien d'autre -->
</xsl:for-each>

s'écrirait plutôt :

Code :

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

1
2
3
<xsl:for-each select="ElementB[ElementC1 = 'IndexRechercher']">
  <!-- traitement -->
</xsl:for-each>

Mais bon, c'est pas super important non plus.

[kacedda]

Bonjour,
Merci pour vos commentaires, j'ai amélioré mon script!
a+