RE: [xsl] How to find parent's-parent's-sibling-sibling name

Subject: RE: [xsl] How to find parent's-parent's-sibling-sibling name
From: "Michael Kay" <michael.h.kay@xxxxxxxxxxxx>
Date: Thu, 19 Sep 2002 15:06:50 +0100
> You could use variable:
> <xsl:variable name="FOO">
>    <xsl:value-of select="BehaviourInfo/Trap/Name"/>
> </xsl:variable>
> 

An xsl:variable that contains a single xsl:value-of is almost always bad
coding. Use

<xsl:variable name="FOO" select="string(BehaviourInfo/Trap/Name)"/>

Apart from the fact that it's one line of code instead of three, it also
defines the variable as a string rather than as a tree. The only useful
thing you can do with this kind of tree in practice is to convert it to
a string, so it's much simpler and more efficient to create it as a
string in the first place.

I thought it worth mentioning because I'm seeing more and more examples
of this bad coding style.

Of course,

<xsl:variable name="FOO">
   <xsl:copy-of select="BehaviourInfo/Trap/Name"/>
</xsl:variable>

is even worse, and also very common.

Michael Kay
Software AG
home: Michael.H.Kay@xxxxxxxxxxxx
work: Michael.Kay@xxxxxxxxxxxxxx 


 XSL-List info and archive:  http://www.mulberrytech.com/xsl/xsl-list


Current Thread